On Sun, Feb 23, 2014 at 9:59 PM, Kenneth Graunke <kenn...@whitecape.org> wrote:
> On earlier hardware, we had to implement math in the shader to translate
> Y-tiled or untiled coordinates to W-tiled coordinates (which is what
> BLORP does today in order to texture from stencil buffers).
>
> On Broadwell, we can simply state that it's W-tiled in SURFACE_STATE,
> and adjust the pitch. This is much easier.
>
> In the surface state code, I chose to handle the "should we sample depth
> or stencil?" question separately from the setup for sampling from
> stencil. This should make it work with the BindRenderbufferTexImage
> hook as well, and hopefully be reusable for GL_ARB_texture_stencil8
> someday.
>
> Signed-off-by: Kenneth Graunke <kenn...@whitecape.org>
> ---
> src/mesa/drivers/dri/i965/brw_defines.h | 1 +
> src/mesa/drivers/dri/i965/brw_surface_formats.c | 2 +-
> src/mesa/drivers/dri/i965/gen8_surface_state.c | 17 ++++++++++++++---
> src/mesa/drivers/dri/i965/intel_extensions.c | 4 ++++
> 4 files changed, 20 insertions(+), 4 deletions(-)
>
> Passes my Piglit test (which Topi reviewed and I already pushed).
>
> diff --git a/src/mesa/drivers/dri/i965/brw_defines.h
> b/src/mesa/drivers/dri/i965/brw_defines.h
> index 01d3cb6..8a4879b 100644
> --- a/src/mesa/drivers/dri/i965/brw_defines.h
> +++ b/src/mesa/drivers/dri/i965/brw_defines.h
> @@ -264,6 +264,7 @@
> #define GEN8_SURFACE_HALIGN_8 (2 << 14)
> #define GEN8_SURFACE_HALIGN_16 (3 << 14)
> #define GEN8_SURFACE_TILING_NONE (0 << 12)
> +#define GEN8_SURFACE_TILING_W (1 << 12)
> #define GEN8_SURFACE_TILING_X (2 << 12)
> #define GEN8_SURFACE_TILING_Y (3 << 12)
> #define BRW_SURFACE_RC_READ_WRITE (1 << 8)
> diff --git a/src/mesa/drivers/dri/i965/brw_surface_formats.c
> b/src/mesa/drivers/dri/i965/brw_surface_formats.c
> index 9acece5..0c95e86 100644
> --- a/src/mesa/drivers/dri/i965/brw_surface_formats.c
> +++ b/src/mesa/drivers/dri/i965/brw_surface_formats.c
> @@ -363,7 +363,7 @@ brw_format_for_mesa_format(mesa_format mesa_format)
> [MESA_FORMAT_Z24_UNORM_X8_UINT] = 0,
> [MESA_FORMAT_X8Z24_UNORM] = 0,
> [MESA_FORMAT_Z_UNORM32] = 0,
> - [MESA_FORMAT_S_UINT8] = 0,
> + [MESA_FORMAT_S_UINT8] = BRW_SURFACEFORMAT_R8_UINT,
>
> [MESA_FORMAT_BGR_SRGB8] = 0,
> [MESA_FORMAT_A8B8G8R8_SRGB] = 0,
> diff --git a/src/mesa/drivers/dri/i965/gen8_surface_state.c
> b/src/mesa/drivers/dri/i965/gen8_surface_state.c
> index 594e531..27af6ad 100644
> --- a/src/mesa/drivers/dri/i965/gen8_surface_state.c
> +++ b/src/mesa/drivers/dri/i965/gen8_surface_state.c
> @@ -139,6 +139,18 @@ gen8_update_texture_surface(struct gl_context *ctx,
> return;
> }
>
> + if (tObj->StencilSampling && firstImage->_BaseFormat == GL_DEPTH_STENCIL)
> + mt = mt->stencil_mt;
> +
> + unsigned tiling_mode, pitch;
> + if (mt->format == MESA_FORMAT_S_UINT8) {
> + tiling_mode = GEN8_SURFACE_TILING_W;
> + pitch = 2 * mt->region->pitch;
Why do we multiply by two?
You updated GL3.txt in the last patch. Need to update it here too?
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
