Re: [Mesa-dev] [PATCH 0/7] mesa, intel: Support s8z24 pure renderbuffers when using separate stencil

2011-06-16 Thread Chad Versace
On Wed, 15 Jun 2011 17:34:37 -0700, Chad Versace  wrote:
> Only patch 1 affects core mesa; the remaining patches are in the intel shared 
> driver.
> 
> Patch 6 is just a clean-up leading to patch 7.
> 
> Chad Versace (7):
>   mesa: Add field gl_renderbuffer.Unwrapped
>   intel: Unconditionally enable support for S8_Z24 texture format
>   intel: Support renderbuffer unwrappers in intel_get_renderbuffer()
>   intel: Fix intel_framebuffer_map/unamp to use intel_get_renderbuffer()
>   intel: Fix region refcounting in intel_alloc_renderbuffer_storage
>   intel: Unobfuscate intel_alloc_renderbuffer_storage
>   intel: Support s8z24 pure renderbuffers when using separate stencil

NAK this patch series. A comment from Chris Wilson revealed that patch
5/7 is incorrect. I've resubmitted with that patch removed.

- Chad

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH 0/7] mesa, intel: Support s8z24 pure renderbuffers when using separate stencil

2011-06-15 Thread Chad Versace
On 06/15/2011 05:34 PM, Chad Versace wrote:
> Only patch 1 affects core mesa; the remaining patches are in the intel shared 
> driver.
> 
> Patch 6 is just a clean-up leading to patch 7.
> 
> Chad Versace (7):
>   mesa: Add field gl_renderbuffer.Unwrapped
>   intel: Unconditionally enable support for S8_Z24 texture format
>   intel: Support renderbuffer unwrappers in intel_get_renderbuffer()
>   intel: Fix intel_framebuffer_map/unamp to use intel_get_renderbuffer()
>   intel: Fix region refcounting in intel_alloc_renderbuffer_storage
>   intel: Unobfuscate intel_alloc_renderbuffer_storage
>   intel: Support s8z24 pure renderbuffers when using separate stencil
> 
>  src/mesa/drivers/dri/intel/intel_context.c |2 +-
>  src/mesa/drivers/dri/intel/intel_fbo.c |  139 
> ++--
>  src/mesa/drivers/dri/intel/intel_fbo.h |   21 -
>  src/mesa/drivers/dri/intel/intel_span.c|6 +-
>  src/mesa/main/fbobject.c   |   26 +-
>  src/mesa/main/framebuffer.c|   54 ---
>  src/mesa/main/mtypes.h |   11 ++
>  src/mesa/main/renderbuffer.c   |6 +
>  8 files changed, 213 insertions(+), 52 deletions(-)
> 

NAK this patch series. A comment from Chris Wilson revealed that patch
5/7 is incorrect. I've resubmitted with that patch removed.

-- 
Chad Versace
c...@chad-versace.us
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev