[Mesa-dev] [PATCH 1/2] gallium/util: add u_transfer_helper

Rob Clark robdclark at gmail.com
Fri Dec 1 13:43:04 UTC 2017


On Thu, Nov 30, 2017 at 10:33 AM, Rob Clark <robdclark at gmail.com> wrote:
> On Thu, Nov 30, 2017 at 10:04 AM, Nicolai Hähnle <nhaehnle at gmail.com> wrote:
>> On 29.11.2017 14:48, Rob Clark wrote:
>>>
>>> Add a new helper that drivers can use to emulate various things that
>>> need special handling in particular in transfer_map:
>>>
>>>   1) z32_s8x24.. gl/gallium treats this as a single buffer with depth
>>>      and stencil interleaved but hardware frequently treats this as
>>>      separate z32 and s8 buffers.  Special pack/unpack handling is
>>>      needed in transfer_map/unmap to pack/unpack the exposed buffer
>>>
>>>   2) fake RGTC.. GPUs designed with GLES in mind, but which can other-
>>>      wise do GL3, if native RGTC is not supported it can be emulated
>>>      by converting to uncompressed internally, but needs pack/unpack
>>>      in transfer_map/unmap
>>>
>>> This could be possibly extended to handle MSAA resolve in map/unmap.
>>
>>
>> This looks mostly fine (though the MSAA thing needs to be resolved one way
>
> I was hoping if Eric was interested in using this, that maybe he could
> add in the MSAA resolve bits.. otherwise that might have to wait until
> I implement non-fake MSAA.
>
>> or the other), except there's a clash if you ever want to use threaded
>> contexts, which have their own threaded_resource and threaded_transfer.
>>
>> I haven't really given any thought yet to how to reconcile those.
>
> hmm, ok, I see the issue.. maybe it is ok though, if the driver
> installs the u_transfer_helper_* pipe fxns in the threaded_context
> that wraps the real context, and the vtbl used by u_transfer_helper
> has the threaded_context fxns instead of actual driver fxns.  (Since
> the transfer_helper is installed into the pipe_screen, it means you
> couldn't mix threaded and non-threaded contexts under a single
> screen.. not sure if there is ever a reason to do that?)

hmm, so the issue actually is there but not w/ threaded_transfer vs
u_transfer.. instead with threaded_resource vs u_transfer_resource.
:-/

u_transfer_helper really only needs to track the additional stencil
buffer and internal_format (although maybe adding MSAA resolves could
add more to that).. possibly I could track that a different way
(setter/getter fxns in vtbl?)

BR,
-R

> There are two cases with the transfer-helper:
>
>  1) the transfer is passed directly through to the driver and not
> intercepted.  In which case the pipe_transfer returned from
> ->transfer_map() is the driver's transfer_map (which doesn't subclass
> u_transfer.. which I should probably move to the .c file)
>
>  2) the transfer is handled by the helper, in which case the helper
> calls the driver's original ->transfer_map() via vtbl.  In this case
> the pipe_transfer passed back to st is u_transfer, but it has pointers
> to the driver's real pipe_transfer(s).  The pointers to the real
> pipe_transfer's could just as well be threaded_transfer's.
>
> In either case, the driver doesn't need to subclass u_transfer.. which
> I should move to .c to make move obvious, I guess.
>
> So possibly threaded_context_create() might need to gain a 'bool
> use_transfer_helper' arg, or something like that.  But doesn't seem
> like a major problem.
>
> BR,
> -R
>
>> Cheers,
>> Nicolai
>>
>>
>>
>>>
>>> Signed-off-by: Rob Clark <robdclark at gmail.com>
>>> ---
>>>   src/gallium/auxiliary/Makefile.sources         |   2 +
>>>   src/gallium/auxiliary/meson.build              |   2 +
>>>   src/gallium/auxiliary/util/u_transfer_helper.c | 364
>>> +++++++++++++++++++++++++
>>>   src/gallium/auxiliary/util/u_transfer_helper.h | 121 ++++++++
>>>   src/gallium/include/pipe/p_screen.h            |   8 +-
>>>   5 files changed, 496 insertions(+), 1 deletion(-)
>>>   create mode 100644 src/gallium/auxiliary/util/u_transfer_helper.c
>>>   create mode 100644 src/gallium/auxiliary/util/u_transfer_helper.h
>>>
>>> diff --git a/src/gallium/auxiliary/Makefile.sources
>>> b/src/gallium/auxiliary/Makefile.sources
>>> index f40c4723fae..a2dae04698c 100644
>>> --- a/src/gallium/auxiliary/Makefile.sources
>>> +++ b/src/gallium/auxiliary/Makefile.sources
>>> @@ -304,6 +304,8 @@ C_SOURCES := \
>>>         util/u_tile.h \
>>>         util/u_transfer.c \
>>>         util/u_transfer.h \
>>> +       util/u_transfer_helper.c \
>>> +       util/u_transfer_helper.h \
>>>         util/u_threaded_context.c \
>>>         util/u_threaded_context.h \
>>>         util/u_threaded_context_calls.h \
>>> diff --git a/src/gallium/auxiliary/meson.build
>>> b/src/gallium/auxiliary/meson.build
>>> index 3e623fd099f..8c242ec1a05 100644
>>> --- a/src/gallium/auxiliary/meson.build
>>> +++ b/src/gallium/auxiliary/meson.build
>>> @@ -324,6 +324,8 @@ files_libgallium = files(
>>>     'util/u_tile.h',
>>>     'util/u_transfer.c',
>>>     'util/u_transfer.h',
>>> +  'util/u_transfer_helper.c',
>>> +  'util/u_transfer_helper.h',
>>>     'util/u_threaded_context.c',
>>>     'util/u_threaded_context.h',
>>>     'util/u_threaded_context_calls.h',
>>> diff --git a/src/gallium/auxiliary/util/u_transfer_helper.c
>>> b/src/gallium/auxiliary/util/u_transfer_helper.c
>>> new file mode 100644
>>> index 00000000000..fb4024db35b
>>> --- /dev/null
>>> +++ b/src/gallium/auxiliary/util/u_transfer_helper.c
>>> @@ -0,0 +1,364 @@
>>> +/*
>>> + * Copyright © 2017 Red Hat
>>> + *
>>> + * 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 AUTHORS OR COPYRIGHT HOLDERS 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 "pipe/p_screen.h"
>>> +
>>> +#include "util/u_box.h"
>>> +#include "util/u_format.h"
>>> +#include "util/u_format_rgtc.h"
>>> +#include "util/u_format_zs.h"
>>> +#include "util/u_inlines.h"
>>> +#include "util/u_transfer_helper.h"
>>> +
>>> +
>>> +struct u_transfer_helper {
>>> +   const struct u_transfer_vtbl *vtbl;
>>> +   bool separate_z32s8;
>>> +   bool fake_rgtc;
>>> +   bool msaa_map;
>>> +};
>>> +
>>> +static inline bool handle_transfer(struct pipe_resource *prsc)
>>> +{
>>> +   struct u_transfer_helper *helper = prsc->screen->transfer_helper;
>>> +   struct u_transfer_resource *rsc = u_transfer_resource(prsc);
>>> +   return (rsc->internal_format != prsc->format) ||
>>> +         (helper->msaa_map && (prsc->nr_samples > 1));
>>> +}
>>> +
>>> +/* The pipe_transfer ptr could either be the driver's, or u_transfer,
>>> + * depending on whether we are intervening or not.  Check
>>> handle_transfer()
>>> + * before dereferencing.
>>> + */
>>> +struct u_transfer {
>>> +   struct pipe_transfer base;
>>> +   struct pipe_transfer *trans;   /* driver's transfer */
>>> +   struct pipe_transfer *trans2;  /* 2nd transfer for z32s8 */
>>> +   void *ptr, *ptr2;              /* ptr to trans, and trans2 */
>>> +   void *staging;                 /* staging buffer */
>>> +};
>>> +
>>> +static inline struct u_transfer *
>>> +u_transfer(struct pipe_transfer *ptrans)
>>> +{
>>> +   debug_assert(handle_transfer(ptrans->resource));
>>> +   return (struct u_transfer *)ptrans;
>>> +}
>>> +
>>> +struct pipe_resource *
>>> +u_transfer_helper_resource_create(struct pipe_screen *pscreen,
>>> +                                  const struct pipe_resource *templ)
>>> +{
>>> +   struct u_transfer_helper *helper = pscreen->transfer_helper;
>>> +   enum pipe_format format = templ->format;
>>> +   struct pipe_resource *prsc;
>>> +   struct u_transfer_resource *rsc;
>>> +
>>> +   if ((format == PIPE_FORMAT_Z32_FLOAT_S8X24_UINT) &&
>>> helper->separate_z32s8) {
>>> +      struct pipe_resource t = *templ;
>>> +      t.format = PIPE_FORMAT_Z32_FLOAT;
>>> +
>>> +      prsc = helper->vtbl->resource_create(pscreen, &t);
>>> +      if (!prsc)
>>> +         return NULL;
>>> +
>>> +      rsc = u_transfer_resource(prsc);
>>> +
>>> +      prsc->format = format;  /* frob the format back to the "external"
>>> format */
>>> +      rsc->internal_format = t.format;
>>> +
>>> +      t.format = PIPE_FORMAT_S8_UINT;
>>> +      rsc->stencil = helper->vtbl->resource_create(pscreen, &t);
>>> +
>>> +      if (!rsc->stencil) {
>>> +         helper->vtbl->resource_destroy(pscreen, prsc);
>>> +         return NULL;
>>> +      }
>>> +   } else if ((util_format_description(format)->layout ==
>>> UTIL_FORMAT_LAYOUT_RGTC) &&
>>> +         helper->fake_rgtc) {
>>> +      struct pipe_resource t = *templ;
>>> +      t.format = PIPE_FORMAT_R8G8B8A8_UNORM;
>>> +
>>> +      prsc = helper->vtbl->resource_create(pscreen, &t);
>>> +      if (!prsc)
>>> +         return NULL;
>>> +
>>> +      rsc = u_transfer_resource(prsc);
>>> +
>>> +      prsc->format = format;  /* frob the format back to the "external"
>>> format */
>>> +      rsc->internal_format = t.format;
>>> +   } else {
>>> +      /* normal case, no special handling: */
>>> +      prsc = helper->vtbl->resource_create(pscreen, templ);
>>> +      if (!prsc)
>>> +         return NULL;
>>> +
>>> +      rsc = u_transfer_resource(prsc);
>>> +      rsc->stencil = NULL;
>>> +      rsc->internal_format = prsc->format;
>>> +   }
>>> +
>>> +   return prsc;
>>> +}
>>> +
>>> +void
>>> +u_transfer_helper_resource_destroy(struct pipe_screen *pscreen,
>>> +                                   struct pipe_resource *prsc)
>>> +{
>>> +   struct u_transfer_helper *helper = pscreen->transfer_helper;
>>> +   struct u_transfer_resource *rsc = u_transfer_resource(prsc);
>>> +
>>> +   if (rsc->stencil)
>>> +      helper->vtbl->resource_destroy(pscreen, rsc->stencil);
>>> +
>>> +   helper->vtbl->resource_destroy(pscreen, prsc);
>>> +}
>>> +
>>> +static bool needs_pack(unsigned usage)
>>> +{
>>> +   return (usage & PIPE_TRANSFER_READ) &&
>>> +      !(usage & (PIPE_TRANSFER_DISCARD_WHOLE_RESOURCE |
>>> PIPE_TRANSFER_DISCARD_RANGE));
>>> +}
>>> +
>>> +void *
>>> +u_transfer_helper_transfer_map(struct pipe_context *pctx,
>>> +                               struct pipe_resource *prsc,
>>> +                               unsigned level,
>>> +                               unsigned usage,
>>> +                               const struct pipe_box *box,
>>> +                               struct pipe_transfer **pptrans)
>>> +{
>>> +   struct u_transfer_helper *helper = pctx->screen->transfer_helper;
>>> +   struct u_transfer_resource *rsc = u_transfer_resource(prsc);
>>> +   struct u_transfer *trans;
>>> +   struct pipe_transfer *ptrans;
>>> +   enum pipe_format format = prsc->format;
>>> +   unsigned width = box->width;
>>> +   unsigned height = box->height;
>>> +
>>> +   if (!handle_transfer(prsc))
>>> +      return helper->vtbl->transfer_map(pctx, prsc, level, usage, box,
>>> pptrans);
>>> +
>>> +   debug_assert(box->depth == 1);
>>> +
>>> +   trans = calloc(1, sizeof(*trans));
>>> +   if (!trans)
>>> +      return NULL;
>>> +
>>> +   ptrans = &trans->base;
>>> +   pipe_resource_reference(&ptrans->resource, prsc);
>>> +   ptrans->level = level;
>>> +   ptrans->usage = usage;
>>> +   ptrans->box   = *box;
>>> +   ptrans->stride = util_format_get_stride(format, box->width);
>>> +   ptrans->layer_stride = ptrans->stride * box->height;
>>> +
>>> +   trans->staging = malloc(ptrans->layer_stride);
>>> +   if (!trans->staging)
>>> +      goto fail;
>>> +
>>> +   trans->ptr = helper->vtbl->transfer_map(pctx, prsc, level, usage, box,
>>> +                                           &trans->trans);
>>> +   if (!trans->ptr)
>>> +      goto fail;
>>> +
>>> +   if (prsc->format == PIPE_FORMAT_Z32_FLOAT_S8X24_UINT) {
>>> +      trans->ptr2 = helper->vtbl->transfer_map(pctx, rsc->stencil, level,
>>> +                                               usage, box,
>>> &trans->trans2);
>>> +
>>> +      if (needs_pack(usage)) {
>>> +         util_format_z32_float_s8x24_uint_pack_z_float(trans->staging,
>>> +                                                       ptrans->stride,
>>> +                                                       trans->ptr,
>>> +
>>> trans->trans->stride,
>>> +                                                       width, height);
>>> +         util_format_z32_float_s8x24_uint_pack_s_8uint(trans->staging,
>>> +                                                       ptrans->stride,
>>> +                                                       trans->ptr2,
>>> +
>>> trans->trans2->stride,
>>> +                                                       width, height);
>>> +      }
>>> +   } else if (needs_pack(usage) &&
>>> +              util_format_description(prsc->format)->layout ==
>>> UTIL_FORMAT_LAYOUT_RGTC) {
>>> +      switch (prsc->format) {
>>> +      case PIPE_FORMAT_RGTC1_UNORM:
>>> +      case PIPE_FORMAT_RGTC1_SNORM:
>>> +      case PIPE_FORMAT_LATC1_UNORM:
>>> +      case PIPE_FORMAT_LATC1_SNORM:
>>> +         util_format_rgtc1_unorm_pack_rgba_8unorm(trans->staging,
>>> +                                                  ptrans->stride,
>>> +                                                  trans->ptr,
>>> +                                                  trans->trans->stride,
>>> +                                                  width, height);
>>> +         break;
>>> +      case PIPE_FORMAT_RGTC2_UNORM:
>>> +      case PIPE_FORMAT_RGTC2_SNORM:
>>> +      case PIPE_FORMAT_LATC2_UNORM:
>>> +      case PIPE_FORMAT_LATC2_SNORM:
>>> +         util_format_rgtc2_unorm_pack_rgba_8unorm(trans->staging,
>>> +                                                  ptrans->stride,
>>> +                                                  trans->ptr,
>>> +                                                  trans->trans->stride,
>>> +                                                  width, height);
>>> +         break;
>>> +      default:
>>> +         assert(!"Unexpected format");
>>> +         break;
>>> +      }
>>> +   } else {
>>> +      unreachable("bleh");
>>> +   }
>>> +
>>> +   *pptrans = ptrans;
>>> +   return trans->staging;
>>> +
>>> +fail:
>>> +   if (trans->trans)
>>> +      helper->vtbl->transfer_unmap(pctx, trans->trans);
>>> +   if (trans->trans2)
>>> +      helper->vtbl->transfer_unmap(pctx, trans->trans2);
>>> +   pipe_resource_reference(&ptrans->resource, NULL);
>>> +   free(trans->staging);
>>> +   free(trans);
>>> +   return NULL;
>>> +}
>>> +
>>> +static void
>>> +flush_region(struct pipe_transfer *ptrans, const struct pipe_box *box)
>>> +{
>>> +   struct u_transfer *trans = u_transfer(ptrans);
>>> +   enum pipe_format format = ptrans->resource->format;
>>> +   unsigned width = ptrans->box.width;
>>> +   unsigned height = ptrans->box.height;
>>> +
>>> +   if (!(ptrans->usage & PIPE_TRANSFER_WRITE))
>>> +      return;
>>> +
>>> +   switch (format) {
>>> +   case PIPE_FORMAT_Z32_FLOAT_S8X24_UINT:
>>> +      util_format_z32_float_s8x24_uint_unpack_z_float(trans->ptr,
>>> +
>>> trans->trans->stride,
>>> +                                                      trans->staging,
>>> +                                                      ptrans->stride,
>>> +                                                      width, height);
>>> +      /* fallthru */
>>> +   case PIPE_FORMAT_X32_S8X24_UINT:
>>> +      util_format_z32_float_s8x24_uint_unpack_s_8uint(trans->ptr2,
>>> +
>>> trans->trans2->stride,
>>> +                                                      trans->staging,
>>> +                                                      ptrans->stride,
>>> +                                                      width, height);
>>> +      break;
>>> +   case PIPE_FORMAT_RGTC1_UNORM:
>>> +   case PIPE_FORMAT_RGTC1_SNORM:
>>> +   case PIPE_FORMAT_LATC1_UNORM:
>>> +   case PIPE_FORMAT_LATC1_SNORM:
>>> +      util_format_rgtc1_unorm_unpack_rgba_8unorm(trans->ptr,
>>> +                                                 trans->trans->stride,
>>> +                                                 trans->staging,
>>> +                                                 ptrans->stride,
>>> +                                                 width, height);
>>> +      break;
>>> +   case PIPE_FORMAT_RGTC2_UNORM:
>>> +   case PIPE_FORMAT_RGTC2_SNORM:
>>> +   case PIPE_FORMAT_LATC2_UNORM:
>>> +   case PIPE_FORMAT_LATC2_SNORM:
>>> +      util_format_rgtc2_unorm_unpack_rgba_8unorm(trans->ptr,
>>> +                                                 trans->trans->stride,
>>> +                                                 trans->staging,
>>> +                                                 ptrans->stride,
>>> +                                                 width, height);
>>> +      break;
>>> +   default:
>>> +      assert(!"Unexpected staging transfer type");
>>> +      break;
>>> +   }
>>> +}
>>> +
>>> +void
>>> +u_transfer_helper_transfer_flush_region(struct pipe_context *pctx,
>>> +                                        struct pipe_transfer *ptrans,
>>> +                                        const struct pipe_box *box)
>>> +{
>>> +   struct u_transfer_helper *helper = pctx->screen->transfer_helper;
>>> +
>>> +   if (handle_transfer(ptrans->resource)) {
>>> +      struct u_transfer *trans = u_transfer(ptrans);
>>> +
>>> +      flush_region(ptrans, box);
>>> +
>>> +      helper->vtbl->transfer_flush_region(pctx, trans->trans, box);
>>> +      if (trans->trans2)
>>> +         helper->vtbl->transfer_flush_region(pctx, trans->trans2, box);
>>> +
>>> +   } else {
>>> +      helper->vtbl->transfer_flush_region(pctx, ptrans, box);
>>> +   }
>>> +}
>>> +
>>> +void
>>> +u_transfer_helper_transfer_unmap(struct pipe_context *pctx,
>>> +                                 struct pipe_transfer *ptrans)
>>> +{
>>> +   struct u_transfer_helper *helper = pctx->screen->transfer_helper;
>>> +
>>> +   if (handle_transfer(ptrans->resource)) {
>>> +      struct u_transfer *trans = u_transfer(ptrans);
>>> +
>>> +      if (!(ptrans->usage & PIPE_TRANSFER_FLUSH_EXPLICIT)) {
>>> +         struct pipe_box box;
>>> +         u_box_2d(0, 0, ptrans->box.width, ptrans->box.height, &box);
>>> +         flush_region(ptrans, &box);
>>> +      }
>>> +
>>> +      helper->vtbl->transfer_unmap(pctx, trans->trans);
>>> +      if (trans->trans2)
>>> +         helper->vtbl->transfer_unmap(pctx, trans->trans2);
>>> +
>>> +      free(trans);
>>> +   } else {
>>> +      helper->vtbl->transfer_unmap(pctx, ptrans);
>>> +   }
>>> +}
>>> +
>>> +struct u_transfer_helper *
>>> +u_transfer_helper_create(const struct u_transfer_vtbl *vtbl,
>>> +                         bool separate_z32s8,
>>> +                         bool fake_rgtc,
>>> +                         bool msaa_map)
>>> +{
>>> +   struct u_transfer_helper *helper = calloc(1, sizeof(*helper));
>>> +
>>> +   helper->vtbl = vtbl;
>>> +   helper->separate_z32s8 = separate_z32s8;
>>> +   helper->fake_rgtc = fake_rgtc;
>>> +   helper->msaa_map = msaa_map;
>>> +
>>> +   return helper;
>>> +}
>>> +
>>> +void
>>> +u_transfer_helper_destroy(struct u_transfer_helper *helper)
>>> +{
>>> +   free(helper);
>>> +}
>>> diff --git a/src/gallium/auxiliary/util/u_transfer_helper.h
>>> b/src/gallium/auxiliary/util/u_transfer_helper.h
>>> new file mode 100644
>>> index 00000000000..40848e0d8e5
>>> --- /dev/null
>>> +++ b/src/gallium/auxiliary/util/u_transfer_helper.h
>>> @@ -0,0 +1,121 @@
>>> +/*
>>> + * Copyright © 2017 Red Hat
>>> + *
>>> + * 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 AUTHORS OR COPYRIGHT HOLDERS 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.
>>> + */
>>> +
>>> +#ifndef _U_TRANSFER_HELPER_H
>>> +#define _U_TRANSFER_HELPER_H
>>> +
>>> +#include "pipe/p_state.h"
>>> +#include "pipe/p_context.h"
>>> +
>>> +#ifdef __cplusplus
>>> +extern "C" {
>>> +#endif
>>> +
>>> +/* A helper to implement various "lowering" for transfers:
>>> + *
>>> + *  - exposing separate z32 and s8 as z32x24s8
>>> + *  - fake RGTC support for GLES class hardware which needs it to expose
>>> GL3+
>>> + *  - MSAA resolves (TODO)
>>> + *
>>> + * To use this, drivers should:
>>> + *
>>> + *  1) subclass u_transfer_resource instead of pipe_resource directly
>>> + *  2) populate u_transfer_vtbl and plug that into
>>> pipe_screen::transfer_helper
>>> + *  3) plug the the transfer helpers into pipe_screen/pipe_context
>>> + */
>>> +
>>> +struct u_transfer_resource {
>>> +   struct pipe_resource b;
>>> +   enum pipe_format internal_format;
>>> +   struct pipe_resource *stencil;     /* separate stencil for z32x24s8 */
>>> +};
>>> +
>>> +static inline struct u_transfer_resource *
>>> +u_transfer_resource(struct pipe_resource *prsc)
>>> +{
>>> +   return (struct u_transfer_resource *)prsc;
>>> +}
>>> +
>>> +struct u_transfer_vtbl {
>>> +   /* NOTE I am not expecting resource_create_from_handle() or
>>> +    * resource_create_with_modifiers() paths to be creating any
>>> +    * resources that need special handling.  Otherwise they would
>>> +    * need to be wrapped too.
>>> +    */
>>> +   struct pipe_resource * (*resource_create)(struct pipe_screen *pscreen,
>>> +                                             const struct pipe_resource
>>> *templ);
>>> +
>>> +   void (*resource_destroy)(struct pipe_screen *pscreen,
>>> +                            struct pipe_resource *prsc);
>>> +
>>> +   void *(*transfer_map)(struct pipe_context *pctx,
>>> +                         struct pipe_resource *prsc,
>>> +                         unsigned level,
>>> +                         unsigned usage,
>>> +                         const struct pipe_box *box,
>>> +                         struct pipe_transfer **pptrans);
>>> +
>>> +
>>> +   void (*transfer_flush_region)(struct pipe_context *pctx,
>>> +                                 struct pipe_transfer *ptrans,
>>> +                                 const struct pipe_box *box);
>>> +
>>> +   void (*transfer_unmap)(struct pipe_context *pctx,
>>> +                          struct pipe_transfer *ptrans);
>>> +};
>>> +
>>> +struct pipe_resource *u_transfer_helper_resource_create(
>>> +      struct pipe_screen *pscreen, const struct pipe_resource *templ);
>>> +
>>> +void u_transfer_helper_resource_destroy(struct pipe_screen *pscreen,
>>> +                                        struct pipe_resource *prsc);
>>> +
>>> +void *u_transfer_helper_transfer_map(struct pipe_context *pctx,
>>> +                                     struct pipe_resource *prsc,
>>> +                                     unsigned level,
>>> +                                     unsigned usage,
>>> +                                     const struct pipe_box *box,
>>> +                                     struct pipe_transfer **pptrans);
>>> +
>>> +
>>> +void u_transfer_helper_transfer_flush_region(struct pipe_context *pctx,
>>> +                                             struct pipe_transfer
>>> *ptrans,
>>> +                                             const struct pipe_box *box);
>>> +
>>> +void u_transfer_helper_transfer_unmap(struct pipe_context *pctx,
>>> +                                      struct pipe_transfer *ptrans);
>>> +
>>> +struct u_transfer_helper;
>>> +
>>> +struct u_transfer_helper * u_transfer_helper_create(const struct
>>> u_transfer_vtbl *vtbl,
>>> +                                                    bool separate_z32s8,
>>> +                                                    bool fake_rgtc,
>>> +                                                    bool msaa_map);
>>> +
>>> +void u_transfer_helper_destroy(struct u_transfer_helper *helper);
>>> +
>>> +#ifdef __cplusplus
>>> +} // extern "C" {
>>> +#endif
>>> +
>>> +#endif /* _U_TRANSFER_HELPER_H */
>>> diff --git a/src/gallium/include/pipe/p_screen.h
>>> b/src/gallium/include/pipe/p_screen.h
>>> index c249c7d63b4..101e229088b 100644
>>> --- a/src/gallium/include/pipe/p_screen.h
>>> +++ b/src/gallium/include/pipe/p_screen.h
>>> @@ -60,7 +60,7 @@ struct pipe_box;
>>>   struct pipe_memory_info;
>>>   struct disk_cache;
>>>   struct driOptionCache;
>>> -
>>> +struct u_transfer_helper;
>>>     /**
>>>    * Gallium screen/adapter context.  Basically everything
>>> @@ -68,6 +68,12 @@ struct driOptionCache;
>>>    * context.
>>>    */
>>>   struct pipe_screen {
>>> +
>>> +   /**
>>> +    * For drivers using u_transfer_helper:
>>> +    */
>>> +   struct u_transfer_helper *transfer_helper;
>>> +
>>>      void (*destroy)( struct pipe_screen * );
>>>        const char *(*get_name)( struct pipe_screen * );
>>>
>>
>>
>> --
>> Lerne, wie die Welt wirklich ist,
>> Aber vergiss niemals, wie sie sein sollte.


More information about the mesa-dev mailing list