powerpc/vphn: move endianness fixing to vphn_unpack_associativity()
authorGreg Kurz <gkurz@linux.vnet.ibm.com>
Mon, 23 Feb 2015 15:14:25 +0000 (16:14 +0100)
committerMichael Ellerman <mpe@ellerman.id.au>
Tue, 17 Mar 2015 23:48:59 +0000 (10:48 +1100)
The first argument to vphn_unpack_associativity() is a const long *, but the
parsing code expects __be64 values actually. Let's move the endian fixing
down for consistency.

Signed-off-by: Greg Kurz <gkurz@linux.vnet.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/mm/numa.c

index 7a5bc21152f6d1c1726b610d550c94e32fb489d3..59196c5a0984baa01930f83067851addc4215498 100644 (file)
@@ -1263,13 +1263,18 @@ static int update_cpu_associativity_changes_mask(void)
  */
 static int vphn_unpack_associativity(const long *packed, __be32 *unpacked)
 {
+       __be64 be_packed[VPHN_REGISTER_COUNT];
        int i, nr_assoc_doms = 0;
-       const __be16 *field = (const __be16 *) packed;
+       const __be16 *field = (const __be16 *) be_packed;
 
 #define VPHN_FIELD_UNUSED      (0xffff)
 #define VPHN_FIELD_MSB         (0x8000)
 #define VPHN_FIELD_MASK                (~VPHN_FIELD_MSB)
 
+       /* Let's recreate the original stream. */
+       for (i = 0; i < VPHN_REGISTER_COUNT; i++)
+               be_packed[i] = cpu_to_be64(packed[i]);
+
        for (i = 1; i < VPHN_ASSOC_BUFSIZE; i++) {
                if (be16_to_cpup(field) == VPHN_FIELD_UNUSED) {
                        /* All significant fields processed, and remaining
@@ -1310,11 +1315,8 @@ static long hcall_vphn(unsigned long cpu, __be32 *associativity)
        long retbuf[PLPAR_HCALL9_BUFSIZE] = {0};
        u64 flags = 1;
        int hwcpu = get_hard_smp_processor_id(cpu);
-       int i;
 
        rc = plpar_hcall9(H_HOME_NODE_ASSOCIATIVITY, retbuf, flags, hwcpu);
-       for (i = 0; i < VPHN_REGISTER_COUNT; i++)
-               retbuf[i] = cpu_to_be64(retbuf[i]);
        vphn_unpack_associativity(retbuf, associativity);
 
        return rc;