From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756669AbcDDUmN (ORCPT ); Mon, 4 Apr 2016 16:42:13 -0400 Received: from mga11.intel.com ([192.55.52.93]:45356 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756492AbcDDUmF (ORCPT ); Mon, 4 Apr 2016 16:42:05 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,442,1455004800"; d="scan'208";a="951519410" From: Andi Kleen To: peterz@infradead.org, acme@kernel.org Cc: jolsa@kernel.org, linux-kernel@vger.kernel.org, mingo@kernel.org, Andi Kleen Subject: [PATCH 01/11] x86, perf: Support sysfs files depending on SMT status Date: Mon, 4 Apr 2016 13:41:44 -0700 Message-Id: <1459802514-31501-2-git-send-email-andi@firstfloor.org> X-Mailer: git-send-email 2.5.5 In-Reply-To: <1459802514-31501-1-git-send-email-andi@firstfloor.org> References: <1459802514-31501-1-git-send-email-andi@firstfloor.org> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Andi Kleen Add a way to show different sysfs events attributes depending on HyperThreading is on or off. This is difficult to determine early at boot, so we just do it dynamically when the sysfs attribute is read. v2: Compute HT status only once in CPU online/offline hooks. Signed-off-by: Andi Kleen --- arch/x86/events/core.c | 35 +++++++++++++++++++++++++++++++++++ arch/x86/events/perf_event.h | 19 +++++++++++++++++++ include/linux/perf_event.h | 7 +++++++ 3 files changed, 61 insertions(+) diff --git a/arch/x86/events/core.c b/arch/x86/events/core.c index 002b2eadd600..4a9523426990 100644 --- a/arch/x86/events/core.c +++ b/arch/x86/events/core.c @@ -1477,6 +1477,7 @@ x86_pmu_notifier(struct notifier_block *self, unsigned long action, void *hcpu) unsigned int cpu = (long)hcpu; struct cpu_hw_events *cpuc = &per_cpu(cpu_hw_events, cpu); int i, ret = NOTIFY_OK; + bool ht_on; switch (action & ~CPU_TASKS_FROZEN) { case CPU_UP_PREPARE: @@ -1496,6 +1497,7 @@ x86_pmu_notifier(struct notifier_block *self, unsigned long action, void *hcpu) kfree(cpuc->kfree_on_online[i]); cpuc->kfree_on_online[i] = NULL; } + x86_pmu.ht_on = cpumask_weight(topology_sibling_cpumask(cpu)) > 1; break; case CPU_DYING: @@ -1507,6 +1509,15 @@ x86_pmu_notifier(struct notifier_block *self, unsigned long action, void *hcpu) case CPU_DEAD: if (x86_pmu.cpu_dead) x86_pmu.cpu_dead(cpu); + /* Recompute HT state for all CPUs on offline */ + ht_on = false; + for_each_online_cpu (cpu) { + if (cpumask_weight(topology_sibling_cpumask(cpu)) > 1) { + ht_on = true; + break; + } + } + x86_pmu.ht_on = ht_on; break; default: @@ -1616,6 +1627,30 @@ ssize_t events_sysfs_show(struct device *dev, struct device_attribute *attr, cha } EXPORT_SYMBOL_GPL(events_sysfs_show); +ssize_t events_ht_sysfs_show(struct device *dev, struct device_attribute *attr, + char *page) +{ + struct perf_pmu_events_ht_attr *pmu_attr = + container_of(attr, struct perf_pmu_events_ht_attr, attr); + + /* + * Report conditional events depending on Hyper-Threading. + * + * This is overly conservative as usually the HT special + * handling is not needed if the other CPU thread is idle. + * + * Note this does not (cannot) handle the case when thread + * siblings are invisible, for example with virtualization + * if they are owned by some other guest. The user tool + * has to re-read when a thread sibling gets onlined later. + */ + + return sprintf(page, "%s", + x86_pmu.ht_on ? + pmu_attr->event_str_ht : + pmu_attr->event_str_noht); +} + EVENT_ATTR(cpu-cycles, CPU_CYCLES ); EVENT_ATTR(instructions, INSTRUCTIONS ); EVENT_ATTR(cache-references, CACHE_REFERENCES ); diff --git a/arch/x86/events/perf_event.h b/arch/x86/events/perf_event.h index a6771e2303d2..86538527def3 100644 --- a/arch/x86/events/perf_event.h +++ b/arch/x86/events/perf_event.h @@ -617,6 +617,11 @@ struct x86_pmu { * Intel host/guest support (KVM) */ struct perf_guest_switch_msr *(*guest_get_msrs)(int *nr); + + /* + * Hyper Threading on? + */ + bool ht_on; }; struct x86_perf_task_context { @@ -662,6 +667,14 @@ static struct perf_pmu_events_attr event_attr_##v = { \ .event_str = str, \ }; +#define EVENT_ATTR_STR_HT(_name, v, noht, ht) \ +static struct perf_pmu_events_ht_attr event_attr_##v = { \ + .attr = __ATTR(_name, 0444, events_ht_sysfs_show, NULL),\ + .id = 0, \ + .event_str_noht = noht, \ + .event_str_ht = ht, \ +} + extern struct x86_pmu x86_pmu __read_mostly; static inline bool x86_pmu_has_lbr_callstack(void) @@ -928,6 +941,12 @@ int p6_pmu_init(void); int knc_pmu_init(void); +ssize_t events_sysfs_show(struct device *dev, struct device_attribute *attr, + char *page); + +ssize_t events_ht_sysfs_show(struct device *dev, struct device_attribute *attr, + char *page); + static inline int is_ht_workaround_enabled(void) { return !!(x86_pmu.flags & PMU_FL_EXCL_ENABLED); diff --git a/include/linux/perf_event.h b/include/linux/perf_event.h index 4065ca2d7149..e02ab677ab3c 100644 --- a/include/linux/perf_event.h +++ b/include/linux/perf_event.h @@ -1170,6 +1170,13 @@ struct perf_pmu_events_attr { const char *event_str; }; +struct perf_pmu_events_ht_attr { + struct device_attribute attr; + u64 id; + const char *event_str_ht; + const char *event_str_noht; +}; + ssize_t perf_event_sysfs_show(struct device *dev, struct device_attribute *attr, char *page); -- 2.5.5