diff options
| author | 晏艳(采苓) <yanyan.yan@antgroup.com> | 2023-05-06 15:42:53 +0800 |
|---|---|---|
| committer | Peter Zijlstra <peterz@infradead.org> | 2023-05-08 10:58:39 +0200 |
| commit | a6fcdd8d95f7486150b3faadfea119fc3dfc3b74 (patch) | |
| tree | 855ebc124be72a6d5a0def233150e5b9b7b3985a /kernel/sched | |
| parent | bf2dc42d6beb890c995b8b09f881ef1b37259107 (diff) | |
| download | lwn-a6fcdd8d95f7486150b3faadfea119fc3dfc3b74.tar.gz lwn-a6fcdd8d95f7486150b3faadfea119fc3dfc3b74.zip | |
sched/debug: Correct printing for rq->nr_uninterruptible
Commit e6fe3f422be1 ("sched: Make multiple runqueue task counters
32-bit") changed the type for rq->nr_uninterruptible from "unsigned
long" to "unsigned int", but left wrong cast print to
/sys/kernel/debug/sched/debug and to the console.
For example, nr_uninterruptible's value is fffffff7 with type
"unsigned int", (long)nr_uninterruptible shows 4294967287 while
(int)nr_uninterruptible prints -9. So using int cast fixes wrong
printing.
Signed-off-by: Yan Yan <yanyan.yan@antgroup.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20230506074253.44526-1-yanyan.yan@antgroup.com
Diffstat (limited to 'kernel/sched')
| -rw-r--r-- | kernel/sched/debug.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/sched/debug.c b/kernel/sched/debug.c index 0b2340a79b65..066ff1c8ae4e 100644 --- a/kernel/sched/debug.c +++ b/kernel/sched/debug.c @@ -777,7 +777,7 @@ static void print_cpu(struct seq_file *m, int cpu) #define P(x) \ do { \ if (sizeof(rq->x) == 4) \ - SEQ_printf(m, " .%-30s: %ld\n", #x, (long)(rq->x)); \ + SEQ_printf(m, " .%-30s: %d\n", #x, (int)(rq->x)); \ else \ SEQ_printf(m, " .%-30s: %Ld\n", #x, (long long)(rq->x));\ } while (0) |