Re: [PATCH v7 1/2] drm: Unplug drm device when unregistering it

2017-04-12 Thread jeffy
Hi Daniel, On 04/12/2017 02:33 PM, Daniel Vetter wrote: On Tue, Apr 11, 2017 at 11:31:41AM +0800, Jeffy Chen wrote: After unbinding drm, the user space may still owns the drm dev fd, and may still be able to call drm ioctl. We're using an unplugged state to prevent something like that, so

Re: [PATCH v7 1/2] drm: Unplug drm device when unregistering it

2017-04-12 Thread jeffy
Hi Daniel, missed some questions... On 04/12/2017 04:17 PM, jeffy wrote: Hi Daniel, On 04/12/2017 02:33 PM, Daniel Vetter wrote: On Tue, Apr 11, 2017 at 11:31:41AM +0800, Jeffy Chen wrote: After unbinding drm, the user space may still owns the drm dev fd, and may still be able to call drm

Re: [PATCH v7 1/2] drm: Unplug drm device when unregistering it

2017-04-12 Thread Daniel Vetter
On Tue, Apr 11, 2017 at 11:31:41AM +0800, Jeffy Chen wrote: > After unbinding drm, the user space may still owns the drm dev fd, > and may still be able to call drm ioctl. > > We're using an unplugged state to prevent something like that, so > let's reuse it here. > > Signed-off-by: Jeffy Chen

[PATCH v7 1/2] drm: Unplug drm device when unregistering it

2017-04-11 Thread Jeffy Chen
After unbinding drm, the user space may still owns the drm dev fd, and may still be able to call drm ioctl. We're using an unplugged state to prevent something like that, so let's reuse it here. Signed-off-by: Jeffy Chen Reviewed-by: Sean Paul