From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1762766AbcLPXQQ (ORCPT ); Fri, 16 Dec 2016 18:16:16 -0500 Received: from mga02.intel.com ([134.134.136.20]:36940 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760795AbcLPXNX (ORCPT ); Fri, 16 Dec 2016 18:13:23 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,360,1477983600"; d="scan'208";a="40673337" From: Vikas Shivappa To: vikas.shivappa@intel.com, vikas.shivappa@linux.intel.com Cc: linux-kernel@vger.kernel.org, x86@kernel.org, tglx@linutronix.de, peterz@infradead.org, ravi.v.shankar@intel.com, tony.luck@intel.com, fenghua.yu@intel.com, andi.kleen@intel.com, davidcc@google.com, eranian@google.com, hpa@zytor.com Subject: [PATCH 05/14] x86/cqm,perf/core: Cgroup support prepare Date: Fri, 16 Dec 2016 15:12:59 -0800 Message-Id: <1481929988-31569-6-git-send-email-vikas.shivappa@linux.intel.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1481929988-31569-1-git-send-email-vikas.shivappa@linux.intel.com> References: <1481929988-31569-1-git-send-email-vikas.shivappa@linux.intel.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: David Carrillo-Cisneros cgroup hierarchy monitoring is not supported currently. This patch builds all the necessary datastructures, cgroup APIs like alloc, free etc and necessary quirks for supporting cgroup hierarchy monitoring in later patches. - Introduce a architecture specific data structure arch_info in perf_cgroup to keep track of RMIDs and cgroup hierarchical monitoring. - perf sched_in calls all the cgroup ancestors when a cgroup is scheduled in. This will not work with cqm as we have a common per pkg rmid associated with one task and hence cannot write different RMIds into the MSR for each event. cqm driver enables a flag PERF_EV_CGROUP_NO_RECURSION which indicates the perf to not call all ancestor cgroups for each event and let the driver handle the hierarchy monitoring for cgroup. - Introduce event_terminate as event_destroy is called after cgrp is disassociated from the event to support rmid handling of the cgroup. This helps cqm clean up the cqm specific arch_info. - Add the cgroup APIs for alloc,free,attach and can_attach The above framework will be used to build different cgroup features in later patches. Tests: Same as before. Cgroup still doesnt work but we did the prep to get it to work Patch modified/refactored by Vikas Shivappa to support recycling removal. Signed-off-by: Vikas Shivappa --- arch/x86/events/intel/cqm.c | 19 ++++++++++++++++++- arch/x86/include/asm/perf_event.h | 32 ++++++++++++++++++++++++++++++++ include/linux/perf_event.h | 36 ++++++++++++++++++++++++++++++++++++ kernel/events/core.c | 29 ++++++++++++++++++++++++++++- 4 files changed, 114 insertions(+), 2 deletions(-) diff --git a/arch/x86/events/intel/cqm.c b/arch/x86/events/intel/cqm.c index a0719af..536c8ad 100644 --- a/arch/x86/events/intel/cqm.c +++ b/arch/x86/events/intel/cqm.c @@ -741,7 +741,13 @@ static int intel_cqm_event_init(struct perf_event *event) INIT_LIST_HEAD(&event->hw.cqm_group_entry); INIT_LIST_HEAD(&event->hw.cqm_groups_entry); - event->destroy = intel_cqm_event_destroy; + /* + * CQM driver handles cgroup recursion and since only noe + * RMID can be programmed at the time in each core, then + * it is incompatible with the way generic code handles + * cgroup hierarchies. + */ + event->event_caps |= PERF_EV_CAP_CGROUP_NO_RECURSION; mutex_lock(&cache_mutex); @@ -918,6 +924,17 @@ static int intel_cqm_event_init(struct perf_event *event) .read = intel_cqm_event_read, .count = intel_cqm_event_count, }; +#ifdef CONFIG_CGROUP_PERF +int perf_cgroup_arch_css_alloc(struct cgroup_subsys_state *parent_css, + struct cgroup_subsys_state *new_css) +{} +void perf_cgroup_arch_css_free(struct cgroup_subsys_state *css) +{} +void perf_cgroup_arch_attach(struct cgroup_taskset *tset) +{} +int perf_cgroup_arch_can_attach(struct cgroup_taskset *tset) +{} +#endif static inline void cqm_pick_event_reader(int cpu) { diff --git a/arch/x86/include/asm/perf_event.h b/arch/x86/include/asm/perf_event.h index f353061..43c8e5e 100644 --- a/arch/x86/include/asm/perf_event.h +++ b/arch/x86/include/asm/perf_event.h @@ -299,4 +299,36 @@ static inline void perf_check_microcode(void) { } #define arch_perf_out_copy_user copy_from_user_nmi +/* + * Hooks for architecture specific features of perf_event cgroup. + * Currently used by Intel's CQM. + */ +#ifdef CONFIG_INTEL_RDT_M +#ifdef CONFIG_CGROUP_PERF + +#define perf_cgroup_arch_css_alloc perf_cgroup_arch_css_alloc + +int perf_cgroup_arch_css_alloc(struct cgroup_subsys_state *parent_css, + struct cgroup_subsys_state *new_css); + +#define perf_cgroup_arch_css_free perf_cgroup_arch_css_free + +void perf_cgroup_arch_css_free(struct cgroup_subsys_state *css); + +#define perf_cgroup_arch_attach perf_cgroup_arch_attach + +void perf_cgroup_arch_attach(struct cgroup_taskset *tset); + +#define perf_cgroup_arch_can_attach perf_cgroup_arch_can_attach + +int perf_cgroup_arch_can_attach(struct cgroup_taskset *tset); + +extern struct cftype perf_event_cgrp_arch_subsys_cftypes[]; + +#define PERF_CGROUP_ARCH_CGRP_SUBSYS_ATTS \ + .dfl_cftypes = perf_event_cgrp_arch_subsys_cftypes, \ + .legacy_cftypes = perf_event_cgrp_arch_subsys_cftypes, +#endif + +#endif #endif /* _ASM_X86_PERF_EVENT_H */ diff --git a/include/linux/perf_event.h b/include/linux/perf_event.h index a8f4749..abeacb5 100644 --- a/include/linux/perf_event.h +++ b/include/linux/perf_event.h @@ -300,6 +300,12 @@ struct pmu { int (*event_init) (struct perf_event *event); /* + * Terminate the event for this PMU. Optional complement for a + * successful event_init. Called before the event fields are tear down. + */ + void (*event_terminate) (struct perf_event *event); + + /* * Notification that the event was mapped or unmapped. Called * in the context of the mapping task. */ @@ -516,9 +522,13 @@ typedef void (*perf_overflow_handler_t)(struct perf_event *, * PERF_EV_CAP_SOFTWARE: Is a software event. * PERF_EV_CAP_READ_ACTIVE_PKG: A CPU event (or cgroup event) that can be read * from any CPU in the package where it is active. + * PERF_EV_CAP_CGROUP_NO_RECURSION: A cgroup event that handles its own + * cgroup scoping. It does not need to be enabled for all of its descendants + * cgroups. */ #define PERF_EV_CAP_SOFTWARE BIT(0) #define PERF_EV_CAP_READ_ACTIVE_PKG BIT(1) +#define PERF_EV_CAP_CGROUP_NO_RECURSION BIT(2) #define SWEVENT_HLIST_BITS 8 #define SWEVENT_HLIST_SIZE (1 << SWEVENT_HLIST_BITS) @@ -823,6 +833,8 @@ struct perf_cgroup_info { }; struct perf_cgroup { + /* Architecture specific information. */ + void *arch_info; struct cgroup_subsys_state css; struct perf_cgroup_info __percpu *info; }; @@ -844,6 +856,7 @@ struct perf_cgroup { #ifdef CONFIG_PERF_EVENTS +extern int is_cgroup_event(struct perf_event *event); extern void *perf_aux_output_begin(struct perf_output_handle *handle, struct perf_event *event); extern void perf_aux_output_end(struct perf_output_handle *handle, @@ -1387,4 +1400,27 @@ ssize_t perf_event_sysfs_show(struct device *dev, struct device_attribute *attr, #define perf_event_exit_cpu NULL #endif +/* + * Hooks for architecture specific extensions for perf_cgroup. + */ +#ifndef perf_cgroup_arch_css_alloc +#define perf_cgroup_arch_css_alloc(parent_css, new_css) 0 +#endif + +#ifndef perf_cgroup_arch_css_free +#define perf_cgroup_arch_css_free(css) do { } while (0) +#endif + +#ifndef perf_cgroup_arch_attach +#define perf_cgroup_arch_attach(tskset) do { } while (0) +#endif + +#ifndef perf_cgroup_arch_can_attach +#define perf_cgroup_arch_can_attach(tskset) 0 +#endif + +#ifndef PERF_CGROUP_ARCH_CGRP_SUBSYS_ATTS +#define PERF_CGROUP_ARCH_CGRP_SUBSYS_ATTS +#endif + #endif /* _LINUX_PERF_EVENT_H */ diff --git a/kernel/events/core.c b/kernel/events/core.c index 0e29213..a290c53 100644 --- a/kernel/events/core.c +++ b/kernel/events/core.c @@ -590,6 +590,9 @@ static inline u64 perf_event_clock(struct perf_event *event) if (!cpuctx->cgrp) return false; + if (event->event_caps & PERF_EV_CAP_CGROUP_NO_RECURSION) + return cpuctx->cgrp->css.cgroup == event->cgrp->css.cgroup; + /* * Cgroup scoping is recursive. An event enabled for a cgroup is * also enabled for all its descendant cgroups. If @cpuctx's @@ -606,7 +609,7 @@ static inline void perf_detach_cgroup(struct perf_event *event) event->cgrp = NULL; } -static inline int is_cgroup_event(struct perf_event *event) +int is_cgroup_event(struct perf_event *event) { return event->cgrp != NULL; } @@ -4011,6 +4014,9 @@ static void _free_event(struct perf_event *event) mutex_unlock(&event->mmap_mutex); } + if (event->pmu->event_terminate) + event->pmu->event_terminate(event); + if (is_cgroup_event(event)) perf_detach_cgroup(event); @@ -9236,6 +9242,8 @@ static void account_event(struct perf_event *event) exclusive_event_destroy(event); err_pmu: + if (event->pmu->event_terminate) + event->pmu->event_terminate(event); if (event->destroy) event->destroy(event); module_put(pmu->module); @@ -10738,6 +10746,7 @@ static int __init perf_event_sysfs_init(void) perf_cgroup_css_alloc(struct cgroup_subsys_state *parent_css) { struct perf_cgroup *jc; + int ret; jc = kzalloc(sizeof(*jc), GFP_KERNEL); if (!jc) @@ -10749,6 +10758,12 @@ static int __init perf_event_sysfs_init(void) return ERR_PTR(-ENOMEM); } + jc->arch_info = NULL; + + ret = perf_cgroup_arch_css_alloc(parent_css, &jc->css); + if (ret) + return ERR_PTR(ret); + return &jc->css; } @@ -10756,6 +10771,8 @@ static void perf_cgroup_css_free(struct cgroup_subsys_state *css) { struct perf_cgroup *jc = container_of(css, struct perf_cgroup, css); + perf_cgroup_arch_css_free(css); + free_percpu(jc->info); kfree(jc); } @@ -10776,11 +10793,21 @@ static void perf_cgroup_attach(struct cgroup_taskset *tset) cgroup_taskset_for_each(task, css, tset) task_function_call(task, __perf_cgroup_move, task); + + perf_cgroup_arch_attach(tset); +} + +static int perf_cgroup_can_attach(struct cgroup_taskset *tset) +{ + return perf_cgroup_arch_can_attach(tset); } + struct cgroup_subsys perf_event_cgrp_subsys = { .css_alloc = perf_cgroup_css_alloc, .css_free = perf_cgroup_css_free, + .can_attach = perf_cgroup_can_attach, .attach = perf_cgroup_attach, + PERF_CGROUP_ARCH_CGRP_SUBSYS_ATTS }; #endif /* CONFIG_CGROUP_PERF */ -- 1.9.1