[PATCH 1/2] drm: Make control nodes master-less

2014-03-12 Thread Thomas Hellstrom
Hi, On 03/12/2014 02:16 PM, David Herrmann wrote: > Hi > You didn't miss any patches. It was I who missed the usage in drm_crtc.c. I guess this, and Daniel's reply prompts a discussion about control nodes and the master concept. First I'd like to give some background about

[PATCH 1/2] drm: Make control nodes master-less

2014-03-12 Thread David Herrmann
Hi >>> You didn't miss any patches. It was I who missed the usage in drm_crtc.c. >>> I guess this, and Daniel's reply prompts a discussion about control >>> nodes and the master concept. >>> >>> First I'd like to give some background about the use-case: I'd like to >>> use the control node for a d

[PATCH 1/2] drm: Make control nodes master-less

2014-03-12 Thread Thomas Hellstrom
On 03/12/2014 10:45 AM, David Herrmann wrote: > Hi > >> You didn't miss any patches. It was I who missed the usage in drm_crtc.c. >> I guess this, and Daniel's reply prompts a discussion about control >> nodes and the master concept. >> >> First I'd like to give some background about the use-case:

[PATCH 1/2] drm: Make control nodes master-less

2014-03-12 Thread David Herrmann
Hi > You didn't miss any patches. It was I who missed the usage in drm_crtc.c. > I guess this, and Daniel's reply prompts a discussion about control > nodes and the master concept. > > First I'd like to give some background about the use-case: I'd like to > use the control node for a daemon that t

[PATCH 1/2] drm: Make control nodes master-less

2014-03-10 Thread Thomas Hellstrom
On 03/05/2014 03:32 PM, David Herrmann wrote: > Hi > > On Wed, Feb 19, 2014 at 2:40 PM, Thomas Hellstrom > wrote: >> Like for render-nodes, there is no point in maintaining the master concept >> for control nodes, so set the struct drm_file::master pointer to NULL. >> >> At the same time, make su

[PATCH 1/2] drm: Make control nodes master-less

2014-03-05 Thread David Herrmann
Hi On Wed, Feb 19, 2014 at 2:40 PM, Thomas Hellstrom wrote: > Like for render-nodes, there is no point in maintaining the master concept > for control nodes, so set the struct drm_file::master pointer to NULL. > > At the same time, make sure DRM_MASTER | DRM_CONTROL_ALLOW ioctls are always > all

[PATCH 1/2] drm: Make control nodes master-less

2014-03-04 Thread Daniel Vetter
On Wed, Feb 19, 2014 at 02:40:41PM +0100, Thomas Hellstrom wrote: > Like for render-nodes, there is no point in maintaining the master concept > for control nodes, so set the struct drm_file::master pointer to NULL. > > At the same time, make sure DRM_MASTER | DRM_CONTROL_ALLOW ioctls are always >

[PATCH 1/2] drm: Make control nodes master-less

2014-02-28 Thread Thomas Hellstrom
Hi! Could someone please review these two patches? Thanks, Thomas On 02/19/2014 02:40 PM, Thomas Hellstrom wrote: > Like for render-nodes, there is no point in maintaining the master concept > for control nodes, so set the struct drm_file::master pointer to NULL. > > At the same time, make sure

[PATCH 1/2] drm: Make control nodes master-less

2014-02-19 Thread Thomas Hellstrom
Like for render-nodes, there is no point in maintaining the master concept for control nodes, so set the struct drm_file::master pointer to NULL. At the same time, make sure DRM_MASTER | DRM_CONTROL_ALLOW ioctls are always allowed when called through the control node. Previously the caller also ne