On Mon, 2 Oct 2017, Don Zickus wrote: > On Mon, Oct 02, 2017 at 07:32:57PM +0000, Thomas Gleixner wrote: > > static void softlockup_reconfigure_threads(void) > > There is a second copy of ^^^^, you will need to add identical locking there > too.
Good catch, but then we might move it further out > > + cpus_read_unlock(); > > + /* > > + * Must be called outside the cpus locked section to prevent > > + * recursive locking in the perf code. > > + */ > > + __lockup_detector_cleanup(); Or not due to this .... Thanks, tglx