From: Thomas Richter Date: Mon, 7 Oct 2024 05:45:16 +0000 (+0200) Subject: s390/cpum_sf: Set bit PMU_F_ENABLED enabled after lpp() invocation X-Git-Tag: v6.13-rc1~206^2~123 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=31d1d8a35eeb9cd2a2ecd840304a6935e461c500;p=thirdparty%2Fkernel%2Flinux.git s390/cpum_sf: Set bit PMU_F_ENABLED enabled after lpp() invocation Set PMU enabled bit after lpp() call. This ensures the proper task PID is loaded by the llp instruction into Program Parameter register from where it is copied to each sampling data buffer (SDB) sample entry after the sampling was enabled. The barrier() instruction is not needed as cpumsf_pmu_enable() changes a CPU specific variable. Only the CPU that task is running on changes structure members. Signed-off-by: Thomas Richter Reviewed-by: Hendrik Brueckner Signed-off-by: Heiko Carstens --- diff --git a/arch/s390/kernel/perf_cpum_sf.c b/arch/s390/kernel/perf_cpum_sf.c index 5b765e3ccf0ca..e806693c8473d 100644 --- a/arch/s390/kernel/perf_cpum_sf.c +++ b/arch/s390/kernel/perf_cpum_sf.c @@ -942,18 +942,15 @@ static void cpumsf_pmu_enable(struct pmu *pmu) } /* (Re)enable the PMU and sampling facility */ - cpuhw->flags |= PMU_F_ENABLED; - barrier(); - err = lsctl(&cpuhw->lsctl); if (err) { - cpuhw->flags &= ~PMU_F_ENABLED; pr_err("Loading sampling controls failed: op 1 err %i\n", err); return; } /* Load current program parameter */ lpp(&get_lowcore()->lpp); + cpuhw->flags |= PMU_F_ENABLED; } static void cpumsf_pmu_disable(struct pmu *pmu)