drm/i915: Sanitize i915_get_bridge_dev() error path
authorImre Deak <imre.deak@intel.com>
Tue, 19 Jan 2016 13:26:27 +0000 (15:26 +0200)
committerImre Deak <imre.deak@intel.com>
Wed, 27 Jan 2016 15:43:14 +0000 (17:43 +0200)
Clarify the name of the label on the error path, making it clear what's
being cleaned up. The kmem_cache_destroy() calls are NOPs on the
corresponding error path.

Signed-off-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: David Weinehall <david.weinehall@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1453209992-25995-3-git-send-email-imre.deak@intel.com
drivers/gpu/drm/i915/i915_dma.c

index 4b2c073f20e7e9073ac93e9b08e58c12998506b7..5caea9e97fcf28ba4e93196148f43e988e4267d0 100644 (file)
@@ -916,7 +916,7 @@ int i915_driver_load(struct drm_device *dev, unsigned long flags)
 
        if (i915_get_bridge_dev(dev)) {
                ret = -EIO;
-               goto free_priv;
+               goto out_runtime_pm_put;
        }
 
        mmio_bar = IS_GEN2(dev) ? 1 : 0;
@@ -1121,11 +1121,10 @@ out_uncore_fini:
        pci_iounmap(dev->pdev, dev_priv->regs);
 put_bridge:
        pci_dev_put(dev_priv->bridge_dev);
-free_priv:
        kmem_cache_destroy(dev_priv->requests);
        kmem_cache_destroy(dev_priv->vmas);
        kmem_cache_destroy(dev_priv->objects);
-
+out_runtime_pm_put:
        intel_runtime_pm_put(dev_priv);
 
        kfree(dev_priv);