Re: [PATCH] sparseirq: Enable early irq_desc allocation.

[Paul Mundt]

On Thu, May 21, 2009 at 01:26:26PM -0700, Yinghai Lu wrote:
> can you check tip?
> we change _cpu to node already.
> 
> also only sh have after_bootmem now.
> arch/sh/mm/init.c:int after_bootmem = 0;
> arch/sh/mm/init.c:      after_bootmem = 1;
> arch/sh/mm/ioremap_64.c:        extern int after_bootmem;
> arch/sh/mm/ioremap_64.c:        if (after_bootmem) {
> include/linux/mm.h:extern int after_bootmem;
> 
> for x86 we have bootmem_state ...
> arch/x86/include/asm/page_types.h:enum bootmem_state {
> arch/x86/include/asm/page_types.h:extern enum bootmem_state bootmem_state;
> arch/x86/kernel/setup.c:        bootmem_state = DURING_BOOTMEM;
> arch/x86/mm/init.c:enum bootmem_state bootmem_state = BEFORE_BOOTMEM;
> arch/x86/mm/init.c:     if (bootmem_state == BEFORE_BOOTMEM)
> arch/x86/mm/init.c:     if (bootmem_state == BEFORE_BOOTMEM)
> arch/x86/mm/init.c:     if (bootmem_state == BEFORE_BOOTMEM && !start) {
> arch/x86/mm/init.c:     if (bootmem_state == BEFORE_BOOTMEM &&
> arch/x86/mm/init.c:     if (bootmem_state == BEFORE_BOOTMEM)
> 
> Andrew,
> do we need to move bootmem_state back to linux/mm.h?

No need, I just switched to slab_is_available() in the respin. Here is
v2, against -tip:

--

sparseirq: Enable early irq_desc allocation, v2.

Presently non-legacy IRQs have their irq_desc allocated with
kzalloc_node(). This assumes that all callers of irq_to_desc_node_alloc()
will be sufficiently late in the boot process that kmalloc is available.

While porting sparseirq support to sh this blew up immediately, as at the
time that we register the CPU's interrupt vector map only bootmem is
available. Check slab_is_available() to work out which path to use.

Cc: Yinghai Lu <yinghai@xxxxxxxxxx>
Cc: Ingo Molnar <mingo@xxxxxxx>
Signed-off-by: Paul Mundt <lethal@xxxxxxxxxxxx>

---

 V2 updated for -tip, and switched to slab_is_available().

 kernel/irq/handle.c |   19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/kernel/irq/handle.c b/kernel/irq/handle.c
index 33ff8f5..b5a47d6 100644
--- a/kernel/irq/handle.c
+++ b/kernel/irq/handle.c
@@ -18,8 +18,8 @@
 #include <linux/rculist.h>
 #include <linux/hash.h>
 #include <linux/bootmem.h>
+#include <linux/slab.h>
 #include <trace/events/irq.h>
-
 #include "internals.h"
 
 /*
@@ -81,11 +81,16 @@ static struct irq_desc irq_desc_init = {
 	.lock       = __SPIN_LOCK_UNLOCKED(irq_desc_init.lock),
 };
 
-void init_kstat_irqs(struct irq_desc *desc, int node, int nr)
+void __ref init_kstat_irqs(struct irq_desc *desc, int node, int nr)
 {
 	void *ptr;
 
-	ptr = kzalloc_node(nr * sizeof(*desc->kstat_irqs), GFP_ATOMIC, node);
+	if (slab_is_available())
+		ptr = kzalloc_node(nr * sizeof(*desc->kstat_irqs),
+				   GFP_ATOMIC, node);
+	else
+		ptr = alloc_bootmem_node(NODE_DATA(node),
+				nr * sizeof(*desc->kstat_irqs));
 
 	/*
 	 * don't overwite if can not get new one
@@ -186,7 +191,7 @@ struct irq_desc *irq_to_desc(unsigned int irq)
 	return NULL;
 }
 
-struct irq_desc *irq_to_desc_alloc_node(unsigned int irq, int node)
+struct irq_desc * __ref irq_to_desc_alloc_node(unsigned int irq, int node)
 {
 	struct irq_desc *desc;
 	unsigned long flags;
@@ -208,7 +213,11 @@ struct irq_desc *irq_to_desc_alloc_node(unsigned int irq, int node)
 	if (desc)
 		goto out_unlock;
 
-	desc = kzalloc_node(sizeof(*desc), GFP_ATOMIC, node);
+	if (slab_is_available())
+		desc = kzalloc_node(sizeof(*desc), GFP_ATOMIC, node);
+	else
+		desc = alloc_bootmem_node(NODE_DATA(node), sizeof(*desc));
+
 	printk(KERN_DEBUG "  alloc irq_desc for %d on node %d\n", irq, node);
 	if (!desc) {
 		printk(KERN_ERR "can not alloc irq_desc\n");
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/