drm: ttm: virtio-gpu: dma-buf: Constify ttm_place structures.
authorArvind Yadav <arvind.yadav.cs@gmail.com>
Sun, 2 Jul 2017 07:41:02 +0000 (13:11 +0530)
committerGerd Hoffmann <kraxel@redhat.com>
Mon, 3 Jul 2017 08:31:14 +0000 (10:31 +0200)
ttm_place are not supposed to change at runtime. All functions
working with ttm_place provided by <drm/ttm/ttm_placement.h> work
with const ttm_place. So mark the non-const structs as const.

File size before:
   text    data     bss     dec     hex filename
   2315     184       0    2499     9c3 drivers/gpu/drm/virtio/virtgpu_ttm.o

File size After adding 'const':
   text    data     bss     dec     hex filename
   2347     152       0    2499     9c3 drivers/gpu/drm/virtio/virtgpu_ttm.o

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Link: http://patchwork.freedesktop.org/patch/msgid/25a189402a516a0142d9a4412da0a597c660a96a.1498981093.git.arvind.yadav.cs@gmail.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
drivers/gpu/drm/virtio/virtgpu_ttm.c

index c1f2af4ca4ca9ab7b8ea8a775dbe3f6d9c40a389..e695d74eaa9f08761eccf88ca3df2246f8c398c9 100644 (file)
@@ -234,7 +234,7 @@ static int virtio_gpu_init_mem_type(struct ttm_bo_device *bdev, uint32_t type,
 static void virtio_gpu_evict_flags(struct ttm_buffer_object *bo,
                                struct ttm_placement *placement)
 {
-       static struct ttm_place placements = {
+       static const struct ttm_place placements = {
                .fpfn  = 0,
                .lpfn  = 0,
                .flags = TTM_PL_MASK_CACHING | TTM_PL_FLAG_SYSTEM,