arch/i386/kernel/irq.c: do_IRQ()

Though a comment in arch/i386/kernel/irq.c: do_IRQ() clearly states:

* 0 return value means that this irq is already being
* handled by some other CPU. (or is disabled)

it seems that the function can only ever return (1). We wrote some low-level interrupt handling code that depends on the correct value of this function. Is the following patch what was initially desired? (patched against 2.4.18 tarball from kernel.org...)

Thanks,
William Jhun

---
*** irq.c.orig Mon May 20 16:55:42 2002
--- irq.c Mon May 20 16:57:00 2002
***************
*** 639,645 ****

if (softirq_pending(cpu))
do_softirq();
! return 1;
}

/**
--- 639,645 ----

if (softirq_pending(cpu))
do_softirq();
! return (action != NULL);
}

/**