Messages in this thread | | | Date | Thu, 25 Aug 2011 14:35:42 +0200 | From | Tejun Heo <> | Subject | Re: try_to_freeze() called with IRQs disabled on ARM |
| |
Hello,
On Thu, Aug 25, 2011 at 01:25:43PM +0100, Russell King - ARM Linux wrote: > No. Stop bodging and hiding problems. Anywhere which does this: > > local_irq_enable(); > do something > local_irq_disable(); > > is a bug. Things are called with IRQs disabled for a reason - randomly > re-enabling IRQs does not "fix" stuff, it merely introduces subtle bugs > while hiding warnings of those bugs. > > Please go back and read my response to Mark at the beginning of this > thread, where I describe why IRQs are disabled here. > > The only solution here is to fix the problem properly, and I'm working > on a patch to fix the problem I highlighted in my earlier response to > Mark. Once we have that problem fixed, we can then (more) safely call > do_signal() with IRQs enabled.
Arrrrrrrrgghhhhhhhhhhhh, why is communication so difficult with you? When did I ever suggest that as a proper fix. ARM is frigging broken in that path so don't push it over to PM and just deal with it inside ARM arch code. For now, we need to make the warning go away until it's properly fixed so turn off and on IRQ around get_signal_to_deliver() and mark it with giant FIXME comment. How many times did I write that? I don't know how to make that any clearer to you. Gees...
-- tejun
| |