diff options
| author |   Lucas De Marchi <lucas.demarchi@intel.com> | 2024-11-04 13:35:12 -0800 |
|---|---|---|
| committer | Lucas De Marchi <lucas.demarchi@intel.com> | 2024-11-05 11:24:33 -0800 |
| commit | 79367b7a58c82d0b1c0a7b0ef748f7aafa91d048 (patch) | |
| tree | d89be9d28615a6ba9ebd85fd8dbcd2b67d931493 | |
| parent | drm/i915/pmu: Replace closed with registered (diff) | |
| download | wireguard-linux-79367b7a58c82d0b1c0a7b0ef748f7aafa91d048.tar.xz wireguard-linux-79367b7a58c82d0b1c0a7b0ef748f7aafa91d048.zip | |
drm/i915/pmu: Remove pointless synchronize_rcu() call
This is already done inside perf_pmu_unregister() - no need to do it
before.
Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20241104213512.2314930-5-lucas.demarchi@intel.com
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
| -rw-r--r-- | drivers/gpu/drm/i915/i915_pmu.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/gpu/drm/i915/i915_pmu.c b/drivers/gpu/drm/i915/i915_pmu.c index bd476297ed0a..e55db036be1b 100644 --- a/drivers/gpu/drm/i915/i915_pmu.c +++ b/drivers/gpu/drm/i915/i915_pmu.c @@ -1335,13 +1335,8 @@ void i915_pmu_unregister(struct drm_i915_private *i915) if (!pmu->registered) return; - /* - * "Disconnect" the PMU callbacks - since all are atomic synchronize_rcu - * ensures all currently executing ones will have exited before we - * proceed with unregistration. - */ + /* Disconnect the PMU callbacks */ pmu->registered = false; - synchronize_rcu(); hrtimer_cancel(&pmu->timer); |