From 5d8faef80575b5f670369e3c552cfff9fdec9e3c Mon Sep 17 00:00:00 2001 From: Furquan Shaikh Date: Wed, 7 Mar 2018 23:16:57 -0800 Subject: cpu/x86/mp_init: Print amount of time it takes in bsp_do_flight_plan Since the timeout in bsp_do_flight_plan is bumped up to 1 second, this change adds a print to indicate the amount of time it takes for all the APs to check-in. TEST=Verified on Nami that it prints: "bsp_do_flight_plan done after 395 msecs." Change-Id: I4c8380e94305ed58453ed18b341b3b923949d7a8 Signed-off-by: Furquan Shaikh Reviewed-on: https://review.coreboot.org/25044 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Patrick Georgi Reviewed-by: Aaron Durbin --- src/cpu/x86/mp_init.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/cpu/x86/mp_init.c b/src/cpu/x86/mp_init.c index 23aea14683..409caa5a83 100644 --- a/src/cpu/x86/mp_init.c +++ b/src/cpu/x86/mp_init.c @@ -547,6 +547,9 @@ static int bsp_do_flight_plan(struct mp_params *mp_params) const int timeout_us = 1000000; const int step_us = 100; int num_aps = mp_params->num_cpus - 1; + struct stopwatch sw; + + stopwatch_init(&sw); for (i = 0; i < mp_params->num_records; i++) { struct mp_flight_record *rec = &mp_params->flight_plan[i]; @@ -566,6 +569,9 @@ static int bsp_do_flight_plan(struct mp_params *mp_params) release_barrier(&rec->barrier); } + + printk(BIOS_INFO, "%s done after %ld msecs.\n", __func__, + stopwatch_duration_msecs(&sw)); return ret; } -- cgit v1.2.3