| Message ID | 20180220060815.19303-1-stewart@linux.vnet.ibm.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [1/2] cpu_wait_job: Correctly report time spent waiting for job | expand |
Stewart Smith <stewart@linux.vnet.ibm.com> writes: > 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@linux.vnet.ibm.com> Series merged to master as of 03e90a0629bdbd3bae70fd4c531b2c800d4b922c
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);
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@linux.vnet.ibm.com> --- core/cpu.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)