diff options
author | Bas Nieuwenhuizen <[email protected]> | 2018-12-02 23:58:54 +0100 |
---|---|---|
committer | Bas Nieuwenhuizen <[email protected]> | 2019-04-25 19:56:20 +0000 |
commit | b2cfa231d0dd46ade036eb76ed0ad55b3f4cf462 (patch) | |
tree | dd9012f655b8b6d73dfd7c8c05e41266690aa8ef /src/amd/vulkan/radv_image.c | |
parent | 65c4f612aa21008a7140d3d6399832b58374e6c3 (diff) |
radv: Add support for image views with multiple planes.
Reviewed-by: Samuel Pitoiset <[email protected]>
Diffstat (limited to 'src/amd/vulkan/radv_image.c')
-rw-r--r-- | src/amd/vulkan/radv_image.c | 41 |
1 files changed, 25 insertions, 16 deletions
diff --git a/src/amd/vulkan/radv_image.c b/src/amd/vulkan/radv_image.c index dd2c46ae417..a487a5576a6 100644 --- a/src/amd/vulkan/radv_image.c +++ b/src/amd/vulkan/radv_image.c @@ -345,11 +345,12 @@ static void si_set_mutable_tex_desc_fields(struct radv_device *device, struct radv_image *image, const struct legacy_surf_level *base_level_info, + unsigned plane_id, unsigned base_level, unsigned first_level, unsigned block_width, bool is_stencil, bool is_storage_image, uint32_t *state) { - struct radv_image_plane *plane = &image->planes[0]; + struct radv_image_plane *plane = &image->planes[plane_id]; uint64_t gpu_address = image->bo ? radv_buffer_get_va(image->bo) + image->offset : 0; uint64_t va = gpu_address + plane->offset; enum chip_class chip_class = device->physical_device->rad_info.chip_class; @@ -712,7 +713,7 @@ radv_query_opaque_metadata(struct radv_device *device, image->info.depth, desc, NULL); - si_set_mutable_tex_desc_fields(device, image, &image->planes[0].surface.u.legacy.level[0], 0, 0, + si_set_mutable_tex_desc_fields(device, image, &image->planes[0].surface.u.legacy.level[0], 0, 0, 0, image->planes[0].surface.blk_w, false, false, desc); /* Clear the base address and set the relative DCC offset. */ @@ -1108,31 +1109,34 @@ radv_image_create(VkDevice _device, static void radv_image_view_make_descriptor(struct radv_image_view *iview, struct radv_device *device, + VkFormat vk_format, const VkComponentMapping *components, - bool is_storage_image, unsigned plane_id) + bool is_storage_image, unsigned plane_id, + unsigned descriptor_plane_id) { struct radv_image *image = iview->image; struct radv_image_plane *plane = &image->planes[plane_id]; const struct vk_format_description *format_desc = vk_format_description(image->vk_format); bool is_stencil = iview->aspect_mask == VK_IMAGE_ASPECT_STENCIL_BIT; uint32_t blk_w; - uint32_t *descriptor; + union radv_descriptor *descriptor; uint32_t hw_level = 0; if (is_storage_image) { - descriptor = iview->storage_descriptor; + descriptor = &iview->storage_descriptor; } else { - descriptor = iview->descriptor; + descriptor = &iview->descriptor; } - assert(plane->surface.blk_w % vk_format_get_blockwidth(image->vk_format) == 0); - blk_w = plane->surface.blk_w / vk_format_get_blockwidth(image->vk_format) * vk_format_get_blockwidth(iview->vk_format); + assert(vk_format_get_plane_count(vk_format) == 1); + assert(plane->surface.blk_w % vk_format_get_blockwidth(plane->format) == 0); + blk_w = plane->surface.blk_w / vk_format_get_blockwidth(plane->format) * vk_format_get_blockwidth(vk_format); if (device->physical_device->rad_info.chip_class >= GFX9) hw_level = iview->base_mip; si_make_texture_descriptor(device, image, is_storage_image, iview->type, - iview->vk_format, + vk_format, components, hw_level, hw_level + iview->level_count - 1, iview->base_layer, @@ -1140,8 +1144,8 @@ radv_image_view_make_descriptor(struct radv_image_view *iview, iview->extent.width / (plane_id ? format_desc->width_divisor : 1), iview->extent.height / (plane_id ? format_desc->height_divisor : 1), iview->extent.depth, - descriptor, - descriptor + 8); + descriptor->plane_descriptors[descriptor_plane_id], + descriptor_plane_id ? NULL : descriptor->fmask_descriptor); const struct legacy_surf_level *base_level_info = NULL; if (device->physical_device->rad_info.chip_class <= GFX9) { @@ -1152,9 +1156,10 @@ radv_image_view_make_descriptor(struct radv_image_view *iview, } si_set_mutable_tex_desc_fields(device, image, base_level_info, + plane_id, iview->base_mip, iview->base_mip, - blk_w, is_stencil, is_storage_image, descriptor); + blk_w, is_stencil, is_storage_image, descriptor->plane_descriptors[descriptor_plane_id]); } static unsigned @@ -1212,9 +1217,10 @@ radv_image_view_init(struct radv_image_view *iview, iview->image = image; iview->bo = image->bo; iview->type = pCreateInfo->viewType; - iview->vk_format = pCreateInfo->format; - iview->aspect_mask = pCreateInfo->subresourceRange.aspectMask; iview->plane_id = radv_plane_from_aspect(pCreateInfo->subresourceRange.aspectMask); + iview->aspect_mask = pCreateInfo->subresourceRange.aspectMask; + iview->multiple_planes = vk_format_get_plane_count(image->vk_format) > 1 && iview->aspect_mask == VK_IMAGE_ASPECT_COLOR_BIT; + iview->vk_format = pCreateInfo->format; if (iview->aspect_mask == VK_IMAGE_ASPECT_STENCIL_BIT) { iview->vk_format = vk_format_stencil_only(iview->vk_format); @@ -1291,8 +1297,11 @@ radv_image_view_init(struct radv_image_view *iview, iview->base_mip = range->baseMipLevel; iview->level_count = radv_get_levelCount(image, range); - radv_image_view_make_descriptor(iview, device, &pCreateInfo->components, false, iview->plane_id); - radv_image_view_make_descriptor(iview, device, &pCreateInfo->components, true, iview->plane_id); + for (unsigned i = 0; i < (iview->multiple_planes ? vk_format_get_plane_count(image->vk_format) : 1); ++i) { + VkFormat format = vk_format_get_plane_format(iview->vk_format, i); + radv_image_view_make_descriptor(iview, device, format, &pCreateInfo->components, false, iview->plane_id + i, i); + radv_image_view_make_descriptor(iview, device, format, &pCreateInfo->components, true, iview->plane_id + i, i); + } } bool radv_layout_has_htile(const struct radv_image *image, |