linux-kernel - Re: [RFC -tip] perf, x86: Add PERF_COUNT_HW_NMI

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <BANLkTink5nOTqvqQYr4kx0fnBoOpoq7CiQ@mail.gmail.com>
Date:	Tue, 21 Jun 2011 19:10:05 +0200
From:	Stephane Eranian <eranian@...gle.com>
To:	Cyrill Gorcunov <gorcunov@...il.com>
Cc:	Peter Zijlstra <a.p.zijlstra@...llo.nl>,
	Don Zickus <dzickus@...hat.com>, Ingo Molnar <mingo@...e.hu>,
	Lin Ming <ming.m.lin@...el.com>,
	Arnaldo Carvalho de Melo <acme@...hat.com>,
	Frederic Weisbecker <fweisbec@...il.com>,
	Vince Weaver <vweaver1@...s.utk.edu>,
	lkml <linux-kernel@...r.kernel.org>
Subject: Re: [RFC -tip] perf, x86: Add PERF_COUNT_HW_NMI_WATCHDOG event v2

On Tue, Jun 21, 2011 at 6:48 PM, Cyrill Gorcunov <gorcunov@...il.com> wrote:
> On Tue, Jun 21, 2011 at 06:20:56PM +0200, Stephane Eranian wrote:
>> Cyrill,
>> [repost because of MIME crap]
>
> I saw first as well, sorry for delay, my isp is broken today :/
>
>>
>> I admit I don't quite understand how this patch works around
>> the limitation. In the end you are still going to program some cycle
>> event into a P4 PMU register. So how it is going to free more counters
>> for regular users?
>>
>>
>
>  The key here is that we use that named non-sleeping ticks (as oprofile
> did) for nmi-watchdog and it allows us to free "cpu-cycles" counter for
> user needs. Of course we pick up one counter for this but it doesn't intersect
> with "cpu-cycles" counter (because counters are grouped and can count only
> specified events in each group). Stepane, should I post more details?
>
Ok, so you're using a different PMU event for the watchdog. But then, in this
case why not simply change kernel/watchdog.c to hardcode something specific
for P4, i.e, change the wd_hw_attr.config/type fields to match the
event you want.
You could create a arch specific callback to setup config/type.

I don't think it makes sense to expose yet another generic PMU event, especially
given the name you gave to it and what it actually does. People might wonder
what good is that for?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/