Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/57342 )
Change subject: cpu/x86: Use MP_RUN_ON_ALL_CPUS macro instead of hardcoding ......................................................................
cpu/x86: Use MP_RUN_ON_ALL_CPUS macro instead of hardcoding
This patch ensures mp_run_on_all_aps() is passing 'MP_RUN_ON_ALL_CPUS' macro rather hardcoding `0` while running `func` on all APs.
Change-Id: Icd34371c0d4349e1eefe945958eda957c4794707 Signed-off-by: Subrata Banik subrata.banik@intel.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/57342 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Tim Wawrzynczak twawrzynczak@chromium.org --- M src/cpu/x86/mp_init.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Tim Wawrzynczak: Looks good to me, approved
diff --git a/src/cpu/x86/mp_init.c b/src/cpu/x86/mp_init.c index 3eef355..aa037f3 100644 --- a/src/cpu/x86/mp_init.c +++ b/src/cpu/x86/mp_init.c @@ -997,7 +997,7 @@ int ap_index, bsp_index;
if (run_parallel) - return mp_run_on_aps(func, arg, 0, expire_us); + return mp_run_on_aps(func, arg, MP_RUN_ON_ALL_CPUS, expire_us);
bsp_index = cpu_index();
1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.