Hi Miao, Please remove the qemu tag from the commit title, as this is a change to common x86 codes.
On Mon, Dec 28, 2015 at 5:18 PM, Miao Yan <yanmiaob...@gmail.com> wrote: > Use actual CPU number , instead of maximum cpu configured, nits: remove the space after 'number' > to allocate stack memory in 'load_sipi_vector' > > Signed-off-by: Miao Yan <yanmiaob...@gmail.com> > --- > arch/x86/cpu/mp_init.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/arch/x86/cpu/mp_init.c b/arch/x86/cpu/mp_init.c > index 2f34317..2a3ce48 100644 > --- a/arch/x86/cpu/mp_init.c > +++ b/arch/x86/cpu/mp_init.c > @@ -210,7 +210,7 @@ static int save_bsp_msrs(char *start, int size) > return msr_count; > } > > -static int load_sipi_vector(atomic_t **ap_countp) > +static int load_sipi_vector(atomic_t **ap_countp, int num_cpus) > { > struct sipi_params_16bit *params16; > struct sipi_params *params; > @@ -239,7 +239,7 @@ static int load_sipi_vector(atomic_t **ap_countp) > params->idt_ptr = (uint32_t)x86_get_idt(); > > params->stack_size = CONFIG_AP_STACK_SIZE; > - size = params->stack_size * CONFIG_MAX_CPUS; > + size = params->stack_size * num_cpus; > stack = memalign(size, 4096); > if (!stack) > return -ENOMEM; > @@ -483,7 +483,7 @@ int mp_init(struct mp_params *p) > mp_info.records = p->flight_plan; > > /* Load the SIPI vector */ > - ret = load_sipi_vector(&ap_count); > + ret = load_sipi_vector(&ap_count, num_cpus); > if (ap_count == NULL) > return -1; > > -- Other than that, Reviewed-by: Bin Meng <bmeng...@gmail.com> Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot