[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