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

David Woodhouse dwmw2 at infradead.org
Wed Jul 17 01:14:10 PDT 2024


On Tue, 2024-07-16 at 13:54 +0200, Peter Hilber wrote:
> On 08.07.24 11:27, David Woodhouse wrote:
> > +
> > +       /*
> > +        * Time according to time_type field above.
> > +        */
> > +       uint64_t time_sec;              /* Seconds since time_type epoch */
> > +       uint64_t time_frac_sec;         /* (seconds >> 64) */
> > +       uint64_t time_esterror_picosec; /* (± picoseconds) */
> > +       uint64_t time_maxerror_picosec; /* (± picoseconds) */
> 
> Is this unsigned or signed?

The field itself is unsigned, as it provides the absolute value of the
error (which can be in either direction). Probably better just to drop
the ± from the comment.

Julien is now back from vacation and I'm expecting to see his opinion
on whether we can change that to nanoseconds for consistency.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 5965 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20240717/3a801613/attachment.p7s>


More information about the linux-arm-kernel mailing list