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

From: Lennart Sorensen
Date: Tue Jul 10 2007 - 12:01:59 EST


On Sun, Jul 01, 2007 at 09:24:41PM +0200, Rodolfo Giometti wrote:
> struct pps_timedata_s {
> __32 sec;
> __32 nsec;
> }
>
> Ok? I think 32 bits are enought for keeping seconds... :)

You want to purposely define an API that will break in 23 years (or is
that 83 years since you made it unsigned potentially)? Why not 64bit
for seconds and 32bit for nsec. That should cover it for long enough.
When the unix time format was created 37 years ago, I could see thinking
32bit seemed reasonable, but why do it now. We have ram enough for
64bits.

--
Len Sorensen
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/