linux-kernel - Re: [RFE PATCH 1/2] x86, rtc, ntp, Do full rtc synchronization with ntp

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <5127D853.6030703@linaro.org>
Date:	Fri, 22 Feb 2013 12:42:59 -0800
From:	John Stultz <john.stultz@...aro.org>
To:	Prarit Bhargava <prarit@...hat.com>
CC:	linux-kernel@...r.kernel.org, Thomas Gleixner <tglx@...utronix.de>,
	x86@...nel.org, Matt Fleming <matt.fleming@...el.com>,
	David Vrabel <david.vrabel@...rix.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Andi Kleen <ak@...ux.intel.com>, linux-efi@...r.kernel.org
Subject: Re: [RFE PATCH 1/2] x86, rtc, ntp, Do full rtc synchronization with
 ntp

On 02/14/2013 09:02 AM, Prarit Bhargava wrote:
> Every 11 minutes ntp attempts to update the x86 rtc with the current
> system time.  Currently, the x86 code only updates the rtc if the system
> time is within +/-15 minutes of the current value of the rtc.  Other
> architectures do a full synchronization and there is no reason that x86
> should be software limited to a 30 minute window.
>
> This patch changes the behavior of the kernel to do a full synchronization
> (year, month, day, hour, minute, and second) of the rtc when ntp requests
> a synchronization between the system time and the rtc.
>
> I've used the RTC library functions in this patchset as they do all the
> required bounds checking.
>
> Signed-off-by: Prarit Bhargava <prarit@...hat.com>
> Cc: Thomas Gleixner <tglx@...utronix.de>
> Cc: John Stultz <john.stultz@...aro.org>
> Cc: x86@...nel.org
> Cc: Matt Fleming <matt.fleming@...el.com>
> Cc: David Vrabel <david.vrabel@...rix.com>
> Cc: Andrew Morton <akpm@...ux-foundation.org>
> Cc: Andi Kleen <ak@...ux.intel.com>
> Cc: linux-efi@...r.kernel.org


This looks reasonable to me.

Though I want to make sure we get this thoroughly tested by the various 
distros so we don't surprise anyone, since it has to potential to cause 
problems where folks are dualbooting windows (using a localtime RTC) and 
do not have their OS setup to trigger warp_clock to adjust for the 
localtime rtc (instead getting a time correction later via NTP).

I'll queue it and see about getting it merged to -tip & -next. Then 
we'll have to decide if 3.10 or 3.11 is the right time frame to land it.

thanks
-john


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/