freedreno: Remove the "active" member of queries.
[mesa.git] / src / gallium / drivers / freedreno / freedreno_resource.c
index 97bead52686fb669306fb23a07773063f470cb0e..56f5c478b275edfc43c9c73b29f690bf1a7ee7a2 100644 (file)
@@ -24,9 +24,9 @@
  *    Rob Clark <robclark@freedesktop.org>
  */
 
-#include "util/u_format.h"
-#include "util/u_format_rgtc.h"
-#include "util/u_format_zs.h"
+#include "util/format/u_format.h"
+#include "util/format/u_format_rgtc.h"
+#include "util/format/u_format_zs.h"
 #include "util/u_inlines.h"
 #include "util/u_transfer.h"
 #include "util/u_string.h"
 /* XXX this should go away, needed for 'struct winsys_handle' */
 #include "state_tracker/drm_driver.h"
 
+/* A private modifier for now, so we have a way to request tiled but not
+ * compressed.  It would perhaps be good to get real modifiers for the
+ * tiled formats, but would probably need to do some work to figure out
+ * the layout(s) of the tiled modes, and whether they are the same
+ * across generations.
+ */
+#define FD_FORMAT_MOD_QCOM_TILED       fourcc_mod_code(QCOM, 0xffffffff)
+
 /**
  * Go through the entire state and see if the resource is bound
  * anywhere. If it is, mark the relevant state as dirty. This is
@@ -86,6 +94,15 @@ rebind_resource(struct fd_context *ctx, struct pipe_resource *prsc)
                                ctx->dirty_shader[stage] |= FD_DIRTY_SHADER_TEX;
                }
 
+               /* Images */
+               const unsigned num_images = util_last_bit(ctx->shaderimg[stage].enabled_mask);
+               for (unsigned i = 0; i < num_images; i++) {
+                       if (ctx->dirty_shader[stage] & FD_DIRTY_SHADER_IMAGE)
+                               break;
+                       if (ctx->shaderimg[stage].si[i].resource == prsc)
+                               ctx->dirty_shader[stage] |= FD_DIRTY_SHADER_IMAGE;
+               }
+
                /* SSBOs */
                const unsigned num_ssbos = util_last_bit(ctx->shaderbuf[stage].enabled_mask);
                for (unsigned i = 0; i < num_ssbos; i++) {
@@ -115,7 +132,7 @@ realloc_bo(struct fd_resource *rsc, uint32_t size)
                fd_bo_del(rsc->bo);
 
        rsc->bo = fd_bo_new(screen->dev, size, flags, "%ux%ux%u@%u:%x",
-                       prsc->width0, prsc->height0, prsc->depth0, rsc->cpp, prsc->bind);
+                       prsc->width0, prsc->height0, prsc->depth0, rsc->layout.cpp, prsc->bind);
        rsc->seqno = p_atomic_inc_return(&screen->rsc_seqno);
        util_range_set_empty(&rsc->valid_buffer_range);
        fd_bc_invalidate_resource(rsc, true);
@@ -206,10 +223,7 @@ fd_try_shadow_resource(struct fd_context *ctx, struct fd_resource *rsc,
        /* TODO valid_buffer_range?? */
        swap(rsc->bo,        shadow->bo);
        swap(rsc->write_batch,   shadow->write_batch);
-       swap(rsc->offset, shadow->offset);
-       swap(rsc->ubwc_offset, shadow->ubwc_offset);
-       swap(rsc->ubwc_pitch, shadow->ubwc_pitch);
-       swap(rsc->ubwc_size, shadow->ubwc_size);
+       swap(rsc->layout, shadow->layout);
        rsc->seqno = p_atomic_inc_return(&ctx->screen->rsc_seqno);
 
        /* at this point, the newly created shadow buffer is not referenced
@@ -251,7 +265,10 @@ fd_try_shadow_resource(struct fd_context *ctx, struct fd_resource *rsc,
                set_box(box.height, u_minify(prsc->height0, l));
                set_box(box.depth,  u_minify(prsc->depth0, l));
 
-               do_blit(ctx, &blit, fallback);
+               for (int i = 0; i < prsc->array_size; i++) {
+                       set_box(box.z, i);
+                       do_blit(ctx, &blit, fallback);
+               }
        }
 
        /* deal w/ current level specially, since we might need to split
@@ -295,6 +312,34 @@ fd_try_shadow_resource(struct fd_context *ctx, struct fd_resource *rsc,
        return true;
 }
 
+/**
+ * Uncompress an UBWC compressed buffer "in place".  This works basically
+ * like resource shadowing, creating a new resource, and doing an uncompress
+ * blit, and swapping the state between shadow and original resource so it
+ * appears to the state tracker as if nothing changed.
+ */
+void
+fd_resource_uncompress(struct fd_context *ctx, struct fd_resource *rsc)
+{
+       bool success =
+               fd_try_shadow_resource(ctx, rsc, 0, NULL, FD_FORMAT_MOD_QCOM_TILED);
+
+       /* shadow should not fail in any cases where we need to uncompress: */
+       debug_assert(success);
+
+       /*
+        * TODO what if rsc is used in other contexts, we don't currently
+        * have a good way to rebind_resource() in other contexts.  And an
+        * app that is reading one resource in multiple contexts, isn't
+        * going to expect that the resource is modified.
+        *
+        * Hopefully the edge cases where we need to uncompress are rare
+        * enough that they mostly only show up in deqp.
+        */
+
+       rebind_resource(ctx, &rsc->base);
+}
+
 static struct fd_resource *
 fd_alloc_staging(struct fd_context *ctx, struct fd_resource *rsc,
                unsigned level, const struct pipe_box *box)
@@ -308,6 +353,8 @@ fd_alloc_staging(struct fd_context *ctx, struct fd_resource *rsc,
         * for 3d textures, it is the depth:
         */
        if (tmpl.array_size > 1) {
+               if (tmpl.target == PIPE_TEXTURE_CUBE)
+                       tmpl.target = PIPE_TEXTURE_2D_ARRAY;
                tmpl.array_size = box->depth;
                tmpl.depth0 = 1;
        } else {
@@ -372,7 +419,7 @@ static void fd_resource_transfer_flush_region(struct pipe_context *pctx,
        struct fd_resource *rsc = fd_resource(ptrans->resource);
 
        if (ptrans->resource->target == PIPE_BUFFER)
-               util_range_add(&rsc->valid_buffer_range,
+               util_range_add(&rsc->base, &rsc->valid_buffer_range,
                                           ptrans->box.x + box->x,
                                           ptrans->box.x + box->x + box->width);
 }
@@ -402,15 +449,14 @@ flush_resource(struct fd_context *ctx, struct fd_resource *rsc, unsigned usage)
                mtx_unlock(&ctx->screen->lock);
 
                foreach_batch(batch, &ctx->screen->batch_cache, batch_mask)
-                       fd_batch_flush(batch, false, false);
+                       fd_batch_flush(batch);
 
                foreach_batch(batch, &ctx->screen->batch_cache, batch_mask) {
-                       fd_batch_sync(batch);
                        fd_batch_reference(&batches[batch->idx], NULL);
                }
                assert(rsc->batch_mask == 0);
        } else if (write_batch) {
-               fd_batch_flush(write_batch, true, false);
+               fd_batch_flush(write_batch);
        }
 
        fd_batch_reference(&write_batch, NULL);
@@ -442,7 +488,7 @@ fd_resource_transfer_unmap(struct pipe_context *pctx,
                fd_bo_cpu_fini(rsc->bo);
        }
 
-       util_range_add(&rsc->valid_buffer_range,
+       util_range_add(&rsc->base, &rsc->valid_buffer_range,
                                   ptrans->box.x,
                                   ptrans->box.x + ptrans->box.width);
 
@@ -459,7 +505,7 @@ fd_resource_transfer_map(struct pipe_context *pctx,
 {
        struct fd_context *ctx = fd_context(pctx);
        struct fd_resource *rsc = fd_resource(prsc);
-       struct fd_resource_slice *slice = fd_resource_slice(rsc, level);
+       struct fdl_slice *slice = fd_resource_slice(rsc, level);
        struct fd_transfer *trans;
        struct pipe_transfer *ptrans;
        enum pipe_format format = prsc->format;
@@ -483,8 +529,8 @@ fd_resource_transfer_map(struct pipe_context *pctx,
        ptrans->level = level;
        ptrans->usage = usage;
        ptrans->box = *box;
-       ptrans->stride = util_format_get_nblocksx(format, slice->pitch) * rsc->cpp;
-       ptrans->layer_stride = rsc->layer_first ? rsc->layer_size : slice->size0;
+       ptrans->stride = util_format_get_nblocksx(format, slice->pitch) * rsc->layout.cpp;
+       ptrans->layer_stride = fd_resource_layer_stride(rsc, level);
 
        /* we always need a staging texture for tiled buffers:
         *
@@ -492,17 +538,18 @@ fd_resource_transfer_map(struct pipe_context *pctx,
         * splitting a batch.. for ex, mid-frame texture uploads to a tiled
         * texture.
         */
-       if (rsc->tile_mode) {
+       if (rsc->layout.tile_mode) {
                struct fd_resource *staging_rsc;
 
                staging_rsc = fd_alloc_staging(ctx, rsc, level, box);
                if (staging_rsc) {
+                       struct fdl_slice *staging_slice =
+                               fd_resource_slice(staging_rsc, 0);
                        // TODO for PIPE_TRANSFER_READ, need to do untiling blit..
                        trans->staging_prsc = &staging_rsc->base;
                        trans->base.stride = util_format_get_nblocksx(format,
-                               staging_rsc->slices[0].pitch) * staging_rsc->cpp;
-                       trans->base.layer_stride = staging_rsc->layer_first ?
-                               staging_rsc->layer_size : staging_rsc->slices[0].size0;
+                               staging_slice->pitch) * staging_rsc->layout.cpp;
+                       trans->base.layer_stride = fd_resource_layer_stride(staging_rsc, 0);
                        trans->staging_box = *box;
                        trans->staging_box.x = 0;
                        trans->staging_box.y = 0;
@@ -511,21 +558,6 @@ fd_resource_transfer_map(struct pipe_context *pctx,
                        if (usage & PIPE_TRANSFER_READ) {
                                fd_blit_to_staging(ctx, trans);
 
-                               struct fd_batch *batch = NULL;
-
-                               fd_context_lock(ctx);
-                               fd_batch_reference_locked(&batch, staging_rsc->write_batch);
-                               fd_context_unlock(ctx);
-
-                               /* we can't fd_bo_cpu_prep() until the blit to staging
-                                * is submitted to kernel.. in that case write_batch
-                                * wouldn't be NULL yet:
-                                */
-                               if (batch) {
-                                       fd_batch_sync(batch);
-                                       fd_batch_reference(&batch, NULL);
-                               }
-
                                fd_bo_cpu_prep(staging_rsc->bo, ctx->pipe,
                                                DRM_FREEDRENO_PREP_READ);
                        }
@@ -613,11 +645,13 @@ fd_resource_transfer_map(struct pipe_context *pctx,
                                 */
                                staging_rsc = fd_alloc_staging(ctx, rsc, level, box);
                                if (staging_rsc) {
+                                       struct fdl_slice *staging_slice =
+                                               fd_resource_slice(staging_rsc, 0);
                                        trans->staging_prsc = &staging_rsc->base;
                                        trans->base.stride = util_format_get_nblocksx(format,
-                                               staging_rsc->slices[0].pitch) * staging_rsc->cpp;
-                                       trans->base.layer_stride = staging_rsc->layer_first ?
-                                               staging_rsc->layer_size : staging_rsc->slices[0].size0;
+                                               staging_slice->pitch) * staging_rsc->layout.cpp;
+                                       trans->base.layer_stride =
+                                               fd_resource_layer_stride(staging_rsc, 0);
                                        trans->staging_box = *box;
                                        trans->staging_box.x = 0;
                                        trans->staging_box.y = 0;
@@ -657,7 +691,7 @@ fd_resource_transfer_map(struct pipe_context *pctx,
        buf = fd_bo_map(rsc->bo);
        offset =
                box->y / util_format_get_blockheight(format) * ptrans->stride +
-               box->x / util_format_get_blockwidth(format) * rsc->cpp +
+               box->x / util_format_get_blockwidth(format) * rsc->layout.cpp +
                fd_resource_offset(rsc, level, box->z);
 
        if (usage & PIPE_TRANSFER_WRITE)
@@ -690,17 +724,17 @@ fd_resource_destroy(struct pipe_screen *pscreen,
 static uint64_t
 fd_resource_modifier(struct fd_resource *rsc)
 {
-       if (!rsc->tile_mode)
+       if (!rsc->layout.tile_mode)
                return DRM_FORMAT_MOD_LINEAR;
 
-       if (rsc->ubwc_size)
+       if (rsc->layout.ubwc_layer_size)
                return DRM_FORMAT_MOD_QCOM_COMPRESSED;
 
        /* TODO invent a modifier for tiled but not UBWC buffers: */
        return DRM_FORMAT_MOD_INVALID;
 }
 
-static boolean
+static bool
 fd_resource_get_handle(struct pipe_screen *pscreen,
                struct pipe_context *pctx,
                struct pipe_resource *prsc,
@@ -712,7 +746,7 @@ fd_resource_get_handle(struct pipe_screen *pscreen,
        handle->modifier = fd_resource_modifier(rsc);
 
        return fd_screen_bo_get_handle(pscreen, rsc->bo, rsc->scanout,
-                       rsc->slices[0].pitch * rsc->cpp, handle);
+                       fd_resource_slice(rsc, 0)->pitch * rsc->layout.cpp, handle);
 }
 
 static uint32_t
@@ -729,10 +763,10 @@ setup_slices(struct fd_resource *rsc, uint32_t alignment, enum pipe_format forma
        /* in layer_first layout, the level (slice) contains just one
         * layer (since in fact the layer contains the slices)
         */
-       uint32_t layers_in_level = rsc->layer_first ? 1 : prsc->array_size;
+       uint32_t layers_in_level = rsc->layout.layer_first ? 1 : prsc->array_size;
 
        for (level = 0; level <= prsc->last_level; level++) {
-               struct fd_resource_slice *slice = fd_resource_slice(rsc, level);
+               struct fdl_slice *slice = fd_resource_slice(rsc, level);
                uint32_t blocks;
 
                if (layout == UTIL_FORMAT_LAYOUT_ASTC)
@@ -750,12 +784,12 @@ setup_slices(struct fd_resource *rsc, uint32_t alignment, enum pipe_format forma
                 */
                if (prsc->target == PIPE_TEXTURE_3D && (
                                        level == 1 ||
-                                       (level > 1 && rsc->slices[level - 1].size0 > 0xf000)))
-                       slice->size0 = align(blocks * rsc->cpp, alignment);
-               else if (level == 0 || rsc->layer_first || alignment == 1)
-                       slice->size0 = align(blocks * rsc->cpp, alignment);
+                                       (level > 1 && fd_resource_slice(rsc, level - 1)->size0 > 0xf000)))
+                       slice->size0 = align(blocks * rsc->layout.cpp, alignment);
+               else if (level == 0 || rsc->layout.layer_first || alignment == 1)
+                       slice->size0 = align(blocks * rsc->layout.cpp, alignment);
                else
-                       slice->size0 = rsc->slices[level - 1].size0;
+                       slice->size0 = fd_resource_slice(rsc, level - 1)->size0;
 
                size += slice->size0 * depth * layers_in_level;
 
@@ -801,10 +835,10 @@ fd_setup_slices(struct fd_resource *rsc)
        if (is_a4xx(screen)) {
                switch (rsc->base.target) {
                case PIPE_TEXTURE_3D:
-                       rsc->layer_first = false;
+                       rsc->layout.layer_first = false;
                        break;
                default:
-                       rsc->layer_first = true;
+                       rsc->layout.layer_first = true;
                        alignment = 1;
                        break;
                }
@@ -827,23 +861,18 @@ fd_resource_resize(struct pipe_resource *prsc, uint32_t sz)
        realloc_bo(rsc, fd_screen(prsc->screen)->setup_slices(rsc));
 }
 
-// TODO common helper?
-static bool
-has_depth(enum pipe_format format)
+static void
+fd_resource_layout_init(struct pipe_resource *prsc)
 {
-       switch (format) {
-       case PIPE_FORMAT_Z16_UNORM:
-       case PIPE_FORMAT_Z32_UNORM:
-       case PIPE_FORMAT_Z32_FLOAT:
-       case PIPE_FORMAT_Z32_FLOAT_S8X24_UINT:
-       case PIPE_FORMAT_Z24_UNORM_S8_UINT:
-       case PIPE_FORMAT_S8_UINT_Z24_UNORM:
-       case PIPE_FORMAT_Z24X8_UNORM:
-       case PIPE_FORMAT_X8Z24_UNORM:
-               return true;
-       default:
-               return false;
-       }
+       struct fd_resource *rsc = fd_resource(prsc);
+       struct fdl_layout *layout = &rsc->layout;
+
+       layout->width0 = prsc->width0;
+       layout->height0 = prsc->height0;
+       layout->depth0 = prsc->depth0;
+
+       layout->cpp = util_format_get_blocksize(prsc->format);
+       layout->cpp *= fd_resource_nr_samples(prsc);
 }
 
 /**
@@ -871,6 +900,9 @@ fd_resource_create_with_modifiers(struct pipe_screen *pscreen,
                struct renderonly_scanout *scanout;
                struct winsys_handle handle;
 
+               /* apply freedreno alignment requirement */
+               scanout_templat.width0 = align(tmpl->width0, screen->gmem_alignw);
+
                scanout = renderonly_scanout_for_resource(&scanout_templat,
                                                                                                  screen->ro, &handle);
                if (!scanout)
@@ -903,6 +935,7 @@ fd_resource_create_with_modifiers(struct pipe_screen *pscreen,
                return NULL;
 
        *prsc = *tmpl;
+       fd_resource_layout_init(prsc);
 
 #define LINEAR \
        (PIPE_BIND_SCANOUT | \
@@ -913,6 +946,9 @@ fd_resource_create_with_modifiers(struct pipe_screen *pscreen,
        if (tmpl->bind & LINEAR)
                linear = true;
 
+       if (fd_mesa_debug & FD_DBG_NOTILE)
+               linear = true;
+
        /* Normally, for non-shared buffers, allow buffer compression if
         * not shared, otherwise only allow if QCOM_COMPRESSED modifier
         * is requested:
@@ -925,70 +961,32 @@ fd_resource_create_with_modifiers(struct pipe_screen *pscreen,
        if (tmpl->bind & PIPE_BIND_SHARED)
                allow_ubwc = drm_find_modifier(DRM_FORMAT_MOD_QCOM_COMPRESSED, modifiers, count);
 
-       /* TODO turn on UBWC for all internal buffers
-        *
-        * There are still some regressions in deqp with UBWC enabled.  I
-        * think it is mostly related to sampler/image views using a format
-        * that doesn't support compression with a resource created with
-        * a format that does.  We need to track the compression state of
-        * a buffer and do an (in-place, hopefully?) resolve if it is re-
-        * interpreted with a format that does not support compression.
-        *
-        * It is possible (likely?) that we can't do atomic ops on a
-        * compressed buffer as well, so this would also require transition
-        * to a compressed state.
-        */
-       allow_ubwc &= !!(fd_mesa_debug & FD_DBG_UBWC);
+       allow_ubwc &= !(fd_mesa_debug & FD_DBG_NOUBWC);
+
+       pipe_reference_init(&prsc->reference, 1);
+
+       prsc->screen = pscreen;
 
        if (screen->tile_mode &&
                        (tmpl->target != PIPE_BUFFER) &&
                        !linear) {
-               rsc->tile_mode = screen->tile_mode(tmpl);
+               rsc->layout.tile_mode = screen->tile_mode(prsc);
        }
 
-       pipe_reference_init(&prsc->reference, 1);
-
-       prsc->screen = pscreen;
-
        util_range_init(&rsc->valid_buffer_range);
 
        rsc->internal_format = format;
-       rsc->cpp = util_format_get_blocksize(format);
-       rsc->cpp *= fd_resource_nr_samples(prsc);
-
-       assert(rsc->cpp);
-
-       // XXX probably need some extra work if we hit rsc shadowing path w/ lrz..
-       if ((is_a5xx(screen) || is_a6xx(screen)) &&
-                (fd_mesa_debug & FD_DBG_LRZ) && has_depth(format)) {
-               const uint32_t flags = DRM_FREEDRENO_GEM_CACHE_WCOMBINE |
-                               DRM_FREEDRENO_GEM_TYPE_KMEM; /* TODO */
-               unsigned lrz_pitch  = align(DIV_ROUND_UP(tmpl->width0, 8), 64);
-               unsigned lrz_height = DIV_ROUND_UP(tmpl->height0, 8);
-
-               /* LRZ buffer is super-sampled: */
-               switch (prsc->nr_samples) {
-               case 4:
-                       lrz_pitch *= 2;
-               case 2:
-                       lrz_height *= 2;
-               }
-
-               unsigned size = lrz_pitch * lrz_height * 2;
 
-               size += 0x1000; /* for GRAS_LRZ_FAST_CLEAR_BUFFER */
+       rsc->layout.ubwc = rsc->layout.tile_mode && is_a6xx(screen) && allow_ubwc;
 
-               rsc->lrz_height = lrz_height;
-               rsc->lrz_width = lrz_pitch;
-               rsc->lrz_pitch = lrz_pitch;
-               rsc->lrz = fd_bo_new(screen->dev, size, flags, "lrz");
+       if (prsc->target == PIPE_BUFFER) {
+               assert(prsc->format == PIPE_FORMAT_R8_UNORM);
+               size = prsc->width0;
+               fdl_layout_buffer(&rsc->layout, size);
+       } else {
+               size = screen->setup_slices(rsc);
        }
 
-       size = screen->setup_slices(rsc);
-
-       if (allow_ubwc && screen->fill_ubwc_buffer_sizes && rsc->tile_mode)
-               size += screen->fill_ubwc_buffer_sizes(rsc);
-
        /* special case for hw-query buffer, which we need to allocate before we
         * know the size:
         */
@@ -998,11 +996,15 @@ fd_resource_create_with_modifiers(struct pipe_screen *pscreen,
                return prsc;
        }
 
-       if (rsc->layer_first) {
-               rsc->layer_size = align(size, 4096);
-               size = rsc->layer_size * prsc->array_size;
+       /* Set the layer size if the (non-a6xx) backend hasn't done so. */
+       if (rsc->layout.layer_first && !rsc->layout.layer_size) {
+               rsc->layout.layer_size = align(size, 4096);
+               size = rsc->layout.layer_size * prsc->array_size;
        }
 
+       if (fd_mesa_debug & FD_DBG_LAYOUT)
+               fdl_dump_layout(&rsc->layout);
+
        realloc_bo(rsc, size);
        if (!rsc->bo)
                goto fail;
@@ -1021,26 +1023,6 @@ fd_resource_create(struct pipe_screen *pscreen,
        return fd_resource_create_with_modifiers(pscreen, tmpl, &mod, 1);
 }
 
-static bool
-is_supported_modifier(struct pipe_screen *pscreen, enum pipe_format pfmt,
-               uint64_t mod)
-{
-       int count;
-
-       /* Get the count of supported modifiers: */
-       pscreen->query_dmabuf_modifiers(pscreen, pfmt, 0, NULL, NULL, &count);
-
-       /* Get the supported modifiers: */
-       uint64_t modifiers[count];
-       pscreen->query_dmabuf_modifiers(pscreen, pfmt, count, modifiers, NULL, &count);
-
-       for (int i = 0; i < count; i++)
-               if (modifiers[i] == mod)
-                       return true;
-
-       return false;
-}
-
 /**
  * Create a texture from a winsys_handle. The handle is often created in
  * another process by first creating a pipe texture and then calling
@@ -1053,7 +1035,7 @@ fd_resource_from_handle(struct pipe_screen *pscreen,
 {
        struct fd_screen *screen = fd_screen(pscreen);
        struct fd_resource *rsc = CALLOC_STRUCT(fd_resource);
-       struct fd_resource_slice *slice = &rsc->slices[0];
+       struct fdl_slice *slice = fd_resource_slice(rsc, 0);
        struct pipe_resource *prsc = &rsc->base;
        uint32_t pitchalign = fd_screen(pscreen)->gmem_alignw;
 
@@ -1068,6 +1050,7 @@ fd_resource_from_handle(struct pipe_screen *pscreen,
                return NULL;
 
        *prsc = *tmpl;
+       fd_resource_layout_init(prsc);
 
        pipe_reference_init(&prsc->reference, 1);
 
@@ -1080,9 +1063,7 @@ fd_resource_from_handle(struct pipe_screen *pscreen,
                goto fail;
 
        rsc->internal_format = tmpl->format;
-       rsc->cpp = util_format_get_blocksize(tmpl->format);
-       rsc->cpp *= fd_resource_nr_samples(prsc);
-       slice->pitch = handle->stride / rsc->cpp;
+       slice->pitch = handle->stride / rsc->layout.cpp;
        slice->offset = handle->offset;
        slice->size0 = handle->stride * prsc->height0;
 
@@ -1090,20 +1071,10 @@ fd_resource_from_handle(struct pipe_screen *pscreen,
                        (slice->pitch & (pitchalign - 1)))
                goto fail;
 
-       if (handle->modifier == DRM_FORMAT_MOD_QCOM_COMPRESSED) {
-               if (!is_supported_modifier(pscreen, tmpl->format,
-                               DRM_FORMAT_MOD_QCOM_COMPRESSED)) {
-                       DBG("bad modifier: %"PRIx64, handle->modifier);
-                       goto fail;
-               }
-               debug_assert(screen->fill_ubwc_buffer_sizes);
-               screen->fill_ubwc_buffer_sizes(rsc);
-       } else if (handle->modifier &&
-                       (handle->modifier != DRM_FORMAT_MOD_INVALID)) {
-               goto fail;
-       }
+       assert(rsc->layout.cpp);
 
-       assert(rsc->cpp);
+       if (screen->layout_resource_for_modifier(rsc, handle->modifier) < 0)
+               goto fail;
 
        if (screen->ro) {
                rsc->scanout =
@@ -1206,6 +1177,21 @@ static const struct u_transfer_vtbl transfer_vtbl = {
                .get_stencil              = fd_resource_get_stencil,
 };
 
+static const uint64_t supported_modifiers[] = {
+       DRM_FORMAT_MOD_LINEAR,
+};
+
+static int
+fd_layout_resource_for_modifier(struct fd_resource *rsc, uint64_t modifier)
+{
+       switch (modifier) {
+       case DRM_FORMAT_MOD_LINEAR:
+               return 0;
+       default:
+               return -1;
+       }
+}
+
 void
 fd_resource_screen_init(struct pipe_screen *pscreen)
 {
@@ -1226,6 +1212,12 @@ fd_resource_screen_init(struct pipe_screen *pscreen)
 
        if (!screen->setup_slices)
                screen->setup_slices = fd_setup_slices;
+       if (!screen->layout_resource_for_modifier)
+               screen->layout_resource_for_modifier = fd_layout_resource_for_modifier;
+       if (!screen->supported_modifiers) {
+               screen->supported_modifiers = supported_modifiers;
+               screen->num_supported_modifiers = ARRAY_SIZE(supported_modifiers);
+       }
 }
 
 static void