[Intel-gfx] [PATCH 07/14] drm: Only do the hw.lock cleanup in master_relase for !MODESET
Chris Wilson
chris at chris-wilson.co.uk
Wed Jun 15 11:48:23 UTC 2016
On Tue, Jun 14, 2016 at 08:51:02PM +0200, Daniel Vetter wrote:
> Another place gone where modern drivers could have hit
> dev->struct_mutex.
>
> To avoid too deeply nesting control flow rework it a bit.
>
> Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>
> ---
> drivers/gpu/drm/drm_auth.c | 20 ++++++++++++--------
> 1 file changed, 12 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_auth.c b/drivers/gpu/drm/drm_auth.c
> index e015a7edb154..a58b2eb0d004 100644
> --- a/drivers/gpu/drm/drm_auth.c
> +++ b/drivers/gpu/drm/drm_auth.c
> @@ -246,11 +246,14 @@ int drm_master_open(struct drm_file *file_priv)
> void drm_master_release(struct drm_file *file_priv)
> {
> struct drm_device *dev = file_priv->minor->dev;
> + struct drm_master *master = file_priv->master;
> +
Spare newline.
> mutex_lock(&dev->master_mutex);
> - if (file_priv->is_master) {
> - struct drm_master *master = file_priv->master;
> + if (!file_priv->is_master)
> + goto out_unlock;
>
> + if (!drm_core_check_feature(dev, DRIVER_MODESET)) {
> /*
> * Since the master is disappearing, so is the
> * possibility to lock.
> @@ -264,19 +267,20 @@ void drm_master_release(struct drm_file *file_priv)
> wake_up_interruptible_all(&master->lock.lock_queue);
> }
> mutex_unlock(&dev->struct_mutex);
> + }
>
> - if (file_priv->minor->master == file_priv->master) {
> - /* drop the reference held my the minor */
> - if (dev->driver->master_drop)
> - dev->driver->master_drop(dev, file_priv, true);
> - drm_master_put(&file_priv->minor->master);
> - }
> + if (file_priv->minor->master == file_priv->master) {
> + /* drop the reference held my the minor */
> + if (dev->driver->master_drop)
> + dev->driver->master_drop(dev, file_priv, true);
> + drm_master_put(&file_priv->minor->master);
> }
>
> /* drop the master reference held by the file priv */
> if (file_priv->master)
> drm_master_put(&file_priv->master);
> file_priv->is_master = 0;
> +out_unlock:
This changes the reference counting, and from my quick scan we can have
a reference on file_priv->master but file_priv->is_master == 0.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx
mailing list