| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <1209409750.3367.41.camel@localhost.localdomain> Date: Mon, 28 Apr 2008 15:09:10 -0400 From: James Bottomley <James.Bottomley@...senPartnership.com> To: Ingo Molnar <mingo@...e.hu> Cc: Adrian Bunk <bunk@...nel.org>, Alexey Starikovskiy <astarikovskiy@...e.de>, tglx@...utronix.de, hpa@...or.com, linux-kernel@...r.kernel.org Subject: Re: Voyager phys_cpu_present_map compile error On Mon, 2008-04-28 at 19:59 +0200, Ingo Molnar wrote: > * James Bottomley <James.Bottomley@...senPartnership.com> wrote: > > > Oh ... oops ... unfortunately one I wouldn't spot in a voyager build. > > > > This should be the corrected patch; thanks. > > randconfig testing on x86.git found the build breakage below, i bisected > it down to your patch. The config is: > > http://redhat.com/~mingo/misc/config-Mon_Apr_28_19_39_30_CEST_2008.bad > > reverted the patch for now. > > Ingo > > ------------> > arch/x86/kernel/built-in.o: In function `physflat_cpu_mask_to_apicid': > genapic_flat_64.c:(.text+0x14373): undefined reference to `per_cpu__x86_cpu_to_apicid' > arch/x86/kernel/built-in.o: In function `physflat_send_IPI_mask': > genapic_flat_64.c:(.text+0x1447f): undefined reference to `per_cpu__x86_cpu_to_apicid' > arch/x86/kernel/built-in.o: In function `flat_apic_id_registered': > genapic_flat_64.c:(.text+0x14558): undefined reference to `phys_cpu_present_map' > genapic_flat_64.c:(.text+0x1455f): undefined reference to `phys_cpu_present_map' > arch/x86/kernel/built-in.o: In function `uv_cpu_mask_to_apicid': > genx2apic_uv_x.c:(.text+0x14627): undefined reference to `per_cpu__x86_cpu_to_apicid' > arch/x86/kernel/built-in.o: In function `uv_send_IPI_mask': > genx2apic_uv_x.c:(.text+0x14658): undefined reference to `per_cpu__x86_cpu_to_apicid' > arch/x86/kernel/built-in.o: In function `acpi_register_lapic_address': > boot.c:(.init.text+0x3b1b): undefined reference to `boot_cpu_physical_apicid' > boot.c:(.init.text+0x3b2c): undefined reference to `boot_cpu_physical_apicid' > arch/x86/kernel/built-in.o: In function `__get_smp_config': > mpparse.c:(.init.text+0x53ad): undefined reference to `num_processors' > mpparse.c:(.init.text+0x53c4): undefined reference to `num_processors' > mpparse.c:(.init.text+0x543c): undefined reference to `num_processors' > arch/x86/kernel/built-in.o: In function `init_apic_mappings': > : undefined reference to `boot_cpu_physical_apicid' > arch/x86/kernel/built-in.o: In function `init_apic_mappings': > : undefined reference to `boot_cpu_physical_apicid' > arch/x86/kernel/built-in.o: In function `APIC_init_uniprocessor': > : undefined reference to `boot_cpu_physical_apicid' > arch/x86/kernel/built-in.o: In function `APIC_init_uniprocessor': > : undefined reference to `phys_cpu_present_map' > arch/x86/kernel/built-in.o: In function `APIC_init_uniprocessor': > : undefined reference to `boot_cpu_physical_apicid' > arch/x86/kernel/built-in.o: In function `early_init_lapic_mapping': > : undefined reference to `boot_cpu_physical_apicid' > arch/x86/kernel/built-in.o: In function `acpi_register_lapic': > boot.c:(.cpuinit.text+0x177a): undefined reference to `disabled_cpus' > arch/x86/kernel/built-in.o: In function `MP_processor_info': > mpparse.c:(.cpuinit.text+0x179d): undefined reference to `disabled_cpus' > mpparse.c:(.cpuinit.text+0x17b4): undefined reference to `boot_cpu_physical_apicid' > arch/x86/kernel/built-in.o: In function `setup_secondary_APIC_clock': > : undefined reference to `boot_cpu_physical_apicid' > arch/x86/kernel/built-in.o: In function `generic_processor_info': > : undefined reference to `num_processors' > arch/x86/kernel/built-in.o: In function `generic_processor_info': > : undefined reference to `num_processors' > arch/x86/kernel/built-in.o: In function `generic_processor_info': > : undefined reference to `phys_cpu_present_map' > arch/x86/kernel/built-in.o: In function `generic_processor_info': > : undefined reference to `per_cpu__x86_cpu_to_apicid' > arch/x86/kernel/built-in.o: In function `uv_cpu_init': > : undefined reference to `per_cpu__x86_cpu_to_apicid' > arch/x86/kernel/built-in.o: In function `uv_cpu_init': > : undefined reference to `per_cpu__x86_cpu_to_apicid' > arch/x86/kernel/built-in.o: In function `uv_cpu_init': > : undefined reference to `per_cpu__x86_cpu_to_apicid' Hmm, that's nasty. What it's showing is that the non-SMP local APIC configuration pulls in large numbers of SMP variables. This was all working right a while ago ... as in you need mpparse and the apic files but not the SMP ones or the SMP variables. The quickest fix is probably this one, since in these days of multi-core I suspect optimising the non-SMP but use APIC case for size has a lot less relevance. James --- diff --git a/arch/x86/kernel/setup.c b/arch/x86/kernel/setup.c index c0c68c1..808daf1 100644 --- a/arch/x86/kernel/setup.c +++ b/arch/x86/kernel/setup.c @@ -12,6 +12,7 @@ #include <asm/mpspec.h> #include <asm/apicdef.h> +#ifdef CONFIG_X86_MPPARSE unsigned int num_processors; unsigned disabled_cpus __cpuinitdata; /* Processor that is doing the boot up */ @@ -23,8 +24,9 @@ EXPORT_PER_CPU_SYMBOL(x86_cpu_to_apicid); /* Bitmask of physically existing CPUs */ physid_mask_t phys_cpu_present_map; +#endif -#if defined(CONFIG_HAVE_SETUP_PER_CPU_AREA) && defined(CONFIG_SMP) +#if defined(CONFIG_HAVE_SETUP_PER_CPU_AREA) && defined(CONFIG_X86_SMP) /* * Copy data used in early init routines from the initial arrays to the * per cpu data areas. These arrays then become expendable and the -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists