diff options
| author | Vasant Hegde <hegdevasant@linux.vnet.ibm.com> | 2018-02-28 15:29:08 +0530 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2018-02-28 20:28:39 -0600 |
| commit | 1dacecd2b28b71b45982e3f063918dab78279cf5 (patch) | |
| tree | 471fddbaa95c025bdcd80f984c2ccee251ac514f /core/init.c | |
| parent | ac4272bf5e73de78bfb4cdffac8f57222b059f72 (diff) | |
| download | skiboot-1dacecd2b28b71b45982e3f063918dab78279cf5.zip skiboot-1dacecd2b28b71b45982e3f063918dab78279cf5.tar.gz skiboot-1dacecd2b28b71b45982e3f063918dab78279cf5.tar.bz2 | |
core: Fix 'opal-runtime-size' property
We are populating 'opal-runtime-size' before calculating actual stack size.
Hence we endup having wrong runtime size (ex: on P9 it shows ~540MB while
actual size is around ~40MB). Note that only device tree property is shows
wrong value, but reserved-memory reflects correct size.
init_all_cpus() calculates and updates actual stack size. Hence move this
function call before add_opal_node().
Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'core/init.c')
| -rw-r--r-- | core/init.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/core/init.c b/core/init.c index 5633a9d..69f83ed 100644 --- a/core/init.c +++ b/core/init.c @@ -937,6 +937,11 @@ void __noreturn __nomcount main_cpu_entry(const void *fdt) /* Reserve HOMER and OCC area */ homer_init(); + /* Initialize the rest of the cpu thread structs */ + init_all_cpus(); + if (proc_gen == proc_gen_p9) + cpu_set_ipi_enable(true); + /* Add the /opal node to the device-tree */ add_opal_node(); @@ -948,11 +953,6 @@ void __noreturn __nomcount main_cpu_entry(const void *fdt) */ probe_platform(); - /* Initialize the rest of the cpu thread structs */ - init_all_cpus(); - if (proc_gen == proc_gen_p9) - cpu_set_ipi_enable(true); - /* Allocate our split trace buffers now. Depends add_opal_node() */ init_trace_buffers(); |