[Mesa-dev] [PATCH] gallium/util: remove u_staging
Jose Fonseca
jfonseca at vmware.com
Mon Jun 6 09:37:15 UTC 2016
On 02/06/16 15:15, Rob Clark wrote:
> From: Rob Clark <robclark at freedesktop.org>
>
> Unused, and fixes a couple of coverity warnings: CID 1362171, 1362170
>
> Signed-off-by: Rob Clark <robclark at freedesktop.org>
> ---
> src/gallium/auxiliary/Makefile.sources | 2 -
> src/gallium/auxiliary/util/u_staging.c | 136 ---------------------------------
> src/gallium/auxiliary/util/u_staging.h | 67 ----------------
> 3 files changed, 205 deletions(-)
> delete mode 100644 src/gallium/auxiliary/util/u_staging.c
> delete mode 100644 src/gallium/auxiliary/util/u_staging.h
>
> diff --git a/src/gallium/auxiliary/Makefile.sources b/src/gallium/auxiliary/Makefile.sources
> index 9b0c9a3..26a06b2 100644
> --- a/src/gallium/auxiliary/Makefile.sources
> +++ b/src/gallium/auxiliary/Makefile.sources
> @@ -288,8 +288,6 @@ C_SOURCES := \
> util/u_slab.h \
> util/u_split_prim.h \
> util/u_sse.h \
> - util/u_staging.c \
> - util/u_staging.h \
> util/u_string.h \
> util/u_suballoc.c \
> util/u_suballoc.h \
> diff --git a/src/gallium/auxiliary/util/u_staging.c b/src/gallium/auxiliary/util/u_staging.c
> deleted file mode 100644
> index 5b61f5e..0000000
> --- a/src/gallium/auxiliary/util/u_staging.c
> +++ /dev/null
> @@ -1,136 +0,0 @@
> -/**************************************************************************
> - *
> - * Copyright 2010 Luca Barbieri
> - *
> - * Permission is hereby granted, free of charge, to any person obtaining
> - * a copy of this software and associated documentation files (the
> - * "Software"), to deal in the Software without restriction, including
> - * without limitation the rights to use, copy, modify, merge, publish,
> - * distribute, sublicense, and/or sell copies of the Software, and to
> - * permit persons to whom the Software is furnished to do so, subject to
> - * the following conditions:
> - *
> - * The above copyright notice and this permission notice (including the
> - * next paragraph) shall be included in all copies or substantial
> - * portions of the Software.
> - *
> - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> - * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> - * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
> - * IN NO EVENT SHALL THE COPYRIGHT OWNER(S) AND/OR ITS SUPPLIERS BE
> - * LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
> - * OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
> - * WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
> - *
> - **************************************************************************/
> -
> -#include "util/u_staging.h"
> -#include "pipe/p_context.h"
> -#include "util/u_memory.h"
> -#include "util/u_inlines.h"
> -
> -
> -static void
> -util_staging_resource_template(struct pipe_resource *pt, unsigned width,
> - unsigned height, unsigned depth,
> - struct pipe_resource *template)
> -{
> - memset(template, 0, sizeof(struct pipe_resource));
> - if (pt->target != PIPE_BUFFER && depth <= 1)
> - template->target = PIPE_TEXTURE_RECT;
> - else
> - template->target = pt->target;
> - template->format = pt->format;
> - template->width0 = width;
> - template->height0 = height;
> - template->depth0 = depth;
> - template->array_size = 1;
> - template->last_level = 0;
> - template->nr_samples = pt->nr_samples;
> - template->bind = 0;
> - template->usage = PIPE_USAGE_STAGING;
> - template->flags = 0;
> -}
> -
> -
> -struct util_staging_transfer *
> -util_staging_transfer_init(struct pipe_context *pipe,
> - struct pipe_resource *pt,
> - unsigned level, enum pipe_resource_usage usage,
> - const struct pipe_box *box,
> - boolean direct, struct util_staging_transfer *tx)
> -{
> - struct pipe_screen *pscreen = pipe->screen;
> - struct pipe_resource staging_resource_template;
> -
> - pipe_resource_reference(&tx->base.resource, pt);
> - tx->base.level = level;
> - tx->base.usage = usage;
> - tx->base.box = *box;
> -
> - if (direct) {
> - tx->staging_resource = pt;
> - return tx;
> - }
> -
> - util_staging_resource_template(pt, box->width, box->height,
> - box->depth, &staging_resource_template);
> - tx->staging_resource = pscreen->resource_create(pscreen,
> - &staging_resource_template);
> - if (!tx->staging_resource) {
> - pipe_resource_reference(&tx->base.resource, NULL);
> - FREE(tx);
> - return NULL;
> - }
> -
> - if (usage & PIPE_TRANSFER_READ) {
> - /* XXX this looks wrong dst is always the same but looping over src z? */
> - int zi;
> - struct pipe_box sbox;
> - sbox.x = box->x;
> - sbox.y = box->y;
> - sbox.z = box->z;
> - sbox.width = box->width;
> - sbox.height = box->height;
> - sbox.depth = 1;
> - for (zi = 0; zi < box->depth; ++zi) {
> - sbox.z = sbox.z + zi;
> - pipe->resource_copy_region(pipe, tx->staging_resource, 0, 0, 0, 0,
> - tx->base.resource, level, &sbox);
> - }
> - }
> -
> - return tx;
> -}
> -
> -
> -void
> -util_staging_transfer_destroy(struct pipe_context *pipe,
> - struct pipe_transfer *ptx)
> -{
> - struct util_staging_transfer *tx = (struct util_staging_transfer *)ptx;
> -
> - if (tx->staging_resource != tx->base.resource) {
> - if (tx->base.usage & PIPE_TRANSFER_WRITE) {
> - /* XXX this looks wrong src is always the same but looping over dst z? */
> - int zi;
> - struct pipe_box sbox;
> - sbox.x = 0;
> - sbox.y = 0;
> - sbox.z = 0;
> - sbox.width = tx->base.box.width;
> - sbox.height = tx->base.box.height;
> - sbox.depth = 1;
> - for (zi = 0; zi < tx->base.box.depth; ++zi)
> - pipe->resource_copy_region(pipe, tx->base.resource, tx->base.level,
> - tx->base.box.x, tx->base.box.y,
> - tx->base.box.z + zi,
> - tx->staging_resource, 0, &sbox);
> - }
> -
> - pipe_resource_reference(&tx->staging_resource, NULL);
> - }
> -
> - pipe_resource_reference(&ptx->resource, NULL);
> - FREE(ptx);
> -}
> diff --git a/src/gallium/auxiliary/util/u_staging.h b/src/gallium/auxiliary/util/u_staging.h
> deleted file mode 100644
> index eed5584..0000000
> --- a/src/gallium/auxiliary/util/u_staging.h
> +++ /dev/null
> @@ -1,67 +0,0 @@
> -/**************************************************************************
> - *
> - * Copyright 2010 Luca Barbieri
> - *
> - * Permission is hereby granted, free of charge, to any person obtaining
> - * a copy of this software and associated documentation files (the
> - * "Software"), to deal in the Software without restriction, including
> - * without limitation the rights to use, copy, modify, merge, publish,
> - * distribute, sublicense, and/or sell copies of the Software, and to
> - * permit persons to whom the Software is furnished to do so, subject to
> - * the following conditions:
> - *
> - * The above copyright notice and this permission notice (including the
> - * next paragraph) shall be included in all copies or substantial
> - * portions of the Software.
> - *
> - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> - * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> - * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
> - * IN NO EVENT SHALL THE COPYRIGHT OWNER(S) AND/OR ITS SUPPLIERS BE
> - * LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
> - * OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
> - * WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
> - *
> - **************************************************************************/
> -
> -/* Direct3D 10/11 has no concept of transfers. Applications instead
> - * create resources with a STAGING or DYNAMIC usage, copy between them
> - * and the real resource and use Map to map the STAGING/DYNAMIC resource.
> - *
> - * This util module allows to implement Gallium drivers as a Direct3D
> - * driver would be implemented: transfers allocate a resource with
> - * PIPE_USAGE_STAGING, and copy the data between it and the real resource
> - * with resource_copy_region.
> - */
> -
> -#ifndef U_STAGING_H
> -#define U_STAGING_H
> -
> -#include "pipe/p_state.h"
> -
> -struct util_staging_transfer {
> - struct pipe_transfer base;
> -
> - /* if direct, same as base.resource, otherwise the temporary staging
> - * resource
> - */
> - struct pipe_resource *staging_resource;
> -};
> -
> -/* user must be stride, slice_stride and offset.
> - * pt->usage == PIPE_USAGE_DYNAMIC || pt->usage == PIPE_USAGE_STAGING
> - * should be a good value to pass for direct staging resource is currently
> - * created with PIPE_USAGE_STAGING
> - */
> -struct util_staging_transfer *
> -util_staging_transfer_init(struct pipe_context *pipe,
> - struct pipe_resource *pt,
> - unsigned level, enum pipe_resource_usage usage,
> - const struct pipe_box *box,
> - boolean direct, struct util_staging_transfer *tx);
> -
> -void
> -util_staging_transfer_destroy(struct pipe_context *pipe,
> - struct pipe_transfer *ptx);
> -
> -#endif
>
We don't use it internally AFAICT.
Acked-by: Jose Fonseca <jfonseca at vmware.com>
More information about the mesa-dev
mailing list