From: "Rafael J. Wysocki" <rjw@rjwysocki.net>
To: Thomas Gleixner <tglx@linutronix.de>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
Mark Salyzyn <salyzyn@android.com>,
Ingo Molnar <mingo@kernel.org>,
Peter Zijlstra <peterz@infradead.org>,
John Stultz <john.stultz@linaro.org>, Peter Anvin <hpa@zytor.com>,
Michael Kerrisk <mtk.manpages@gmail.com>,
Prarit Bhargava <prarit@redhat.com>,
Steven Rostedt <rostedt@goodmis.org>, Pavel Machek <pavel@ucw.cz>,
Kevin Easton <kevin@guarana.org>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
Petr Mladek <pmladek@suse.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
SergeySenozhatsky <sergey.senozhatsky@gmail.com>,
Jonathan Corbet <corbet@lwn.net>,
linux-tip-commits@vger.kernel.org, Mike Galbraith <efault@gmx.de>
Subject: Re: [tip:timers/urgent] Revert: Unify CLOCK_MONOTONIC and CLOCK_BOOTTIME
Date: Thu, 26 Apr 2018 10:00:22 +0200 [thread overview]
Message-ID: <1937172.K7anUBJQAz@aspire.rjw.lan> (raw)
In-Reply-To: <2814513.Yf7f2L7MUC@aspire.rjw.lan>
On Wednesday, April 25, 2018 11:28:27 PM CEST Rafael J. Wysocki wrote:
> On Wednesday, April 25, 2018 9:09:29 PM CEST Thomas Gleixner wrote:
> > On Wed, 25 Apr 2018, Linus Torvalds wrote:
> > > On Wed, Apr 25, 2018 at 6:45 AM, tip-bot for Thomas Gleixner
> > > <tipbot@zytor.com> wrote:
> > > >
> > > > As stated in the pull request for the unification of CLOCK_MONOTONIC and
> > > > CLOCK_BOOTTIME, it was clear that we might have to revert the change.
> > >
> > > I'm ok with the revert., but I do want the revert message to talk
> > > about *exactly* what broke.
> > >
> > > I don't see the reported watchdog things, and I run systemd and I ran
> > > these patches.
> >
> > With suspend/resume?
> >
> > > So it's not just "systemd uses this for a watchdog". It must be
> > > something specific that made it trigger for some people and not
> > > others, and it's that specific thing that we should document so that
> > > we know why this simplification failed.
> >
> > Fair enough. Do the reporters have more information or do I have to chase
> > it down myself?
>
> Well, if CLOCK_MONOTONIC did not count when the system was suspended and now
> it does that due to the change in question, then this is a significant
> difference in behavior likely to confuse things even in principle.
FWIW, here's a list of issues reported and narrowed down to the commit
in question so far I could find:
* systemd kills daemons on resume, after >WatchdogSec seconds
of suspending (Genki Sky). [Verified that that's because systemd uses
CLOCK_MONOTONIC and expects it to not include the suspend time.]
* systemd-journald misbehaves after resume:
systemd-journald[7266]: File /var/log/journal/016627c3c4784cd4812d4b7e96a34226/system.journal corrupted or uncleanly shut down, renaming and replacing.
( Mike Galbraith).
* NetworkManager reports "networking disabled" and networking is broken
after resume 50% of the time (Pavel). [May be because of systemd.]
* MATE desktop dims the display and starts the screensaver right after
system resume (Pavel).
* Full system hang during resume (me). [May be due to systemd or NM or both.]
I'm not sure if all of the above can be connected to one specific thing (other
than the fact that things get confused if they expect CLOCK_MONOTONIC to not
include system suspend time).
next prev parent reply other threads:[~2018-04-26 8:00 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <tip-757d5a9b3052a4eebd029f207f1e1df59bef7b7d@git.kernel.org>
2018-04-25 17:31 ` Linus Torvalds
2018-04-25 19:09 ` Thomas Gleixner
2018-04-25 19:41 ` Linus Torvalds
2018-04-25 20:19 ` Pavel Machek
2018-04-26 6:53 ` Rafael J. Wysocki
2018-04-26 7:19 ` Pavel Machek
2018-04-26 7:26 ` Rafael J. Wysocki
2018-04-25 20:57 ` Rafael J. Wysocki
2018-04-25 22:41 ` Steven Rostedt
2018-04-26 6:48 ` Rafael J. Wysocki
2018-04-25 21:28 ` Rafael J. Wysocki
2018-04-26 8:00 ` Rafael J. Wysocki [this message]
2018-04-26 8:53 ` Thomas Gleixner
2018-04-26 20:38 ` Pavel Machek
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=1937172.K7anUBJQAz@aspire.rjw.lan \
--to=rjw@rjwysocki.net \
--cc=corbet@lwn.net \
--cc=dmitry.torokhov@gmail.com \
--cc=efault@gmx.de \
--cc=hpa@zytor.com \
--cc=john.stultz@linaro.org \
--cc=kevin@guarana.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tip-commits@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=mtk.manpages@gmail.com \
--cc=pavel@ucw.cz \
--cc=peterz@infradead.org \
--cc=pmladek@suse.com \
--cc=prarit@redhat.com \
--cc=rostedt@goodmis.org \
--cc=salyzyn@android.com \
--cc=sergey.senozhatsky@gmail.com \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.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
Powered by JetHome