[PATCH] arch/i386/kernel/irq.c: do_IRQ()

William Jhun (wjhun@ayrnetworks.com)
Mon, 20 May 2002 17:05:23 -0700


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);
  }
  
  /**

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/