drm/i915: Add locks around audio component bind/unbind
authorDavid Henningsson <david.henningsson@canonical.com>
Thu, 3 Sep 2015 09:51:35 +0000 (11:51 +0200)
committerTakashi Iwai <tiwai@suse.de>
Thu, 3 Sep 2015 10:14:45 +0000 (12:14 +0200)
This will make sure that audio callbacks do not race with
component bind/unbind.

[Note: this is an update patch to commit [51e1d83cab99: drm/i915: Call
 audio pin/ELD notify function] where I mistakenly applied the older
 version.  Jani and Daniel's review tags were to the latest version,
 so I add them below, too -- tiwai]

Signed-off-by: David Henningsson <david.henningsson@canonical.com>
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
Acked-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
drivers/gpu/drm/i915/intel_audio.c

index 2d52d96b549385fa2966b92ee5a4c5338126c44b..678a34f87c1c2e32ab9bf3558bf38e2a64fd447b 100644 (file)
@@ -543,9 +543,11 @@ static int i915_audio_component_bind(struct device *i915_dev,
        if (WARN_ON(acomp->ops || acomp->dev))
                return -EEXIST;
 
+       drm_modeset_lock_all(dev_priv->dev);
        acomp->ops = &i915_audio_component_ops;
        acomp->dev = i915_dev;
        dev_priv->audio_component = acomp;
+       drm_modeset_unlock_all(dev_priv->dev);
 
        return 0;
 }
@@ -556,9 +558,11 @@ static void i915_audio_component_unbind(struct device *i915_dev,
        struct i915_audio_component *acomp = data;
        struct drm_i915_private *dev_priv = dev_to_i915(i915_dev);
 
+       drm_modeset_lock_all(dev_priv->dev);
        acomp->ops = NULL;
        acomp->dev = NULL;
        dev_priv->audio_component = NULL;
+       drm_modeset_unlock_all(dev_priv->dev);
 }
 
 static const struct component_ops i915_audio_component_bind_ops = {