Commit 802620f5 authored by Tvrtko Ursulin's avatar Tvrtko Ursulin Committed by Tvrtko Ursulin
Browse files

drm/ttm: Tidy ttm_operation_ctx initialization 



No need to initialize a subset of fields to zero.

Signed-off-by: default avatarTvrtko Ursulin <tvrtko.ursulin@igalia.com>
Acked-by: default avatarThadeu Lima de Souza Cascardo <cascardo@igalia.com>
Link: https://lore.kernel.org/r/20250919131530.91247-4-tvrtko.ursulin@igalia.com


Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarTvrtko Ursulin <tursulin@ursulin.net>
[tursulin: fixup conflict in ttm_resource_manager_evict_all]
parent feb06515

drivers/gpu/drm/ttm/ttm_bo_util.c

+2 −8
Original line number Diff line number Diff line
@@ -343,10 +343,7 @@ static int ttm_bo_kmap_ttm(struct ttm_buffer_object *bo, 
			   struct ttm_bo_kmap_obj *map)

{

	struct ttm_resource *mem = bo->resource;

	struct ttm_operation_ctx ctx = {

		.interruptible = false,

		.no_wait_gpu = false

	};

	struct ttm_operation_ctx ctx = { };

	struct ttm_tt *ttm = bo->ttm;

	struct ttm_resource_manager *man =

			ttm_manager_type(bo->bdev, bo->resource->mem_type);
@@ -530,10 +527,7 @@ int ttm_bo_vmap(struct ttm_buffer_object *bo, struct iosys_map *map) 
		iosys_map_set_vaddr_iomem(map, vaddr_iomem);



	} else {

		struct ttm_operation_ctx ctx = {

			.interruptible = false,

			.no_wait_gpu = false

		};

		struct ttm_operation_ctx ctx = { };

		struct ttm_tt *ttm = bo->ttm;

		pgprot_t prot;

		void *vaddr;

drivers/gpu/drm/ttm/ttm_device.c

+1 −4
Original line number Diff line number Diff line
@@ -135,10 +135,7 @@ static int ttm_global_init(void) 
 */

int ttm_device_prepare_hibernation(struct ttm_device *bdev)

{

	struct ttm_operation_ctx ctx = {

		.interruptible = false,

		.no_wait_gpu = false,

	};

	struct ttm_operation_ctx ctx = { };

	int ret;



	do {

drivers/gpu/drm/ttm/ttm_resource.c

+1 −4
Original line number Diff line number Diff line
@@ -545,10 +545,7 @@ EXPORT_SYMBOL(ttm_resource_manager_init); 
int ttm_resource_manager_evict_all(struct ttm_device *bdev,

				   struct ttm_resource_manager *man)

{

	struct ttm_operation_ctx ctx = {

		.interruptible = false,

		.no_wait_gpu = false,

	};

	struct ttm_operation_ctx ctx = { };

	struct dma_fence *fence;

	int ret, i;

drivers/gpu/drm/ttm/ttm_tt.c

+1 −1
Original line number Diff line number Diff line
@@ -456,7 +456,7 @@ EXPORT_SYMBOL_FOR_TESTS_ONLY(ttm_tt_unpopulate); 
/* Test the shrinker functions and dump the result */

static int ttm_tt_debugfs_shrink_show(struct seq_file *m, void *data)

{

	struct ttm_operation_ctx ctx = { false, false };

	struct ttm_operation_ctx ctx = { };



	seq_printf(m, "%d\n", ttm_global_swapout(&ctx, GFP_KERNEL));

	return 0;