[RESEND PATCH] ARM: fix 'unannotated irqs-on' lockdep warning

Ming Lei tom.leiming at gmail.com
Sun May 23 11:07:50 EDT 2010


2010/5/23 Russell King - ARM Linux <linux at arm.linux.org.uk>:
> On Sun, May 23, 2010 at 09:44:20PM +0800, Ming Lei wrote:
>> 2010/5/23 Russell King - ARM Linux <linux at arm.linux.org.uk>:
>> >>  ENTRY(ret_to_user)
>> >>  ret_slow_syscall:
>> >> -     disable_irq                             @ disable interrupts
>> >> +     disable_irq_notrace                             @ disable interrupts
>> >
>> > I think this one does need to be traced - the pending work functions are
>> > all C code which could call back into lockdep.
>>
>> If there are pending works, schedule will be called to give cpu to it,
>> I wonder why the work function to be scheduled will be run with irq
>> disabled. Seems we should enable irq again before calling schedule,
>> not sure.
>
> No.  I'm talking about things like do_notify_resume().
>
> I think the above should be left as-is, so that as far as lockdep is
> concerned, IRQs are off while userspace runs.  What happens between
> returning to userspace and re-entering the kernel has no bearing what
> so ever on lockdep.
>

Oh, trace_ret_hardirqs_on has to be added before returning to user-space to
remove the warning, like x86 and mips. If you agree, I'd like to post
a new version
patch.

Thanks,

-- 
Lei Ming



More information about the linux-arm-kernel mailing list