s390/smp,topology: add polarization member to pcpu struct

The cpu polarization member is the only per cpu state that is not part
of the pcpu structure. So add it there and have everything in one place.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>

diff --git a/arch/s390/include/asm/smp.h b/arch/s390/include/asm/smp.h
index da7dcd7edf8e1644d698f895ceeb74cc0560f422..b64f15c3b4cc739a13e76ec3c1287328315ecb74 100644 (file)
--- a/arch/s390/include/asm/smp.h
+++ b/arch/s390/include/asm/smp.h
@@ -30,6 +30,8 @@ extern int smp_vcpu_scheduled(int cpu);
 extern void smp_yield_cpu(int cpu);
 extern void smp_yield(void);
 extern void smp_stop_cpu(void);
+extern void smp_cpu_set_polarization(int cpu, int val);
+extern int smp_cpu_get_polarization(int cpu);
 
 #else /* CONFIG_SMP */
 

diff --git a/arch/s390/include/asm/topology.h b/arch/s390/include/asm/topology.h
index 1e6c446e67e100c1afd3e3b4590dab24b7b24eb9..9ca30538376001ffe6b07e9cac9db311707e195f 100644 (file)
--- a/arch/s390/include/asm/topology.h
+++ b/arch/s390/include/asm/topology.h
@@ -51,24 +51,6 @@ static inline void topology_expect_change(void) { }
 #define POLARIZATION_VM                (2)
 #define POLARIZATION_VH                (3)
 
-extern int cpu_polarization[];
-
-static inline void cpu_set_polarization(int cpu, int val)
-{
-#ifdef CONFIG_SCHED_BOOK
-       cpu_polarization[cpu] = val;
-#endif
-}
-
-static inline int cpu_read_polarization(int cpu)
-{
-#ifdef CONFIG_SCHED_BOOK
-       return cpu_polarization[cpu];
-#else
-       return POLARIZATION_HRZ;
-#endif
-}
-
 #ifdef CONFIG_SCHED_BOOK
 void s390_init_cpu_topology(void);
 #else

diff --git a/arch/s390/kernel/smp.c b/arch/s390/kernel/smp.c
index c88fe8569e62e35f8db5550f71b16e8227ca0224..8676aab6ea6aa4c8c02bc5769c09ca95d7854721 100644 (file)
--- a/arch/s390/kernel/smp.c
+++ b/arch/s390/kernel/smp.c
@@ -66,6 +66,7 @@ struct pcpu {
        unsigned long panic_stack;      /* panic stack for the cpu */
        unsigned long ec_mask;          /* bit mask for ec_xxx functions */
        int state;                      /* physical cpu state */
+       int polarization;               /* physical polarization */
        u16 address;                    /* physical cpu address */
 };
 
@@ -73,6 +74,10 @@ static u8 boot_cpu_type;
 static u16 boot_cpu_address;
 static struct pcpu pcpu_devices[NR_CPUS];
 
+/*
+ * The smp_cpu_state_mutex must be held when changing the state or polarization
+ * member of a pcpu data structure within the pcpu_devices arreay.
+ */
 DEFINE_MUTEX(smp_cpu_state_mutex);
 
 /*
@@ -587,6 +592,16 @@ static inline void smp_get_save_area(int cpu, u16 address) { }
 
 #endif /* CONFIG_ZFCPDUMP || CONFIG_CRASH_DUMP */
 
+void smp_cpu_set_polarization(int cpu, int val)
+{
+       pcpu_devices[cpu].polarization = val;
+}
+
+int smp_cpu_get_polarization(int cpu)
+{
+       return pcpu_devices[cpu].polarization;
+}
+
 static struct sclp_cpu_info *smp_get_cpu_info(void)
 {
        static int use_sigp_detection;
@@ -629,7 +644,7 @@ static int __devinit __smp_rescan_cpus(struct sclp_cpu_info *info,
                pcpu->address = info->cpu[i].address;
                pcpu->state = (cpu >= info->configured) ?
                        CPU_STATE_STANDBY : CPU_STATE_CONFIGURED;
-               cpu_set_polarization(cpu, POLARIZATION_UNKNOWN);
+               smp_cpu_set_polarization(cpu, POLARIZATION_UNKNOWN);
                set_cpu_present(cpu, true);
                if (sysfs_add && smp_add_present_cpu(cpu) != 0)
                        set_cpu_present(cpu, false);
@@ -797,7 +812,7 @@ void __init smp_prepare_boot_cpu(void)
        pcpu->async_stack = S390_lowcore.async_stack - ASYNC_SIZE;
        pcpu->panic_stack = S390_lowcore.panic_stack - PAGE_SIZE;
        S390_lowcore.percpu_offset = __per_cpu_offset[0];
-       cpu_set_polarization(0, POLARIZATION_UNKNOWN);
+       smp_cpu_set_polarization(0, POLARIZATION_UNKNOWN);
        set_cpu_present(0, true);
        set_cpu_online(0, true);
 }
@@ -863,7 +878,7 @@ static ssize_t cpu_configure_store(struct device *dev,
                if (rc)
                        break;
                pcpu->state = CPU_STATE_STANDBY;
-               cpu_set_polarization(cpu, POLARIZATION_UNKNOWN);
+               smp_cpu_set_polarization(cpu, POLARIZATION_UNKNOWN);
                topology_expect_change();
                break;
        case 1:
@@ -873,7 +888,7 @@ static ssize_t cpu_configure_store(struct device *dev,
                if (rc)
                        break;
                pcpu->state = CPU_STATE_CONFIGURED;
-               cpu_set_polarization(cpu, POLARIZATION_UNKNOWN);
+               smp_cpu_set_polarization(cpu, POLARIZATION_UNKNOWN);
                topology_expect_change();
                break;
        default:

diff --git a/arch/s390/kernel/topology.c b/arch/s390/kernel/topology.c
index 80ca4ba759f65fdd3e42b019ae88d3adec91cd58..30c8533b8d4e0930debf730c669d1791c47baec6 100644 (file)
--- a/arch/s390/kernel/topology.c
+++ b/arch/s390/kernel/topology.c
@@ -45,9 +45,6 @@ static struct mask_info book_info;
 cpumask_t cpu_book_map[NR_CPUS];
 unsigned char cpu_book_id[NR_CPUS];
 
-/* smp_cpu_state_mutex must be held when accessing this array */
-int cpu_polarization[NR_CPUS];
-
 static cpumask_t cpu_group_map(struct mask_info *info, unsigned int cpu)
 {
        cpumask_t mask;
@@ -95,7 +92,7 @@ static struct mask_info *add_cpus_to_mask(struct topology_cpu *tl_cpu,
                        } else {
                                cpu_core_id[lcpu] = core->id;
                        }
-                       cpu_set_polarization(lcpu, tl_cpu->pp);
+                       smp_cpu_set_polarization(lcpu, tl_cpu->pp);
                }
        }
        return core;
@@ -202,7 +199,7 @@ static void topology_update_polarization_simple(void)
 
        mutex_lock(&smp_cpu_state_mutex);
        for_each_possible_cpu(cpu)
-               cpu_set_polarization(cpu, POLARIZATION_HRZ);
+               smp_cpu_set_polarization(cpu, POLARIZATION_HRZ);
        mutex_unlock(&smp_cpu_state_mutex);
 }
 
@@ -232,7 +229,7 @@ int topology_set_cpu_management(int fc)
        if (rc)
                return -EBUSY;
        for_each_possible_cpu(cpu)
-               cpu_set_polarization(cpu, POLARIZATION_UNKNOWN);
+               smp_cpu_set_polarization(cpu, POLARIZATION_UNKNOWN);
        return rc;
 }
 
@@ -414,7 +411,7 @@ static ssize_t cpu_polarization_show(struct device *dev,
        ssize_t count;
 
        mutex_lock(&smp_cpu_state_mutex);
-       switch (cpu_read_polarization(cpu)) {
+       switch (smp_cpu_get_polarization(cpu)) {
        case POLARIZATION_HRZ:
                count = sprintf(buf, "horizontal\n");
                break;