Re: [RFC PATCH v2] ptp: Add vDSO-style vmclock support

From: Peter Hilber
Date: Fri Jul 05 2024 - 04:12:55 EST


On 03.07.24 12:40, David Woodhouse wrote:

[...]

>
>
> This is what I currently have for 'struct vmclock_abi' that I'd like to
> persuade you to adopt. I need to tweak it some more, for at least the
> following reasons, as well as any more you can see:
>
> • size isn't big enough for 64KiB pages
> • Should be explicitly little-endian
> • Does it need esterror as well as maxerror?

I have no opinion about this. I can drop esterror if unwanted.

> • Why is maxerror in picoseconds? It's the only use of that unit
> • Where do the clock_status values come from? Do they make sense?
> • Are signed integers OK? (I think so!).

Signed integers would need to be introduced to Virtio, which so far only
uses explicitly unsigned types: u8, le16 etc.