Re: [Mesa-dev] [PATCH 2/2] i965: Use IVB specific formula for depthbuffer

2013-10-09 Thread Brian Paul
On 10/09/2013 05:13 PM, Chad Versace wrote: On 10/09/2013 03:14 PM, Ian Romanick wrote: On 10/09/2013 02:49 PM, Chad Versace wrote: On 10/08/2013 04:27 PM, Ben Widawsky wrote: + } + hz_height /= 2; + break; + default: + perf_debug("Unknown depthbuffer texture type (%d).",

Re: [Mesa-dev] [PATCH 2/2] i965: Use IVB specific formula for depthbuffer

2013-10-09 Thread Chad Versace
On 10/09/2013 03:14 PM, Ian Romanick wrote: On 10/09/2013 02:49 PM, Chad Versace wrote: On 10/08/2013 04:27 PM, Ben Widawsky wrote: + } + hz_height /= 2; + break; + default: + perf_debug("Unknown depthbuffer texture type (%d).", mt->target); I thought I commented on the

Re: [Mesa-dev] [PATCH 2/2] i965: Use IVB specific formula for depthbuffer

2013-10-09 Thread Ian Romanick
On 10/09/2013 02:49 PM, Chad Versace wrote: > On 10/08/2013 04:27 PM, Ben Widawsky wrote: >> + } >> + hz_height /= 2; >> + break; >> + default: >> + perf_debug("Unknown depthbuffer texture type (%d).", mt->target); > > I thought I commented on the perf_debug() before, but may

Re: [Mesa-dev] [PATCH 2/2] i965: Use IVB specific formula for depthbuffer

2013-10-09 Thread Chad Versace
On 10/08/2013 04:27 PM, Ben Widawsky wrote: After the last patch, we can replace the region allocated in the miptree creation with a more straightforward (and hopefully smaller resulting) buffer based on the bspec's allocation formula. Since I am relatively new to this part of the bspec, I would

[Mesa-dev] [PATCH 2/2] i965: Use IVB specific formula for depthbuffer

2013-10-08 Thread Ben Widawsky
After the last patch, we can replace the region allocated in the miptree creation with a more straightforward (and hopefully smaller resulting) buffer based on the bspec's allocation formula. Since I am relatively new to this part of the bspec, I would very much appreciate scrutiny during review o

Re: [Mesa-dev] [PATCH 2/2] i965: Use IVB specific formula for depthbuffer

2013-09-29 Thread Ben Widawsky
On Thu, Sep 26, 2013 at 01:28:47PM -0700, Chad Versace wrote: > On 09/22/2013 10:37 AM, Ben Widawsky wrote: > >After the last patch, we can replace the region allocated in the miptree > >creation with a more straightforward (and hopefully smaller resulting) > >buffer based on the bspec's allocation

Re: [Mesa-dev] [PATCH 2/2] i965: Use IVB specific formula for depthbuffer

2013-09-26 Thread Chad Versace
On 09/22/2013 10:37 AM, Ben Widawsky wrote: After the last patch, we can replace the region allocated in the miptree creation with a more straightforward (and hopefully smaller resulting) buffer based on the bspec's allocation formula. Since I am relatively new to this part of the bspec, I would

Re: [Mesa-dev] [PATCH 2/2] i965: Use IVB specific formula for depthbuffer

2013-09-23 Thread Pohjolainen, Topi
On Sun, Sep 22, 2013 at 10:37:19AM -0700, Ben Widawsky wrote: > After the last patch, we can replace the region allocated in the miptree > creation with a more straightforward (and hopefully smaller resulting) > buffer based on the bspec's allocation formula. > > Since I am relatively new to this

[Mesa-dev] [PATCH 2/2] i965: Use IVB specific formula for depthbuffer

2013-09-22 Thread Ben Widawsky
After the last patch, we can replace the region allocated in the miptree creation with a more straightforward (and hopefully smaller resulting) buffer based on the bspec's allocation formula. Since I am relatively new to this part of the bspec, I would very much appreciate scrutiny during review o