| From daa647d418615cf69081d90c2cbb498246df1572 Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Wed, 5 Sep 2018 11:36:36 +0200 |
| Subject: sched/debug: Use symbolic names for task state constants |
| MIME-Version: 1.0 |
| Content-Type: text/plain; charset=UTF-8 |
| Content-Transfer-Encoding: 8bit |
| |
| From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> |
| |
| [ Upstream commit ff28915fd31ccafc0d38e6f84b66df280ed9e86a ] |
| |
| include/trace/events/sched.h includes <linux/sched.h> (via |
| <linux/sched/numa_balancing.h>) and so knows about the TASK_* constants |
| used to interpret .prev_state. So instead of duplicating the magic |
| numbers make use of the defined macros to ease understanding the |
| mapping from state bits to letters which isn't completely intuitive for |
| an outsider. |
| |
| Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> |
| Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org> |
| Cc: Linus Torvalds <torvalds@linux-foundation.org> |
| Cc: Peter Zijlstra <peterz@infradead.org> |
| Cc: Sebastian Andrzej Siewior <bigeasy@linutronix.de> |
| Cc: Thomas Gleixner <tglx@linutronix.de> |
| Cc: kernel@pengutronix.de |
| Link: http://lkml.kernel.org/r/20180905093636.24068-1-u.kleine-koenig@pengutronix.de |
| Signed-off-by: Ingo Molnar <mingo@kernel.org> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| include/trace/events/sched.h | 11 ++++++++--- |
| 1 file changed, 8 insertions(+), 3 deletions(-) |
| |
| diff --git a/include/trace/events/sched.h b/include/trace/events/sched.h |
| index 5e1a7578c9edd..9a4bdfadab077 100644 |
| --- a/include/trace/events/sched.h |
| +++ b/include/trace/events/sched.h |
| @@ -169,9 +169,14 @@ TRACE_EVENT(sched_switch, |
| |
| (__entry->prev_state & (TASK_REPORT_MAX - 1)) ? |
| __print_flags(__entry->prev_state & (TASK_REPORT_MAX - 1), "|", |
| - { 0x01, "S" }, { 0x02, "D" }, { 0x04, "T" }, |
| - { 0x08, "t" }, { 0x10, "X" }, { 0x20, "Z" }, |
| - { 0x40, "P" }, { 0x80, "I" }) : |
| + { TASK_INTERRUPTIBLE, "S" }, |
| + { TASK_UNINTERRUPTIBLE, "D" }, |
| + { __TASK_STOPPED, "T" }, |
| + { __TASK_TRACED, "t" }, |
| + { EXIT_DEAD, "X" }, |
| + { EXIT_ZOMBIE, "Z" }, |
| + { TASK_PARKED, "P" }, |
| + { TASK_DEAD, "I" }) : |
| "R", |
| |
| __entry->prev_state & TASK_REPORT_MAX ? "+" : "", |
| -- |
| 2.20.1 |
| |