All of lore.kernel.org
 help / color / mirror / Atom feed
From: linux@armlinux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] rtc: Allow rtc drivers to specify the tv_nsec value for ntp
Date: Tue, 28 Nov 2017 10:20:25 +0000	[thread overview]
Message-ID: <20171128102025.GT31757@n2100.armlinux.org.uk> (raw)
In-Reply-To: <20171127193135.GA21126@piout.net>

On Mon, Nov 27, 2017 at 08:31:35PM +0100, Alexandre Belloni wrote:
> On 27/11/2017 at 18:53:52 +0000, Russell King - ARM Linux wrote:
> > You are, yet again, wrong.
> > 
> > I am in a position to make the comment because it was me who identified
> > the problem, put in the hours to work on, develop and extensively test
> > Jason's patch.  So, it's partly my time that you seem to be wasting,
> > and that gives me every right to complain at this point.
> > 
> > You, on the other hand, were copied with every single email, and did
> > nothing to discuss the issue except for the "easy" bits when I posted
> > a relatively smaller patch - but you ignored the bigger issue.
> 
> And this is exactly what you do with other people patches/time when you
> don't like their changes.
> You simply ignore the patch series until they go away.

That is not intentional.

> I would really expect people merging code in any subsystem to wait for
> the ack of the maintainer of that subsystem.
> 
> I didn't complain about any missing email addresses, I said the RTC ML
> was not copied but that is didn't matter.

A mailing list is an email address.

> You're not even happy about the patch that was merged because it was the
> wrong one!

That's not what I said.  I was concerned that an _old_ patch had been
merged, but as it turns out, it was not an old patch, but a newer
patch that I'd missed.

> > And now that someone dare criticise your abilities, you decide to revert
> > the change and restore Linux back to a crippled state.
> > 
> > Honestly, I don't _care_ if you revert it and if you want to cripple
> > the kernel as a result in regards to this issue, I can carry the patch
> > ad infinitum, no skin off my back.  You're only going to be hurting
> > yourself and other people through your spite by doing that revert.
> > 
> > I suggest you take a good long hard look at what you're about to do and
> > ask whether you are being reasonable, given that it's taken you over
> > two months whole months to raise any _technical_ issues with the approach
> > that Jason and myself came up with.
> > 
> 
> What I don't get is that it has been broken for almost 5 years and now
> you seem to think it has to be fixed urgently.

Now you're making crap up.  I've not made any comment about this being
something that needs fixing urgently.  In fact, as I've said several
times already, I really don't care what you do with the mainline kernel,
because I have a fix here locally that I intend to use and maintain
into the future.  For me, the issue is fixed and resolved, and I intend
to spend no further time developing any fixes for it.

My comments are about the _two months_ its taken to get to the stage of
finding out that you don't like the approach.

The result of this is, most likely, one of:
1. you'll revert the patch (which, incidentally, has no real effect until
   my other patches get merged) and everyone will either be stuck with a
   kernel that sets their RTC time wrong when they have NTP installed

2. you'll remove the systohc code, people won't realise that their kernel
   no longer sync's the RTC time, and systems that are on the Internet
   (eg, providing stratum 1 NTP servers) will be unreliable if they are
   rebooted even more so than they are today.

> On my side, I want to take the opportunity to think about systohc before
> adding an ABI that we will maybe regret later.

Fine, but bear in mind that I don't care anymore, because I have
perfectly good fixes locally, and I'm not wasting any further time
in this issue.

While you think about it, I would encourage anyone who wishing to run
or running a public NTP server using rtclib to ditch Linux and use
FreeBSD instead to avoid their NTP server supplying false time, even
when synchronised to a reference clock.  A reboot of such a server
will supply false time for a period of a few hours depending on how
far the time is out (which can be up to a couple of seconds) after
such an event.  In such a scenario, NTP will not step-correct the
time but will gradually slew it instead.

This goes for systems that some ISPs deploy as stratum 1 NTP servers,
which today tend to be Raspberry Pis.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 8.8Mbps down 630kbps up
According to speedtest.net: 8.21Mbps down 510kbps up

  reply	other threads:[~2017-11-28 10:20 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-13 17:54 [PATCH] rtc: Allow rtc drivers to specify the tv_nsec value for ntp Jason Gunthorpe
2017-11-23  9:54 ` Alexandre Belloni
2017-11-23 11:23   ` Russell King - ARM Linux
2017-11-23 12:04     ` Alexandre Belloni
2017-11-23 12:04       ` Alexandre Belloni
2017-11-23 12:53       ` Russell King - ARM Linux
2017-11-23 12:53         ` Russell King - ARM Linux
2017-11-24  0:13         ` J William Piggott
2017-11-24  0:13           ` J William Piggott
2017-11-27 20:18         ` Alexandre Belloni
2017-11-27 20:18           ` Alexandre Belloni
2017-11-27 20:29           ` Jason Gunthorpe
2017-11-27 20:29             ` Jason Gunthorpe
2017-11-28 10:03           ` Russell King - ARM Linux
2017-11-28 10:03             ` Russell King - ARM Linux
2017-11-23 15:04   ` Jason Gunthorpe
2017-11-23 15:36     ` Alexandre Belloni
2017-11-23 16:10       ` Russell King - ARM Linux
2017-11-23 16:25         ` Russell King - ARM Linux
2017-11-27 18:48         ` Alexandre Belloni
2017-11-23 16:33       ` Jason Gunthorpe
2017-11-27 15:43     ` Russell King - ARM Linux
2017-11-27 17:43       ` Russell King - ARM Linux
2017-11-27 18:59         ` Jason Gunthorpe
2017-11-27 17:35   ` John Stultz
2017-11-27 17:52     ` Russell King - ARM Linux
2017-11-27 18:44       ` Alexandre Belloni
2017-11-27 18:53         ` Russell King - ARM Linux
2017-11-27 19:18           ` Russell King - ARM Linux
2017-11-27 19:31           ` Alexandre Belloni
2017-11-28 10:20             ` Russell King - ARM Linux [this message]
2017-11-30 19:39               ` Alexandre Belloni
2017-11-30 19:39               ` Alexandre Belloni
2017-11-30 19:43                 ` Alexandre Belloni

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=20171128102025.GT31757@n2100.armlinux.org.uk \
    --to=linux@armlinux.org.uk \
    --cc=linux-arm-kernel@lists.infradead.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.