From: Thomas Richter Date: Mon, 15 Apr 2024 12:40:03 +0000 (+0200) Subject: s390/pai_ext: Enable concurrent system-wide counting/sampling X-Git-Tag: v6.11-rc1~126^2~40 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=14e3768435da557079216f51d774165c70aad9b1;p=thirdparty%2Fkernel%2Flinux.git s390/pai_ext: Enable concurrent system-wide counting/sampling The PMU for PAI NNPA counters enforces the following restriction: - No system wide counting while system wide sampling is active. This restriction is removed. One or more system wide counting events can now be active at the same time while at most one system wide sampling event is active. Acked-by: Sumanth Korikkar Signed-off-by: Thomas Richter Signed-off-by: Alexander Gordeev --- diff --git a/arch/s390/kernel/perf_pai_ext.c b/arch/s390/kernel/perf_pai_ext.c index a6da7e0cc7a66..5d00f77a64a5f 100644 --- a/arch/s390/kernel/perf_pai_ext.c +++ b/arch/s390/kernel/perf_pai_ext.c @@ -120,8 +120,10 @@ static void paiext_event_destroy(struct perf_event *event) struct paiext_mapptr *mp = per_cpu_ptr(paiext_root.mapptr, event->cpu); struct paiext_map *cpump = mp->mapptr; - free_page(PAI_SAVE_AREA(event)); mutex_lock(&paiext_reserve_mutex); + if (event->attr.sample_period) + cpump->mode &= ~PAI_MODE_SAMPLING; + free_page(PAI_SAVE_AREA(event)); if (refcount_dec_and_test(&cpump->refcnt)) /* Last reference gone */ paiext_free(mp); paiext_root_free(); @@ -186,21 +188,19 @@ static int paiext_alloc(struct perf_event_attr *a, struct perf_event *event) goto undo; } refcount_set(&cpump->refcnt, 1); - cpump->mode = a->sample_period ? PAI_MODE_SAMPLING - : PAI_MODE_COUNTING; } else { /* Multiple invocation, check what is active. - * Supported are multiple counter events or only one sampling + * Supported are multiple counter events and only one sampling * event concurrently at any one time. */ - if (cpump->mode == PAI_MODE_SAMPLING || - (cpump->mode == PAI_MODE_COUNTING && a->sample_period)) { + if (a->sample_period && (cpump->mode & PAI_MODE_SAMPLING)) { rc = -EBUSY; goto undo; } refcount_inc(&cpump->refcnt); } - + if (a->sample_period) + cpump->mode |= PAI_MODE_SAMPLING; rc = 0; undo: @@ -335,6 +335,8 @@ static void paiext_start(struct perf_event *event, int flags) local64_set(&event->hw.prev_count, sum); } else { /* Sampling */ cpump->event = event; + memcpy((void *)PAI_SAVE_AREA(event), cpump->area, + PAIE1_CTRBLOCK_SZ); perf_sched_cb_inc(event->pmu); } } @@ -493,7 +495,7 @@ static int paiext_have_sample(void) static void paiext_sched_task(struct perf_event_pmu_context *pmu_ctx, bool sched_in) { /* We started with a clean page on event installation. So read out - * results on schedule_out and if page was dirty, clear values. + * results on schedule_out and if page was dirty, save old values. */ if (!sched_in) paiext_have_sample();