Re: [PATCH 4.19 097/125] perf/x86/uncore: Fix event group support

From: Pavel Machek
Date: Wed Nov 13 2019 - 05:35:36 EST


Hi!

> return ret;
> }
>
> +static void uncore_pmu_enable(struct pmu *pmu)
> +{
> + struct intel_uncore_pmu *uncore_pmu;
> + struct intel_uncore_box *box;
> +
> + uncore_pmu = container_of(pmu, struct intel_uncore_pmu, pmu);
> + if (!uncore_pmu)
> + return;

AFAICT this test can never trigger.

> +static void uncore_pmu_disable(struct pmu *pmu)
> +{
> + struct intel_uncore_pmu *uncore_pmu;
> + struct intel_uncore_box *box;
> +
> + uncore_pmu = container_of(pmu, struct intel_uncore_pmu, pmu);
> + if (!uncore_pmu)
> + return;

And neither can this one.

Best regards,
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Attachment: signature.asc
Description: Digital signature