Should activate_page()/__set_page_dirty_buffers() use _irqsave locking?

[Richard Purdie <rpurdie@rpsys.net>]

I've been experimenting with oprofile on an arm system without a PMU.
Whenever I enable callgraphing I see a BUG from run_posix_cpu_timers()
due to irqs being enabled when they should be disabled.

Tracing this back shows interrupts are enabled after the arm backtrace
code completes. Further tracing reveals its the call to
check_user_page_readable() (within an interrupt) that is causing the
problem.

check_user_page_readable() can potentially result in calls to
activate_page() (mm/swap.c) and __set_page_dirty_buffers()
(fs/buffer.c). Both functions use *_lock_irq()/*_unlock_irq rather than
the *_lock_irqsave/*_unlock_irqrestore counterparts.

Switching them to use the save/restore locks makes everything work. Is
there a reason for not using these here? Would such a patch be accepted?

Both the arm and i386 backtrace code would seem to be vulnerable to this
problem.

Richard