[pnfs] [PATCH] update exchange id reply to draft 9

Talpey, Thomas Thomas.Talpey at netapp.com
Thu Mar 8 12:07:33 EST 2007


At 12:05 PM 3/8/2007, Benny Halevy wrote:
>FWIW, all other time values are stored in a timespec struct.
>I'm not sure where the conversion to native structures should take
>place for every use case.  In some other cases we may want to keep
>the time value in a nfstime structure in core and when converting
>into a timespec, with a 32-bit time_t, we should check for
>overflow.

Storing a local time in an nfstime4 directly would be what we
call a bug.

Tom.

>
>Benny
>
>Talpey, Thomas wrote:
>> At 03:36 AM 3/8/2007, Benny Halevy wrote:
>>> Marc, this is not strictly related to this patch but why are
>>> we defining struct nfstime4 and using it in struct nfs_impl_id4?
>>> why not just use struct timespec in core?
>> 
>> Because a struct timespec will have different size and alignment
>> depending on architecture? Not good for marshalling packets.
>> 
>> Tom.
>> 
>
>_______________________________________________
>pNFS mailing list
>pNFS at linux-nfs.org
>http://linux-nfs.org/cgi-bin/mailman/listinfo/pnfs



More information about the pNFS mailing list