Skip to content

Commit

Permalink
x86, apic: clean up target_cpus methods
Browse files Browse the repository at this point in the history
Impact: cleanup

Clean up all the target_cpus() namespace overlap that exists
between bigsmp, es7000, mach-default, numaq and summit - by
separating the different functions into different names.

Signed-off-by: Ingo Molnar <mingo@elte.hu>
  • Loading branch information
Ingo Molnar committed Jan 28, 2009
1 parent 7fe7328 commit 0a9cc20
Show file tree
Hide file tree
Showing 10 changed files with 12 additions and 12 deletions.
2 changes: 1 addition & 1 deletion arch/x86/include/asm/bigsmp/apic.h
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ static inline int bigsmp_apic_id_registered(void)
return 1;
}

static inline const cpumask_t *target_cpus(void)
static inline const cpumask_t *bigsmp_target_cpus(void)
{
#ifdef CONFIG_SMP
return &cpu_online_map;
Expand Down
4 changes: 2 additions & 2 deletions arch/x86/include/asm/es7000/apic.h
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ static inline const cpumask_t *target_cpus_cluster(void)
return &CPU_MASK_ALL;
}

static inline const cpumask_t *target_cpus(void)
static inline const cpumask_t *es7000_target_cpus(void)
{
return &cpumask_of_cpu(smp_processor_id());
}
Expand Down Expand Up @@ -83,7 +83,7 @@ static inline void setup_apic_routing(void)
printk("Enabling APIC mode: %s. Using %d I/O APICs, target cpus %lx\n",
(apic_version[apic] == 0x14) ?
"Physical Cluster" : "Logical Cluster",
nr_ioapics, cpus_addr(*target_cpus())[0]);
nr_ioapics, cpus_addr(*es7000_target_cpus())[0]);
}

static inline int multi_timer_check(int apic, int irq)
Expand Down
4 changes: 2 additions & 2 deletions arch/x86/include/asm/mach-default/mach_apic.h
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@

#define APIC_DFR_VALUE (APIC_DFR_FLAT)

static inline const struct cpumask *target_cpus(void)
static inline const struct cpumask *default_target_cpus(void)
{
#ifdef CONFIG_SMP
return cpu_online_mask;
Expand All @@ -33,7 +33,7 @@ static inline const struct cpumask *target_cpus(void)
#define wakeup_secondary_cpu (apic->wakeup_cpu)
extern void setup_apic_routing(void);
#else
#define TARGET_CPUS (target_cpus())
#define TARGET_CPUS (default_target_cpus())
#define wakeup_secondary_cpu wakeup_secondary_cpu_via_init
/*
* Set up the logical destination ID.
Expand Down
2 changes: 1 addition & 1 deletion arch/x86/include/asm/numaq/apic.h
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

#define APIC_DFR_VALUE (APIC_DFR_CLUSTER)

static inline const cpumask_t *target_cpus(void)
static inline const cpumask_t *numaq_target_cpus(void)
{
return &CPU_MASK_ALL;
}
Expand Down
2 changes: 1 addition & 1 deletion arch/x86/include/asm/summit/apic.h
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@

#define APIC_DFR_VALUE (APIC_DFR_CLUSTER)

static inline const cpumask_t *target_cpus(void)
static inline const cpumask_t *summit_target_cpus(void)
{
/* CPU_MASK_ALL (0xff) has undefined behaviour with
* dest_LowestPrio mode logical clustered apic interrupt routing
Expand Down
2 changes: 1 addition & 1 deletion arch/x86/mach-generic/bigsmp.c
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ struct genapic apic_bigsmp = {
/* phys delivery to target CPU: */
.irq_dest_mode = 0,

.target_cpus = target_cpus,
.target_cpus = bigsmp_target_cpus,
.ESR_DISABLE = esr_disable,
.apic_destination_logical = APIC_DEST_LOGICAL,
.check_apicid_used = check_apicid_used,
Expand Down
2 changes: 1 addition & 1 deletion arch/x86/mach-generic/default.c
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ struct genapic apic_default = {
/* logical delivery broadcast to all CPUs: */
.irq_dest_mode = 1,

.target_cpus = target_cpus,
.target_cpus = default_target_cpus,
.ESR_DISABLE = esr_disable,
.apic_destination_logical = APIC_DEST_LOGICAL,
.check_apicid_used = check_apicid_used,
Expand Down
2 changes: 1 addition & 1 deletion arch/x86/mach-generic/es7000.c
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ struct genapic apic_es7000 = {
/* phys delivery to target CPUs: */
.irq_dest_mode = 0,

.target_cpus = target_cpus,
.target_cpus = es7000_target_cpus,
.ESR_DISABLE = esr_disable,
.apic_destination_logical = APIC_DEST_LOGICAL,
.check_apicid_used = check_apicid_used,
Expand Down
2 changes: 1 addition & 1 deletion arch/x86/mach-generic/numaq.c
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ struct genapic apic_numaq = {
/* physical delivery on LOCAL quad: */
.irq_dest_mode = 0,

.target_cpus = target_cpus,
.target_cpus = numaq_target_cpus,
.ESR_DISABLE = esr_disable,
.apic_destination_logical = APIC_DEST_LOGICAL,
.check_apicid_used = check_apicid_used,
Expand Down
2 changes: 1 addition & 1 deletion arch/x86/mach-generic/summit.c
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ struct genapic apic_summit = {
/* logical delivery broadcast to all CPUs: */
.irq_dest_mode = 1,

.target_cpus = target_cpus,
.target_cpus = summit_target_cpus,
.ESR_DISABLE = esr_disable,
.apic_destination_logical = APIC_DEST_LOGICAL,
.check_apicid_used = check_apicid_used,
Expand Down

0 comments on commit 0a9cc20

Please sign in to comment.