Re: v2.6.28-rc7: Fail compile arch/x86/kernel/cpu/common.c

From: Jianjun Kong
Date: Wed Dec 03 2008 - 06:27:46 EST

On Wed, Dec 03, 2008 at 08:35:04AM +0100, Ingo Molnar wrote:
>* Jianjun Kong <jianjun@xxxxxxxxx> wrote:
>> arch/x86/kernel/cpu/common.c: In function âearly_identify_cpuâ:
>> arch/x86/kernel/cpu/common.c:555: error: âboot_cpu_idâ undeclared (first
>> use in this function)
>> arch/x86/kernel/cpu/common.c:555: error: (Each undeclared identifier is
>> reported only once
>> arch/x86/kernel/cpu/common.c:555: error: for each function it appears
>> in.)
>> arch/x86/kernel/cpu/common.c: In function âcpu_initâ:
>> arch/x86/kernel/cpu/common.c:1142: error: âboot_cpu_idâ undeclared
>> (first use in this function)
>> make[2]: *** [arch/x86/kernel/cpu/common.o] Error 1
>> make[1]: *** [arch/x86/kernel/cpu] Error 2
>> make: *** [arch/x86/kernel] Error 2
>i cannot reproduce this build error neither with v2.6.28-rc7 nor
>with latest -git (commit f6f7b52), using your config.
>could you send me your arch/x86/include/asm/smp.h file? Could
>you also double-check that you have the kernel source tree
>checked out properly? If you track it as a remote as per:

My kernel source tree checked out from git://, and there is no problem of check out.

This problem is also exist on the latest code of linux-2.6.git.

$ cat arch/x86/include/asm/smp.h

#ifndef _ASM_X86_SMP_H
#define _ASM_X86_SMP_H
#ifndef __ASSEMBLY__
#include <linux/cpumask.h>
#include <linux/init.h>
#include <asm/percpu.h>

* We need the APIC definitions automatically as part of 'smp.h'
# include <asm/mpspec.h>
# include <asm/apic.h>
# ifdef CONFIG_X86_IO_APIC
# include <asm/io_apic.h>
# endif
#include <asm/pda.h>
#include <asm/thread_info.h>

extern cpumask_t cpu_callout_map;
extern cpumask_t cpu_initialized;
extern cpumask_t cpu_callin_map;

extern void (*mtrr_hook)(void);
extern void zap_low_mappings(void);

extern int __cpuinit get_local_pda(int cpu);

extern int smp_num_siblings;
extern unsigned int num_processors;
extern cpumask_t cpu_initialized;

DECLARE_PER_CPU(cpumask_t, cpu_sibling_map);
DECLARE_PER_CPU(cpumask_t, cpu_core_map);
DECLARE_PER_CPU(u16, cpu_llc_id);
#ifdef CONFIG_X86_32
DECLARE_PER_CPU(int, cpu_number);

DECLARE_EARLY_PER_CPU(u16, x86_cpu_to_apicid);
DECLARE_EARLY_PER_CPU(u16, x86_bios_cpu_apicid);

/* Static state in head.S used to set up a CPU */
extern struct {
void *sp;
unsigned short ss;
} stack_start;

struct smp_ops {
void (*smp_prepare_boot_cpu)(void);
void (*smp_prepare_cpus)(unsigned max_cpus);
void (*smp_cpus_done)(unsigned max_cpus);

void (*smp_send_stop)(void);
void (*smp_send_reschedule)(int cpu);

int (*cpu_up)(unsigned cpu);
int (*cpu_disable)(void);
void (*cpu_die)(unsigned int cpu);
void (*play_dead)(void);

void (*send_call_func_ipi)(cpumask_t mask);
void (*send_call_func_single_ipi)(int cpu);

/* Globals due to paravirt */
extern void set_cpu_sibling_map(int cpu);

#define startup_ipi_hook(phys_apicid, start_eip, start_esp) do { } while (0)
extern struct smp_ops smp_ops;

static inline void smp_send_stop(void)

static inline void smp_prepare_boot_cpu(void)

static inline void smp_prepare_cpus(unsigned int max_cpus)

static inline void smp_cpus_done(unsigned int max_cpus)

static inline int __cpu_up(unsigned int cpu)
return smp_ops.cpu_up(cpu);

static inline int __cpu_disable(void)
return smp_ops.cpu_disable();

static inline void __cpu_die(unsigned int cpu)

static inline void play_dead(void)

static inline void smp_send_reschedule(int cpu)

static inline void arch_send_call_function_single_ipi(int cpu)

static inline void arch_send_call_function_ipi(cpumask_t mask)

void cpu_disable_common(void);
void native_smp_prepare_boot_cpu(void);
void native_smp_prepare_cpus(unsigned int max_cpus);
void native_smp_cpus_done(unsigned int max_cpus);
int native_cpu_up(unsigned int cpunum);
int native_cpu_disable(void);
void native_cpu_die(unsigned int cpu);
void native_play_dead(void);
void play_dead_common(void);

void native_send_call_func_ipi(cpumask_t mask);
void native_send_call_func_single_ipi(int cpu);

extern void prefill_possible_map(void);

void smp_store_cpu_info(int id);
#define cpu_physical_id(cpu) per_cpu(x86_cpu_to_apicid, cpu)

/* We don't mark CPUs online until __cpu_up(), so we need another measure */
static inline int num_booting_cpus(void)
return cpus_weight(cpu_callout_map);
static inline void prefill_possible_map(void)
#endif /* CONFIG_SMP */

extern unsigned disabled_cpus __cpuinitdata;

#ifdef CONFIG_X86_32_SMP
* This function is needed by all SMP systems. It must _always_ be valid
* from the initial startup. We map APIC_BASE very early in page_setup(),
* so this is correct in the x86 case.
#define raw_smp_processor_id() (x86_read_percpu(cpu_number))
extern int safe_smp_processor_id(void);

#elif defined(CONFIG_X86_64_SMP)
#define raw_smp_processor_id() read_pda(cpunumber)

#define stack_smp_processor_id() \
({ \
struct thread_info *ti; \
__asm__("andq %%rsp,%0; ":"=r" (ti) : "0" (CURRENT_MASK)); \
ti->cpu; \
#define safe_smp_processor_id() smp_processor_id()

#else /* !CONFIG_X86_32_SMP && !CONFIG_X86_64_SMP */
#define cpu_physical_id(cpu) boot_cpu_physical_apicid
#define safe_smp_processor_id() 0
#define stack_smp_processor_id() 0


#ifndef CONFIG_X86_64
static inline int logical_smp_processor_id(void)
/* we don't want to mark this access volatile - bad code generation */

#include <mach_apicdef.h>
static inline unsigned int read_apic_id(void)
unsigned int reg;

reg = *(u32 *)(APIC_BASE + APIC_ID);

return GET_APIC_ID(reg);

# if defined(APIC_DEFINITION) || defined(CONFIG_X86_64)
extern int hard_smp_processor_id(void);
# else
#include <mach_apicdef.h>
static inline int hard_smp_processor_id(void)
/* we don't want to mark this access volatile - bad code generation */
return read_apic_id();
# endif /* APIC_DEFINITION */

#else /* CONFIG_X86_LOCAL_APIC */

# ifndef CONFIG_SMP
# define hard_smp_processor_id() 0
# endif

#endif /* CONFIG_X86_LOCAL_APIC */

extern unsigned char boot_cpu_id;
#define boot_cpu_id 0

#endif /* __ASSEMBLY__ */
#endif /* _ASM_X86_SMP_H */
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at