Re: [PATCH] drm/drm_ioctl.c: Test client capability value early when setting.

2018-03-06 Thread Daniel Vetter
On Wed, Feb 28, 2018 at 04:34:30PM +, Liviu Dudau wrote: > On Wed, Feb 28, 2018 at 05:57:15PM +0200, Ville Syrjälä wrote: > > On Wed, Feb 28, 2018 at 03:44:44PM +, Liviu Dudau wrote: > > > On Wed, Feb 28, 2018 at 05:40:41PM +0200, Ville Syrjälä wrote: > > > > On Wed, Feb 28, 2018 at 03:27:4

Re: [PATCH] drm/drm_ioctl.c: Test client capability value early when setting.

2018-02-28 Thread Liviu Dudau
On Wed, Feb 28, 2018 at 05:57:15PM +0200, Ville Syrjälä wrote: > On Wed, Feb 28, 2018 at 03:44:44PM +, Liviu Dudau wrote: > > On Wed, Feb 28, 2018 at 05:40:41PM +0200, Ville Syrjälä wrote: > > > On Wed, Feb 28, 2018 at 03:27:41PM +, Liviu Dudau wrote: > > > > The drm_setclientcap() function

Re: [PATCH] drm/drm_ioctl.c: Test client capability value early when setting.

2018-02-28 Thread Ville Syrjälä
On Wed, Feb 28, 2018 at 03:44:44PM +, Liviu Dudau wrote: > On Wed, Feb 28, 2018 at 05:40:41PM +0200, Ville Syrjälä wrote: > > On Wed, Feb 28, 2018 at 03:27:41PM +, Liviu Dudau wrote: > > > The drm_setclientcap() function implementing the DRM_IOCTL_SET_CLIENT_CAP > > > ioctl expects that any

Re: [PATCH] drm/drm_ioctl.c: Test client capability value early when setting.

2018-02-28 Thread Liviu Dudau
On Wed, Feb 28, 2018 at 05:40:41PM +0200, Ville Syrjälä wrote: > On Wed, Feb 28, 2018 at 03:27:41PM +, Liviu Dudau wrote: > > The drm_setclientcap() function implementing the DRM_IOCTL_SET_CLIENT_CAP > > ioctl expects that any capability set by the client will have a value of 1. > > Make the ch

Re: [PATCH] drm/drm_ioctl.c: Test client capability value early when setting.

2018-02-28 Thread Ville Syrjälä
On Wed, Feb 28, 2018 at 03:27:41PM +, Liviu Dudau wrote: > The drm_setclientcap() function implementing the DRM_IOCTL_SET_CLIENT_CAP > ioctl expects that any capability set by the client will have a value of 1. > Make the check early so that we don't have to test the value for each > capability

[PATCH] drm/drm_ioctl.c: Test client capability value early when setting.

2018-02-28 Thread Liviu Dudau
The drm_setclientcap() function implementing the DRM_IOCTL_SET_CLIENT_CAP ioctl expects that any capability set by the client will have a value of 1. Make the check early so that we don't have to test the value for each capability. Signed-off-by: Liviu Dudau --- drivers/gpu/drm/drm_ioctl.c | 9 +