From: dzickus@redhat.com (Don Zickus)
To: linux-arm-kernel@lists.infradead.org
Subject: mpcore watchdogs questions
Date: Fri, 3 Dec 2010 09:58:03 -0500 [thread overview]
Message-ID: <20101203145803.GQ18100@redhat.com> (raw)
In-Reply-To: <4CF8F702.1070502@stericsson.com>
On Fri, Dec 03, 2010 at 02:56:18PM +0100, Per Fransson wrote:
> Hi,
>
> I have some questions regarding the ARM mpcore watchdogs and the kernel
> watchdog API in the case of a Cortex-A9. These local watchdogs
> have two properties which appear to make them less well-suited to the
> existing framework:
>
> 1) Their clocking is tied to that of the cores, in the sense that the WDs are
> unclocked if the interrupt controller is. There's nothing we can do
> about this, but it does mean that they can't be used to watch over a
> sleeping system and therefore that we don't want the user space kicker to
> be driven by a timer which will cause a wake-up - not when power
> management is an issue. In the kernel there are deferred timers to use
> for these cases, but the kicker doesn't live there.
>
> 2) They are local to each core, which gives us at least these alternatives:
>
> * Use only one of them and...
> - set the affinity of the user space kicker to the corresponding core, or
> - let the kicker migrate, but get the message to the correct core in
> kernel space through IPI
>
> * Use all of them and...
> - One user space kicker per core, or
> - One user space kicker, but with "cyclic affinity", or
> - One user-space kicker, but each kick causes all
> the cores to get the message in kernel space,
> again using IPIs
>
> All of the above assumes these local watchdogs should be shoe-horned into the
> existing framework in the first place. Should they? Another alternative is to
> somehow use them to watch over the lockup detectors in
>
> kernel/{softlockup,watchdog}.c
>
> Maybe there are other options as well.
I am not entirely sure what you are looking for, but the
kernel/{softlockup,watchdog}.c takes a different approach to a normal
watchdog. Normal watchdogs have somebody kick them so they stay asleep,
with the intention that if they wake up bad things happened.
The hard/soft lockup detector will always wake up at a periodic rate and
check to see if they system has progressed since the last check.
Unfortunately, because of its periodic rate, it will cause wakeup events
(though every 60 seconds isn't that bad is it? :-) ), except for the
hardlockup case which uses the NMI. That is tied to cpu activity and
won't fire an NMI if the cpu is sleeping.
There have been talks about trying to tie the hrtimer and the kthread to
the power management layer with the idea that if the system is sleeping
there is no need to increment interrupt stats (using hrtimer) or check
process usage (using the kthread). That would cut down the number of
wakeups.
Cheers,
Don
next prev parent reply other threads:[~2010-12-03 14:58 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-03 13:56 mpcore watchdogs questions Per Fransson
2010-12-03 14:58 ` Don Zickus [this message]
2010-12-06 9:34 ` Per Fransson
2010-12-06 13:56 ` Don Zickus
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=20101203145803.GQ18100@redhat.com \
--to=dzickus@redhat.com \
--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 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).