From: Karel Zak <kzak@redhat.com>
To: JWP <elseifthen@gmx.com>
Cc: util-linux@vger.kernel.org
Subject: Re: * RECALL * [PATCH 1/2] hwclock: hctosys drift compensation
Date: Wed, 17 Sep 2014 11:55:28 +0200 [thread overview]
Message-ID: <20140917095528.GJ7867@x2.net.home> (raw)
In-Reply-To: <54183647.2080101@gmx.com>
On Tue, Sep 16, 2014 at 09:08:23AM -0400, JWP wrote:
> Hello Karel,
>
> On 09/16/2014 05:35 AM, Karel Zak wrote:
> > See Documentation/TODO, the writable /etc/adjtime sucks, because in
> > many cases we want to keep /etc read-only. I see two possible ways:
>
> Yes, I saw that and have it on my todo list. However, it is down my list a ways.
> I had not planned on attempting to integrate it into the hctosys/show patch.
>
> > -- the problem is that the file (specially last UTC/LOCAL line)
> > may be expected by another tools
>
> True, init scripts depend upon that information too. So it is an important
> consideration.
>
> My initial thought when I read u-l's todo list was that adjtimex [ -c | -a ]
> depend upon /etc/adjtime *drift* data. Do we care about breaking that?
I'm not sure, but I guess adjtimex reads /etc/adjtime to get info
about UTC/LOCAL only. Well, it seems we need to check adjtimex code.
Anyway, if adjtimex depends on drift data then we can change it too to
use /var.
Note that we have hwclock --compare to provide adjtimex -c functionality
with in hwclock.
> I have often thought that hwclock should have a separate 'configuration' file,
> for example, to allow specific init behavior customization without requiring
> system admins to modify the init scripts.
The old RHEL/Fedora has /etc/sysconfig/hwclock where is possible
to specify hwclock command line, so no one is forced to modify init
scripts. All you need is to read the file from your init scripts.
BTW, slowly growing number of systemd based distributions where
hwclock --hctosys and --adjust is no more used and we usually assume
that NTP + kernel is enough to update CMOS. From this point of view
hwclock(8) is more about very basic low-level HW clock manipulation
than about systime and hwtime relationship :-)
(but yes, we (upstream) still support the original hwclock(8) use-cases
and non-systemd installations)
Karel
--
Karel Zak <kzak@redhat.com>
http://karelzak.blogspot.com
next prev parent reply other threads:[~2014-09-17 9:55 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-14 19:29 [PATCH 1/2] hwclock: hctosys drift compensation JWP
2014-09-14 19:46 ` [PATCH 2/2] hwclock: hctosys drift compensation man page JWP
2014-09-15 14:03 ` * RECALL * [PATCH 1/2] hwclock: hctosys drift compensation JWP
2014-09-16 9:35 ` Karel Zak
2014-09-16 13:08 ` JWP
2014-09-16 16:32 ` Bruce Dubbs
2014-09-16 23:08 ` JWP
2014-09-17 9:55 ` Karel Zak [this message]
2014-09-17 13:34 ` elseifthen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140917095528.GJ7867@x2.net.home \
--to=kzak@redhat.com \
--cc=elseifthen@gmx.com \
--cc=util-linux@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).