[PATCH v3 3/4] drm/i915: Fix asymmetry in PMU register/unregister step order

[Janusz Krzysztofik]

To simplify i915_driver_unregister() code, make sure reverts of driver
registration steps executed before potentially unsuccessful device
registration are symmetrically called after drm_dev_unplug().  There is
one case that doesn't follow that rule, introduced by commit b46a33e271ed
("drm/i915/pmu: Expose a PMU interface for perf queries"), with no
justification for asymmetry provided in commit description, then assumed
not intentional.

Cc: Chris Wilson <chris.p.wil...@linux.intel.com>
Cc: Tvrtko Ursulin <tursu...@ursulin.net>
Signed-off-by: Janusz Krzysztofik <janusz.krzyszto...@linux.intel.com>
---
 drivers/gpu/drm/i915/i915_driver.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_driver.c 
b/drivers/gpu/drm/i915/i915_driver.c
index d865e90f54704..7980bb880c7dc 100644
--- a/drivers/gpu/drm/i915/i915_driver.c
+++ b/drivers/gpu/drm/i915/i915_driver.c
@@ -689,20 +689,17 @@ static void i915_driver_unregister(struct 
drm_i915_private *dev_priv)
                intel_gt_driver_unregister(gt);
 
        if (!dev_priv->do_unregister)
-               goto do_pmu;
+               goto do_unplug;
 
        i915_hwmon_unregister(dev_priv);
 
        i915_perf_unregister(dev_priv);
-do_pmu:
-       i915_pmu_unregister(dev_priv);
-       if (!dev_priv->do_unregister)
-               goto do_unplug;
 
        i915_teardown_sysfs(dev_priv);
 do_unplug:
        drm_dev_unplug(&dev_priv->drm);
 
+       i915_pmu_unregister(dev_priv);
        i915_gem_driver_unregister(dev_priv);
 }
 
-- 
2.48.1