Re: [PATCH 08/14] drm: Move authmagic cleanup into drm_master_release

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 14, 2016 at 08:51:03PM +0200, Daniel Vetter wrote:
> It's related, and soon authmagic will also use the master_mutex.
> 
> There is an ever-so-slightly semantic change here:
> - authmagic will only be cleaned up for primary_client drm_minors. But
>   it's impossible to create authmagic on render/control nodes, so this
>   is fine.

For reference: file_priv->magic is created by the drm_getmagic() ioctl,
only available on the legacy device node.

> - The cleanup is moved down a bit in the release processing. Doesn't
>   matter at all since authmagic is purely internal logic used by the
>   core ioctl access checks, and when we're in a file's release
>   callback no one can do ioctls any more.
> 
> Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx>
Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux