patch for hp-timing.h

Wilco Dijkstra Wilco.Dijkstra@arm.com
Wed Feb 8 14:58:39 GMT 2023


Hi Jun,

> There are 7 zero bits at the end of freq (1050000000).  So the time can be extended to
> 2048 second before losing resolution.  Below is the updated patch for 1024 sec -
>
> -   (Diff) = ((End) - (Start)) * (UINT64_C(1000000000) / freq); \
> +   (Diff) = (((End) - (Start)) * UINT64_C(1000000000>>6)) / (freq>>6); \
 
Minor codestyle: there should be a space before/after '>>'.

Apart from that it looks good and works fine.

Can you resend an updated patch with comment header etc in a new email with
subject like "[PATCH v2] AArch64: Fix HP_TIMING_DIFF computation" or similar?

I guess this is your first patch and you don't have commit rights?

Cheers,
Wilco


More information about the Libc-alpha mailing list