Re: [PATCH 2/2 v5] perf sched: adapt TASK_STATE_TO_CHAR_STR to the latest kernel

From: Masami Hiramatsu
Date: Tue Aug 02 2016 - 09:33:13 EST


On Tue, 2 Aug 2016 14:35:29 +0900
Tomoki Sekiyama <tomoki.sekiyama.qu@xxxxxxxxxxx> wrote:

> Update TASK_STATE_TO_CHAR_STR macro to one from sched.h in the latest
> kernel.
>
> Related kernel commits:
> - commit ad86622b478e ("wait: swap EXIT_ZOMBIE and EXIT_DEAD to hide
> EXIT_TRACE from user-space"):
> 'Z' and 'X' are swapped
>
> - commit 80ed87c8a9ca ("sched/wait: Introduce TASK_NOLOAD and TASK_IDLE"):
> Introduces new state 'N'
>
> - commit 7dc603c9028e ("sched/fair: Fix PELT integrity for new tasks"):
> Introduces new state 'n'
>

Ah, here is the newer one :)

Acked-by: Masami Hiramatsu <mhiramat@xxxxxxxxxx>

Thanks!

> Signed-off-by: Tomoki Sekiyama <tomoki.sekiyama.qu@xxxxxxxxxxx>
> Cc: Jiri Olsa <jolsa@xxxxxxxxxx>
> Cc: David Ahern <dsahern@xxxxxxxxx>
> Cc: Namhyung Kim <namhyung@xxxxxxxxxx>
> Cc: Peter Zijlstra <a.p.zijlstra@xxxxxxxxx>
> Cc: Masami Hiramatsu <mhiramat@xxxxxxxxxx>
> ---
> Changes from v4:
> Removed 'Fixes:' tag and added related commits in the commit message
>
> tools/perf/builtin-sched.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c
> index ce9bef6..5776263 100644
> --- a/tools/perf/builtin-sched.c
> +++ b/tools/perf/builtin-sched.c
> @@ -70,7 +70,7 @@ struct sched_atom {
> struct task_desc *wakee;
> };
>
> -#define TASK_STATE_TO_CHAR_STR "RSDTtZXxKWP"
> +#define TASK_STATE_TO_CHAR_STR "RSDTtXZxKWPNn"
>
> enum thread_state {
> THREAD_SLEEPING = 0,
> --
> 2.7.4
>


--
Masami Hiramatsu <mhiramat@xxxxxxxxxx>