[kernel] r13588 - in dists/sid/linux-2.6/debian: . patches/bugfix/mips patches/series

[Martin Michlmayr]

Author: tbm
Date: Fri May  8 07:38:10 2009
New Revision: 13588

Log:
Broadcom SB: fix locking in set_irq_affinity.

Added:
   dists/sid/linux-2.6/debian/patches/bugfix/mips/sb-irq.patch
Modified:
   dists/sid/linux-2.6/debian/changelog
   dists/sid/linux-2.6/debian/patches/series/5

Modified: dists/sid/linux-2.6/debian/changelog
==============================================================================
--- dists/sid/linux-2.6/debian/changelog	Thu May  7 14:16:23 2009	(r13587)
+++ dists/sid/linux-2.6/debian/changelog	Fri May  8 07:38:10 2009	(r13588)
@@ -6,6 +6,9 @@
   [ maximilian attems ]
   * qla1280: Fix off-by-some error in firmware loading. (closes: #527265)
 
+  [ Martin Michlmayr ]
+  * Broadcom SB: fix locking in set_irq_affinity.
+
  -- dann frazier <dannf at debian.org>  Mon, 04 May 2009 13:40:51 -0600
 
 linux-2.6 (2.6.29-4) unstable; urgency=low

Added: dists/sid/linux-2.6/debian/patches/bugfix/mips/sb-irq.patch
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ dists/sid/linux-2.6/debian/patches/bugfix/mips/sb-irq.patch	Fri May  8 07:38:10 2009	(r13588)
@@ -0,0 +1,74 @@
+Locking of irq_desc is now done in irq_set_affinity; Don't lock it
+again in chip specific set_affinity function.
+
+Signed-off-by: Thomas Bogendoerfer <tsbogend at alpha.franken.de>
+---
+
+ arch/mips/sibyte/bcm1480/irq.c |    7 ++-----
+ arch/mips/sibyte/sb1250/irq.c  |    7 ++-----
+ 2 files changed, 4 insertions(+), 10 deletions(-)
+
+diff --git a/arch/mips/sibyte/bcm1480/irq.c b/arch/mips/sibyte/bcm1480/irq.c
+index 352352b..c147c4b 100644
+--- a/arch/mips/sibyte/bcm1480/irq.c
++++ b/arch/mips/sibyte/bcm1480/irq.c
+@@ -113,7 +113,6 @@ static void bcm1480_set_affinity(unsigned int irq, const struct cpumask *mask)
+ {
+ 	int i = 0, old_cpu, cpu, int_on, k;
+ 	u64 cur_ints;
+-	struct irq_desc *desc = irq_desc + irq;
+ 	unsigned long flags;
+ 	unsigned int irq_dirty;
+ 
+@@ -127,8 +126,7 @@ static void bcm1480_set_affinity(unsigned int irq, const struct cpumask *mask)
+ 	cpu = cpu_logical_map(i);
+ 
+ 	/* Protect against other affinity changers and IMR manipulation */
+-	spin_lock_irqsave(&desc->lock, flags);
+-	spin_lock(&bcm1480_imr_lock);
++	spin_lock_irqsave(&bcm1480_imr_lock, flags);
+ 
+ 	/* Swizzle each CPU's IMR (but leave the IP selection alone) */
+ 	old_cpu = bcm1480_irq_owner[irq];
+@@ -153,8 +151,7 @@ static void bcm1480_set_affinity(unsigned int irq, const struct cpumask *mask)
+ 			____raw_writeq(cur_ints, IOADDR(A_BCM1480_IMR_MAPPER(cpu) + R_BCM1480_IMR_INTERRUPT_MASK_H + (k*BCM1480_IMR_HL_SPACING)));
+ 		}
+ 	}
+-	spin_unlock(&bcm1480_imr_lock);
+-	spin_unlock_irqrestore(&desc->lock, flags);
++	spin_unlock_irqrestore(&bcm1480_imr_lock, flags);
+ }
+ #endif
+ 
+diff --git a/arch/mips/sibyte/sb1250/irq.c b/arch/mips/sibyte/sb1250/irq.c
+index c08ff58..38cb998 100644
+--- a/arch/mips/sibyte/sb1250/irq.c
++++ b/arch/mips/sibyte/sb1250/irq.c
+@@ -107,7 +107,6 @@ static void sb1250_set_affinity(unsigned int irq, const struct cpumask *mask)
+ {
+ 	int i = 0, old_cpu, cpu, int_on;
+ 	u64 cur_ints;
+-	struct irq_desc *desc = irq_desc + irq;
+ 	unsigned long flags;
+ 
+ 	i = cpumask_first(mask);
+@@ -121,8 +120,7 @@ static void sb1250_set_affinity(unsigned int irq, const struct cpumask *mask)
+ 	cpu = cpu_logical_map(i);
+ 
+ 	/* Protect against other affinity changers and IMR manipulation */
+-	spin_lock_irqsave(&desc->lock, flags);
+-	spin_lock(&sb1250_imr_lock);
++	spin_lock_irqsave(&sb1250_imr_lock, flags);
+ 
+ 	/* Swizzle each CPU's IMR (but leave the IP selection alone) */
+ 	old_cpu = sb1250_irq_owner[irq];
+@@ -144,8 +142,7 @@ static void sb1250_set_affinity(unsigned int irq, const struct cpumask *mask)
+ 		____raw_writeq(cur_ints, IOADDR(A_IMR_MAPPER(cpu) +
+ 					R_IMR_INTERRUPT_MASK));
+ 	}
+-	spin_unlock(&sb1250_imr_lock);
+-	spin_unlock_irqrestore(&desc->lock, flags);
++	spin_unlock_irqrestore(&sb1250_imr_lock, flags);
+ }
+ #endif
+ 

Modified: dists/sid/linux-2.6/debian/patches/series/5
==============================================================================
--- dists/sid/linux-2.6/debian/patches/series/5	Thu May  7 14:16:23 2009	(r13587)
+++ dists/sid/linux-2.6/debian/patches/series/5	Fri May  8 07:38:10 2009	(r13588)
@@ -1,3 +1,4 @@
 + features/ia64/rtc-efi.patch
 + features/ia64/rtc-inline-LEAP_YEAR.patch
 + bugfix/all/0001-qla1280-Fix-off-by-some-error-in-firmware-loading.patch
++ bugfix/mips/sb-irq.patch