Re: [PATCH] drm/vgem: create a render node for vgem

2018-11-06 Thread Daniel Vetter
On Mon, Nov 05, 2018 at 03:35:54PM +, Emil Velikov wrote: > On Mon, 5 Nov 2018 at 15:30, Chris Wilson wrote: > > > > Quoting kbuild test robot (2018-10-26 16:55:25) > > > Hi Emil, > > > > > > Thank you for the patch! Yet something to improve: > > > > > > [auto build test ERROR on sof-driver-fu

Re: [PATCH] drm/vgem: create a render node for vgem

2018-11-05 Thread Emil Velikov
On Mon, 5 Nov 2018 at 15:30, Chris Wilson wrote: > > Quoting kbuild test robot (2018-10-26 16:55:25) > > Hi Emil, > > > > Thank you for the patch! Yet something to improve: > > > > [auto build test ERROR on sof-driver-fuweitax/master] > > [also build test ERROR on v4.19 next-20181019] > > [if your

Re: [PATCH] drm/vgem: create a render node for vgem

2018-11-05 Thread Chris Wilson
Quoting kbuild test robot (2018-10-26 16:55:25) > Hi Emil, > > Thank you for the patch! Yet something to improve: > > [auto build test ERROR on sof-driver-fuweitax/master] > [also build test ERROR on v4.19 next-20181019] > [if your patch is applied to the wrong git tree, please drop us a note to

Re: [PATCH] drm/vgem: create a render node for vgem

2018-10-29 Thread Daniel Vetter
On Mon, Oct 29, 2018 at 12:03:46PM +, Emil Velikov wrote: > On Fri, 26 Oct 2018 at 14:40, Daniel Vetter wrote: > > > > On Fri, Oct 26, 2018 at 01:06:47PM +0100, Emil Velikov wrote: > > > From: Emil Velikov > > > > > > VGEM doesn't do anything modeset specific, so in a way exposing a > > > pri

Re: [PATCH] drm/vgem: create a render node for vgem

2018-10-29 Thread Chris Wilson
Quoting Emil Velikov (2018-10-29 12:00:17) > On Fri, 26 Oct 2018 at 15:40, Chris Wilson wrote: > > > > Quoting Daniel Vetter (2018-10-26 14:40:36) > > > On Fri, Oct 26, 2018 at 01:06:47PM +0100, Emil Velikov wrote: > > > > From: Emil Velikov > > > > > > > > VGEM doesn't do anything modeset specif

Re: [PATCH] drm/vgem: create a render node for vgem

2018-10-29 Thread Emil Velikov
On Fri, 26 Oct 2018 at 14:40, Daniel Vetter wrote: > > On Fri, Oct 26, 2018 at 01:06:47PM +0100, Emil Velikov wrote: > > From: Emil Velikov > > > > VGEM doesn't do anything modeset specific, so in a way exposing a > > primary node is 'wrong'. At the same time, we extensively use if for > > creati

Re: [PATCH] drm/vgem: create a render node for vgem

2018-10-29 Thread Emil Velikov
On Fri, 26 Oct 2018 at 15:40, Chris Wilson wrote: > > Quoting Daniel Vetter (2018-10-26 14:40:36) > > On Fri, Oct 26, 2018 at 01:06:47PM +0100, Emil Velikov wrote: > > > From: Emil Velikov > > > > > > VGEM doesn't do anything modeset specific, so in a way exposing a > > > primary node is 'wrong'.

Re: [PATCH] drm/vgem: create a render node for vgem

2018-10-26 Thread kbuild test robot
Hi Emil, Thank you for the patch! Yet something to improve: [auto build test ERROR on sof-driver-fuweitax/master] [also build test ERROR on v4.19 next-20181019] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci

Re: [PATCH] drm/vgem: create a render node for vgem

2018-10-26 Thread Chris Wilson
Quoting Daniel Vetter (2018-10-26 14:40:36) > On Fri, Oct 26, 2018 at 01:06:47PM +0100, Emil Velikov wrote: > > From: Emil Velikov > > > > VGEM doesn't do anything modeset specific, so in a way exposing a > > primary node is 'wrong'. At the same time, we extensively use if for > > creating dumb b

Re: [PATCH] drm/vgem: create a render node for vgem

2018-10-26 Thread Daniel Vetter
On Fri, Oct 26, 2018 at 01:06:47PM +0100, Emil Velikov wrote: > From: Emil Velikov > > VGEM doesn't do anything modeset specific, so in a way exposing a > primary node is 'wrong'. At the same time, we extensively use if for > creating dumb buffers, fences, prime fd <> handle imports/exports. > >

[PATCH] drm/vgem: create a render node for vgem

2018-10-26 Thread Emil Velikov
From: Emil Velikov VGEM doesn't do anything modeset specific, so in a way exposing a primary node is 'wrong'. At the same time, we extensively use if for creating dumb buffers, fences, prime fd <> handle imports/exports. To the point that we explicitly annotate the vgem fence ioctls as DRM_RENDE