Hi Vinay, > Add these redundant utils to core lib. Also fix the bug in I think "Move" would better describe what you do in this patch. > + slept = igt_measured_usleep(batch_duration_ns / 1000) * NSEC_PER_USEC; Since you return usec instead of nsec, you had to add "* NSEC_PER_USEC" in many instances that this function is called. How about dropping returning usec alltogether and stick to returning nsec value? (and rename/modify the function to handle nsec values as well) Krzysztof