Andrew and Tony,
This is a patch for IA64 code.
Please note that this patch depends on [patch 2/3]. So [patch 2/3]
needs to be applied before applying this patch.
Thanks,
Kenji Kaneshige
----
Name: IRQ_deallocation_ia64.patch
Kernel Version: 2.6.9-rc3
Depends: IRQ_deallocation_acpi.patch
Change Log:
- Ported to 2.6.9-rc3
- Changed pcibios_disable_device() implementation because now that
pcibios_disalbe_device() hook is defined by using __attribute__
((weak)) instead of modifying all arch specific code.
- Fixed a bug in iosapic_unregister_intr(). Missing
"spin_unlock(&iosapic_lock);" was added.
- Changed the argument of iosapic_unregister_intr() from Linux IRQ
number to GSI number.
Description:
This is an ia64 portion of IRQ resource deallocation. It implements
pcibios_disable_device() and acpi_unregister_gsi() for ia64.
o acpi_unregister_gsi()
Summary of changes for implementing this interface:
- Add new function iosapic_unregister_intr() into
arch/ia64/kernel/iosapic.c. This function frees an interrupt
vector and related data structures.
- Add new function free_irq_vector() into
arch/ia64/kernel/irq_ia64.c. This frees an unused vector.
- Change assign_irq_vector() to be able to support
free_irq_vector().
o pcibios_disable_device()
This calls acpi_pci_irq_disable() to deallocate IRQ resources.
Signed-off-by: Kenji Kaneshige <[email protected]>
---
linux-2.6.9-rc3-kanesige/arch/ia64/Kconfig | 9 +
linux-2.6.9-rc3-kanesige/arch/ia64/kernel/acpi.c | 9 +
linux-2.6.9-rc3-kanesige/arch/ia64/kernel/iosapic.c | 96 +++++++++++++++++--
linux-2.6.9-rc3-kanesige/arch/ia64/kernel/irq.c | 9 +
linux-2.6.9-rc3-kanesige/arch/ia64/kernel/irq_ia64.c | 27 ++++-
linux-2.6.9-rc3-kanesige/arch/ia64/pci/pci.c | 8 +
linux-2.6.9-rc3-kanesige/include/asm-ia64/acpi.h | 4
linux-2.6.9-rc3-kanesige/include/asm-ia64/hw_irq.h | 2
linux-2.6.9-rc3-kanesige/include/asm-ia64/iosapic.h | 4
9 files changed, 153 insertions(+), 15 deletions(-)
diff -puN arch/ia64/Kconfig~IRQ_deallocation_ia64 arch/ia64/Kconfig
--- linux-2.6.9-rc3/arch/ia64/Kconfig~IRQ_deallocation_ia64 2004-10-04 17:03:45.496807086 +0900
+++ linux-2.6.9-rc3-kanesige/arch/ia64/Kconfig 2004-10-04 17:03:45.512432173 +0900
@@ -299,6 +299,15 @@ config IA64_PALINFO
To use this option, you have to ensure that the "/proc file system
support" (CONFIG_PROC_FS) is enabled, too.
+config IA64_DEALLOCATE_IRQ
+ bool "IRQ resource deallocation support (EXPERIMENTAL)"
+ depends on IOSAPIC && EXPERIMENTAL
+ default n
+ help
+ Say Y here to experiment with deallocating IRQ resources
+ dynamically.
+ Say N if you want to disable IRQ resource deallocation.
+
source "drivers/firmware/Kconfig"
source "fs/Kconfig.binfmt"
diff -puN arch/ia64/kernel/acpi.c~IRQ_deallocation_ia64 arch/ia64/kernel/acpi.c
--- linux-2.6.9-rc3/arch/ia64/kernel/acpi.c~IRQ_deallocation_ia64 2004-10-04 17:03:45.498760221 +0900
+++ linux-2.6.9-rc3-kanesige/arch/ia64/kernel/acpi.c 2004-10-04 17:03:45.512432173 +0900
@@ -515,6 +515,15 @@ acpi_register_gsi (u32 gsi, int edge_lev
}
EXPORT_SYMBOL(acpi_register_gsi);
+#ifdef CONFIG_IA64_DEALLOCATE_IRQ
+void
+acpi_unregister_gsi (u32 gsi)
+{
+ iosapic_unregister_intr(gsi);
+}
+EXPORT_SYMBOL(acpi_unregister_gsi);
+#endif
+
static int __init
acpi_parse_fadt (unsigned long phys_addr, unsigned long size)
{
diff -puN arch/ia64/kernel/iosapic.c~IRQ_deallocation_ia64 arch/ia64/kernel/iosapic.c
--- linux-2.6.9-rc3/arch/ia64/kernel/iosapic.c~IRQ_deallocation_ia64 2004-10-04 17:03:45.500713357 +0900
+++ linux-2.6.9-rc3-kanesige/arch/ia64/kernel/iosapic.c 2004-10-04 17:03:45.513408740 +0900
@@ -111,6 +111,7 @@ static struct iosapic_intr_info {
unsigned char dmode : 3; /* delivery mode (see iosapic.h) */
unsigned char polarity: 1; /* interrupt polarity (see iosapic.h) */
unsigned char trigger : 1; /* trigger mode (see iosapic.h) */
+ int refcnt; /* reference counter */
} iosapic_intr_info[IA64_NUM_VECTORS];
static struct iosapic {
@@ -177,7 +178,7 @@ gsi_to_irq (unsigned int gsi)
static void
set_rte (unsigned int vector, unsigned int dest, int mask)
{
- unsigned long pol, trigger, dmode, flags;
+ unsigned long pol, trigger, dmode;
u32 low32, high32;
char *addr;
int rte_index;
@@ -218,13 +219,9 @@ set_rte (unsigned int vector, unsigned i
/* dest contains both id and eid */
high32 = (dest << IOSAPIC_DEST_SHIFT);
- spin_lock_irqsave(&iosapic_lock, flags);
- {
- iosapic_write(addr, IOSAPIC_RTE_HIGH(rte_index), high32);
- iosapic_write(addr, IOSAPIC_RTE_LOW(rte_index), low32);
- iosapic_intr_info[vector].low32 = low32;
- }
- spin_unlock_irqrestore(&iosapic_lock, flags);
+ iosapic_write(addr, IOSAPIC_RTE_HIGH(rte_index), high32);
+ iosapic_write(addr, IOSAPIC_RTE_LOW(rte_index), low32);
+ iosapic_intr_info[vector].low32 = low32;
}
static void
@@ -475,6 +472,7 @@ register_intr (unsigned int gsi, int vec
iosapic_intr_info[vector].addr = iosapic_address;
iosapic_intr_info[vector].gsi_base = gsi_base;
iosapic_intr_info[vector].trigger = trigger;
+ iosapic_intr_info[vector].refcnt++;
if (trigger == IOSAPIC_EDGE)
irq_type = &irq_type_iosapic_edge;
@@ -576,6 +574,7 @@ iosapic_register_intr (unsigned int gsi,
{
vector = gsi_to_vector(gsi);
if (vector > 0) {
+ iosapic_intr_info[vector].refcnt++;
spin_unlock_irqrestore(&iosapic_lock, flags);
return vector;
}
@@ -584,6 +583,8 @@ iosapic_register_intr (unsigned int gsi,
dest = get_target_cpu(gsi, vector);
register_intr(gsi, vector, IOSAPIC_LOWEST_PRIORITY,
polarity, trigger);
+
+ set_rte(vector, dest, 1);
}
spin_unlock_irqrestore(&iosapic_lock, flags);
@@ -592,10 +593,87 @@ iosapic_register_intr (unsigned int gsi,
(polarity == IOSAPIC_POL_HIGH ? "high" : "low"),
cpu_logical_id(dest), dest, vector);
- set_rte(vector, dest, 1);
return vector;
}
+#ifdef CONFIG_IA64_DEALLOCATE_IRQ
+void
+iosapic_unregister_intr (unsigned int gsi)
+{
+ unsigned long flags;
+ int irq, vector;
+ irq_desc_t *idesc;
+ int rte_index;
+ unsigned long trigger, polarity;
+
+ /*
+ * If the irq associated with the gsi is not found,
+ * iosapic_unregister_intr() is unbalanced. We need to check
+ * this again after getting locks.
+ */
+ irq = gsi_to_irq(gsi);
+ if (unlikely(irq < 0)) {
+ printk(KERN_ERR "iosapic_unregister_intr(%u) unbalanced\n", gsi);
+ WARN_ON(1);
+ return;
+ }
+ vector = irq_to_vector(irq);
+
+ idesc = irq_descp(irq);
+ spin_lock_irqsave(&idesc->lock, flags);
+ spin_lock(&iosapic_lock);
+ {
+ rte_index = iosapic_intr_info[vector].rte_index;
+ if (unlikely(rte_index < 0)) {
+ spin_unlock(&iosapic_lock);
+ spin_unlock_irqrestore(&idesc->lock, flags);
+ printk(KERN_ERR "iosapic_unregister_intr(%u) unbalanced\n", gsi);
+ WARN_ON(1);
+ return;
+ }
+
+ if (--iosapic_intr_info[vector].refcnt > 0) {
+ spin_unlock(&iosapic_lock);
+ spin_unlock_irqrestore(&idesc->lock, flags);
+ return;
+ }
+
+ /*
+ * If interrupt handlers still exist on the irq
+ * associated with the gsi, don't unregister the
+ * interrupt.
+ */
+ if (unlikely(idesc->action)) {
+ iosapic_intr_info[vector].refcnt++;
+ spin_unlock(&iosapic_lock);
+ spin_unlock_irqrestore(&idesc->lock, flags);
+ printk(KERN_WARNING "Cannot unregister GSI. IRQ %u is still in use.\n", irq);
+ return;
+ }
+
+ /* Clear the interrupt controller descriptor. */
+ idesc->handler = &no_irq_type;
+
+ trigger = iosapic_intr_info[vector].trigger;
+ polarity = iosapic_intr_info[vector].polarity;
+
+ /* Clear the interrupt information. */
+ memset(&iosapic_intr_info[vector], 0, sizeof(struct iosapic_intr_info));
+ iosapic_intr_info[vector].rte_index = -1; /* mark as unused */
+ }
+ spin_unlock(&iosapic_lock);
+ spin_unlock_irqrestore(&idesc->lock, flags);
+
+ /* Free the interrupt vector */
+ free_irq_vector(vector);
+
+ printk(KERN_INFO "GSI %u (%s, %s) -> vector %d unregisterd.\n",
+ gsi, (trigger == IOSAPIC_EDGE ? "edge" : "level"),
+ (polarity == IOSAPIC_POL_HIGH ? "high" : "low"),
+ vector);
+}
+#endif /* CONFIG_IA64_DEALLOCATE_IRQ */
+
/*
* ACPI calls this when it finds an entry for a platform interrupt.
* Note that the irq_base and IOSAPIC address must be set in iosapic_init().
diff -puN arch/ia64/kernel/irq.c~IRQ_deallocation_ia64 arch/ia64/kernel/irq.c
--- linux-2.6.9-rc3/arch/ia64/kernel/irq.c~IRQ_deallocation_ia64 2004-10-04 17:03:45.502666493 +0900
+++ linux-2.6.9-rc3-kanesige/arch/ia64/kernel/irq.c 2004-10-04 17:03:45.514385308 +0900
@@ -874,8 +874,11 @@ int setup_irq(unsigned int irq, struct i
struct irqaction *old, **p;
irq_desc_t *desc = irq_descp(irq);
- if (desc->handler == &no_irq_type)
+ spin_lock_irqsave(&desc->lock,flags);
+ if (desc->handler == &no_irq_type) {
+ spin_unlock_irqrestore(&desc->lock,flags);
return -ENOSYS;
+ }
/*
* Some drivers like serial.c use request_irq() heavily,
* so we have to be careful not to interfere with a
@@ -901,7 +904,6 @@ int setup_irq(unsigned int irq, struct i
/*
* The following block of code has to be executed atomically
*/
- spin_lock_irqsave(&desc->lock,flags);
p = &desc->action;
if ((old = *p) != NULL) {
/* Can't share interrupts unless both agree to */
@@ -1035,6 +1037,9 @@ void move_irq(int irq)
irq_desc_t *desc = irq_descp(irq);
int redir = test_bit(irq, pending_irq_redir);
+ if (unlikely(!desc->handler->set_affinity))
+ return;
+
if (!cpus_empty(pending_irq_cpumask[irq])) {
cpus_and(tmp, pending_irq_cpumask[irq], cpu_online_map);
if (unlikely(!cpus_empty(tmp))) {
diff -puN arch/ia64/kernel/irq_ia64.c~IRQ_deallocation_ia64 arch/ia64/kernel/irq_ia64.c
--- linux-2.6.9-rc3/arch/ia64/kernel/irq_ia64.c~IRQ_deallocation_ia64 2004-10-04 17:03:45.503643061 +0900
+++ linux-2.6.9-rc3-kanesige/arch/ia64/kernel/irq_ia64.c 2004-10-04 17:03:45.515361876 +0900
@@ -74,15 +74,34 @@ irq_exit (void)
preempt_enable_no_resched();
}
+static unsigned long ia64_vector_mask[BITS_TO_LONGS(IA64_NUM_DEVICE_VECTORS)];
+
int
assign_irq_vector (int irq)
{
- static int next_vector = IA64_FIRST_DEVICE_VECTOR;
-
- if (next_vector > IA64_LAST_DEVICE_VECTOR)
+ int pos, vector;
+ again:
+ pos = find_first_zero_bit(ia64_vector_mask, IA64_NUM_DEVICE_VECTORS);
+ vector = IA64_FIRST_DEVICE_VECTOR + pos;
+ if (vector > IA64_LAST_DEVICE_VECTOR)
/* XXX could look for sharable vectors instead of panic'ing... */
panic("assign_irq_vector: out of interrupt vectors!");
- return next_vector++;
+ if (test_and_set_bit(pos, ia64_vector_mask))
+ goto again;
+ return vector;
+}
+
+void
+free_irq_vector (int vector)
+{
+ int pos;
+
+ if (vector < IA64_FIRST_DEVICE_VECTOR || vector > IA64_LAST_DEVICE_VECTOR)
+ return;
+
+ pos = vector - IA64_FIRST_DEVICE_VECTOR;
+ if (!test_and_clear_bit(pos, ia64_vector_mask))
+ printk(KERN_WARNING "%s: double free!\n", __FUNCTION__);
}
extern unsigned int do_IRQ(unsigned long irq, struct pt_regs *regs);
diff -puN arch/ia64/pci/pci.c~IRQ_deallocation_ia64 arch/ia64/pci/pci.c
--- linux-2.6.9-rc3/arch/ia64/pci/pci.c~IRQ_deallocation_ia64 2004-10-04 17:03:45.505596197 +0900
+++ linux-2.6.9-rc3-kanesige/arch/ia64/pci/pci.c 2004-10-04 17:03:45.515361876 +0900
@@ -431,6 +431,14 @@ pcibios_enable_device (struct pci_dev *d
return acpi_pci_irq_enable(dev);
}
+#ifdef CONFIG_IA64_DEALLOCATE_IRQ
+void
+pcibios_disable_device (struct pci_dev *dev)
+{
+ acpi_pci_irq_disable(dev);
+}
+#endif /* CONFIG_IA64_DEALLOCATE_IRQ */
+
void
pcibios_align_resource (void *data, struct resource *res,
unsigned long size, unsigned long align)
diff -puN include/asm-ia64/acpi.h~IRQ_deallocation_ia64 include/asm-ia64/acpi.h
--- linux-2.6.9-rc3/include/asm-ia64/acpi.h~IRQ_deallocation_ia64 2004-10-04 17:03:45.506572765 +0900
+++ linux-2.6.9-rc3-kanesige/include/asm-ia64/acpi.h 2004-10-04 17:03:45.516338444 +0900
@@ -112,7 +112,11 @@ extern u16 ia64_acpiid_to_sapicid[];
* If this matches the last registration, any IRQ resources for gsi
* are freed.
*/
+#ifdef CONFIG_IA64_DEALLOCATE_IRQ
+void acpi_unregister_gsi (u32 gsi);
+#else
static inline void acpi_unregister_gsi (u32 gsi) { }
+#endif /* CONFIG_IA64_DEALLOCATE_IRQ */
#endif /*__KERNEL__*/
diff -puN include/asm-ia64/hw_irq.h~IRQ_deallocation_ia64 include/asm-ia64/hw_irq.h
--- linux-2.6.9-rc3/include/asm-ia64/hw_irq.h~IRQ_deallocation_ia64 2004-10-04 17:03:45.508525901 +0900
+++ linux-2.6.9-rc3-kanesige/include/asm-ia64/hw_irq.h 2004-10-04 17:03:45.515361876 +0900
@@ -50,6 +50,7 @@ typedef u8 ia64_vector;
*/
#define IA64_FIRST_DEVICE_VECTOR 0x30
#define IA64_LAST_DEVICE_VECTOR 0xe7
+#define IA64_NUM_DEVICE_VECTORS (IA64_LAST_DEVICE_VECTOR - IA64_FIRST_DEVICE_VECTOR + 1)
#define IA64_MCA_RENDEZ_VECTOR 0xe8 /* MCA rendez interrupt */
#define IA64_PERFMON_VECTOR 0xee /* performanc monitor interrupt vector */
@@ -83,6 +84,7 @@ extern unsigned long ipi_base_addr;
extern struct hw_interrupt_type irq_type_ia64_lsapic; /* CPU-internal interrupt controller */
extern int assign_irq_vector (int irq); /* allocate a free vector */
+extern void free_irq_vector (int vector);
extern void ia64_send_ipi (int cpu, int vector, int delivery_mode, int redirect);
extern void register_percpu_irq (ia64_vector vec, struct irqaction *action);
diff -puN include/asm-ia64/iosapic.h~IRQ_deallocation_ia64 include/asm-ia64/iosapic.h
--- linux-2.6.9-rc3/include/asm-ia64/iosapic.h~IRQ_deallocation_ia64 2004-10-04 17:03:45.510479037 +0900
+++ linux-2.6.9-rc3-kanesige/include/asm-ia64/iosapic.h 2004-10-04 17:03:45.516338444 +0900
@@ -78,6 +78,9 @@ extern int gsi_to_irq (unsigned int gsi)
extern void iosapic_enable_intr (unsigned int vector);
extern int iosapic_register_intr (unsigned int gsi, unsigned long polarity,
unsigned long trigger);
+#ifdef CONFIG_IA64_DEALLOCATE_IRQ
+extern void iosapic_unregister_intr (unsigned int irq);
+#endif
extern void __init iosapic_override_isa_irq (unsigned int isa_irq, unsigned int gsi,
unsigned long polarity,
unsigned long trigger);
@@ -97,6 +100,7 @@ extern void __init map_iosapic_to_node (
#define iosapic_system_init(pcat_compat) do { } while (0)
#define iosapic_init(address,gsi_base) do { } while (0)
#define iosapic_register_intr(gsi,polarity,trigger) (gsi)
+#define iosapic_unregister_intr(irq) do { } while (0)
#define iosapic_override_isa_irq(isa_irq,gsi,polarity,trigger) do { } while (0)
#define iosapic_register_platform_intr(type,gsi,pmi,eid,id, \
polarity,trigger) (gsi)
_