Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-28 Thread Pohjolainen, Topi
On Tue, Jan 28, 2014 at 02:20:47PM +0200, Pohjolainen, Topi wrote: > On Tue, Jan 28, 2014 at 11:48:18AM +0200, Pohjolainen, Topi wrote: > > On Mon, Jan 27, 2014 at 06:00:54PM -0800, Kenneth Graunke wrote: > > > On 01/27/2014 11:06 AM, Pohjolainen, Topi wrote: > > > > On Mon, Jan 27, 2014 at 10:20:5

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-28 Thread Pohjolainen, Topi
On Tue, Jan 28, 2014 at 11:48:18AM +0200, Pohjolainen, Topi wrote: > On Mon, Jan 27, 2014 at 06:00:54PM -0800, Kenneth Graunke wrote: > > On 01/27/2014 11:06 AM, Pohjolainen, Topi wrote: > > > On Mon, Jan 27, 2014 at 10:20:54AM -0800, Eric Anholt wrote: > > >> Topi Pohjolainen writes: > > >> > > >

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-28 Thread Pohjolainen, Topi
On Tue, Jan 28, 2014 at 11:48:18AM +0200, Pohjolainen, Topi wrote: > On Mon, Jan 27, 2014 at 06:00:54PM -0800, Kenneth Graunke wrote: > > On 01/27/2014 11:06 AM, Pohjolainen, Topi wrote: > > > On Mon, Jan 27, 2014 at 10:20:54AM -0800, Eric Anholt wrote: > > >> Topi Pohjolainen writes: > > >> > > >

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-28 Thread Pohjolainen, Topi
On Tue, Jan 28, 2014 at 11:48:18AM +0200, Pohjolainen, Topi wrote: > On Mon, Jan 27, 2014 at 06:00:54PM -0800, Kenneth Graunke wrote: > > On 01/27/2014 11:06 AM, Pohjolainen, Topi wrote: > > > On Mon, Jan 27, 2014 at 10:20:54AM -0800, Eric Anholt wrote: > > >> Topi Pohjolainen writes: > > >> > > >

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-28 Thread Pohjolainen, Topi
On Mon, Jan 27, 2014 at 06:00:54PM -0800, Kenneth Graunke wrote: > On 01/27/2014 11:06 AM, Pohjolainen, Topi wrote: > > On Mon, Jan 27, 2014 at 10:20:54AM -0800, Eric Anholt wrote: > >> Topi Pohjolainen writes: > >> > >>> This is really not needed as blorp blit programs already sample > >>> XRGB n

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-27 Thread Kenneth Graunke
On 01/27/2014 11:06 AM, Pohjolainen, Topi wrote: > On Mon, Jan 27, 2014 at 10:20:54AM -0800, Eric Anholt wrote: >> Topi Pohjolainen writes: >> >>> This is really not needed as blorp blit programs already sample >>> XRGB normally and get alpha channel set to 1.0 automatically by >>> the sampler eng

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-27 Thread Pohjolainen, Topi
On Mon, Jan 27, 2014 at 10:20:54AM -0800, Eric Anholt wrote: > Topi Pohjolainen writes: > > > This is really not needed as blorp blit programs already sample > > XRGB normally and get alpha channel set to 1.0 automatically by > > the sampler engine. This is simply copied directly to the payload >

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-27 Thread Kenneth Graunke
On 01/27/2014 02:46 AM, Topi Pohjolainen wrote: > This is really not needed as blorp blit programs already sample > XRGB normally and get alpha channel set to 1.0 automatically by > the sampler engine. This is simply copied directly to the payload > of the render target write message and hence ther

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-27 Thread Eric Anholt
Topi Pohjolainen writes: > This is really not needed as blorp blit programs already sample > XRGB normally and get alpha channel set to 1.0 automatically by > the sampler engine. This is simply copied directly to the payload > of the render target write message and hence there is no need for > an

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-27 Thread Eric Anholt
Topi Pohjolainen writes: > This is really not needed as blorp blit programs already sample > XRGB normally and get alpha channel set to 1.0 automatically by > the sampler engine. This is simply copied directly to the payload > of the render target write message and hence there is no need for > an