* [BUG?][NOMMU] microblaze: scheduling with interrupts disabled
@ 2010-03-05 4:02 Steven J. Magnani
0 siblings, 0 replies; only message in thread
From: Steven J. Magnani @ 2010-03-05 4:02 UTC (permalink / raw)
To: microblaze-uclinux, monstr; +Cc: linux-kernel
I ran across the following code in kernel/entry-nommu.S:
work_pending:
andi r11, r19, _TIF_NEED_RESCHED
beqi r11, 1f
bralid r15, schedule
nop
1: andi r11, r19, _TIF_SIGPENDING
beqi r11, no_work_pending
addk r5, r1, r0
addik r7, r0, 1
bralid r15, do_signal
addk r6, r0, r0
bri no_work_pending
ENTRY(ret_to_user)
disable_irq
swi r4, r1, PT_R4 /* return val */
swi r3, r1, PT_R3 /* return val */
lwi r6, r31, TS_THREAD_INFO /* get thread info */
lwi r19, r6, TI_FLAGS /* get flags in thread info */
bnei r19, work_pending
no_work_pending:
disable_irq
If I am reading this correctly, it looks like schedule() and do_signal()
can get called with interrupts disabled. This looks like a bug to me. I
would propose to fix it by putting an enable_irq at work_pending.
Comments?
------------------------------------------------------------------------
Steven J. Magnani "I claim this network for MARS!
www.digidescorp.com Earthling, return my space modulator!"
#include <standard.disclaimer>
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2010-03-05 4:02 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-03-05 4:02 [BUG?][NOMMU] microblaze: scheduling with interrupts disabled Steven J. Magnani
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®