sched/tracing: Remove the redundant 'success' in the sched tracepoint
authorEd Tsai <ed.tsai@mediatek.com>
Thu, 22 Apr 2021 12:22:26 +0000 (20:22 +0800)
committerSteven Rostedt (VMware) <rostedt@goodmis.org>
Thu, 10 Jun 2021 15:16:20 +0000 (11:16 -0400)
'success' is left here for a long time and also it is meaningless
for the upper user. Just remove it.

[ There were some tools expecting this, and this may break them. But
  hopefully they've been fixed in the mean time. Otherwise this may be
  likely reverted - SDR ]

Link: https://lkml.kernel.org/r/20210422122226.9415-1-ed.tsai@mediatek.com
Cc: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Ed Tsai <ed.tsai@mediatek.com>
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
include/trace/events/sched.h

index 1eca230..9464048 100644 (file)
@@ -148,7 +148,6 @@ DECLARE_EVENT_CLASS(sched_wakeup_template,
                __array(        char,   comm,   TASK_COMM_LEN   )
                __field(        pid_t,  pid                     )
                __field(        int,    prio                    )
-               __field(        int,    success                 )
                __field(        int,    target_cpu              )
        ),
 
@@ -156,7 +155,6 @@ DECLARE_EVENT_CLASS(sched_wakeup_template,
                memcpy(__entry->comm, p->comm, TASK_COMM_LEN);
                __entry->pid            = p->pid;
                __entry->prio           = p->prio; /* XXX SCHED_DEADLINE */
-               __entry->success        = 1; /* rudiment, kill when possible */
                __entry->target_cpu     = task_cpu(p);
        ),