Commit c9ac408b authored by Thomas Falcon's avatar Thomas Falcon Committed by Michael Ellerman

powerpc/pseries: Fix endian issues in memory hotplug

Values acquired from Open Firmware are in 32-bit big endian format
and need to be handled on little endian architectures.  This patch
ensures values are in cpu endian when hotplugging memory.
Signed-off-by: default avatarThomas Falcon <tlfalcon@linux.vnet.ibm.com>
parent 2ce7598c
...@@ -113,7 +113,7 @@ static int pseries_remove_memblock(unsigned long base, unsigned int memblock_siz ...@@ -113,7 +113,7 @@ static int pseries_remove_memblock(unsigned long base, unsigned int memblock_siz
static int pseries_remove_mem_node(struct device_node *np) static int pseries_remove_mem_node(struct device_node *np)
{ {
const char *type; const char *type;
const unsigned int *regs; const __be32 *regs;
unsigned long base; unsigned long base;
unsigned int lmb_size; unsigned int lmb_size;
int ret = -EINVAL; int ret = -EINVAL;
...@@ -132,8 +132,8 @@ static int pseries_remove_mem_node(struct device_node *np) ...@@ -132,8 +132,8 @@ static int pseries_remove_mem_node(struct device_node *np)
if (!regs) if (!regs)
return ret; return ret;
base = *(unsigned long *)regs; base = be64_to_cpu(*(unsigned long *)regs);
lmb_size = regs[3]; lmb_size = be32_to_cpu(regs[3]);
pseries_remove_memblock(base, lmb_size); pseries_remove_memblock(base, lmb_size);
return 0; return 0;
...@@ -153,7 +153,7 @@ static inline int pseries_remove_mem_node(struct device_node *np) ...@@ -153,7 +153,7 @@ static inline int pseries_remove_mem_node(struct device_node *np)
static int pseries_add_mem_node(struct device_node *np) static int pseries_add_mem_node(struct device_node *np)
{ {
const char *type; const char *type;
const unsigned int *regs; const __be32 *regs;
unsigned long base; unsigned long base;
unsigned int lmb_size; unsigned int lmb_size;
int ret = -EINVAL; int ret = -EINVAL;
...@@ -172,8 +172,8 @@ static int pseries_add_mem_node(struct device_node *np) ...@@ -172,8 +172,8 @@ static int pseries_add_mem_node(struct device_node *np)
if (!regs) if (!regs)
return ret; return ret;
base = *(unsigned long *)regs; base = be64_to_cpu(*(unsigned long *)regs);
lmb_size = regs[3]; lmb_size = be32_to_cpu(regs[3]);
/* /*
* Update memory region to represent the memory add * Update memory region to represent the memory add
...@@ -187,14 +187,14 @@ static int pseries_update_drconf_memory(struct of_prop_reconfig *pr) ...@@ -187,14 +187,14 @@ static int pseries_update_drconf_memory(struct of_prop_reconfig *pr)
struct of_drconf_cell *new_drmem, *old_drmem; struct of_drconf_cell *new_drmem, *old_drmem;
unsigned long memblock_size; unsigned long memblock_size;
u32 entries; u32 entries;
u32 *p; __be32 *p;
int i, rc = -EINVAL; int i, rc = -EINVAL;
memblock_size = pseries_memory_block_size(); memblock_size = pseries_memory_block_size();
if (!memblock_size) if (!memblock_size)
return -EINVAL; return -EINVAL;
p = (u32 *) pr->old_prop->value; p = (__be32 *) pr->old_prop->value;
if (!p) if (!p)
return -EINVAL; return -EINVAL;
...@@ -203,28 +203,30 @@ static int pseries_update_drconf_memory(struct of_prop_reconfig *pr) ...@@ -203,28 +203,30 @@ static int pseries_update_drconf_memory(struct of_prop_reconfig *pr)
* entries. Get the niumber of entries and skip to the array of * entries. Get the niumber of entries and skip to the array of
* of_drconf_cell's. * of_drconf_cell's.
*/ */
entries = *p++; entries = be32_to_cpu(*p++);
old_drmem = (struct of_drconf_cell *)p; old_drmem = (struct of_drconf_cell *)p;
p = (u32 *)pr->prop->value; p = (__be32 *)pr->prop->value;
p++; p++;
new_drmem = (struct of_drconf_cell *)p; new_drmem = (struct of_drconf_cell *)p;
for (i = 0; i < entries; i++) { for (i = 0; i < entries; i++) {
if ((old_drmem[i].flags & DRCONF_MEM_ASSIGNED) && if ((be32_to_cpu(old_drmem[i].flags) & DRCONF_MEM_ASSIGNED) &&
(!(new_drmem[i].flags & DRCONF_MEM_ASSIGNED))) { (!(be32_to_cpu(new_drmem[i].flags) & DRCONF_MEM_ASSIGNED))) {
rc = pseries_remove_memblock(old_drmem[i].base_addr, rc = pseries_remove_memblock(
be64_to_cpu(old_drmem[i].base_addr),
memblock_size); memblock_size);
break; break;
} else if ((!(old_drmem[i].flags & DRCONF_MEM_ASSIGNED)) && } else if ((!(be32_to_cpu(old_drmem[i].flags) &
(new_drmem[i].flags & DRCONF_MEM_ASSIGNED)) { DRCONF_MEM_ASSIGNED)) &&
rc = memblock_add(old_drmem[i].base_addr, (be32_to_cpu(new_drmem[i].flags) &
DRCONF_MEM_ASSIGNED)) {
rc = memblock_add(be64_to_cpu(old_drmem[i].base_addr),
memblock_size); memblock_size);
rc = (rc < 0) ? -EINVAL : 0; rc = (rc < 0) ? -EINVAL : 0;
break; break;
} }
} }
return rc; return rc;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment