On 7/20/22 17:21, Steven Rostedt wrote: >> +static void disable_all_monitors(void) >> +{ >> + struct rv_monitor_def *mdef; >> + > I think you need: > > mutex_lock(&rv_interface_lock); > >> + list_for_each_entry(mdef, &rv_monitors_list, list) >> + disable_monitor(mdef); > mutex_unlock(&rv_interface_lock); > >> +} yes, we need. -- Daniel