Skip to content

Commit

Permalink
[POWERPC] Fix section mismatch in PCI code
Browse files Browse the repository at this point in the history
Create a helper function (alloc_maybe_bootmem) that is marked __init_refok
to limit the chances of mistakenly referring to other __init routines.

WARNING: vmlinux.o(.text+0x2a9c4): Section mismatch: reference to .init.text:.__alloc_bootmem (between '.update_dn_pci_info' and '.pci_dn_reconfig_notifier')
WARNING: vmlinux.o(.text+0x36430): Section mismatch: reference to .init.text:.__alloc_bootmem (between '.mpic_msi_init_allocator' and '.find_ht_magic_addr')
WARNING: vmlinux.o(.text+0x5e804): Section mismatch: reference to .init.text:.__alloc_bootmem (between '.celleb_setup_phb' and '.celleb_fake_pci_write_config')
WARNING: vmlinux.o(.text+0x5e8e8): Section mismatch: reference to .init.text:.__alloc_bootmem (between '.celleb_setup_phb' and '.celleb_fake_pci_write_config')
WARNING: vmlinux.o(.text+0x5e968): Section mismatch: reference to .init.text:.__alloc_bootmem (between '.celleb_setup_phb' and '.celleb_fake_pci_write_config')

Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Paul Mackerras <paulus@samba.org>
  • Loading branch information
Stephen Rothwell authored and Paul Mackerras committed Sep 19, 2007
1 parent ee98307 commit 7b2c3c5
Show file tree
Hide file tree
Showing 6 changed files with 24 additions and 26 deletions.
7 changes: 1 addition & 6 deletions arch/powerpc/kernel/pci_dn.c
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,6 @@
#include <linux/pci.h>
#include <linux/string.h>
#include <linux/init.h>
#include <linux/slab.h>
#include <linux/bootmem.h>

#include <asm/io.h>
#include <asm/prom.h>
Expand All @@ -45,10 +43,7 @@ static void * __devinit update_dn_pci_info(struct device_node *dn, void *data)
const u32 *regs;
struct pci_dn *pdn;

if (mem_init_done)
pdn = kmalloc(sizeof(*pdn), GFP_KERNEL);
else
pdn = alloc_bootmem(sizeof(*pdn));
pdn = alloc_maybe_bootmem(sizeof(*pdn), GFP_KERNEL);
if (pdn == NULL)
return NULL;
memset(pdn, 0, sizeof(*pdn));
Expand Down
2 changes: 1 addition & 1 deletion arch/powerpc/lib/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ EXTRA_CFLAGS += -mno-minimal-toc
endif

ifeq ($(CONFIG_PPC_MERGE),y)
obj-y := string.o
obj-y := string.o alloc.o
obj-$(CONFIG_PPC32) += div64.o copy_32.o checksum_32.o
endif

Expand Down
14 changes: 14 additions & 0 deletions arch/powerpc/lib/alloc.c
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
#include <linux/types.h>
#include <linux/init.h>
#include <linux/slab.h>
#include <linux/bootmem.h>

#include <asm/system.h>

void * __init_refok alloc_maybe_bootmem(size_t size, gfp_t mask)
{
if (mem_init_done)
return kmalloc(size, mask);
else
return alloc_bootmem(size);
}
19 changes: 5 additions & 14 deletions arch/powerpc/platforms/celleb/pci.c
Original file line number Diff line number Diff line change
Expand Up @@ -327,10 +327,7 @@ static int __init celleb_setup_fake_pci_device(struct device_node *node,

size = 256;
config = &private->fake_config[devno][fn];
if (mem_init_done)
*config = kzalloc(size, GFP_KERNEL);
else
*config = alloc_bootmem(size);
*config = alloc_maybe_bootmem(size, GFP_KERNEL);
if (*config == NULL) {
printk(KERN_ERR "PCI: "
"not enough memory for fake configuration space\n");
Expand All @@ -341,10 +338,7 @@ static int __init celleb_setup_fake_pci_device(struct device_node *node,

size = sizeof(struct celleb_pci_resource);
res = &private->res[devno][fn];
if (mem_init_done)
*res = kzalloc(size, GFP_KERNEL);
else
*res = alloc_bootmem(size);
*res = alloc_maybe_bootmem(size, GFP_KERNEL);
if (*res == NULL) {
printk(KERN_ERR
"PCI: not enough memory for resource data space\n");
Expand Down Expand Up @@ -436,12 +430,9 @@ static int __init phb_set_bus_ranges(struct device_node *dev,

static void __init celleb_alloc_private_mem(struct pci_controller *hose)
{
if (mem_init_done)
hose->private_data =
kzalloc(sizeof(struct celleb_pci_private), GFP_KERNEL);
else
hose->private_data =
alloc_bootmem(sizeof(struct celleb_pci_private));
hose->private_data =
alloc_maybe_bootmem(sizeof(struct celleb_pci_private),
GFP_KERNEL);
}

int __init celleb_setup_phb(struct pci_controller *phb)
Expand Down
6 changes: 1 addition & 5 deletions arch/powerpc/sysdev/mpic_msi.c
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
*/

#include <linux/irq.h>
#include <linux/bootmem.h>
#include <linux/bitmap.h>
#include <linux/msi.h>
#include <asm/mpic.h>
Expand Down Expand Up @@ -152,10 +151,7 @@ int mpic_msi_init_allocator(struct mpic *mpic)
size = BITS_TO_LONGS(mpic->irq_count) * sizeof(long);
pr_debug("mpic: allocator bitmap size is 0x%x bytes\n", size);

if (mem_init_done)
mpic->hwirq_bitmap = kmalloc(size, GFP_KERNEL);
else
mpic->hwirq_bitmap = alloc_bootmem(size);
mpic->hwirq_bitmap = alloc_maybe_bootmem(size, GFP_KERNEL);

if (!mpic->hwirq_bitmap) {
pr_debug("mpic: ENOMEM allocating allocator bitmap!\n");
Expand Down
2 changes: 2 additions & 0 deletions include/asm-powerpc/system.h
Original file line number Diff line number Diff line change
Expand Up @@ -189,6 +189,8 @@ extern int mem_init_done; /* set on boot once kmalloc can be called */
extern unsigned long memory_limit;
extern unsigned long klimit;

extern void *alloc_maybe_bootmem(size_t size, gfp_t mask);

extern int powersave_nap; /* set if nap mode can be used in idle loop */

/*
Expand Down

0 comments on commit 7b2c3c5

Please sign in to comment.