mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Alessandro Zummo <alessandro.zummo@towertech.it>
To: David Woodhouse <dwmw2@infradead.org>
Cc: rtc-linux@googlegroups.com, paulus@samba.org,
	<linux-kernel@vger.kernel.org>
Subject: Re: [rtc-linux] Re: [PATCH] RTC class driver for ppc_md RTC functions
Date: Wed, 30 Apr 2008 11:39:33 +0200	[thread overview]
Message-ID: <20080430113933.14651edc@i1501.lan.towertech.it> (raw)
In-Reply-To: <1209380882.25560.127.camel@pmac.infradead.org>

On Mon, 28 Apr 2008 12:08:02 +0100
David Woodhouse <dwmw2@infradead.org> wrote:

> 
> I suspect that actually covers quite a lot of devices, but the RTC class
> could happily cope with other types of devices too -- just giving an

 If we are talking about milliseconds I think a suspect is not enough.


> infrastructure for them to be notified when NTP comes into sync
> (cf. notify_cmos_timer()) should be sufficient, and after that we can
> leave them to their own devices. We should probably provide the
> common-case 'call me at half past the second' functionality too, though,
> rather than expecting each driver to implement it for itself.

 A common thing is something that can be used on a lot of devices. Given that
 the rtc class support more than 20 drivers and probably more than 30 devices
 and that this "half second" functionality is eventually useful only on one
 I won't call it common.


> >  And I suspect that the RTC of most PCs is pretty useless for
> >  any serious NTP related use.
> 
> RTCs in computers are notoriously unreliable. All the more reason to
> want them updated, when the computer is deriving its time sync from
> elsewhere.


 The hardware RTC does not need to be updated every time the system
 clock is. In fact, as long as the system is running, it's pretty useless.

 Most distributions update the RTC from the system timer on shutdown.

 This kind of update will also be unreliable on any not-memory-mapped
 RTC (think of SPI and I2C) .

 My opinion is that such NTP mode could be removed from the kernel/not
 implemented in rtc class without loosing anything.

 However, there should still be a way to tell the kernel/userspace
 that it is being kept in sync with a reliable time source, but that's
 not related to RTC.


-- 

 Best regards,

 Alessandro Zummo,
  Tower Technologies - Torino, Italy

  http://www.towertech.it


       reply	other threads:[~2008-04-30  9:39 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <18449.48370.434716.293552@cargo.ozlabs.ibm.com>
     [not found] ` <20080428102629.6e702137@i1501.lan.towertech.it>
     [not found]   ` <1209371478.25560.114.camel@pmac.infradead.org>
     [not found]     ` <20080428105245.5c9fff85@i1501.lan.towertech.it>
     [not found]       ` <1209378191.25560.121.camel@pmac.infradead.org>
     [not found]         ` <20080428124033.70e78dcf@i1501.lan.towertech.it>
     [not found]           ` <1209380882.25560.127.camel@pmac.infradead.org>
2008-04-30  9:39             ` Alessandro Zummo [this message]
2008-04-30 17:34               ` David Woodhouse
2008-04-30 20:29                 ` Alessandro Zummo

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=20080430113933.14651edc@i1501.lan.towertech.it \
    --to=alessandro.zummo@towertech.it \
    --cc=dwmw2@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=paulus@samba.org \
    --cc=rtc-linux@googlegroups.com \
    /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

Powered by JetHome