Re: [PATCH] LinuxPPS (with new syscalls API) - new version

David Woodhouse
2007-07-01T12:03:04Z

On Sat, 2007-06-30 at 19:13 +0200, Rodolfo Giometti wrote:
> Maybe I can define a special struct for exchanging time data as:
>
> struct pps_timedata_s {
> long sec;
> long nsec;
> }

Seems reasonable enough in principle -- but whatever you do, don't use
"long" for it. That would definitely need different behaviour for 32-bit
vs. 64-bit. Use explicitly sized types such as uint32_t or uint64_t.

--
dwmw2