]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
4.14-stable patches
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 20 Jul 2020 09:54:41 +0000 (11:54 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 20 Jul 2020 09:54:41 +0000 (11:54 +0200)
added patches:
perf-stat-zero-all-the-ena-and-run-array-slot-stats-for-interval-mode.patch

queue-4.14/perf-stat-zero-all-the-ena-and-run-array-slot-stats-for-interval-mode.patch [new file with mode: 0644]
queue-4.14/series

diff --git a/queue-4.14/perf-stat-zero-all-the-ena-and-run-array-slot-stats-for-interval-mode.patch b/queue-4.14/perf-stat-zero-all-the-ena-and-run-array-slot-stats-for-interval-mode.patch
new file mode 100644 (file)
index 0000000..1f2445d
--- /dev/null
@@ -0,0 +1,49 @@
+From 0e0bf1ea1147fcf74eab19c2d3c853cc3740a72f Mon Sep 17 00:00:00 2001
+From: Jin Yao <yao.jin@linux.intel.com>
+Date: Thu, 9 Apr 2020 15:07:55 +0800
+Subject: perf stat: Zero all the 'ena' and 'run' array slot stats for interval mode
+
+From: Jin Yao <yao.jin@linux.intel.com>
+
+commit 0e0bf1ea1147fcf74eab19c2d3c853cc3740a72f upstream.
+
+As the code comments in perf_stat_process_counter() say, we calculate
+counter's data every interval, and the display code shows ps->res_stats
+avg value. We need to zero the stats for interval mode.
+
+But the current code only zeros the res_stats[0], it doesn't zero the
+res_stats[1] and res_stats[2], which are for ena and run of counter.
+
+This patch zeros the whole res_stats[] for interval mode.
+
+Fixes: 51fd2df1e882 ("perf stat: Fix interval output values")
+Signed-off-by: Jin Yao <yao.jin@linux.intel.com>
+Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
+Cc: Andi Kleen <ak@linux.intel.com>
+Cc: Jin Yao <yao.jin@intel.com>
+Cc: Jiri Olsa <jolsa@kernel.org>
+Cc: Kan Liang <kan.liang@linux.intel.com>
+Cc: Peter Zijlstra <peterz@infradead.org>
+Link: http://lore.kernel.org/lkml/20200409070755.17261-1-yao.jin@linux.intel.com
+Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+
+---
+ tools/perf/util/stat.c |    6 ++++--
+ 1 file changed, 4 insertions(+), 2 deletions(-)
+
+--- a/tools/perf/util/stat.c
++++ b/tools/perf/util/stat.c
+@@ -350,8 +350,10 @@ int perf_stat_process_counter(struct per
+        * interval mode, otherwise overall avg running
+        * averages will be shown for each interval.
+        */
+-      if (config->interval)
+-              init_stats(ps->res_stats);
++      if (config->interval) {
++              for (i = 0; i < 3; i++)
++                      init_stats(&ps->res_stats[i]);
++      }
+       if (counter->per_pkg)
+               zero_per_pkg(counter);
index 9b94a1c91366fe0b35931c7df344cce50ab757e7..9a1dd2c4377022916c0643ee7ceb56f1e7340c27 100644 (file)
@@ -87,3 +87,4 @@ staging-comedi-verify-array-index-is-correct-before-.patch
 revert-thermal-mediatek-fix-register-index-error.patch
 arm-dts-socfpga-align-l2-cache-controller-nodename-w.patch
 copy_xstate_to_kernel-fix-typo-which-caused-gdb-regression.patch
+perf-stat-zero-all-the-ena-and-run-array-slot-stats-for-interval-mode.patch