RE: [PATCH] bug w/ shared interrupts

From: Kenji Kaneshige <kaneshige.kenji_at_jp.fujitsu.com>
Date: 2004-04-27 21:57:19
Hi,

I don't know whether you are referring to my patch. But if so,
please try a following patch. The patch for sharing vector I sent
before contains a bug and I think this might cause a similar problem.

Thanks,
Kenji Kaneshige


This is a patch to fix the following bug:

Register_intr() changes pin->low32 even when it overrides the
interrupt info. This destroys the value of pin->low32 which is already
set by set_rte() or other staffs.

This patch is against linux-2.6.6-rc2 with share_vector.patch applied.


---

 linux-2.6.6-rc2-kanesige/arch/ia64/kernel/iosapic.c |    1 -
 1 files changed, 1 deletion(-)

diff -puN arch/ia64/kernel/iosapic.c~share_vector_bugfix
arch/ia64/kernel/iosapic.c
--- linux-2.6.6-rc2/arch/ia64/kernel/iosapic.c~share_vector_bugfix
2004-04-27 18:12:56.000000000 +0900
+++ linux-2.6.6-rc2-kanesige/arch/ia64/kernel/iosapic.c	2004-04-27
18:12:56.000000000 +0900
@@ -605,7 +605,6 @@ register_intr (unsigned int gsi, int vec
 		pin->polarity = polarity;
 		pin->addr     = iosapic_address;
 		pin->gsi_base = gsi_base;
-		pin->low32    = IOSAPIC_MASK;
 		iosapic_intr_info[vector].dmode   = delivery;
 		iosapic_intr_info[vector].trigger = trigger;
 		iosapic_intr_info[vector].type    = type;

_


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

Received on Tue Apr 27 07:57:06 2004

This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:25 EST