On Tue, 5 Mar 2019 16:39:24 +0200 Yordan Karadzhov <ykaradzhov@xxxxxxxxxx> wrote: > The sched_events plugin should not be totally disabled when the > wakeup events are missing. The initialization of the plugin must > fail only if the trace.dat file contains no sched_switch events. > I added: Reported-by: Phil Auld <pauld@xxxxxxxxxx> -- Steve > Signed-off-by: Yordan Karadzhov <ykaradzhov@xxxxxxxxxx> > --- > kernel-shark/src/plugins/sched_events.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/kernel-shark/src/plugins/sched_events.c b/kernel-shark/src/plugins/sched_events.c > index fe13e6a..c52fb29 100644 > --- a/kernel-shark/src/plugins/sched_events.c > +++ b/kernel-shark/src/plugins/sched_events.c > @@ -104,9 +104,6 @@ static bool plugin_sched_init_context(struct kshark_context *kshark_ctx) > &plugin_ctx->sched_waking_event, > &plugin_ctx->sched_waking_pid_field); > > - if (!wakeup_found) > - return false; > - > plugin_ctx->second_pass_hash = tracecmd_filter_id_hash_alloc(); > > return true;
|