| 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
| ||
|
Message-ID: <4DB736CB.6040109@gmail.com>
Date: Wed, 27 Apr 2011 01:19:07 +0400
From: Cyrill Gorcunov <gorcunov@...il.com>
To: Vince Weaver <vweaver1@...s.utk.edu>
CC: Peter Zijlstra <peterz@...radead.org>, Ingo Molnar <mingo@...e.hu>,
Arnaldo Carvalho de Melo <acme@...radead.org>,
linux-kernel@...r.kernel.org, Andi Kleen <ak@...ux.intel.com>,
Stephane Eranian <eranian@...il.com>,
Lin Ming <ming.m.lin@...el.com>,
Arnaldo Carvalho de Melo <acme@...hat.com>,
Thomas Gleixner <tglx@...utronix.de>,
Don Zickus <dzickus@...hat.com>
Subject: Re: [PATCH 1/1] perf tools: Add missing user space support for config1/config2
On 04/27/2011 12:33 AM, Vince Weaver wrote:
...
>
> I spent a lot of time trying to fix P4 support back in the 2.6.35 days.
> I only have so much time to spend on this stuff.
>
> When people complain about p4 support, I direct them to Cyrill et al. I
> can't force them to become kernel developers. Usually they want immediate
> results, which they can get with perfctr.
>
Vince I've not read the whole thread so no idea what is all about, but if you
have some p4 machines and have some will to help -- mind to test the patch below,
it should fix nmi-watchdog and cycles conflict. It's utter raw RFC (and i know there
is a nit i should update) but still might be interesting to see the results.
Untested.
--
perf, x86: P4 PMU -- Introduce alternate events v3
Alternate events are used to increase perf subsystem counter usage.
In general the idea is to find an "alternate" event (if there is
one) which do count the same magnitude as former event but use
different counter allowing them to run simultaneously with original
event.
Signed-off-by: Cyrill Gorcunov <gorcunov@...nvz.org>
---
arch/x86/include/asm/perf_event_p4.h | 6 ++
arch/x86/kernel/cpu/perf_event_p4.c | 74 ++++++++++++++++++++++++++++++++++-
2 files changed, 78 insertions(+), 2 deletions(-)
Index: linux-2.6.git/arch/x86/include/asm/perf_event_p4.h
=====================================================================
--- linux-2.6.git.orig/arch/x86/include/asm/perf_event_p4.h
+++ linux-2.6.git/arch/x86/include/asm/perf_event_p4.h
@@ -36,6 +36,10 @@
#define P4_ESCR_T1_OS 0x00000002U
#define P4_ESCR_T1_USR 0x00000001U
+#define P4_ESCR_USR_MASK \
+ (P4_ESCR_T0_OS | P4_ESCR_T0_USR | \
+ P4_ESCR_T1_OS | P4_ESCR_T1_USR)
+
#define P4_ESCR_EVENT(v) ((v) << P4_ESCR_EVENT_SHIFT)
#define P4_ESCR_EMASK(v) ((v) << P4_ESCR_EVENTMASK_SHIFT)
#define P4_ESCR_TAG(v) ((v) << P4_ESCR_TAG_SHIFT)
@@ -839,5 +843,7 @@ enum P4_PEBS_METRIC {
* 31: reserved (HT thread)
*/
+#define P4_INVALID_CONFIG (u64)~0
+
#endif /* PERF_EVENT_P4_H */
Index: linux-2.6.git/arch/x86/kernel/cpu/perf_event_p4.c
=====================================================================
--- linux-2.6.git.orig/arch/x86/kernel/cpu/perf_event_p4.c
+++ linux-2.6.git/arch/x86/kernel/cpu/perf_event_p4.c
@@ -609,6 +609,31 @@ static u64 p4_general_events[PERF_COUNT_
p4_config_pack_cccr(P4_CCCR_EDGE | P4_CCCR_COMPARE),
};
+/*
+ * Alternate events allow us to find substitution for an event if
+ * it's already borrowed, so they may be considered as event aliases.
+ */
+struct p4_alt_event {
+ unsigned int event;
+ u64 config;
+} p4_alternate_events[]= {
+ {
+ .event = P4_EVENT_GLOBAL_POWER_EVENTS,
+ .config =
+ p4_config_pack_escr(P4_ESCR_EVENT(P4_EVENT_EXECUTION_EVENT) |
+ P4_ESCR_EMASK_BIT(P4_EVENT_EXECUTION_EVENT, NBOGUS0) |
+ P4_ESCR_EMASK_BIT(P4_EVENT_EXECUTION_EVENT, NBOGUS1) |
+ P4_ESCR_EMASK_BIT(P4_EVENT_EXECUTION_EVENT, NBOGUS2) |
+ P4_ESCR_EMASK_BIT(P4_EVENT_EXECUTION_EVENT, NBOGUS3) |
+ P4_ESCR_EMASK_BIT(P4_EVENT_EXECUTION_EVENT, BOGUS0) |
+ P4_ESCR_EMASK_BIT(P4_EVENT_EXECUTION_EVENT, BOGUS1) |
+ P4_ESCR_EMASK_BIT(P4_EVENT_EXECUTION_EVENT, BOGUS2) |
+ P4_ESCR_EMASK_BIT(P4_EVENT_EXECUTION_EVENT, BOGUS3)) |
+ p4_config_pack_cccr(P4_CCCR_THRESHOLD(15) | P4_CCCR_COMPLEMENT |
+ P4_CCCR_COMPARE),
+ },
+};
+
static struct p4_event_bind *p4_config_get_bind(u64 config)
{
unsigned int evnt = p4_config_unpack_event(config);
@@ -620,6 +645,18 @@ static struct p4_event_bind *p4_config_g
return bind;
}
+static u64 p4_find_alternate_config(unsigned int evnt)
+{
+ unsigned int i;
+
+ for (i = 0; i < ARRAY_SIZE(p4_alternate_events); i++) {
+ if (evnt == p4_alternate_events[i].event)
+ return p4_alternate_events[i].config;
+ }
+
+ return P4_INVALID_CONFIG;
+}
+
static u64 p4_pmu_event_map(int hw_event)
{
struct p4_event_bind *bind;
@@ -1133,8 +1170,41 @@ static int p4_pmu_schedule_events(struct
}
cntr_idx = p4_next_cntr(thread, used_mask, bind);
- if (cntr_idx == -1 || test_bit(escr_idx, escr_mask))
- goto done;
+ if (cntr_idx == -1 || test_bit(escr_idx, escr_mask)) {
+
+ /*
+ * So the former event already accepted to run
+ * and the only way to success here is to use
+ * an alternate event.
+ */
+ const u64 usr_mask = p4_config_pack_escr(P4_ESCR_USR_MASK);
+ u64 alt_config;
+ unsigned int event;
+
+ event = p4_config_unpack_event(hwc->config);
+ alt_config = p4_find_alternate_config(event);
+
+ if (alt_config == P4_INVALID_CONFIG)
+ goto done;
+
+ bind = p4_config_get_bind(alt_config);
+ escr_idx = p4_get_escr_idx(bind->escr_msr[thread]);
+ if (unlikely(escr_idx == -1))
+ goto done;
+
+ cntr_idx = p4_next_cntr(thread, used_mask, bind);
+ if (cntr_idx == -1 || test_bit(escr_idx, escr_mask))
+ goto done;
+
+ /*
+ * This is a destructive operation we're going
+ * to make. We substitute the former config with
+ * alternate one to continue tracking it after.
+ * Be carefull and don't kill the custom bits
+ * in the former config.
+ */
+ hwc->config = (hwc->config & usr_mask) | alt_config;
+ }
p4_pmu_swap_config_ts(hwc, cpu);
if (assign)
--
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/
Powered by blists - more mailing lists