'sleeping function called from invalid context' bug when mounting an IDE device

[Vitaly Wool <vwool@ru.mvista.com>]

Hi Ingo,

I've come across the following problem during the debugging of IDE driver for
Philips PNX0105 ARM9 platform in RT mode (CONFIG_PREEMPT_RT).
When I mount/unmount a device, the following error is printed out to a terminal:
 
> # mount /dev/discs/disc0/part1 /mnt/
> BUG: sleeping function called from invalid context IRQ 27(655) at
> kernel/rt.c:1449in_atomic():0 [00000000], irqs_disabled():128
 
And gdb shows the following:
 
> (gdb) b kernel/sched.c:5430
> Breakpoint 1 at 0xc0034880: file kernel/sched.c, line 5430.
> (gdb) c
> Continuing.
> 
> Breakpoint 1, __might_sleep (file=0x56 <Address 0x56 out of bounds>,
> line=1449)
>	 at kernel/sched.c:5434
> 5434			     in_atomic(), preempt_count(),
> irqs_disabled());
> (gdb) bt
> #0  __might_sleep (file=0x56 <Address 0x56 out of bounds>, line=1449)
>	 at kernel/sched.c:5434
> #1  0xc004f700 in __spin_lock (lock=0xc01ee10c, eip=3222413300) at
> kernel/rt.c:1449
> #2  0xc004f778 in _spin_lock_irqsave (spin=0x56) at kernel/rt.c:1474
> #3  0xc0121ff4 in ide_intr (irq=27, dev_id=0x0, regs=0x0) at
> drivers/ide/ide-io.c:1416
> #4  0xc00201c0 in __do_irq (irq=27, action=0xc03f8df4, regs=0x0)
>	 at arch/arm/kernel/irq.c:530
> #5  0xc001fec0 in do_hardirq (desc=0xc021f2bc) at
> arch/arm/kernel/irq.c:394
> #6  0xc0020008 in do_irqd (__desc=0xc021f2bc) at
> arch/arm/kernel/irq.c:447
> #7  0xc004c310 in kthread (_create=0xc02abe20) at kernel/kthread.c:95
> #8  0xc0039af4 in exit_notify (tsk=0x0) at kernel/exit.c:785
> Previous frame inner to this frame (corrupt stack?)
> 
So, the problem is in the generic IDE code, namely, in ide_intr() taking ide_lock.

The suggested patch is the following:

Index: linux/drivers/ide/ide-probe.c
===================================================================
--- linux.orig/drivers/ide/ide-probe.c	2005-05-11 19:08:12.000000000 +0000
+++ linux/drivers/ide/ide-probe.c	2005-05-12 16:19:29.000000000 +0000
@@ -1050,16 +1050,13 @@
 	 * Allocate the irq, if not already obtained for another hwif
 	 */
 	if (!match || match->irq != hwif->irq) {
-		int sa = SA_INTERRUPT;
+		int sa = 0;
 #if defined(__mc68000__) || defined(CONFIG_APUS)
 		sa = SA_SHIRQ;
 #endif /* __mc68000__ || CONFIG_APUS */
 
 		if (IDE_CHIPSET_IS_PCI(hwif->chipset)) {
 			sa = SA_SHIRQ;
-#ifndef CONFIG_IDEPCI_SHARE_IRQ
-			sa |= SA_INTERRUPT;
-#endif /* CONFIG_IDEPCI_SHARE_IRQ */
 		}
 
 		if (hwif->io_ports[IDE_CONTROL_OFFSET])

Can you please comment on it?

Thanks,
   Vitaly