For the series:
Reviewed-by: Roland Scheidegger <srol...@vmware.com>
(I'll commit this tomorrow).
Roland
Am 22.02.2017 um 16:56 schrieb Lars Hamre:
> v3: have util_clear_texture mirror the pipe function (Roland Scheidegger)
> v2: rework util clear functions such that they operate on a resource
> instead of a surface (Roland Scheidegger)
>
> Creates a util_clear_texture function for implementing the
> GL_ARB_clear_texture
> in softpipe and llvmpipe.
>
> Signed-off-by: Lars Hamre <cheme...@gmail.com>
>
> ---
>
> CC: Roland Scheidegger <srol...@vmware.com>
>
> NOTE: someone with access will need to commit this post
> review process
>
> src/gallium/auxiliary/util/u_surface.c | 386
> ++++++++++++++++++++-------------
> src/gallium/auxiliary/util/u_surface.h | 7 +
> 2 files changed, 248 insertions(+), 145 deletions(-)
>
> diff --git a/src/gallium/auxiliary/util/u_surface.c
> b/src/gallium/auxiliary/util/u_surface.c
> index a9ed006..f2a471d 100644
> --- a/src/gallium/auxiliary/util/u_surface.c
> +++ b/src/gallium/auxiliary/util/u_surface.c
> @@ -388,6 +388,66 @@ no_src_map:
> ;
> }
>
> +static void
> +util_clear_color_texture_helper(struct pipe_transfer *dst_trans,
> + ubyte *dst_map,
> + enum pipe_format format,
> + const union pipe_color_union *color,
> + unsigned width, unsigned height, unsigned
> depth)
> +{
> + union util_color uc;
> +
> + assert(dst_trans->stride > 0);
> +
> + if (util_format_is_pure_integer(format)) {
> + /*
> + * We expect int/uint clear values here, though some APIs
> + * might disagree (but in any case util_pack_color()
> + * couldn't handle it)...
> + */
> + if (util_format_is_pure_sint(format)) {
> + util_format_write_4i(format, color->i, 0, &uc, 0, 0, 0, 1, 1);
> + } else {
> + assert(util_format_is_pure_uint(format));
> + util_format_write_4ui(format, color->ui, 0, &uc, 0, 0, 0, 1, 1);
> + }
> + } else {
> + util_pack_color(color->f, format, &uc);
> + }
> +
> + util_fill_box(dst_map, format,
> + dst_trans->stride, dst_trans->layer_stride,
> + 0, 0, 0, width, height, depth, &uc);
> +}
> +
> +static void
> +util_clear_color_texture(struct pipe_context *pipe,
> + struct pipe_resource *texture,
> + const union pipe_color_union *color,
> + unsigned level,
> + unsigned dstx, unsigned dsty, unsigned dstz,
> + unsigned width, unsigned height, unsigned depth)
> +{
> + struct pipe_transfer *dst_trans;
> + ubyte *dst_map;
> + enum pipe_format format = texture->format;
> +
> + dst_map = pipe_transfer_map_3d(pipe,
> + texture,
> + level,
> + PIPE_TRANSFER_WRITE,
> + dstx, dsty, dstz,
> + width, height, depth,
> + &dst_trans);
> + if (!dst_map)
> + return;
> +
> + if (dst_trans->stride > 0) {
> + util_clear_color_texture_helper(dst_trans, dst_map, format, color,
> + width, height, depth);
> + }
> + pipe->transfer_unmap(pipe, dst_trans);
> +}
>
>
> #define UBYTE_TO_USHORT(B) ((B) | ((B) << 8))
> @@ -410,8 +470,6 @@ util_clear_render_target(struct pipe_context *pipe,
> {
> struct pipe_transfer *dst_trans;
> ubyte *dst_map;
> - union util_color uc;
> - unsigned max_layer;
>
> assert(dst->texture);
> if (!dst->texture)
> @@ -426,56 +484,202 @@ util_clear_render_target(struct pipe_context *pipe,
> unsigned pixstride = util_format_get_blocksize(dst->format);
> dx = (dst->u.buf.first_element + dstx) * pixstride;
> w = width * pixstride;
> - max_layer = 0;
> dst_map = pipe_transfer_map(pipe,
> dst->texture,
> 0, 0,
> PIPE_TRANSFER_WRITE,
> dx, 0, w, 1,
> &dst_trans);
> + if (dst_map) {
> + util_clear_color_texture_helper(dst_trans, dst_map, dst->format,
> color,
> + width, height, 1);
> + pipe->transfer_unmap(pipe, dst_trans);
> + }
> }
> else {
> - max_layer = dst->u.tex.last_layer - dst->u.tex.first_layer;
> - dst_map = pipe_transfer_map_3d(pipe,
> - dst->texture,
> - dst->u.tex.level,
> - PIPE_TRANSFER_WRITE,
> - dstx, dsty, dst->u.tex.first_layer,
> - width, height, max_layer + 1,
> &dst_trans);
> + unsigned depth = dst->u.tex.last_layer - dst->u.tex.first_layer + 1;
> + util_clear_color_texture(pipe, dst->texture, color, dst->u.tex.level,
> + dstx, dsty, dst->u.tex.first_layer,
> + width, height, depth);
> }
> +}
>
> +static void
> +util_clear_depth_stencil_texture(struct pipe_context *pipe,
> + struct pipe_resource *texture,
> + enum pipe_format format,
> + unsigned clear_flags,
> + uint64_t zstencil, unsigned level,
> + unsigned dstx, unsigned dsty, unsigned dstz,
> + unsigned width, unsigned height, unsigned
> depth)
> +{
> + struct pipe_transfer *dst_trans;
> + ubyte *dst_map;
> + boolean need_rmw = FALSE;
> + unsigned dst_stride;
> + ubyte *dst_layer;
> + unsigned i, j, layer;
> +
> + if ((clear_flags & PIPE_CLEAR_DEPTHSTENCIL) &&
> + ((clear_flags & PIPE_CLEAR_DEPTHSTENCIL) != PIPE_CLEAR_DEPTHSTENCIL)
> &&
> + util_format_is_depth_and_stencil(format))
> + need_rmw = TRUE;
> +
> + dst_map = pipe_transfer_map_3d(pipe,
> + texture,
> + level,
> + (need_rmw ? PIPE_TRANSFER_READ_WRITE :
> + PIPE_TRANSFER_WRITE),
> + dstx, dsty, dstz,
> + width, height, depth, &dst_trans);
> assert(dst_map);
> + if (!dst_map)
> + return;
> +
> + dst_stride = dst_trans->stride;
> + dst_layer = dst_map;
> + assert(dst_trans->stride > 0);
> +
> + for (layer = 0; layer < depth; layer++) {
> + dst_map = dst_layer;
>
> - if (dst_map) {
> - enum pipe_format format = dst->format;
> - assert(dst_trans->stride > 0);
> -
> - if (util_format_is_pure_integer(format)) {
> - /*
> - * We expect int/uint clear values here, though some APIs
> - * might disagree (but in any case util_pack_color()
> - * couldn't handle it)...
> - */
> - if (util_format_is_pure_sint(format)) {
> - util_format_write_4i(format, color->i, 0, &uc, 0, 0, 0, 1, 1);
> + switch (util_format_get_blocksize(format)) {
> + case 1:
> + assert(format == PIPE_FORMAT_S8_UINT);
> + if(dst_stride == width)
> + memset(dst_map, (uint8_t) zstencil, height * width);
> + else {
> + for (i = 0; i < height; i++) {
> + memset(dst_map, (uint8_t) zstencil, width);
> + dst_map += dst_stride;
> + }
> + }
> + break;
> + case 2:
> + assert(format == PIPE_FORMAT_Z16_UNORM);
> + for (i = 0; i < height; i++) {
> + uint16_t *row = (uint16_t *)dst_map;
> + for (j = 0; j < width; j++)
> + *row++ = (uint16_t) zstencil;
> + dst_map += dst_stride;
> + }
> + break;
> + case 4:
> + if (!need_rmw) {
> + for (i = 0; i < height; i++) {
> + uint32_t *row = (uint32_t *)dst_map;
> + for (j = 0; j < width; j++)
> + *row++ = (uint32_t) zstencil;
> + dst_map += dst_stride;
> + }
> + }
> + else {
> + uint32_t dst_mask;
> + if (format == PIPE_FORMAT_Z24_UNORM_S8_UINT)
> + dst_mask = 0x00ffffff;
> + else {
> + assert(format == PIPE_FORMAT_S8_UINT_Z24_UNORM);
> + dst_mask = 0xffffff00;
> + }
> + if (clear_flags & PIPE_CLEAR_DEPTH)
> + dst_mask = ~dst_mask;
> + for (i = 0; i < height; i++) {
> + uint32_t *row = (uint32_t *)dst_map;
> + for (j = 0; j < width; j++) {
> + uint32_t tmp = *row & dst_mask;
> + *row++ = tmp | ((uint32_t) zstencil & ~dst_mask);
> + }
> + dst_map += dst_stride;
> + }
> + }
> + break;
> + case 8:
> + if (!need_rmw) {
> + for (i = 0; i < height; i++) {
> + uint64_t *row = (uint64_t *)dst_map;
> + for (j = 0; j < width; j++)
> + *row++ = zstencil;
> + dst_map += dst_stride;
> + }
> }
> else {
> - assert(util_format_is_pure_uint(format));
> - util_format_write_4ui(format, color->ui, 0, &uc, 0, 0, 0, 1, 1);
> + uint64_t src_mask;
> +
> + if (clear_flags & PIPE_CLEAR_DEPTH)
> + src_mask = 0x00000000ffffffffull;
> + else
> + src_mask = 0x000000ff00000000ull;
> +
> + for (i = 0; i < height; i++) {
> + uint64_t *row = (uint64_t *)dst_map;
> + for (j = 0; j < width; j++) {
> + uint64_t tmp = *row & ~src_mask;
> + *row++ = tmp | (zstencil & src_mask);
> + }
> + dst_map += dst_stride;
> + }
> }
> + break;
> + default:
> + assert(0);
> + break;
> }
> - else {
> - util_pack_color(color->f, format, &uc);
> + dst_layer += dst_trans->layer_stride;
> + }
> +
> + pipe->transfer_unmap(pipe, dst_trans);
> +}
> +
> +
> +void
> +util_clear_texture(struct pipe_context *pipe,
> + struct pipe_resource *tex,
> + unsigned level,
> + const struct pipe_box *box,
> + const void *data)
> +{
> + const struct util_format_description *desc =
> + util_format_description(tex->format);
> +
> + if (level > tex->last_level)
> + return;
> +
> + if (util_format_is_depth_or_stencil(tex->format)) {
> + unsigned clear = 0;
> + float depth = 0.0f;
> + uint8_t stencil = 0;
> + uint64_t zstencil;
> +
> + if (util_format_has_depth(desc)) {
> + clear |= PIPE_CLEAR_DEPTH;
> + desc->unpack_z_float(&depth, 0, data, 0, 1, 1);
> }
>
> - util_fill_box(dst_map, dst->format,
> - dst_trans->stride, dst_trans->layer_stride,
> - 0, 0, 0, width, height, max_layer + 1, &uc);
> + if (util_format_has_stencil(desc)) {
> + clear |= PIPE_CLEAR_STENCIL;
> + desc->unpack_s_8uint(&stencil, 0, data, 0, 1, 1);
> + }
>
> - pipe->transfer_unmap(pipe, dst_trans);
> + zstencil = util_pack64_z_stencil(tex->format, depth, stencil);
> +
> + util_clear_depth_stencil_texture(pipe, tex, tex->format, clear,
> zstencil,
> + level, box->x, box->y, box->z,
> + box->width, box->height, box->depth);
> + } else {
> + union pipe_color_union color;
> + if (util_format_is_pure_uint(tex->format))
> + desc->unpack_rgba_uint(color.ui, 0, data, 0, 1, 1);
> + else if (util_format_is_pure_sint(tex->format))
> + desc->unpack_rgba_sint(color.i, 0, data, 0, 1, 1);
> + else
> + desc->unpack_rgba_float(color.f, 0, data, 0, 1, 1);
> +
> + util_clear_color_texture(pipe, tex, &color, level, box->x, box->y,
> box->z,
> + box->width, box->height, box->depth);
> }
> }
>
> +
> /**
> * Fallback for pipe->clear_stencil() function.
> * sw fallback doesn't look terribly useful here.
> @@ -492,127 +696,19 @@ util_clear_depth_stencil(struct pipe_context *pipe,
> unsigned dstx, unsigned dsty,
> unsigned width, unsigned height)
> {
> - enum pipe_format format = dst->format;
> - struct pipe_transfer *dst_trans;
> - ubyte *dst_map;
> - boolean need_rmw = FALSE;
> - unsigned max_layer, layer;
> -
> - if ((clear_flags & PIPE_CLEAR_DEPTHSTENCIL) &&
> - ((clear_flags & PIPE_CLEAR_DEPTHSTENCIL) != PIPE_CLEAR_DEPTHSTENCIL)
> &&
> - util_format_is_depth_and_stencil(format))
> - need_rmw = TRUE;
> + uint64_t zstencil;
> + unsigned max_layer;
>
> assert(dst->texture);
> if (!dst->texture)
> return;
>
> + zstencil = util_pack64_z_stencil(dst->format, depth, stencil);
> max_layer = dst->u.tex.last_layer - dst->u.tex.first_layer;
> - dst_map = pipe_transfer_map_3d(pipe,
> - dst->texture,
> - dst->u.tex.level,
> - (need_rmw ? PIPE_TRANSFER_READ_WRITE :
> - PIPE_TRANSFER_WRITE),
> - dstx, dsty, dst->u.tex.first_layer,
> - width, height, max_layer + 1, &dst_trans);
> - assert(dst_map);
> -
> - if (dst_map) {
> - unsigned dst_stride = dst_trans->stride;
> - uint64_t zstencil = util_pack64_z_stencil(format, depth, stencil);
> - ubyte *dst_layer = dst_map;
> - unsigned i, j;
> - assert(dst_trans->stride > 0);
> -
> - for (layer = 0; layer <= max_layer; layer++) {
> - dst_map = dst_layer;
> -
> - switch (util_format_get_blocksize(format)) {
> - case 1:
> - assert(format == PIPE_FORMAT_S8_UINT);
> - if(dst_stride == width)
> - memset(dst_map, (uint8_t) zstencil, height * width);
> - else {
> - for (i = 0; i < height; i++) {
> - memset(dst_map, (uint8_t) zstencil, width);
> - dst_map += dst_stride;
> - }
> - }
> - break;
> - case 2:
> - assert(format == PIPE_FORMAT_Z16_UNORM);
> - for (i = 0; i < height; i++) {
> - uint16_t *row = (uint16_t *)dst_map;
> - for (j = 0; j < width; j++)
> - *row++ = (uint16_t) zstencil;
> - dst_map += dst_stride;
> - }
> - break;
> - case 4:
> - if (!need_rmw) {
> - for (i = 0; i < height; i++) {
> - uint32_t *row = (uint32_t *)dst_map;
> - for (j = 0; j < width; j++)
> - *row++ = (uint32_t) zstencil;
> - dst_map += dst_stride;
> - }
> - }
> - else {
> - uint32_t dst_mask;
> - if (format == PIPE_FORMAT_Z24_UNORM_S8_UINT)
> - dst_mask = 0x00ffffff;
> - else {
> - assert(format == PIPE_FORMAT_S8_UINT_Z24_UNORM);
> - dst_mask = 0xffffff00;
> - }
> - if (clear_flags & PIPE_CLEAR_DEPTH)
> - dst_mask = ~dst_mask;
> - for (i = 0; i < height; i++) {
> - uint32_t *row = (uint32_t *)dst_map;
> - for (j = 0; j < width; j++) {
> - uint32_t tmp = *row & dst_mask;
> - *row++ = tmp | ((uint32_t) zstencil & ~dst_mask);
> - }
> - dst_map += dst_stride;
> - }
> - }
> - break;
> - case 8:
> - if (!need_rmw) {
> - for (i = 0; i < height; i++) {
> - uint64_t *row = (uint64_t *)dst_map;
> - for (j = 0; j < width; j++)
> - *row++ = zstencil;
> - dst_map += dst_stride;
> - }
> - }
> - else {
> - uint64_t src_mask;
> -
> - if (clear_flags & PIPE_CLEAR_DEPTH)
> - src_mask = 0x00000000ffffffffull;
> - else
> - src_mask = 0x000000ff00000000ull;
> -
> - for (i = 0; i < height; i++) {
> - uint64_t *row = (uint64_t *)dst_map;
> - for (j = 0; j < width; j++) {
> - uint64_t tmp = *row & ~src_mask;
> - *row++ = tmp | (zstencil & src_mask);
> - }
> - dst_map += dst_stride;
> - }
> - }
> - break;
> - default:
> - assert(0);
> - break;
> - }
> - dst_layer += dst_trans->layer_stride;
> - }
> -
> - pipe->transfer_unmap(pipe, dst_trans);
> - }
> + util_clear_depth_stencil_texture(pipe, dst->texture, dst->format,
> + clear_flags, zstencil, dst->u.tex.level,
> + dstx, dsty, dst->u.tex.first_layer,
> + width, height, max_layer + 1);
> }
>
>
> diff --git a/src/gallium/auxiliary/util/u_surface.h
> b/src/gallium/auxiliary/util/u_surface.h
> index 64a685b..f764abb 100644
> --- a/src/gallium/auxiliary/util/u_surface.h
> +++ b/src/gallium/auxiliary/util/u_surface.h
> @@ -83,6 +83,13 @@ util_resource_copy_region(struct pipe_context *pipe,
> const struct pipe_box *src_box);
>
> extern void
> +util_clear_texture(struct pipe_context *pipe,
> + struct pipe_resource *tex,
> + unsigned level,
> + const struct pipe_box *box,
> + const void *data);
> +
> +extern void
> util_clear_render_target(struct pipe_context *pipe,
> struct pipe_surface *dst,
> const union pipe_color_union *color,
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
