Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 05794eff authored by Shirish S's avatar Shirish S Committed by Alex Deucher
Browse files

drm/amdgpu/gmc: fix compiler errors [-Werror,-Wmissing-braces] (V2) 



Initializing structures with { } is known to be problematic since
it doesn't necessararily initialize all bytes, in case of padding,
causing random failures when structures are memcmp().

This patch fixes the structure initialisation related compiler
error by memset().

V2: rectified missing piece in coding

Signed-off-by: default avatarShirish S <shirish.s@amd.com>
Reviewed-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 32e61361

drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c

+2 −1
Original line number Diff line number Diff line
@@ -1471,8 +1471,9 @@ static int gmc_v8_0_process_interrupt(struct amdgpu_device *adev, 
		gmc_v8_0_set_fault_enable_default(adev, false);



	if (printk_ratelimit()) {

		struct amdgpu_task_info task_info = { 0 };

		struct amdgpu_task_info task_info;



		memset(&task_info, 0, sizeof(struct amdgpu_task_info));

		amdgpu_vm_get_task_info(adev, entry->pasid, &task_info);



		dev_err(adev->dev, "GPU fault detected: %d 0x%08x for process %s pid %d thread %s pid %d\n",

drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c

+2 −1
Original line number Diff line number Diff line
@@ -320,8 +320,9 @@ static int gmc_v9_0_process_interrupt(struct amdgpu_device *adev, 
	}



	if (printk_ratelimit()) {

		struct amdgpu_task_info task_info = { 0 };

		struct amdgpu_task_info task_info;



		memset(&task_info, 0, sizeof(struct amdgpu_task_info));

		amdgpu_vm_get_task_info(adev, entry->pasid, &task_info);



		dev_err(adev->dev,