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-next>] [day] [month] [year] [list]
Message-Id: <1437723969-19558-1-git-send-email-adrian.hunter@intel.com>
Date:	Fri, 24 Jul 2015 10:46:09 +0300
From:	Adrian Hunter <adrian.hunter@...el.com>
To:	Peter Zijlstra <peterz@...radead.org>,
	Ingo Molnar <mingo@...nel.org>
Cc:	Arnaldo Carvalho de Melo <acme@...nel.org>,
	Andy Lutomirski <luto@...capital.net>,
	Vince Weaver <vincent.weaver@...ne.edu>,
	Thomas Gleixner <tglx@...utronix.de>,
	"H. Peter Anvin" <hpa@...ux.intel.com>,
	linux-kernel@...r.kernel.org, Jiri Olsa <jolsa@...hat.com>,
	Stephane Eranian <eranian@...gle.com>,
	Borislav Petkov <bp@...e.de>,
	Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
	Andi Kleen <ak@...ux.intel.com>
Subject: [PATCH RFC] perf: Let PMUs provide supplementary information

This is a follow-up to "[RFC PATCH] perf: Provide status of known PMUs" i.e.

	https://lkml.kernel.org/r/1436428080-3098-1-git-send-email-adrian.hunter@intel.com

There are 3 main changes:
	- uses debugfs not sysfs
	- doesn't require a PMU to be on the list of known PMU's
	- renamed from 'known_pmus' to 'pmu_supplementary_info'

The purpose of the changes is to make optional whether or not PMU
supplementary information is provided for a PMU, thus easing the
maintenance burden.  Maintenance is also eased by having all the
code in a separate file.

Also the status codes have changed as requested:

	[PERF_PMU_STATUS_ENABLED]	= "Enabled",
	[PERF_PMU_STATUS_ERROR]		= "Disabled: Driver error",
	[PERF_PMU_STATUS_NOT_LOADED]	= "Disabled: Driver not loaded",
	[PERF_PMU_STATUS_NOT_CONFIG]	= "Disabled: Driver not in kernel config",
	[PERF_PMU_STATUS_NOT_SUPPORTED]	= "Disabled: Not supported by the kernel",
	[PERF_PMU_STATUS_WRONG_HW]	= "Disabled: Not supported by the hardware",
	[PERF_PMU_STATUS_WRONG_VENDOR]	= "Disabled: Not supported by the hardware vendor",
	[PERF_PMU_STATUS_WRONG_ARCH]	= "Disabled: Not supported by the the architecture",
	[PERF_PMU_STATUS_UNKNOWN]	= "Disabled: Unknown status",

Example:

	$ cat /sys/kernel/debug/perf/pmu_supplementary_info/intel_pt
	Enabled

Last time it wasn't clear that a custom status message can be provided.
For example, this code snippet from Intel PT:

	if (prior_warn) {
		x86_add_exclusive(x86_lbr_exclusive_pt);
		status_msg = "PT is enabled at boot time, doing nothing";
		ret = -EBUSY;
		goto out;
	}
	...
    out:
	if (status_msg)
		pr_warn("%s\n", status_msg);
	perf_pmu_error_status("intel_pt", ret, status_msg);

Would result in:

	$ cat /sys/kernel/debug/perf/pmu_supplementary_info/intel_pt
	Disabled: Driver error - PT is enabled at boot time, doing nothing

Signed-off-by: Adrian Hunter <adrian.hunter@...el.com>
---
 arch/x86/kernel/cpu/perf_event.c           |   3 +
 arch/x86/kernel/cpu/perf_event_intel_bts.c |  20 +-
 arch/x86/kernel/cpu/perf_event_intel_pt.c  |  18 +-
 include/linux/perf_event.h                 |  45 ++++
 kernel/events/Makefile                     |   2 +-
 kernel/events/pmu_supp_info.c              | 384 +++++++++++++++++++++++++++++
 6 files changed, 462 insertions(+), 10 deletions(-)
 create mode 100644 kernel/events/pmu_supp_info.c

diff --git a/arch/x86/kernel/cpu/perf_event.c b/arch/x86/kernel/cpu/perf_event.c
index 3658de47900f..546163a42a30 100644
--- a/arch/x86/kernel/cpu/perf_event.c
+++ b/arch/x86/kernel/cpu/perf_event.c
@@ -1670,12 +1670,15 @@ static int __init init_hw_perf_events(void)
 
 	switch (boot_cpu_data.x86_vendor) {
 	case X86_VENDOR_INTEL:
+		perf_pmu_vendor("Intel");
 		err = intel_pmu_init();
 		break;
 	case X86_VENDOR_AMD:
+		perf_pmu_vendor("AMD");
 		err = amd_pmu_init();
 		break;
 	default:
+		perf_pmu_vendor("Unknown");
 		err = -ENOTSUPP;
 	}
 	if (err != 0) {
diff --git a/arch/x86/kernel/cpu/perf_event_intel_bts.c b/arch/x86/kernel/cpu/perf_event_intel_bts.c
index 54690e885759..7c397cecce65 100644
--- a/arch/x86/kernel/cpu/perf_event_intel_bts.c
+++ b/arch/x86/kernel/cpu/perf_event_intel_bts.c
@@ -511,8 +511,19 @@ static void bts_event_read(struct perf_event *event)
 
 static __init int bts_init(void)
 {
-	if (!boot_cpu_has(X86_FEATURE_DTES64) || !x86_pmu.bts)
-		return -ENODEV;
+	const char *status_msg = NULL;
+	int ret;
+
+	if (!x86_pmu.bts) {
+		ret = -ENODEV;
+		goto out;
+	}
+
+	if (!boot_cpu_has(X86_FEATURE_DTES64)) {
+		status_msg = "requires 64-bit kernel";
+		ret = -ENOTSUPP;
+		goto out;
+	}
 
 	bts_pmu.capabilities	= PERF_PMU_CAP_AUX_NO_SG | PERF_PMU_CAP_ITRACE;
 	bts_pmu.task_ctx_nr	= perf_sw_context;
@@ -525,6 +536,9 @@ static __init int bts_init(void)
 	bts_pmu.setup_aux	= bts_buffer_setup_aux;
 	bts_pmu.free_aux	= bts_buffer_free_aux;
 
-	return perf_pmu_register(&bts_pmu, "intel_bts", -1);
+	ret = perf_pmu_register(&bts_pmu, "intel_bts", -1);
+out:
+	perf_pmu_error_status("intel_bts", ret, status_msg);
+	return ret;
 }
 arch_initcall(bts_init);
diff --git a/arch/x86/kernel/cpu/perf_event_intel_pt.c b/arch/x86/kernel/cpu/perf_event_intel_pt.c
index 5505371b414d..f4aa3831f8b5 100644
--- a/arch/x86/kernel/cpu/perf_event_intel_pt.c
+++ b/arch/x86/kernel/cpu/perf_event_intel_pt.c
@@ -1122,6 +1122,7 @@ static int pt_event_init(struct perf_event *event)
 static __init int pt_init(void)
 {
 	int ret, cpu, prior_warn = 0;
+	const char *status_msg = NULL;
 
 	BUILD_BUG_ON(sizeof(struct topa) > PAGE_SIZE);
 	get_online_cpus();
@@ -1136,18 +1137,19 @@ static __init int pt_init(void)
 
 	if (prior_warn) {
 		x86_add_exclusive(x86_lbr_exclusive_pt);
-		pr_warn("PT is enabled at boot time, doing nothing\n");
-
-		return -EBUSY;
+		status_msg = "PT is enabled at boot time, doing nothing";
+		ret = -EBUSY;
+		goto out;
 	}
 
 	ret = pt_pmu_hw_init();
 	if (ret)
-		return ret;
+		goto out;
 
 	if (!pt_cap_get(PT_CAP_topa_output)) {
-		pr_warn("ToPA output is not supported on this CPU\n");
-		return -ENODEV;
+		status_msg = "ToPA output is not supported on this CPU";
+		ret = -ENODEV;
+		goto out;
 	}
 
 	if (!pt_cap_get(PT_CAP_topa_multiple_entries))
@@ -1166,6 +1168,10 @@ static __init int pt_init(void)
 	pt_pmu.pmu.setup_aux	= pt_buffer_setup_aux;
 	pt_pmu.pmu.free_aux	= pt_buffer_free_aux;
 	ret = perf_pmu_register(&pt_pmu.pmu, "intel_pt", -1);
+out:
+	if (status_msg)
+		pr_warn("%s\n", status_msg);
+	perf_pmu_error_status("intel_pt", ret, status_msg);
 
 	return ret;
 }
diff --git a/include/linux/perf_event.h b/include/linux/perf_event.h
index 0af4c8b4b6d0..13886c2d9e93 100644
--- a/include/linux/perf_event.h
+++ b/include/linux/perf_event.h
@@ -185,6 +185,22 @@ struct perf_event;
 #define PERF_PMU_CAP_ITRACE			0x20
 
 /**
+ * enum perf_pmu_status - the PMU Supplementary Information status of PMUs
+ */
+enum perf_pmu_status {
+	PERF_PMU_STATUS_ENABLED,
+	PERF_PMU_STATUS_ERROR,
+	PERF_PMU_STATUS_NOT_LOADED,
+	PERF_PMU_STATUS_NOT_CONFIG,
+	PERF_PMU_STATUS_NOT_SUPPORTED,
+	PERF_PMU_STATUS_WRONG_HW,
+	PERF_PMU_STATUS_WRONG_VENDOR,
+	PERF_PMU_STATUS_WRONG_ARCH,
+	PERF_PMU_STATUS_UNKNOWN,
+	PERF_PMU_STATUS_MAX,
+};
+
+/**
  * struct pmu - generic performance monitoring unit
  */
 struct pmu {
@@ -639,6 +655,35 @@ extern void *perf_get_aux(struct perf_output_handle *handle);
 extern int perf_pmu_register(struct pmu *pmu, const char *name, int type);
 extern void perf_pmu_unregister(struct pmu *pmu);
 
+#ifdef CONFIG_DEBUG_FS
+
+extern int perf_pmu_update_status(const char *name, enum perf_pmu_status status,
+				  const char *status_msg);
+extern int perf_pmu_error_status(const char *name, int err,
+				 const char *status_msg);
+extern void perf_pmu_vendor(const char *vendor);
+
+#else /* !CONFIG_DEBUG_FS */
+
+static inline int perf_pmu_update_status(const char *name,
+					 enum perf_pmu_status status,
+					 const char *status_msg)
+{
+	return 0;
+}
+
+static inline int perf_pmu_error_status(const char *name, int err,
+					const char *status_msg)
+{
+	return 0;
+}
+
+static inline void perf_pmu_vendor(const char *vendor)
+{
+}
+
+#endif /* CONFIG_DEBUG_FS */
+
 extern int perf_num_counters(void);
 extern const char *perf_pmu_name(void);
 extern void __perf_event_task_sched_in(struct task_struct *prev,
diff --git a/kernel/events/Makefile b/kernel/events/Makefile
index 2925188f50ea..f4ee64411af0 100644
--- a/kernel/events/Makefile
+++ b/kernel/events/Makefile
@@ -6,4 +6,4 @@ obj-y := core.o ring_buffer.o callchain.o
 
 obj-$(CONFIG_HAVE_HW_BREAKPOINT) += hw_breakpoint.o
 obj-$(CONFIG_UPROBES) += uprobes.o
-
+obj-$(CONFIG_DEBUG_FS) += pmu_supp_info.o
diff --git a/kernel/events/pmu_supp_info.c b/kernel/events/pmu_supp_info.c
new file mode 100644
index 000000000000..ac714ad09f97
--- /dev/null
+++ b/kernel/events/pmu_supp_info.c
@@ -0,0 +1,384 @@
+/*
+ * pmu_supp_info.c: PMU Supplementary Information
+ * Copyright (c) 2015, Intel Corporation.
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms and conditions of the GNU General Public License,
+ * version 2, as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
+ * more details.
+ */
+
+#include <linux/rbtree.h>
+#include <linux/slab.h>
+#include <linux/debugfs.h>
+#include <linux/perf_event.h>
+
+struct pmu_supp_info {
+	struct rb_node node;
+	const char *name;
+	const char *vendor;
+	const char *status_msg;
+	int status;
+};
+
+static struct rb_root pmu_supp_info_tree = RB_ROOT;
+static DEFINE_MUTEX(pmu_supp_info_lock);
+static struct dentry *pmu_supp_info_dir;
+
+struct known_pmu {
+	const char *name;
+	const char *vendor;
+	int status;
+};
+
+#define KNOWN_PMU(_name, _vendor, _status) {	\
+	.name   = _name,			\
+	.vendor = _vendor,			\
+	.status = _status,			\
+}
+
+#if defined(CONFIG_X86)
+#define PERF_PMU_STATUS_ARCH_X86	PERF_PMU_STATUS_UNKNOWN
+#else
+#define PERF_PMU_STATUS_ARCH_X86	PERF_PMU_STATUS_WRONG_ARCH
+#endif
+
+static struct known_pmu known_pmus[] = {
+	KNOWN_PMU("intel_pt", "Intel", PERF_PMU_STATUS_ARCH_X86),
+	KNOWN_PMU("intel_bts", "Intel", PERF_PMU_STATUS_ARCH_X86),
+	KNOWN_PMU(NULL, NULL, 0),
+};
+
+static const char *pmu_status_msg[] = {
+	[PERF_PMU_STATUS_ENABLED]	= "Enabled",
+	[PERF_PMU_STATUS_ERROR]		= "Disabled: Driver error",
+	[PERF_PMU_STATUS_NOT_LOADED]	= "Disabled: Driver not loaded",
+	[PERF_PMU_STATUS_NOT_CONFIG]	= "Disabled: Driver not in kernel config",
+	[PERF_PMU_STATUS_NOT_SUPPORTED]	= "Disabled: Not supported by the kernel",
+	[PERF_PMU_STATUS_WRONG_HW]	= "Disabled: Not supported by the hardware",
+	[PERF_PMU_STATUS_WRONG_VENDOR]	= "Disabled: Not supported by the hardware vendor",
+	[PERF_PMU_STATUS_WRONG_ARCH]	= "Disabled: Not supported by the the architecture",
+	[PERF_PMU_STATUS_UNKNOWN]	= "Disabled: Unknown status",
+};
+
+static int pmu_supp_info_show(struct seq_file *seq, void *v)
+{
+	struct pmu_supp_info *pmuinfo = seq->private;
+	const char *msg, *vendor;
+	int status;
+
+	mutex_lock(&pmu_supp_info_lock);
+
+	status = pmuinfo->status;
+	msg    = pmuinfo->status_msg;
+	vendor = pmuinfo->vendor;
+
+	mutex_unlock(&pmu_supp_info_lock);
+
+	if (status < 0 || status >= PERF_PMU_STATUS_MAX)
+		status = PERF_PMU_STATUS_UNKNOWN;
+
+	if (!msg && status == PERF_PMU_STATUS_WRONG_VENDOR && vendor) {
+		seq_printf(seq, "%s - requires %s CPU\n",
+			   pmu_status_msg[status], vendor);
+		return 0;
+	}
+
+	if (!msg) {
+		seq_printf(seq, "%s\n", pmu_status_msg[status]);
+		return 0;
+	}
+
+	seq_printf(seq, "%s - %s\n", pmu_status_msg[status], msg);
+
+	return 0;
+}
+
+static int pmu_supp_info_open(struct inode *inode, struct file *file)
+{
+	struct pmu_supp_info *pmuinfo = inode->i_private;
+
+	return single_open(file, pmu_supp_info_show, pmuinfo);
+}
+
+static const struct file_operations pmu_supp_info_fops = {
+	.open		= pmu_supp_info_open,
+	.read		= seq_read,
+	.llseek		= seq_lseek,
+	.release	= single_release,
+};
+
+static int pmu_supp_info_add_file(struct pmu_supp_info *pmuinfo)
+{
+	if (!pmu_supp_info_dir)
+		return 0;
+
+	if (!debugfs_create_file(pmuinfo->name, S_IRUGO, pmu_supp_info_dir,
+				 pmuinfo, &pmu_supp_info_fops))
+		return -ENODEV;
+
+	return 0;
+}
+
+static struct pmu_supp_info *pmu_supp_info_lookup(const char *name)
+{
+        struct rb_node **node = &(pmu_supp_info_tree.rb_node), *parent = NULL;
+	struct pmu_supp_info *pmuinfo;
+	int cmp;
+
+	if (!name)
+		return NULL;
+
+        while (*node) {
+                pmuinfo = container_of(*node, struct pmu_supp_info, node);
+                cmp = strcmp(name, pmuinfo->name);
+                parent = *node;
+                if (cmp < 0)
+                        node = &((*node)->rb_left);
+                else if (cmp > 0)
+                        node = &((*node)->rb_right);
+                else
+                        return pmuinfo;
+        }
+
+	return NULL;
+}
+
+static int pmu_supp_info_insert(struct pmu_supp_info *new_pmuinfo)
+{
+        struct rb_node **node = &(pmu_supp_info_tree.rb_node), *parent = NULL;
+	struct pmu_supp_info *pmuinfo;
+	int cmp;
+
+	if (!new_pmuinfo->name)
+		return -EINVAL;
+
+        while (*node) {
+                pmuinfo = container_of(*node, struct pmu_supp_info, node);
+                cmp = strcmp(new_pmuinfo->name, pmuinfo->name);
+                parent = *node;
+                if (cmp < 0)
+                        node = &((*node)->rb_left);
+                else if (cmp > 0)
+                        node = &((*node)->rb_right);
+                else
+                        return -EEXIST;
+        }
+
+        rb_link_node(&new_pmuinfo->node, parent, node);
+        rb_insert_color(&new_pmuinfo->node, &pmu_supp_info_tree);
+
+	return 0;
+}
+
+static struct pmu_supp_info *pmu_supp_info_alloc(const char *name)
+{
+	struct pmu_supp_info *pmuinfo;
+
+	pmuinfo = kzalloc(sizeof(struct pmu_supp_info), GFP_KERNEL);
+	if (!pmuinfo)
+		return NULL;
+
+	pmuinfo->name = kstrdup_const(name, GFP_KERNEL);
+	if (!pmuinfo->name) {
+		kfree(pmuinfo);
+		return NULL;
+	}
+
+	pmuinfo->status = PERF_PMU_STATUS_UNKNOWN;
+
+	return pmuinfo;
+}
+
+static void pmu_supp_info_free(struct pmu_supp_info *pmuinfo)
+{
+	kfree_const(pmuinfo->name);
+	kfree(pmuinfo);
+}
+
+static struct pmu_supp_info *pmu_supp_info_add(const char *name)
+{
+	struct pmu_supp_info *pmuinfo;
+	int err;
+
+	pmuinfo = pmu_supp_info_alloc(name);
+	if (!pmuinfo)
+		return ERR_PTR(-ENOMEM);
+
+	err = pmu_supp_info_insert(pmuinfo);
+	if (err)
+		goto out_free;
+
+	err = pmu_supp_info_add_file(pmuinfo);
+	if (err)
+		goto out_remove;
+
+	return pmuinfo;
+
+out_remove:
+	rb_erase(&pmuinfo->node, &pmu_supp_info_tree);
+out_free:
+	pmu_supp_info_free(pmuinfo);
+	return ERR_PTR(err);
+}
+
+int perf_pmu_update_status(const char *name, enum perf_pmu_status status,
+			   const char *status_msg)
+{
+	struct pmu_supp_info *pmuinfo;
+	int err = 0;
+
+	if (!name)
+		return -EINVAL;
+
+	mutex_lock(&pmu_supp_info_lock);
+
+	pmuinfo = pmu_supp_info_lookup(name);
+	if (!pmuinfo) {
+		pmuinfo = pmu_supp_info_add(name);
+		if (IS_ERR(pmuinfo)) {
+			err = PTR_ERR(pmuinfo);
+			goto out_unlock;
+		}
+	}
+
+	pmuinfo->status = status;
+	kfree_const(pmuinfo->status_msg);
+	pmuinfo->status_msg = kstrdup_const(status_msg, GFP_KERNEL);
+
+out_unlock:
+	mutex_unlock(&pmu_supp_info_lock);
+
+	return err;
+}
+
+int perf_pmu_error_status(const char *name, int err, const char *status_msg)
+{
+	enum perf_pmu_status status;
+
+	switch (err) {
+	case 0:
+		status = PERF_PMU_STATUS_ENABLED;
+		break;
+	case -ENODEV:
+		status = PERF_PMU_STATUS_WRONG_HW;
+		break;
+	case -ENOTSUPP:
+		status = PERF_PMU_STATUS_NOT_SUPPORTED;
+		break;
+	default:
+		status = PERF_PMU_STATUS_ERROR;
+	}
+
+	return perf_pmu_update_status(name, status, status_msg);
+}
+
+
+void perf_pmu_vendor(const char *vendor)
+{
+	struct pmu_supp_info *pmuinfo;
+	struct known_pmu *known_pmu;
+	struct rb_node *node;
+
+	if (!vendor)
+		return;
+
+	mutex_lock(&pmu_supp_info_lock);
+
+	for (known_pmu = known_pmus; known_pmu->name; known_pmu++) {
+		if (known_pmu->status == PERF_PMU_STATUS_UNKNOWN &&
+		    known_pmu->vendor && strcmp(known_pmu->vendor, vendor))
+			known_pmu->status = PERF_PMU_STATUS_WRONG_VENDOR;
+	}
+
+	for (node = rb_first(&pmu_supp_info_tree); node; node = rb_next(node)) {
+		pmuinfo = rb_entry(node, struct pmu_supp_info, node);
+		if (pmuinfo->status == PERF_PMU_STATUS_UNKNOWN &&
+		    pmuinfo->vendor && strcmp(pmuinfo->vendor, vendor)) {
+			pmuinfo->status = PERF_PMU_STATUS_WRONG_VENDOR;
+			kfree_const(pmuinfo->status_msg);
+			pmuinfo->status_msg = NULL;
+		}
+	}
+
+	mutex_unlock(&pmu_supp_info_lock);
+}
+
+static void __init pmu_supp_info_add_known_pmus(void)
+{
+	struct pmu_supp_info *pmuinfo;
+	struct known_pmu *known_pmu;
+
+	for (known_pmu = known_pmus; known_pmu->name; known_pmu++) {
+		pmuinfo = pmu_supp_info_lookup(known_pmu->name);
+		if (!pmuinfo) {
+			pmuinfo = pmu_supp_info_add(known_pmu->name);
+			if (IS_ERR(pmuinfo)) {
+				pr_warn("Failed to add pmu_supplementary_info for %s\n",
+					known_pmu->name);
+				continue;
+			}
+			pmuinfo->status = known_pmu->status;
+		}
+		if (known_pmu->vendor && !pmuinfo->vendor)
+			pmuinfo->vendor = kstrdup_const(known_pmu->vendor,
+							GFP_KERNEL);
+	}
+}
+
+static int __init pmu_supp_info_add_files(void)
+{
+	struct pmu_supp_info *pmuinfo;
+	struct rb_node *node;
+	int err;
+
+	for (node = rb_first(&pmu_supp_info_tree); node; node = rb_next(node)) {
+		pmuinfo = rb_entry(node, struct pmu_supp_info, node);
+		err = pmu_supp_info_add_file(pmuinfo);
+		if (err)
+			return err;
+	}
+
+	return 0;
+}
+
+static int __init perf_event_debugfs_init(void)
+{
+	struct dentry *root = NULL, *dentry;
+	int err;
+
+	mutex_lock(&pmu_supp_info_lock);
+
+	root = debugfs_create_dir("perf", NULL);
+	if (IS_ERR_OR_NULL(root))
+		goto out_err;
+
+	dentry = debugfs_create_dir("pmu_supplementary_info", root);
+	if (IS_ERR_OR_NULL(dentry))
+		goto out_err_remove;
+
+	pmu_supp_info_dir = dentry;
+
+	err = pmu_supp_info_add_files();
+	if (err)
+		goto out_err_remove;
+
+	pmu_supp_info_add_known_pmus();
+
+out_unlock:
+	mutex_unlock(&pmu_supp_info_lock);
+
+	return err;
+
+out_err_remove:
+	debugfs_remove_recursive(root);
+out_err:
+	pr_err("perf failed to create debugfs dir/files\n");
+	err = -ENODEV;
+	goto out_unlock;
+}
+device_initcall(perf_event_debugfs_init);
-- 
1.9.1

--
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

Powered by Openwall GNU/*/Linux Powered by OpenVZ