From: Ulrich Obergfell <uobergfe@redhat.com>
To: Peter Zijlstra <peterz@infradead.org>
Cc: Don Zickus <dzickus@redhat.com>, Jiri Olsa <jolsa@kernel.org>,
lkml <linux-kernel@vger.kernel.org>,
Ingo Molnar <mingo@kernel.org>, Michal Hocko <mhocko@suse.cz>,
Stephane Eranian <eranian@google.com>,
Andi Kleen <ak@linux.intel.com>, Paul Bunyan <pbunyan@redhat.com>
Subject: Re: [PATCH] watchdog: Fix race on single cpu boot
Date: Sun, 2 Aug 2015 07:09:13 -0400 (EDT) [thread overview]
Message-ID: <366054547.2544162.1438513753610.JavaMail.zimbra@redhat.com> (raw)
In-Reply-To: <20150801162522.GO2859@worktop.programming.kicks-ass.net>
Peter,
>> I posted the patch set here:
>>
>> https://lkml.org/lkml/2015/8/1/64
>> https://lkml.org/lkml/2015/8/1/65
>> https://lkml.org/lkml/2015/8/1/66
>> https://lkml.org/lkml/2015/8/1/67
>> https://lkml.org/lkml/2015/8/1/68
>
> If only you didn't use lkml.org links, that site is flaky beyond
> unusable.
>
> Please use lkml.kernel.org/r/$msgid, that way people actually have a
> chance of getting a working link and/or finding it in their local
> archive.
I apologize for the inconvenience. The following links redirect to marc.info:
- [PATCH 0/4] watchdog: avoid races in watchdog_nmi_{en|disable} functions
lkml.kernel.org/r/1438433365-2979-1-git-send-email-uobergfe@redhat.com
- [PATCH 1/4] watchdog: introduce watchdog_park_threads() and watchdog_unpark_threads()
lkml.kernel.org/r/1438433365-2979-2-git-send-email-uobergfe@redhat.com
- [PATCH 2/4] watchdog: introduce watchdog_suspend() and watchdog_resume()
lkml.kernel.org/r/1438433365-2979-3-git-send-email-uobergfe@redhat.com
This part of the series has already received a comment. The suggestion is
to introduce different function names to avoid confusion with the watchdog
subsystem, so I propose
lockup_detector_suspend() instead of watchdog_suspend()
lockup_detector_resume() instead of watchdog_resume()
which would be consistent with the existing name lockup_detector_init().
- [PATCH 3/4] watchdog: use park/unpark functions in update_watchdog_all_cpus()
lkml.kernel.org/r/1438433365-2979-4-git-send-email-uobergfe@redhat.com
- [PATCH 4/4] watchdog: use suspend/resume interface in fixup_ht_bug()
lkml.kernel.org/r/1438433365-2979-5-git-send-email-uobergfe@redhat.com
Regards,
Uli
prev parent reply other threads:[~2015-08-02 11:09 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-31 11:56 [PATCH] watchdog: Fix race on single cpu boot Jiri Olsa
2015-07-31 15:36 ` Don Zickus
2015-08-01 13:05 ` Ulrich Obergfell
2015-08-01 16:25 ` Peter Zijlstra
2015-08-02 11:09 ` Ulrich Obergfell [this message]
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=366054547.2544162.1438513753610.JavaMail.zimbra@redhat.com \
--to=uobergfe@redhat.com \
--cc=ak@linux.intel.com \
--cc=dzickus@redhat.com \
--cc=eranian@google.com \
--cc=jolsa@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mhocko@suse.cz \
--cc=mingo@kernel.org \
--cc=pbunyan@redhat.com \
--cc=peterz@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.