Don't use special flag to indicate power-only mode,
just set proc_num to 0.

Signed-off-by: Stanislav Fomichev <stfomic...@yandex-team.ru>
---
 tools/perf/builtin-timechart.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/tools/perf/builtin-timechart.c b/tools/perf/builtin-timechart.c
index 51a5532b3382..74c040abb3af 100644
--- a/tools/perf/builtin-timechart.c
+++ b/tools/perf/builtin-timechart.c
@@ -911,7 +911,7 @@ static int determine_display_tasks(u64 threshold)
                /* no exit marker, task kept running to the end */
                if (p->end_time == 0)
                        p->end_time = last_time;
-               if (p->total_time >= threshold && !power_only)
+               if (p->total_time >= threshold)
                        p->display = 1;
 
                c = p->all;
@@ -922,7 +922,7 @@ static int determine_display_tasks(u64 threshold)
                        if (c->start_time == 1)
                                c->start_time = first_time;
 
-                       if (c->total_time >= threshold && !power_only) {
+                       if (c->total_time >= threshold) {
                                c->display = 1;
                                count++;
                        }
@@ -949,6 +949,8 @@ static void write_svg_file(const char *filename)
 
        numcpus++;
 
+       if (power_only)
+               proc_num = 0;
 
        /* We'd like to show at least proc_num tasks;
         * be less picky if we have fewer */
@@ -966,9 +968,11 @@ static void write_svg_file(const char *filename)
                svg_cpu_box(i, max_freq, turbo_frequency);
 
        draw_cpu_usage();
-       draw_process_bars();
+       if (proc_num)
+               draw_process_bars();
        draw_c_p_states();
-       draw_wakeups();
+       if (proc_num)
+               draw_wakeups();
 
        svg_close();
 }
-- 
1.8.3.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to