Ingo Molnar wrote: > * Hiroshi Shimamoto wrote: > >> Yinghai Lu wrote: >>> Ingo Molnar wrote: >>>> hm, one of my testboxes is rather unhappy about your patch: >>> >>> please check >>> >>> if you agree, will update update_genapic to autodetect other functions or etc. >>> >>> YH >>> >>> --- >>> >>> [PATCH] x86: fix wakeup_cpu with numaq/es7000 v2 - fix >>> >>> Impact: fix wakeup_secondary_cpu with hotplug >>> >>> We can not put that in x86_quirks, because that is __initdata. >>> So try to move that to genapic, and add update_genapic in x86_quirks. >>> >>> later we even could use that stub to >>> 1. autodetect CONFIG_ES7000_CLUSTERED_APIC >>> 2. more correct inquire_remote_apic with apic_verbosity setting. >> Hi, >> >> I encountered boot failure on x86_32 box. >> I think setting wakeup_cpu is missing. >> >> -------- >> From: Hiroshi Shimamoto >> Subject: [PATCH] x86: genapic: fix missing wakeup_cpu setup >> >> Impact: fix boot failure >> >> There is no way to setup wakeup_cpu when boot parameter apic= is not passed. >> >> Signed-off-by: Hiroshi Shimamoto >> --- >> arch/x86/mach-generic/probe.c | 3 +++ >> 1 files changed, 3 insertions(+), 0 deletions(-) > > thanks - i've applied the wider fix from Yinghai - see it below. > > i'm wondering why my tests didnt trigger this boot crash. What makes > your system or .config special to trigger it? I attached the .config I'm using. And here is panic log at the boot. CPU: Trace cache: 12K uops, L1 D cache: 16K CPU: L2 cache: 2048K CPU: Physical Processor ID: 0 CPU: Processor Core ID: 0 Intel machine check architecture supported. Intel machine check reporting enabled on CPU#0. CPU0: Intel P4/Xeon Extended MCE MSRs (24) available CPU0: Thermal monitoring enabled using mwait in idle threads. Checking 'hlt' instruction... OK. ACPI: Core revision 20080926 ..TIMER: vector=0x30 apic1=0 pin1=2 apic2=-1 pin2=-1 CPU0: Intel(R) Xeon(TM) CPU 3.80GHz stepping 0a Booting processor 1 APIC 0x6 ip 0x6000 invalid opcode: 0000 [#1] PREEMPT SMP last sysfs file: Modules linked in: Pid: 1, comm: swapper Not tainted (2.6.28-rc5-tip-00857-g5c47ccc #250) Express5800/120Rf-1 [N8100-1099] EIP: 0060:[<00000003>] EFLAGS: 00010282 CPU: 0 EIP is at 0x3 EAX: 00000006 EBX: 00006000 ECX: c045d8c0 EDX: 00006000 ESI: 00000001 EDI: f7423f8c EBP: f7423fa8 ESP: f7423f50 DS: 007b ES: 007b FS: 00d8 GS: 0000 SS: 0068 Process swapper (pid: 1, ti=f7422000 task=f7430000 task.ti=f7422000) Stack: 00006000 c03482a6 00000000 f7423f64 00000006 00000000 f7423f68 f7423f68 c0347d10 f7436080 00000001 00000101 f7423f80 f7423f80 00000001 00000000 00000000 f7423f80 f7423f80 00000001 00000000 f7423fb0 f7423fc0 c0349309 Call Trace: [] ? native_cpu_up+0x26e/0x3b3 [] ? do_fork_idle+0x0/0x1e [] ? _cpu_up+0x88/0xea [] ? cpu_up+0x44/0x54 [] ? kernel_init+0x94/0x141 [] ? kernel_init+0x0/0x141 [] ? kernel_thread_helper+0x7/0x10 Code: Bad EIP value. EIP: [<00000003>] 0x3 SS:ESP 0068:f7423f50 ---[ end trace 4eaa2a86a8e2da22 ]--- Kernel panic - not syncing: Attempted to kill init! thanks, Hiroshi Shimamoto