On 28.11.2017 22:38, Marek Olšák wrote:
From: Marek Olšák <[email protected]>

---
  src/gallium/drivers/radeon/r600_pipe_common.h |   8 --
  src/gallium/drivers/radeon/r600_texture.c     | 110 ++++++++++++++++++++++++--
  src/gallium/drivers/radeonsi/si_state.c       |  98 -----------------------
  3 files changed, 102 insertions(+), 114 deletions(-)

diff --git a/src/gallium/drivers/radeon/r600_pipe_common.h 
b/src/gallium/drivers/radeon/r600_pipe_common.h
index dea1d6f..236b3bc 100644
--- a/src/gallium/drivers/radeon/r600_pipe_common.h
+++ b/src/gallium/drivers/radeon/r600_pipe_common.h
@@ -427,28 +427,20 @@ struct r600_common_screen {
                /* Context flags to set so that all writes from earlier jobs
                 * that end in L2 are seen by CP.
                 */
                unsigned L2_to_cp;
/* Context flags to set so that all writes from earlier
                 * compute jobs are seen by L2 clients.
                 */
                unsigned compute_to_L2;
        } barrier_flags;
-
-       void (*query_opaque_metadata)(struct r600_common_screen *rscreen,
-                                     struct r600_texture *rtex,
-                                     struct radeon_bo_metadata *md);
-
-       void (*apply_opaque_metadata)(struct r600_common_screen *rscreen,
-                                   struct r600_texture *rtex,
-                                   struct radeon_bo_metadata *md);
  };
/* This encapsulates a state or an operation which can emitted into the GPU
   * command stream. */
  struct r600_atom {
        void (*emit)(struct r600_common_context *ctx, struct r600_atom *state);
        unsigned short          id;
  };
struct r600_ring {
diff --git a/src/gallium/drivers/radeon/r600_texture.c 
b/src/gallium/drivers/radeon/r600_texture.c
index 46962eb..1051a3b 100644
--- a/src/gallium/drivers/radeon/r600_texture.c
+++ b/src/gallium/drivers/radeon/r600_texture.c
@@ -13,27 +13,29 @@
   * 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 NON-INFRINGEMENT. IN NO EVENT SHALL
   * THE AUTHOR(S) AND/OR THEIR 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 "r600_pipe_common.h"
+
+#include "radeonsi/si_pipe.h"

For the other cases you used ../radeonsi. Not sure which is better, but it'd be nice to be consistent.

Cheers,
Nicolai


  #include "r600_cs.h"
  #include "r600_query.h"
  #include "util/u_format.h"
  #include "util/u_log.h"
  #include "util/u_memory.h"
  #include "util/u_pack_color.h"
+#include "util/u_resource.h"
  #include "util/u_surface.h"
  #include "util/os_time.h"
  #include <errno.h>
  #include <inttypes.h>
  #include "state_tracker/drm_driver.h"
  #include "amd/common/sid.h"
static void r600_texture_discard_cmask(struct r600_common_screen *rscreen,
                                       struct r600_texture *rtex);
  static enum radeon_surf_mode
@@ -556,20 +558,116 @@ static void r600_reallocate_texture_inplace(struct 
r600_common_context *rctx,
                assert(!rtex->fmask.size);
                assert(!rtex->dcc_offset);
                assert(!rtex->is_depth);
        }
r600_texture_reference(&new_tex, NULL); p_atomic_inc(&rctx->screen->dirty_tex_counter);
  }
+static uint32_t si_get_bo_metadata_word1(struct r600_common_screen *rscreen)
+{
+       return (ATI_VENDOR_ID << 16) | rscreen->info.pci_id;
+}
+
+static void si_query_opaque_metadata(struct r600_common_screen *rscreen,
+                                    struct r600_texture *rtex,
+                                    struct radeon_bo_metadata *md)
+{
+       struct si_screen *sscreen = (struct si_screen*)rscreen;
+       struct pipe_resource *res = &rtex->resource.b.b;
+       static const unsigned char swizzle[] = {
+               PIPE_SWIZZLE_X,
+               PIPE_SWIZZLE_Y,
+               PIPE_SWIZZLE_Z,
+               PIPE_SWIZZLE_W
+       };
+       uint32_t desc[8], i;
+       bool is_array = util_resource_is_array_texture(res);
+
+       /* DRM 2.x.x doesn't support this. */
+       if (rscreen->info.drm_major != 3)
+               return;
+
+       assert(rtex->dcc_separate_buffer == NULL);
+       assert(rtex->fmask.size == 0);
+
+       /* Metadata image format format version 1:
+        * [0] = 1 (metadata format identifier)
+        * [1] = (VENDOR_ID << 16) | PCI_ID
+        * [2:9] = image descriptor for the whole resource
+        *         [2] is always 0, because the base address is cleared
+        *         [9] is the DCC offset bits [39:8] from the beginning of
+        *             the buffer
+        * [10:10+LAST_LEVEL] = mipmap level offset bits [39:8] for each level
+        */
+
+       md->metadata[0] = 1; /* metadata image format version 1 */
+
+       /* TILE_MODE_INDEX is ambiguous without a PCI ID. */
+       md->metadata[1] = si_get_bo_metadata_word1(rscreen);
+
+       si_make_texture_descriptor(sscreen, rtex, true,
+                                  res->target, res->format,
+                                  swizzle, 0, res->last_level, 0,
+                                  is_array ? res->array_size - 1 : 0,
+                                  res->width0, res->height0, res->depth0,
+                                  desc, NULL);
+
+       si_set_mutable_tex_desc_fields(sscreen, rtex, 
&rtex->surface.u.legacy.level[0],
+                                      0, 0, rtex->surface.blk_w, false, desc);
+
+       /* Clear the base address and set the relative DCC offset. */
+       desc[0] = 0;
+       desc[1] &= C_008F14_BASE_ADDRESS_HI;
+       desc[7] = rtex->dcc_offset >> 8;
+
+       /* Dwords [2:9] contain the image descriptor. */
+       memcpy(&md->metadata[2], desc, sizeof(desc));
+       md->size_metadata = 10 * 4;
+
+       /* Dwords [10:..] contain the mipmap level offsets. */
+       if (rscreen->chip_class <= VI) {
+               for (i = 0; i <= res->last_level; i++)
+                       md->metadata[10+i] = rtex->surface.u.legacy.level[i].offset 
>> 8;
+
+               md->size_metadata += (1 + res->last_level) * 4;
+       }
+}
+
+static void si_apply_opaque_metadata(struct r600_common_screen *rscreen,
+                                    struct r600_texture *rtex,
+                                    struct radeon_bo_metadata *md)
+{
+       uint32_t *desc = &md->metadata[2];
+
+       if (rscreen->chip_class < VI)
+               return;
+
+       /* Return if DCC is enabled. The texture should be set up with it
+        * already.
+        */
+       if (md->size_metadata >= 11 * 4 &&
+           md->metadata[0] != 0 &&
+           md->metadata[1] == si_get_bo_metadata_word1(rscreen) &&
+           G_008F28_COMPRESSION_EN(desc[6])) {
+               rtex->dcc_offset = (uint64_t)desc[7] << 8;
+               return;
+       }
+
+       /* Disable DCC. These are always set by texture_from_handle and must
+        * be cleared here.
+        */
+       rtex->dcc_offset = 0;
+}
+
  static boolean r600_texture_get_handle(struct pipe_screen* screen,
                                       struct pipe_context *ctx,
                                       struct pipe_resource *resource,
                                       struct winsys_handle *whandle,
                                         unsigned usage)
  {
        struct r600_common_screen *rscreen = (struct r600_common_screen*)screen;
        struct r600_common_context *rctx;
        struct r600_resource *res = (struct r600_resource*)resource;
        struct r600_texture *rtex = (struct r600_texture*)resource;
@@ -619,23 +717,21 @@ static boolean r600_texture_get_handle(struct 
pipe_screen* screen,
                        /* Disable CMASK if flush_resource isn't going
                         * to be called.
                         */
                        if (rtex->cmask.size)
                                r600_texture_discard_cmask(rscreen, rtex);
                }
/* Set metadata. */
                if (!res->b.is_shared || update_metadata) {
                        r600_texture_init_metadata(rscreen, rtex, &metadata);
-                       if (rscreen->query_opaque_metadata)
-                               rscreen->query_opaque_metadata(rscreen, rtex,
-                                                              &metadata);
+                       si_query_opaque_metadata(rscreen, rtex, &metadata);
rscreen->ws->buffer_set_metadata(res->buf, &metadata);
                }
if (rscreen->chip_class >= GFX9) {
                        offset = rtex->surface.u.gfx9.surf_offset;
                        stride = rtex->surface.u.gfx9.surf_pitch *
                                 rtex->surface.bpe;
                        slice_size = rtex->surface.u.gfx9.surf_slice_size;
                } else {
@@ -1373,22 +1469,21 @@ static struct pipe_resource 
*r600_texture_from_handle(struct pipe_screen *screen
                return NULL;
        }
rtex = r600_texture_create_object(screen, templ, buf, &surface);
        if (!rtex)
                return NULL;
rtex->resource.b.is_shared = true;
        rtex->resource.external_usage = usage;
- if (rscreen->apply_opaque_metadata)
-               rscreen->apply_opaque_metadata(rscreen, rtex, &metadata);
+       si_apply_opaque_metadata(rscreen, rtex, &metadata);
assert(rtex->surface.tile_swizzle == 0);
        return &rtex->resource.b.b;
  }
bool si_init_flushed_depth_texture(struct pipe_context *ctx,
                                   struct pipe_resource *texture,
                                   struct r600_texture **staging)
  {
        struct r600_texture *rtex = (struct r600_texture*)texture;
@@ -2339,22 +2434,21 @@ r600_texture_from_memobj(struct pipe_screen *screen,
                return NULL;
/* r600_texture_create_object doesn't increment refcount of
         * memobj->buf, so increment it here.
         */
        pb_reference(&buf, memobj->buf);
rtex->resource.b.is_shared = true;
        rtex->resource.external_usage = PIPE_HANDLE_USAGE_READ_WRITE;
- if (rscreen->apply_opaque_metadata)
-               rscreen->apply_opaque_metadata(rscreen, rtex, &metadata);
+       si_apply_opaque_metadata(rscreen, rtex, &metadata);
return &rtex->resource.b.b;
  }
static bool si_check_resource_capability(struct pipe_screen *screen,
                                         struct pipe_resource *resource,
                                         unsigned bind)
  {
        struct r600_texture *tex = (struct r600_texture*)resource;
diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/radeonsi/si_state.c
index cdab8ea..b68dea9 100644
--- a/src/gallium/drivers/radeonsi/si_state.c
+++ b/src/gallium/drivers/radeonsi/si_state.c
@@ -4580,121 +4580,23 @@ void si_init_state_functions(struct si_context *sctx)
        sctx->b.b.set_active_query_state = si_set_active_query_state;
        sctx->b.set_occlusion_query_state = si_set_occlusion_query_state;
        sctx->b.save_qbo_state = si_save_qbo_state;
        sctx->b.need_gfx_cs_space = si_need_gfx_cs_space;
sctx->b.b.draw_vbo = si_draw_vbo; si_init_config(sctx);
  }
-static uint32_t si_get_bo_metadata_word1(struct r600_common_screen *rscreen)
-{
-       return (ATI_VENDOR_ID << 16) | rscreen->info.pci_id;
-}
-
-static void si_query_opaque_metadata(struct r600_common_screen *rscreen,
-                                    struct r600_texture *rtex,
-                                    struct radeon_bo_metadata *md)
-{
-       struct si_screen *sscreen = (struct si_screen*)rscreen;
-       struct pipe_resource *res = &rtex->resource.b.b;
-       static const unsigned char swizzle[] = {
-               PIPE_SWIZZLE_X,
-               PIPE_SWIZZLE_Y,
-               PIPE_SWIZZLE_Z,
-               PIPE_SWIZZLE_W
-       };
-       uint32_t desc[8], i;
-       bool is_array = util_resource_is_array_texture(res);
-
-       /* DRM 2.x.x doesn't support this. */
-       if (rscreen->info.drm_major != 3)
-               return;
-
-       assert(rtex->dcc_separate_buffer == NULL);
-       assert(rtex->fmask.size == 0);
-
-       /* Metadata image format format version 1:
-        * [0] = 1 (metadata format identifier)
-        * [1] = (VENDOR_ID << 16) | PCI_ID
-        * [2:9] = image descriptor for the whole resource
-        *         [2] is always 0, because the base address is cleared
-        *         [9] is the DCC offset bits [39:8] from the beginning of
-        *             the buffer
-        * [10:10+LAST_LEVEL] = mipmap level offset bits [39:8] for each level
-        */
-
-       md->metadata[0] = 1; /* metadata image format version 1 */
-
-       /* TILE_MODE_INDEX is ambiguous without a PCI ID. */
-       md->metadata[1] = si_get_bo_metadata_word1(rscreen);
-
-       si_make_texture_descriptor(sscreen, rtex, true,
-                                  res->target, res->format,
-                                  swizzle, 0, res->last_level, 0,
-                                  is_array ? res->array_size - 1 : 0,
-                                  res->width0, res->height0, res->depth0,
-                                  desc, NULL);
-
-       si_set_mutable_tex_desc_fields(sscreen, rtex, 
&rtex->surface.u.legacy.level[0],
-                                      0, 0, rtex->surface.blk_w, false, desc);
-
-       /* Clear the base address and set the relative DCC offset. */
-       desc[0] = 0;
-       desc[1] &= C_008F14_BASE_ADDRESS_HI;
-       desc[7] = rtex->dcc_offset >> 8;
-
-       /* Dwords [2:9] contain the image descriptor. */
-       memcpy(&md->metadata[2], desc, sizeof(desc));
-       md->size_metadata = 10 * 4;
-
-       /* Dwords [10:..] contain the mipmap level offsets. */
-       if (rscreen->chip_class <= VI) {
-               for (i = 0; i <= res->last_level; i++)
-                       md->metadata[10+i] = rtex->surface.u.legacy.level[i].offset 
>> 8;
-
-               md->size_metadata += (1 + res->last_level) * 4;
-       }
-}
-
-static void si_apply_opaque_metadata(struct r600_common_screen *rscreen,
-                                    struct r600_texture *rtex,
-                                    struct radeon_bo_metadata *md)
-{
-       uint32_t *desc = &md->metadata[2];
-
-       if (rscreen->chip_class < VI)
-               return;
-
-       /* Return if DCC is enabled. The texture should be set up with it
-        * already.
-        */
-       if (md->size_metadata >= 11 * 4 &&
-           md->metadata[0] != 0 &&
-           md->metadata[1] == si_get_bo_metadata_word1(rscreen) &&
-           G_008F28_COMPRESSION_EN(desc[6])) {
-               rtex->dcc_offset = (uint64_t)desc[7] << 8;
-               return;
-       }
-
-       /* Disable DCC. These are always set by texture_from_handle and must
-        * be cleared here.
-        */
-       rtex->dcc_offset = 0;
-}
-
  void si_init_screen_state_functions(struct si_screen *sscreen)
  {
        sscreen->b.b.is_format_supported = si_is_format_supported;
-       sscreen->b.query_opaque_metadata = si_query_opaque_metadata;
-       sscreen->b.apply_opaque_metadata = si_apply_opaque_metadata;
  }
static void si_set_grbm_gfx_index(struct si_context *sctx,
                                  struct si_pm4_state *pm4,  unsigned value)
  {
        unsigned reg = sctx->b.chip_class >= CIK ? R_030800_GRBM_GFX_INDEX :
                                                   R_00802C_GRBM_GFX_INDEX;
        si_pm4_set_reg(pm4, reg, value);
  }


--
Lerne, wie die Welt wirklich ist,
Aber vergiss niemals, wie sie sein sollte.
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to