From: Dave Airlie <airl...@redhat.com>

Work out the width/height from the level manually, as on GFX9
we won't minify the iview width/height.

This fixes:
dEQP-VK.api.image_clearing.core.clear_color_image* on gfx9

Cc: "17.2" <mesa-sta...@lists.freedesktop.org>
Signed-off-by: Dave Airlie <airl...@redhat.com>
---
 src/amd/vulkan/radv_meta_blit.c  | 19 ++++++++++++-------
 src/amd/vulkan/radv_meta_clear.c | 15 +++++++++------
 2 files changed, 21 insertions(+), 13 deletions(-)

diff --git a/src/amd/vulkan/radv_meta_blit.c b/src/amd/vulkan/radv_meta_blit.c
index 3510e87..2c1a132 100644
--- a/src/amd/vulkan/radv_meta_blit.c
+++ b/src/amd/vulkan/radv_meta_blit.c
@@ -275,15 +275,20 @@ meta_emit_blit(struct radv_cmd_buffer *cmd_buffer,
                VkFilter blit_filter)
 {
        struct radv_device *device = cmd_buffer->device;
+       uint32_t src_width = radv_minify(src_iview->image->info.width, 
src_iview->base_mip);
+       uint32_t src_height = radv_minify(src_iview->image->info.height, 
src_iview->base_mip);
+       uint32_t src_depth = radv_minify(src_iview->image->info.depth, 
src_iview->base_mip);
+       uint32_t dst_width = radv_minify(dest_iview->image->info.width, 
dest_iview->base_mip);
+       uint32_t dst_height = radv_minify(dest_iview->image->info.height, 
dest_iview->base_mip);
 
        assert(src_image->info.samples == dest_image->info.samples);
 
        float vertex_push_constants[5] = {
-               (float)src_offset_0.x / (float)src_iview->extent.width,
-               (float)src_offset_0.y / (float)src_iview->extent.height,
-               (float)src_offset_1.x / (float)src_iview->extent.width,
-               (float)src_offset_1.y / (float)src_iview->extent.height,
-               (float)src_offset_0.z / (float)src_iview->extent.depth,
+               (float)src_offset_0.x / (float)src_width,
+               (float)src_offset_0.y / (float)src_height,
+               (float)src_offset_1.x / (float)src_width,
+               (float)src_offset_1.y / (float)src_height,
+               (float)src_offset_0.z / (float)src_depth,
        };
 
        radv_CmdPushConstants(radv_cmd_buffer_to_handle(cmd_buffer),
@@ -310,8 +315,8 @@ meta_emit_blit(struct radv_cmd_buffer *cmd_buffer,
                                               .pAttachments = (VkImageView[]) {
                                               
radv_image_view_to_handle(dest_iview),
                                       },
-                                      .width = dest_iview->extent.width,
-                                      .height = dest_iview->extent.height,
+                                      .width = dst_width,
+                                      .height = dst_height,
                                       .layers = 1,
                                }, &cmd_buffer->pool->alloc, &fb);
        VkPipeline pipeline;
diff --git a/src/amd/vulkan/radv_meta_clear.c b/src/amd/vulkan/radv_meta_clear.c
index b3eb389..08a6278 100644
--- a/src/amd/vulkan/radv_meta_clear.c
+++ b/src/amd/vulkan/radv_meta_clear.c
@@ -1202,6 +1202,9 @@ radv_clear_image_layer(struct radv_cmd_buffer *cmd_buffer,
 {
        VkDevice device_h = radv_device_to_handle(cmd_buffer->device);
        struct radv_image_view iview;
+       uint32_t width = radv_minify(image->info.width, range->baseMipLevel + 
level);
+       uint32_t height = radv_minify(image->info.height, range->baseMipLevel + 
level);
+
        radv_image_view_init(&iview, cmd_buffer->device,
                             &(VkImageViewCreateInfo) {
                                     .sType = 
VK_STRUCTURE_TYPE_IMAGE_VIEW_CREATE_INFO,
@@ -1225,9 +1228,9 @@ radv_clear_image_layer(struct radv_cmd_buffer *cmd_buffer,
                                               .pAttachments = (VkImageView[]) {
                                               
radv_image_view_to_handle(&iview),
                                       },
-                                              .width = iview.extent.width,
-                                                       .height = 
iview.extent.height,
-                                                       .layers = 1
+                                              .width = width,
+                                              .height = height,
+                                              .layers = 1
                               },
                               &cmd_buffer->pool->alloc,
                               &fb);
@@ -1283,8 +1286,8 @@ radv_clear_image_layer(struct radv_cmd_buffer *cmd_buffer,
                                                .renderArea = {
                                                .offset = { 0, 0, },
                                                .extent = {
-                                                       .width = 
iview.extent.width,
-                                                       .height = 
iview.extent.height,
+                                                       .width = width,
+                                                       .height = height,
                                                },
                                        },
                                                .renderPass = pass,
@@ -1303,7 +1306,7 @@ radv_clear_image_layer(struct radv_cmd_buffer *cmd_buffer,
        VkClearRect clear_rect = {
                .rect = {
                        .offset = { 0, 0 },
-                       .extent = { iview.extent.width, iview.extent.height },
+                       .extent = { width, height },
                },
                .baseArrayLayer = range->baseArrayLayer,
                .layerCount = 1, /* FINISHME: clear multi-layer framebuffer */
-- 
2.9.3

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to