Subject: Re: [PATCH 3/4 -rev1] Initilize and populate age field

From: Varun Chandramohan <[email-masked]>
Date: Thu, 26 Jul 2007 12:29:22 +0530

Eric Dumazet wrote:
> Varun Chandramohan a écrit :
>> Eric Dumazet wrote:
>>> Varun Chandramohan a écrit :
>>>> The age field is filled with the current time at the time of creation
>>>> of the route. When the routes are dumped
>>>> then the age value stored in the route structure is subtracted from
>>>> the current time value and the difference is the age expressed in
>>>> secs.
>>>>
>>>>      }
>>>> +
>>>> +    do_gettimeofday(&tv);
>>>> +    if (!*age) {
>>>> +        *age = timeval_to_sec(&tv);
>>>> +        NLA_PUT_U32(skb, RTA_AGE, *age);
>>> here, what happens if sizeof(time_t) is not 4 ?
>>>
>> Same case as above :-) comments?
>
> So you are confident it (truncation from 8 to 4 bytes) works well on
> Big endian arches as well as Little endian arches ? :-)
>
I suppose so. But i must admit i have'nt tested it on Big endian arch
though!!!
>
> -
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to [email-masked]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html