[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8562537.c35I4lEtz8@wuerfel>
Date: Wed, 22 Jun 2016 16:09:24 +0200
From: Arnd Bergmann <arnd@...db.de>
To: Deepa Dinamani <deepa.kernel@...il.com>
Cc: linux-fsdevel@...r.kernel.org, linux-kernel@...r.kernel.org,
tglx@...utronix.de, torvalds@...ux-foundation.org, tytso@....edu,
viro@...iv.linux.org.uk, y2038@...ts.linaro.org,
Hiral Patel <hiralpat@...co.com>,
Suma Ramars <sramars@...co.com>,
Brian Uchino <buchino@...co.com>,
"James E.J. Bottomley" <jejb@...ux.vnet.ibm.com>,
"Martin K. Petersen" <martin.petersen@...cle.com>,
John Stultz <john.stultz@...aro.org>,
linux-scsi@...r.kernel.org
Subject: Re: [PATCH v2 19/24] fnic: Use time64_t to represent trace timestamps
On Sunday, June 19, 2016 5:27:18 PM CEST Deepa Dinamani wrote:
> trace timestamps use struct timespec and CURRENT_TIME which
> are not y2038 safe.
> These timestamps are only part of the trace log on the machine
> and are not shared with the fnic.
> Replace then with y2038 safe struct timespec64 and
> ktime_get_real_ts64(), respectively.
>
> Note that change to add time64_to_tm() is already part of John's
> kernel tree: https://lkml.org/lkml/2016/6/17/875 .
>
While the patch looks good, I think it can't be part of this
series now, since it has to go on top of that first patch,
rather than merged in parallel.
When you send a pull request, please leave it out and submit
this one separately after 4.8-rc1.
Arnd
Powered by blists - more mailing lists