On Wed, Apr 24, 2013 at 5:19 AM, John Stultz <john.stultz@xxxxxxxxxx> wrote:On 04/23/2013 08:05 PM, Kay Sievers wrote:Until the above commits we always needed:On Wed, Apr 24, 2013 at 4:43 AM, John Stultz <john.stultz@xxxxxxxxxx>On x86 the persistent_clock() is backed by the CMOS/EFI/kvm-wall/xen/vrtc
wrote:
On 04/23/2013 06:34 PM, Kay Sievers wrote:Right, and RTC_HCTOSYS is not NTP related. It just reads the time fromHey,
what's the intention of:
e90c83f757fffdacec8b3c5eee5617dcc038338f ?
x86: Select HAS_PERSISTENT_CLOCK on x86
It unconditionally sets:
HAS_PERSISTENT_CLOCK
but:
RTC_SYSTOHC
got a depends on !HAS_PERSISTENT_CLOCK
This makes it impossible to sync the system time from the RTC on x86.
What's going on here?
So I suspect just some confusion, but let me know if thats wrong and
you're
actually seeing an issue.
SYSTOHC is what *sets the RTC* to the system time when we're synced with
NTP.
HCTOSYS is what sets the system time from the RTC.
the RTC_HCTOSYS_DEVICE at bootup so we do not boot in 1970 time mode.
We need that it in all cases, at every bootup on x86. But it's no
longer there with the above commits. :)
clock (all via x86_platform.get_wallclock) should be present and we'll
initialize the time in timekeeping_init() there.
Its only systems where there isn't a persistent_clock is where the RTC layer
and the HCTOSYS is helpful.
Again, if you're having a problem where an x86 system isn't getting its time
initialized correctly, please let me know the details of the system.
CONFIG_RTC_HCTOSYS=y
CONFIG_RTC_HCTOSYS_DEVICE="rtc0"
to get the system time correctly initialized at bootup on x86.
These options are gone now and cannot be selected anymore. You are
saying that this is all fine, that they are gone, but all initial
clock syncing should still work?
Also:
$ cat /sys/class/rtc/rtc0/hctosys
0
always carried "1", and this now breaks setups which expect an
automatically created symlink /dev/rtc to the actual "system rtc".
There was also always a line in dmesg that told the rtc_cmos time it
wrote to the system clock. This is also gone?
I haven't looked what goes wrong, I expected the make(1) errors with
"time in the future" after bootup before the network is up, which I
see since a week or two, to be a fallout of that.