[Mesa-dev] [PATCH] r600g: Implement ARB_texture_view
Ilia Mirkin
imirkin at alum.mit.edu
Thu Oct 15 17:03:58 PDT 2015
On Thu, Oct 15, 2015 at 7:53 PM, Glenn Kennard <glenn.kennard at gmail.com> wrote:
> Signed-off-by: Glenn Kennard <glenn.kennard at gmail.com>
> ---
> See also additional texture view piglit test case posted to piglit ml,
> which tests cases with layer>0. Notably softpipe and llvmpipe fail that
> case but i965/hsw, nv50/nvc0 and r600g pass.
>
> docs/GL3.txt | 2 +-
> docs/relnotes/11.1.0.html | 1 +
> src/gallium/drivers/r600/evergreen_state.c | 23 +++++++++++++++++------
> src/gallium/drivers/r600/r600_pipe.c | 2 +-
> 4 files changed, 20 insertions(+), 8 deletions(-)
>
> diff --git a/docs/GL3.txt b/docs/GL3.txt
> index 6503e2a..c03a574 100644
> --- a/docs/GL3.txt
> +++ b/docs/GL3.txt
> @@ -169,7 +169,7 @@ GL 4.3, GLSL 4.30:
> GL_ARB_texture_buffer_range DONE (nv50, nvc0, i965, r600, radeonsi, llvmpipe)
> GL_ARB_texture_query_levels DONE (all drivers that support GLSL 1.30)
> GL_ARB_texture_storage_multisample DONE (all drivers that support GL_ARB_texture_multisample)
> - GL_ARB_texture_view DONE (i965, nv50, nvc0, llvmpipe, softpipe)
> + GL_ARB_texture_view DONE (i965, nv50, nvc0, r600, llvmpipe, softpipe)
> GL_ARB_vertex_attrib_binding DONE (all drivers)
>
>
> diff --git a/docs/relnotes/11.1.0.html b/docs/relnotes/11.1.0.html
> index dcf425e..cb8715c 100644
> --- a/docs/relnotes/11.1.0.html
> +++ b/docs/relnotes/11.1.0.html
> @@ -53,6 +53,7 @@ Note: some of the new features are only available with certain drivers.
> <li>GL_ARB_texture_query_lod on softpipe</li>
> <li>EGL_KHR_create_context on softpipe, llvmpipe</li>
> <li>EGL_KHR_gl_colorspace on softpipe, llvmpipe</li>
> +<li>GL_ARB_texture_view on r600 for Evergreen and later chips</li>
> </ul>
>
> <h2>Bug fixes</h2>
> diff --git a/src/gallium/drivers/r600/evergreen_state.c b/src/gallium/drivers/r600/evergreen_state.c
> index c6702a9..60747d1 100644
> --- a/src/gallium/drivers/r600/evergreen_state.c
> +++ b/src/gallium/drivers/r600/evergreen_state.c
> @@ -666,6 +666,7 @@ evergreen_create_sampler_view_custom(struct pipe_context *ctx,
> enum pipe_format pipe_format = state->format;
> struct radeon_surf_level *surflevel;
> unsigned base_level, first_level, last_level;
> + unsigned dim, last_layer;
> uint64_t va;
>
> if (view == NULL)
> @@ -679,7 +680,7 @@ evergreen_create_sampler_view_custom(struct pipe_context *ctx,
> view->base.reference.count = 1;
> view->base.context = ctx;
>
> - if (texture->target == PIPE_BUFFER)
> + if (state->target == PIPE_BUFFER)
Not sure, but I'd guess things would have to be pretty messed up if
texture->target == buffer, but state->target == not buffer. I'd throw
in an assert actually.
> return texture_buffer_sampler_view(rctx, view, width0, height0);
>
> swizzle[0] = state->swizzle_r;
> @@ -773,12 +774,12 @@ evergreen_create_sampler_view_custom(struct pipe_context *ctx,
> }
> nbanks = eg_num_banks(rscreen->b.tiling_info.num_banks);
>
> - if (texture->target == PIPE_TEXTURE_1D_ARRAY) {
> + if (state->target == PIPE_TEXTURE_1D_ARRAY) {
> height = 1;
> depth = texture->array_size;
Not sure where depth is used, but doesn't this need to take first/last
layer into account? Does textureSize() return the right thing for
these? I guess there's no piglit for that either.
> - } else if (texture->target == PIPE_TEXTURE_2D_ARRAY) {
> + } else if (state->target == PIPE_TEXTURE_2D_ARRAY) {
> depth = texture->array_size;
> - } else if (texture->target == PIPE_TEXTURE_CUBE_ARRAY)
> + } else if (state->target == PIPE_TEXTURE_CUBE_ARRAY)
> depth = texture->array_size / 6;
>
> va = tmp->resource.gpu_address;
> @@ -790,7 +791,13 @@ evergreen_create_sampler_view_custom(struct pipe_context *ctx,
> view->is_stencil_sampler = true;
>
> view->tex_resource = &tmp->resource;
> - view->tex_resource_words[0] = (S_030000_DIM(r600_tex_dim(texture->target, texture->nr_samples)) |
> +
> + /* array type views and views into array types need to use layer offset */
> + dim = state->target;
> + if (state->target != PIPE_TEXTURE_CUBE)
> + dim = MAX2(state->target, texture->target);
> +
> + view->tex_resource_words[0] = (S_030000_DIM(r600_tex_dim(dim, texture->nr_samples)) |
> S_030000_PITCH((pitch / 8) - 1) |
> S_030000_TEX_WIDTH(width - 1));
> if (rscreen->b.chip_class == CAYMAN)
> @@ -818,10 +825,14 @@ evergreen_create_sampler_view_custom(struct pipe_context *ctx,
> view->tex_resource_words[3] = (surflevel[base_level].offset + va) >> 8;
> }
>
> + last_layer = state->u.tex.last_layer;
> + if (state->target != texture->target && depth == 1) {
> + last_layer = state->u.tex.first_layer;
> + }
> view->tex_resource_words[4] = (word4 |
> S_030010_ENDIAN_SWAP(endian));
> view->tex_resource_words[5] = S_030014_BASE_ARRAY(state->u.tex.first_layer) |
> - S_030014_LAST_ARRAY(state->u.tex.last_layer);
> + S_030014_LAST_ARRAY(last_layer);
> view->tex_resource_words[6] = S_030018_TILE_SPLIT(tile_split);
>
> if (texture->nr_samples > 1) {
> diff --git a/src/gallium/drivers/r600/r600_pipe.c b/src/gallium/drivers/r600/r600_pipe.c
> index 32ce76a..c5da44d 100644
> --- a/src/gallium/drivers/r600/r600_pipe.c
> +++ b/src/gallium/drivers/r600/r600_pipe.c
> @@ -323,6 +323,7 @@ static int r600_get_param(struct pipe_screen* pscreen, enum pipe_cap param)
> case PIPE_CAP_TEXTURE_GATHER_SM5:
> case PIPE_CAP_TEXTURE_QUERY_LOD:
> case PIPE_CAP_TGSI_FS_FINE_DERIVATIVE:
> + case PIPE_CAP_SAMPLER_VIEW_TARGET:
> return family >= CHIP_CEDAR ? 1 : 0;
> case PIPE_CAP_MAX_TEXTURE_GATHER_COMPONENTS:
> return family >= CHIP_CEDAR ? 4 : 0;
> @@ -338,7 +339,6 @@ static int r600_get_param(struct pipe_screen* pscreen, enum pipe_cap param)
> case PIPE_CAP_VERTEX_COLOR_CLAMPED:
> case PIPE_CAP_USER_VERTEX_BUFFERS:
> case PIPE_CAP_TEXTURE_GATHER_OFFSETS:
> - case PIPE_CAP_SAMPLER_VIEW_TARGET:
> case PIPE_CAP_VERTEXID_NOBASE:
> case PIPE_CAP_MAX_SHADER_PATCH_VARYINGS:
> case PIPE_CAP_DEPTH_BOUNDS_TEST:
> --
> 1.9.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
More information about the mesa-dev
mailing list