public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* local_irq_enable() in __do_softirq()?
@ 2005-07-30 10:32 Peter Osterlund
  0 siblings, 0 replies; only message in thread
From: Peter Osterlund @ 2005-07-30 10:32 UTC (permalink / raw)
  To: Andi Kleen; +Cc: linux-kernel

Hi,

The change "x86_64: Switch to the interrupt stack when running a
softirq in local_bh ..." (ed6b676ca8b50e0b538e61c283d52fd04f007abf)
contains this:

--- a/kernel/softirq.c
+++ b/kernel/softirq.c
@@ -86,7 +86,7 @@ restart:
 	/* Reset the pending bitmask before enabling irqs */
 	local_softirq_pending() = 0;
 
-	local_irq_enable();
+	//local_irq_enable();
 
 	h = softirq_vec;
 
@@ -99,7 +99,7 @@ restart:
 		pending >>= 1;
 	} while (pending);
 
-	local_irq_disable();
+	//local_irq_disable();
 
 	pending = local_softirq_pending();
 	if (pending && --max_restart)

Is that intentional? If so, shouldn't the code be removed instead of
commented out?

-- 
Peter Osterlund - petero2@telia.com
http://web.telia.com/~u89404340

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2005-07-30 10:34 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-07-30 10:32 local_irq_enable() in __do_softirq()? Peter Osterlund

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox