[Skiboot] [PATCH 1/2] cpu_wait_job: Correctly report time spent waiting for job

Stewart Smith stewart at linux.vnet.ibm.com
Tue Feb 20 17:08:14 AEDT 2018


Way back when, we got confused between timebase and ms,
so let's just use ms and be done with it.

Fixes: 514406fa44279996bfc9c85c1e4e53689d375e64
Signed-off-by: Stewart Smith <stewart at linux.vnet.ibm.com>
---
 core/cpu.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/core/cpu.c b/core/cpu.c
index 3e110c3c5015..2eab5225531e 100644
--- a/core/cpu.c
+++ b/core/cpu.c
@@ -245,9 +245,9 @@ void cpu_wait_job(struct cpu_job *job, bool free_it)
 	}
 	lwsync();
 
-	if (time_waited > msecs_to_tb(1000))
-		prlog(PR_DEBUG, "cpu_wait_job(%s) for %lu\n",
-		      job->name, tb_to_msecs(time_waited));
+	if (time_waited > 1000)
+		prlog(PR_DEBUG, "cpu_wait_job(%s) for %lums\n",
+		      job->name, time_waited);
 
 	if (free_it)
 		free(job);
-- 
2.14.3



More information about the Skiboot mailing list