On 05/30/2016 06:02 PM, Ed Cashin wrote:
On 05/30/2016 04:01 AM, Tina Ruchandani wrote:
'struct frame' uses two variables to store the sent timestamp - 'struct timeval' and jiffies. jiffies is used to avoid discrepancies caused by updates to system time. 'struct timeval' uses 32-bit representation for seconds which will overflow in year 2038.
...
I don't think it's a good idea to introduce a performance regression, since there's no safety concern for aoe.
If the concern is that we don't know which users of the 32-bit timestamp are buggy, but we *do* know that this use is OK, then a comment in the source code seems more appropriate.
... But after reviewing the previous discussion I think I should try to find people with 32-bit systems who can tell me whether they see a performance regression.
I'll do that and try to get an answer soon.