Re: [Mesa-dev] [PATCH] gallium/xorg st/nv50/i915: add PIPE_BIND_CURSOR

2011-05-08 Thread Marcin Slusarz
On Tue, May 03, 2011 at 12:06:33PM +0200, Daniel Vetter wrote: > On Mon, May 2, 2011 at 8:40 PM, Marcin Slusarz > wrote: > > On Mon, May 02, 2011 at 03:11:00PM +0200, Daniel Vetter wrote: > >> On Mon, May 2, 2011 at 2:56 PM, Benjamin Franzke > >> wrote: > >> > I think in i915g the CURSOR flag sh

Re: [Mesa-dev] [PATCH] gallium/xorg st/nv50/i915: add PIPE_BIND_CURSOR

2011-05-03 Thread Daniel Vetter
On Mon, May 2, 2011 at 8:40 PM, Marcin Slusarz wrote: > On Mon, May 02, 2011 at 03:11:00PM +0200, Daniel Vetter wrote: >> On Mon, May 2, 2011 at 2:56 PM, Benjamin Franzke >> wrote: >> > I think in i915g the CURSOR flag should be used in i9x5_scanout_layout >> > for the "special case for cursors"

Re: [Mesa-dev] [PATCH] gallium/xorg st/nv50/i915: add PIPE_BIND_CURSOR

2011-05-02 Thread Marcin Slusarz
On Mon, May 02, 2011 at 03:11:00PM +0200, Daniel Vetter wrote: > On Mon, May 2, 2011 at 2:56 PM, Benjamin Franzke > wrote: > > I think in i915g the CURSOR flag should be used in i9x5_scanout_layout > > for the "special case for cursors" as well, instead of only checking > > only pt->width0 == 64 &

Re: [Mesa-dev] [PATCH] gallium/xorg st/nv50/i915: add PIPE_BIND_CURSOR

2011-05-02 Thread Daniel Vetter
On Mon, May 2, 2011 at 2:56 PM, Benjamin Franzke wrote: > I think in i915g the CURSOR flag should be used in i9x5_scanout_layout > for the "special case for cursors" as well, instead of only checking > only pt->width0 == 64 && pt->height0 == 64. Oops, so much for actually re-checking the code. On

Re: [Mesa-dev] [PATCH] gallium/xorg st/nv50/i915: add PIPE_BIND_CURSOR

2011-05-02 Thread Benjamin Franzke
I think in i915g the CURSOR flag should be used in i9x5_scanout_layout for the "special case for cursors" as well, instead of only checking only pt->width0 == 64 && pt->height0 == 64. 2011/5/2 Daniel Vetter : > On Sun, May 1, 2011 at 11:59 PM, Marcin Slusarz > wrote: >> We need to distinguish sur

Re: [Mesa-dev] [PATCH] gallium/xorg st/nv50/i915: add PIPE_BIND_CURSOR

2011-05-02 Thread Daniel Vetter
On Sun, May 1, 2011 at 11:59 PM, Marcin Slusarz wrote: > We need to distinguish surfaces for mouse cursors from scanouts, because nv50 > hardware display engine ignores tiling flags. > i915 seems to have similar needs, so fix it too. For the i915g part: Acked-by: Daniel Vetter -- Daniel Vetter

[Mesa-dev] [PATCH] gallium/xorg st/nv50/i915: add PIPE_BIND_CURSOR

2011-05-01 Thread Marcin Slusarz
We need to distinguish surfaces for mouse cursors from scanouts, because nv50 hardware display engine ignores tiling flags. i915 seems to have similar needs, so fix it too. --- src/gallium/drivers/i915/i915_resource_texture.c |6 +- src/gallium/drivers/nv50/nv50_miptree.c | 11 +