From 2981a7bfc9dc3b1b642a868b09a75d011c49063a Mon Sep 17 00:00:00 2001 From: Stewart Smith Date: Wed, 23 Nov 2016 11:45:59 +1100 Subject: [PATCH] wait_for_resource_loaded: don't needlessly sleep for 5ms We would needlessly time_wait_ms_nopoll(5) if opal_run_pollers() managed to crank things enough to load the requested resource. Instead of needlessly sleeping, check the result of resource_loaded() *before* going to sleep for a bit. Signed-off-by: Stewart Smith --- core/platform.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/core/platform.c b/core/platform.c index 587816478c79..ba0636d94242 100644 --- a/core/platform.c +++ b/core/platform.c @@ -205,9 +205,11 @@ int wait_for_resource_loaded(enum resource_id id, uint32_t idx) while(r == OPAL_BUSY) { opal_run_pollers(); + r = resource_loaded(id, idx); + if (r != OPAL_BUSY) + break; time_wait_ms_nopoll(5); waited+=5; - r = resource_loaded(id, idx); } prlog(PR_TRACE, "PLATFORM: wait_for_resource_loaded %x/%x %u ms\n",