From 7b46f09f26c3f4377f46d145344f98d5b27b0b00 Mon Sep 17 00:00:00 2001 From: Alyssa Rosenzweig Date: Thu, 20 Jun 2019 08:19:06 -0700 Subject: [PATCH] panfrost: Fix tiled NPOT textures with bpp<4 Panfrost's tiling routines (incorrectly) ignored the source stride, masking this bug; lima's routines respect this stride, causing issues when tiling NPOT textures whose stride is not a multiple of 64 (for instance, NPOT textures with bpp=1). Signed-off-by: Alyssa Rosenzweig --- src/gallium/drivers/panfrost/pan_resource.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/panfrost/pan_resource.c b/src/gallium/drivers/panfrost/pan_resource.c index 332208d4e2f..4f4499aa403 100644 --- a/src/gallium/drivers/panfrost/pan_resource.c +++ b/src/gallium/drivers/panfrost/pan_resource.c @@ -198,10 +198,10 @@ panfrost_setup_slices(const struct pipe_resource *tmpl, struct panfrost_bo *bo) /* We don't know how to specify a 2D stride for 3D textures */ - bool should_align_stride = + bool can_align_stride = tmpl->target != PIPE_TEXTURE_3D; - should_align &= should_align_stride; + should_align &= can_align_stride; unsigned offset = 0; unsigned size_2d = 0; @@ -226,7 +226,7 @@ panfrost_setup_slices(const struct pipe_resource *tmpl, struct panfrost_bo *bo) unsigned stride = bytes_per_pixel * effective_width; /* ..but cache-line align it for performance */ - if (should_align_stride) + if (can_align_stride && bo->layout == PAN_LINEAR) stride = ALIGN(stride, 64); slice->stride = stride; -- 2.30.2