On Fri, 26 May 2017, Steven Rostedt wrote:
> On Thu, 25 May 2017 10:57:51 +0200 (CEST)
> Thomas Gleixner <t...@linutronix.de> wrote:
>
> > ftrace use module_alloc() to allocate trampoline pages. The mapping of
> > module_alloc() is RWX, which makes sense as the memory is written to right
> > after allocation. But nothing makes these pages RO after writing to them.
> >
> > Add proper set_memory_rw/ro() calls to protect the trampolines after
> > modification.
> >
> > Signed-off-by: Thomas Gleixner <t...@linutronix.de>
> > ---
>
> Unfortunately, this was triggered in my tests:
>
> ftrace: allocating 54840 entries in 215 pages
> Starting tracer 'function'
> ------------[ cut here ]------------
> kernel BUG at
> /work/autotest/nobackup/linux-test.git/arch/x86/mm/pageattr.c:189!
>
> It appears that if you enable function tracing at boot up, calling
> set_memory_ro() with interrupts disabled can cause this. As pageattr.c
> at line 189 has:
>
> BUG_ON(irqs_disabled());
That's very early boot, right? So interrupts have to be disabled.
So this wants to be:
BUG_ON(irqs_disabled() && !early_boot_irqs_disabled);
Thanks,
tglx
