Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Honglei Wang <wanghonglei@didichuxing.com>
To:     Ingo Molnar <mingo@redhat.com>,
        Peter Zijlstra <peterz@infradead.org>,
        Juri Lelli <juri.lelli@redhat.com>,
        Vincent Guittot <vincent.guittot@linaro.org>,
        Dietmar Eggemann <dietmar.eggemann@arm.com>,
        Steven Rostedt <rostedt@goodmis.org>,
        Ben Segall <bsegall@google.com>, Mel Gorman <mgorman@suse.de>,
        Daniel Bristot de Oliveira <bristot@redhat.com>,
        <linux-kernel@vger.kernel.org>
CC:     Honglei Wang <jameshongleiwang@126.com>,
        Honglei Wang <wanghonglei@didichuxing.com>
Subject: [PATCH] sched debug: refine the output of sched-debug
Date:   Fri, 25 Feb 2022 12:01:17 +0800
Message-ID: <20220225040117.40235-1-wanghonglei@didichuxing.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7BIT
Content-Type:   text/plain; charset=US-ASCII
Precedence: bulk

Several items including sum-block, numabalancing associates and task
group path were missed in the header of sched debug info. Let's add them
to make the info clearer.

Signed-off-by: Honglei Wang <wanghonglei@didichuxing.com>
---
 kernel/sched/debug.c | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/kernel/sched/debug.c b/kernel/sched/debug.c
index aa29211de1bf..ca1316f1def6 100644
--- a/kernel/sched/debug.c
+++ b/kernel/sched/debug.c
@@ -548,7 +548,7 @@ print_task(struct seq_file *m, struct rq *rq, struct task_struct *p)
 		SPLIT_NS(schedstat_val_or_zero(p->stats.sum_block_runtime)));
 
 #ifdef CONFIG_NUMA_BALANCING
-	SEQ_printf(m, " %d %d", task_node(p), task_numa_group_id(p));
+	SEQ_printf(m, "        %d           %d", task_node(p), task_numa_group_id(p));
 #endif
 #ifdef CONFIG_CGROUP_SCHED
 	SEQ_printf_task_group_path(m, task_group(p), " %s")
@@ -563,8 +563,16 @@ static void print_rq(struct seq_file *m, struct rq *rq, int rq_cpu)
 
 	SEQ_printf(m, "\n");
 	SEQ_printf(m, "runnable tasks:\n");
-	SEQ_printf(m, " S            task   PID         tree-key  switches  prio"
-		   "     wait-time             sum-exec        sum-sleep\n");
+	SEQ_printf(m, " S            task   PID         tree-key  switches  prio");
+	SEQ_printf(m, "     wait-time             sum-exec        sum-sleep");
+	SEQ_printf(m, "        sum-block");
+#ifdef CONFIG_NUMA_BALANCING
+	SEQ_printf(m, "    node  numa-group");
+#endif
+#ifdef CONFIG_CGROUP_SCHED
+	SEQ_printf(m, " task-group");
+#endif
+	SEQ_printf(m, "\n");
 	SEQ_printf(m, "-------------------------------------------------------"
 		   "------------------------------------------------------\n");
 
-- 
2.14.1