Re: [PATCH net-next 3/3] tcp: use monotonic timestamps for PAWS

From: Arnd Bergmann
Date: Tue Jun 19 2018 - 05:23:56 EST


On Mon, Jun 18, 2018 at 5:59 PM, Eric Dumazet <eric.dumazet@xxxxxxxxx> wrote:
> On 06/18/2018 08:22 AM, Arnd Bergmann wrote:
>
> Please use the time_after32(), since ktime_get_seconds() is time64_t while ts_recent_stamp is int.
>
> Same remark for tcp_twsk_unique()
>
> Lets clean up this stuff, thanks !

Sure, no problem. Note that as ktime_get_seconds() uses CLOCK_MONOTONIC, it is
guaranteed to return a number that fits into a positive 32-bit number
(for about 136
years after boot to be precise). Most users of time_after() or
time_after32() are for
jiffies values that do overflow regularly, but it doesn't hurt to use
time_after32()
here for illustration.

Arnd