From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kalle Valo Subject: Re: wlcore: use boottime for fw time sync Date: Thu, 7 Dec 2017 13:20:46 +0000 (UTC) Message-ID: <20171207132047.9770D6086A@smtp.codeaurora.org> References: <20171127114903.2779545-1-arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Cc: y2038@lists.linaro.org, Arnd Bergmann , Johannes Berg , Arend Van Spriel , Iain Hunter , "Reizer, Eyal" , "David S. Miller" , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: Arnd Bergmann Return-path: In-Reply-To: <20171127114903.2779545-1-arnd@arndb.de> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Arnd Bergmann wrote: > Using getnstimeofday()/timespec_to_ns() causes an overflow on 32-bit > architectures in 2038, and may suffer from time jumps due to > settimeofday() or leap seconds. > > I don't see a reason why this needs to be UTC, so either monotonic > or boot time would be better here. Assuming that the fw time keeps > running during suspend, boottime is better than monotonic, and > ktime_get_boot_ns() will also save the additional conversion to > nanoseconds. > > Signed-off-by: Arnd Bergmann Patch applied to wireless-drivers-next.git, thanks. 99f6996d4cba wlcore: use boottime for fw time sync -- https://patchwork.kernel.org/patch/10076691/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches