Re: [BUG] "sched: Remove rq->lock from the first half of ttwu()" locks up on ARM

Michal Simek
2011-05-31T14:08:55Z

Peter Zijlstra wrote:
> On Tue, 2011-05-31 at 15:37 +0200, Michal Simek wrote:
>
>> I briefly looked at it and it probably come from copy_thread function (process.c
>> - line: childregs->msr |= MSR_IE;)
>> When context switch happen, childregs->msr value is loaded to MSR (machine
>> status register) which caused that IE is enabled ( entry.S:~977 lwi r12, r11,
>> CC_MSR; mts rmsr, r12)
>>
>> NOTE: MSR stores flags for IE, i/d-cache ON/OFF, virtual memory/user mode etc.
>>
>> This is no problem if context switch is done with irq on. But maybe there is
>> another place which is causing some problems.
>
> Ahh, no wonder I didn't find that ;-)

:-)

>
>> Where exactly should be IRQ reenable after context switch?
>
> the tail end of finish_lock_switch(), where it does:
> raw_spin_unlock_irq(&rq->lock).

ok - I see.

>
>> I would like to also check some things.
>> 1. When schedule should be called from arch specific code?
>> Currently we are calling schedule after syscall/exception/interrupt happen.
>> Is there any place where schedule should/shouldn't be called?
>
> It should be called on the return to userspace path when
> TIF_NEED_RESCHED is set.

Yes, we do that. (PTO + PT_MODE stores if return is to kernel or user space)

It should not be called from non-preemptible
> contexts like non-zero preempt_count or IRQ-disabled.

Is this even when the return is to userspace?

PREEMPT is not well tested feature but maybe it is right time to do so.
There is only small part of code (ifdef CONFIG_PREEMPT) when irq happen and
there is return to the kernel. Is this correct?

>
> [ with the exception of CONFIG_PREEMPT which calls preempt_schedule()
> which checks both those things ]

This is called only when IRQ happen right? We call preempt_schedule_irq because
irq are off and IRQ is ON by rtid below IRQ_return label.

>
>> 2. For syscall and exception handling - interrupt is ON but it is only masked.
>
> I'm having trouble understanding: on but masked.

Interrupt can't happen because some masking bits are setup. If you call
irgs_disabled() or others you will get that IRQ is ON but can't happen.

>
>> When schedule is called from that any code has to enable IRQ if generic code
>> doesn't do that. Not sure if it does.
>
> generic code isn't supposed to call schedule() with IRQs disabled (and
> doesn't afaik)

OK. Which means I have to disable IRQ before schedule is called. Is that correct?

Michal

--
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian