radv: remove radeon_surf_level::pitch_bytes
authorNicolai Hähnle <nicolai.haehnle@amd.com>
Wed, 10 May 2017 20:14:39 +0000 (22:14 +0200)
committerDave Airlie <airlied@redhat.com>
Mon, 5 Jun 2017 00:43:53 +0000 (10:43 +1000)
Like radeonsi. This saves memory, and the information can easily be
recomputed on the fly where necessary.

Reviewed-by: Dave Airlie <airlied@redhat.com>
src/amd/vulkan/radv_image.c
src/amd/vulkan/radv_private.h
src/amd/vulkan/radv_radeon_winsys.h
src/amd/vulkan/radv_wsi.c
src/amd/vulkan/winsys/amdgpu/radv_amdgpu_surface.c

index d138988..a1598fc 100644 (file)
@@ -446,7 +446,7 @@ radv_init_metadata(struct radv_device *device,
        metadata->tile_split = surface->tile_split;
        metadata->mtilea = surface->mtilea;
        metadata->num_banks = surface->num_banks;
-       metadata->stride = surface->level[0].pitch_bytes;
+       metadata->stride = surface->level[0].nblk_x * surface->bpe;
        metadata->scanout = (surface->flags & RADEON_SURF_SCANOUT) != 0;
 
        radv_query_opaque_metadata(device, image, metadata);
@@ -677,13 +677,6 @@ radv_image_create(VkDevice _device,
                radv_image_alloc_htile(device, image);
        }
 
-
-       if (create_info->stride && create_info->stride != image->surface.level[0].pitch_bytes) {
-               image->surface.level[0].nblk_x = create_info->stride / image->surface.bpe;
-               image->surface.level[0].pitch_bytes = create_info->stride;
-               image->surface.level[0].slice_size = create_info->stride * image->surface.level[0].nblk_y;
-       }
-
        if (pCreateInfo->flags & VK_IMAGE_CREATE_SPARSE_BINDING_BIT) {
                image->alignment = MAX2(image->alignment, 4096);
                image->size = align64(image->size, image->alignment);
@@ -856,7 +849,7 @@ void radv_GetImageSubresourceLayout(
        struct radeon_surf *surface = &image->surface;
 
        pLayout->offset = surface->level[level].offset + surface->level[level].slice_size * layer;
-       pLayout->rowPitch = surface->level[level].pitch_bytes;
+       pLayout->rowPitch = surface->level[level].nblk_x * surface->bpe;
        pLayout->arrayPitch = surface->level[level].slice_size;
        pLayout->depthPitch = surface->level[level].slice_size;
        pLayout->size = surface->level[level].slice_size;
index d64f801..c442f03 100644 (file)
@@ -1262,7 +1262,6 @@ struct radv_image_view {
 
 struct radv_image_create_info {
        const VkImageCreateInfo *vk_info;
-       uint32_t stride;
        bool scanout;
 };
 
index 84b1d73..df582d8 100644 (file)
@@ -170,7 +170,6 @@ struct radeon_surf_level {
        uint32_t                    nblk_x;
        uint32_t                    nblk_y;
        uint32_t                    nblk_z;
-       uint32_t                    pitch_bytes;
        uint32_t                    mode;
        bool                        dcc_enabled;
        uint64_t                    dcc_offset;
index 5e86612..d0ba377 100644 (file)
@@ -224,7 +224,7 @@ radv_wsi_image_create(VkDevice device_h,
        *memory_p = memory_h;
        *size = image->size;
        *offset = image->offset;
-       *row_pitch = surface->level[0].pitch_bytes;
+       *row_pitch = surface->level[0].nblk_x * surface->bpe;
        return VK_SUCCESS;
  fail_alloc_memory:
        radv_FreeMemory(device_h, memory_h, pAllocator);
index c972ef1..f8e22da 100644 (file)
@@ -203,7 +203,6 @@ static int radv_compute_level(ADDR_HANDLE addrlib,
        surf_level = is_stencil ? &surf->stencil_level[level] : &surf->level[level];
        surf_level->offset = align64(surf->bo_size, AddrSurfInfoOut->baseAlign);
        surf_level->slice_size = AddrSurfInfoOut->sliceSize;
-       surf_level->pitch_bytes = AddrSurfInfoOut->pitch * (is_stencil ? 1 : surf->bpe);
        surf_level->nblk_x = AddrSurfInfoOut->pitch;
        surf_level->nblk_y = AddrSurfInfoOut->height;
        if (type == RADEON_SURF_TYPE_3D)