From: Lin Ming <ming.m.lin@intel.com>
To: Ingo Molnar <mingo@elte.hu>,
Peter Zijlstra <peterz@infradead.org>,
Don Zickus <dzickus@redhat.com>,
Cyrill Gorcunov <gorcunov@gmail.com>,
robert.richter@amd.com, fweisbec@gmail.com
Cc: linux-kernel@vger.kernel.org, "Huang, Ying" <ying.huang@intel.com>
Subject: A question of perf NMI handler
Date: Wed, 04 Aug 2010 17:21:10 +0800 [thread overview]
Message-ID: <1280913670.20797.179.camel@minggr.sh.intel.com> (raw)
With nmi_watchdog enabled, perf_event_nmi_handler always return
NOTIFY_STOP(active_events > 0), and the notifier call chain will not
call further.
If it was not perf NMI, does the perf nmi handler may stop the real NMI
handler get called because NOTIFY_STOP is returned??
static int __kprobes
perf_event_nmi_handler(struct notifier_block *self,
unsigned long cmd, void *__args)
{
struct die_args *args = __args;
struct pt_regs *regs;
if (!atomic_read(&active_events)) ===> With nmi_watchdog enabled, active_events > 0
return NOTIFY_DONE;
switch (cmd) {
case DIE_NMI:
case DIE_NMI_IPI:
break;
default:
return NOTIFY_DONE;
}
regs = args->regs;
apic_write(APIC_LVTPC, APIC_DM_NMI);
/*
* Can't rely on the handled return value to say it was our NMI, two
* events could trigger 'simultaneously' raising two back-to-back NMIs.
*
* If the first NMI handles both, the latter will be empty and daze
* the CPU.
*/
x86_pmu.handle_irq(regs);
return NOTIFY_STOP;
}
Thanks,
Lin Ming
next reply other threads:[~2010-08-04 9:21 UTC|newest]
Thread overview: 85+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-04 9:21 Lin Ming [this message]
2010-08-04 9:50 ` Peter Zijlstra
2010-08-04 10:01 ` Robert Richter
2010-08-04 10:24 ` Peter Zijlstra
2010-08-04 10:29 ` Robert Richter
2010-08-04 14:00 ` Don Zickus
2010-08-04 14:11 ` Peter Zijlstra
2010-08-04 14:52 ` Don Zickus
2010-08-04 15:02 ` Peter Zijlstra
2010-08-04 15:18 ` Cyrill Gorcunov
2010-08-04 15:50 ` Don Zickus
2010-08-04 16:10 ` Cyrill Gorcunov
2010-08-04 16:20 ` Don Zickus
2010-08-04 16:39 ` Cyrill Gorcunov
2010-08-04 18:48 ` Robert Richter
2010-08-04 19:22 ` Andi Kleen
2010-08-04 19:26 ` Cyrill Gorcunov
2010-08-06 6:52 ` Robert Richter
2010-08-06 14:21 ` Don Zickus
2010-08-09 19:48 ` [PATCH] perf, x86: try to handle unknown nmis with running perfctrs Robert Richter
2010-08-09 20:02 ` Cyrill Gorcunov
2010-08-10 7:42 ` Robert Richter
2010-08-10 16:16 ` Cyrill Gorcunov
2010-08-10 16:41 ` Robert Richter
2010-08-10 17:24 ` Cyrill Gorcunov
2010-08-10 19:05 ` Robert Richter
2010-08-10 19:24 ` Cyrill Gorcunov
2010-08-12 13:24 ` Robert Richter
2010-08-12 14:31 ` Cyrill Gorcunov
2010-08-10 20:48 ` Don Zickus
2010-08-11 2:44 ` Frederic Weisbecker
2010-08-11 11:10 ` Robert Richter
2010-08-11 12:44 ` Don Zickus
2010-08-11 14:03 ` Robert Richter
2010-08-11 14:32 ` Don Zickus
2010-08-13 4:37 ` Frederic Weisbecker
2010-08-13 8:22 ` Robert Richter
2010-08-14 1:28 ` Frederic Weisbecker
2010-08-14 2:29 ` Robert Richter
2010-08-11 12:39 ` Don Zickus
2010-08-11 3:19 ` Huang Ying
2010-08-11 12:36 ` Don Zickus
2010-08-16 14:37 ` Peter Zijlstra
2010-08-11 22:00 ` [PATCH -v2] " Robert Richter
2010-08-12 13:10 ` Robert Richter
2010-08-12 18:21 ` Don Zickus
2010-08-16 7:37 ` Robert Richter
2010-08-12 13:52 ` Don Zickus
2010-08-13 4:25 ` Frederic Weisbecker
2010-08-16 14:48 ` Peter Zijlstra
2010-08-16 16:27 ` Cyrill Gorcunov
2010-08-16 17:16 ` Robert Richter
2010-08-16 19:06 ` Cyrill Gorcunov
2010-08-16 19:13 ` Peter Zijlstra
2010-08-16 19:18 ` Cyrill Gorcunov
2010-08-16 22:55 ` Robert Richter
2010-08-17 15:23 ` Cyrill Gorcunov
2010-08-17 15:22 ` [PATCH -v3] " Robert Richter
2010-08-17 16:17 ` Cyrill Gorcunov
2010-08-19 10:45 ` Peter Zijlstra
2010-08-19 12:39 ` Robert Richter
2010-08-19 14:12 ` Don Zickus
2010-08-19 14:27 ` Peter Zijlstra
2010-08-19 15:20 ` Don Zickus
2010-08-19 17:43 ` Cyrill Gorcunov
2010-08-19 17:53 ` Peter Zijlstra
2010-08-19 21:58 ` Don Zickus
2010-08-20 8:50 ` Peter Zijlstra
2010-08-20 1:50 ` Don Zickus
2010-08-20 8:16 ` Ingo Molnar
2010-08-20 10:04 ` Peter Zijlstra
2010-08-20 10:30 ` Cyrill Gorcunov
2010-08-20 12:39 ` Don Zickus
2010-08-20 13:27 ` Ingo Molnar
2010-08-20 13:51 ` Don Zickus
2010-08-20 14:17 ` Ingo Molnar
2010-08-20 20:45 ` Cyrill Gorcunov
2010-08-24 21:48 ` Don Zickus
2010-08-20 8:36 ` Robert Richter
2010-08-20 14:17 ` [tip:perf/urgent] perf, x86: Fix handle_irq return values tip-bot for Peter Zijlstra
2010-08-20 14:17 ` [tip:perf/urgent] perf, x86: Try to handle unknown nmis with an enabled PMU tip-bot for Robert Richter
2010-08-06 15:35 ` A question of perf NMI handler Andi Kleen
2010-08-04 15:45 ` Don Zickus
2010-08-06 15:37 ` Andi Kleen
2010-08-04 13:54 ` 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=1280913670.20797.179.camel@minggr.sh.intel.com \
--to=ming.m.lin@intel.com \
--cc=dzickus@redhat.com \
--cc=fweisbec@gmail.com \
--cc=gorcunov@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=peterz@infradead.org \
--cc=robert.richter@amd.com \
--cc=ying.huang@intel.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