[PATCH] sched_ext: Include remaining task time slice in error state dump

Andrea Righi posted 1 patch 1 year, 1 month ago
kernel/sched/ext.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[PATCH] sched_ext: Include remaining task time slice in error state dump
Posted by Andrea Righi 1 year, 1 month ago
Report the remaining time slice when dumping task information during an
error exit.

This information can be useful for tracking incorrect or excessively
long time slices in schedulers that implement dynamic time slice logic.

Signed-off-by: Andrea Righi <arighi@nvidia.com>
---
 kernel/sched/ext.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/sched/ext.c b/kernel/sched/ext.c
index 0ce116e0f67c..010ce7680bbc 100644
--- a/kernel/sched/ext.c
+++ b/kernel/sched/ext.c
@@ -5218,9 +5218,9 @@ static void scx_dump_task(struct seq_buf *s, struct scx_dump_ctx *dctx,
 		  scx_get_task_state(p), p->scx.flags & ~SCX_TASK_STATE_MASK,
 		  p->scx.dsq_flags, ops_state & SCX_OPSS_STATE_MASK,
 		  ops_state >> SCX_OPSS_QSEQ_SHIFT);
-	dump_line(s, "      sticky/holding_cpu=%d/%d dsq_id=%s dsq_vtime=%llu",
+	dump_line(s, "      sticky/holding_cpu=%d/%d dsq_id=%s dsq_vtime=%llu slice=%llu",
 		  p->scx.sticky_cpu, p->scx.holding_cpu, dsq_id_buf,
-		  p->scx.dsq_vtime);
+		  p->scx.dsq_vtime, p->scx.slice);
 	dump_line(s, "      cpus=%*pb", cpumask_pr_args(p->cpus_ptr));
 
 	if (SCX_HAS_OP(dump_task)) {
-- 
2.47.1
Re: [PATCH] sched_ext: Include remaining task time slice in error state dump
Posted by Tejun Heo 1 year, 1 month ago
On Mon, Jan 06, 2025 at 08:20:34AM +0100, Andrea Righi wrote:
> Report the remaining time slice when dumping task information during an
> error exit.
> 
> This information can be useful for tracking incorrect or excessively
> long time slices in schedulers that implement dynamic time slice logic.
> 
> Signed-off-by: Andrea Righi <arighi@nvidia.com>

Acked-by: Tejun Heo <tj@kernel.org>

Thanks.

-- 
tejun