Re: [Intel-gfx] [i-g-t PATCH v1 07/14] lib: Map dumb buffers

2016-03-05 Thread Daniel Vetter
On Sat, Mar 05, 2016 at 01:24:19PM +0100, Daniel Vetter wrote: > On Wed, Mar 02, 2016 at 02:54:20PM +, Chris Wilson wrote: > > On Wed, Mar 02, 2016 at 02:40:44PM +, Daniel Stone wrote: > > > > > > On Wed, 2016-03-02 at 14:39 +, Chris Wilson wrote: > > > > On Wed, Mar 02, 2016 at

Re: [Intel-gfx] [i-g-t PATCH v1 07/14] lib: Map dumb buffers

2016-03-05 Thread Daniel Vetter
On Wed, Mar 02, 2016 at 02:54:20PM +, Chris Wilson wrote: > On Wed, Mar 02, 2016 at 02:40:44PM +, Daniel Stone wrote: > > > > On Wed, 2016-03-02 at 14:39 +, Chris Wilson wrote: > > > On Wed, Mar 02, 2016 at 02:22:58PM +, Daniel Stone wrote: > > > > On Wed, 2016-03-02 at 14:21

Re: [Intel-gfx] [i-g-t PATCH v1 07/14] lib: Map dumb buffers

2016-03-02 Thread Daniel Stone
On Wed, 2016-03-02 at 14:54 +, Chris Wilson wrote: > On Wed, Mar 02, 2016 at 02:40:44PM +, Daniel Stone wrote: > > On Wed, 2016-03-02 at 14:39 +, Chris Wilson wrote: > > > Don't forget to call dirtyfb then. > > Are you talking about frontbuffer rendering, or pageflipping > > between >

Re: [Intel-gfx] [i-g-t PATCH v1 07/14] lib: Map dumb buffers

2016-03-02 Thread Chris Wilson
On Wed, Mar 02, 2016 at 02:40:44PM +, Daniel Stone wrote: > > On Wed, 2016-03-02 at 14:39 +, Chris Wilson wrote: > > On Wed, Mar 02, 2016 at 02:22:58PM +, Daniel Stone wrote: > > > On Wed, 2016-03-02 at 14:21 +, Chris Wilson wrote: > > > > On Wed, Mar 02, 2016 at 03:00:14PM +0100,

Re: [Intel-gfx] [i-g-t PATCH v1 07/14] lib: Map dumb buffers

2016-03-02 Thread Daniel Stone
On Wed, 2016-03-02 at 14:39 +, Chris Wilson wrote: > On Wed, Mar 02, 2016 at 02:22:58PM +, Daniel Stone wrote: > > On Wed, 2016-03-02 at 14:21 +, Chris Wilson wrote: > > > On Wed, Mar 02, 2016 at 03:00:14PM +0100, Tomeu Vizoso wrote: > > > > - gem_set_domain(fd, fb->gem_handle, >

Re: [Intel-gfx] [i-g-t PATCH v1 07/14] lib: Map dumb buffers

2016-03-02 Thread Chris Wilson
On Wed, Mar 02, 2016 at 02:22:58PM +, Daniel Stone wrote: > On Wed, 2016-03-02 at 14:21 +, Chris Wilson wrote: > > On Wed, Mar 02, 2016 at 03:00:14PM +0100, Tomeu Vizoso wrote: > > > @@ -1006,8 +1019,9 @@ static cairo_surface_t *get_cairo_surface(int > > > fd, struct igt_fb *fb) > > >  

Re: [Intel-gfx] [i-g-t PATCH v1 07/14] lib: Map dumb buffers

2016-03-02 Thread Chris Wilson
On Wed, Mar 02, 2016 at 03:00:14PM +0100, Tomeu Vizoso wrote: > @@ -1006,8 +1019,9 @@ static cairo_surface_t *get_cairo_surface(int fd, > struct igt_fb *fb) > create_cairo_surface__gtt(fd, fb); > } > > - gem_set_domain(fd, fb->gem_handle, > -

Re: [Intel-gfx] [i-g-t PATCH v1 07/14] lib: Map dumb buffers

2016-03-02 Thread Daniel Stone
On Wed, 2016-03-02 at 14:21 +, Chris Wilson wrote: > On Wed, Mar 02, 2016 at 03:00:14PM +0100, Tomeu Vizoso wrote: > > @@ -1006,8 +1019,9 @@ static cairo_surface_t *get_cairo_surface(int > > fd, struct igt_fb *fb) > >   create_cairo_surface__gtt(fd, fb); > >   } > >   > > -

[Intel-gfx] [i-g-t PATCH v1 07/14] lib: Map dumb buffers

2016-03-02 Thread Tomeu Vizoso
If a buffer object is dumb, call DRM_IOCTL_MODE_MAP_DUMB when mapping it. Also, don't call DRM_IOCTL_I915_GEM_SET_DOMAIN on dumb buffers. Signed-off-by: Tomeu Vizoso --- lib/igt_fb.c | 20 +--- 1 file changed, 17 insertions(+), 3 deletions(-) diff