Re: [PATCH 2/2] pseries: Fix endian issues in cpu hot-removal
From: Michael Ellerman <mpe@ellerman.id.au>
Date: 2014-09-12 08:53:39
On Wed, 2014-09-10 at 17:41 -0500, Thomas Falcon wrote:
quoted hunk ↗ jump to hunk
When removing a cpu, this patch makes sure that values gotten from or passed to firmware are in the correct endian format. Signed-off-by: Thomas Falcon <redacted> --- arch/powerpc/platforms/pseries/dlpar.c | 14 +++++++------- arch/powerpc/platforms/pseries/hotplug-cpu.c | 8 ++++---- 2 files changed, 11 insertions(+), 11 deletions(-)diff --git a/arch/powerpc/platforms/pseries/dlpar.c b/arch/powerpc/platforms/pseries/dlpar.c index cd425dc..c5ecfdb 100644 --- a/arch/powerpc/platforms/pseries/dlpar.c +++ b/arch/powerpc/platforms/pseries/dlpar.c@@ -442,7 +442,7 @@ static int dlpar_offline_cpu(struct device_node *dn) int rc = 0; unsigned int cpu; int len, nthreads, i; - const u32 *intserv; + const __be32 *intserv; intserv = of_get_property(dn, "ibm,ppc-interrupt-server#s", &len); if (!intserv)@@ -453,7 +453,7 @@ static int dlpar_offline_cpu(struct device_node *dn) cpu_maps_update_begin(); for (i = 0; i < nthreads; i++) {
Can you please do the conversion once here for each value of i. You can call the converted value "thread" ?
for_each_present_cpu(cpu) {
- if (get_hard_smp_processor_id(cpu) != intserv[i])
+ if (get_hard_smp_processor_id(cpu) != be32_to_cpu(intserv[i]))
continue;Rather than doing it for every cpu in the system for every value of i. Not that performance is really an issue, but it's just ugly. And obviously the other places that use it in the loop should use the converted value.
quoted hunk ↗ jump to hunk
@@ -494,7 +494,7 @@ out: static ssize_t dlpar_cpu_release(const char *buf, size_t count) { struct device_node *dn; - const u32 *drc_index; + const __be32 *drc_index; int rc; dn = of_find_node_by_path(buf);@@ -513,7 +513,7 @@ static ssize_t dlpar_cpu_release(const char *buf, size_t count) return -EINVAL; }
Here again you should do the conversion once. Better still use of_property_read_u32().
quoted hunk ↗ jump to hunk
- rc = dlpar_release_drc(*drc_index); + rc = dlpar_release_drc(be32_to_cpup(drc_index)); if (rc) { of_node_put(dn); return rc;@@ -521,7 +521,7 @@ static ssize_t dlpar_cpu_release(const char *buf, size_t count) rc = dlpar_detach_node(dn); if (rc) { - dlpar_acquire_drc(*drc_index); + dlpar_acquire_drc(be32_to_cpup(drc_index)); return rc; }
cheers