Messages in this thread | | | Date | Wed, 13 Dec 2017 11:31:35 -0500 | From | Steven Rostedt <> | Subject | Re: [tip:x86/urgent] x86/mm/kmmio: Fix mmiotrace for page unaligned addresses |
| |
On Tue, 12 Dec 2017 15:51:05 +0100 Peter Zijlstra <peterz@infradead.org> wrote:
> On Tue, Dec 12, 2017 at 09:47:05AM -0500, Ilia Mirkin wrote: > > > So this 'thing' has huge gaping SMP holes in. > > > > Sure does! Probably why the following happens when mmiotrace is enabled: > > > > void enable_mmiotrace(void) > > { > > mutex_lock(&mmiotrace_mutex); > > if (is_enabled()) > > goto out; > > > > if (nommiotrace) > > pr_info("MMIO tracing disabled.\n"); > > kmmio_init(); > > enter_uniprocessor(); > > ^^^^^ > > Ah! I completely missed that. OK, that makes it much less broken :-)
/me just saw this thread.
Once I saw your initial complaint, my first thought was "I think he doesn't know that it causes the system to turn into a uniprocessor first".
My ftrace tests have a test that switches to each tracer, and this mmiotrace catches the most bugs after an rc1 release. The bugs are triggered by switching to and from uniprocessor mode. Hopefully, with the new hotplug code, that will be a thing of the past.
> > If I don't forget, I'll add some comments to this file to clarify that.
Great, thanks!
-- Steve
>
| |