On Wed, Apr 24, 2013 at 8:55 PM, John Stultz <john.stultz@xxxxxxxxxx> wrote:On 04/24/2013 11:41 AM, Kay Sievers wrote:So far we only created the symlink for an rtc with the hctosys flagOn Wed, Apr 24, 2013 at 8:32 PM, John Stultz <john.stultz@xxxxxxxxxx>Is that actually wanted? What happens to applications that use /dev/rtc?
Kay Sievers noted that the ALWAYS_USE_PERSISTENT_CLOCK config,FWIW, in the light of the original change, I've just removed the
which enables some minor compile time optimization to avoid
uncessary code in mostly the suspend/resume path could cause
problems for userland.
In particular, the dependency for RTC_HCTOSYS on
!ALWAYS_USE_PERSISTENT_CLOCK, which avoids setting the time
twice and simplifies suspend/resume, has the side effect
of causing the /sys/class/rtc/rtcN/hctosys flag to always be
zero, and this flag is commonly used by udev to setup the
/dev/rtc symlink to /dev/rtcN, which can cause pain for
/dev/rtc creation from the default udev rules now, so that thing will
be phased out in the future.
I think setting up the /dev/rtc link is important. Its just that setting it
up exclusively by the hctosys flag is maybe more fragile then we'd like.
Instead the hctosys flag maybe should only be used as a hint if there is
more then one RTC available.
set. It was added as a workaround for ARM, which sometimes has
multiple RTCs. But there is now also logic in udev/systemd anyway to
search for the system's rtc, which does not rely on the symlink. Other
commonly used tools we checked just use /dev/rtc0 directly.
As mentioned in the mail yesterday, I expected that
CONFIG_RTC_HCTOSYS=y was also needed on x86. The current udev logic
would need updating anyway, if we disable CONFIG_RTC_HCTOSYS=y now. So
let's just find out what's really needed here and add it back properly
if something really needs it.
While we are at it, the interface to read and set the persistent clock
from userspace, the clock the kernel internally uses, is still to just
use the /dev/rtc0 device with the ioctls? There is no other native
kernel timer interface or something else for that, right?