drm/amdgpu/mes11: optimize MES pipe FW version fetching

alexdeucher · alexdeucher · commit b71a2bb0ce07 · 2025-04-07T14:30:10.000-04:00
Don't fetch it again if we already have it. It seems the registers don't reliably have the value at resume in some cases. Fixes: 028c3fb ("drm/amdgpu/mes11: initiate mes v11 support") Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/4083 Reviewed-by: Mario Limonciello <mario.limonciello@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> Cc: stable@vger.kernel.org
diff --git a/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c b/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
@@ -894,6 +894,10 @@ static void mes_v11_0_get_fw_version(struct amdgpu_device *adev)
 {
 	int pipe;
 
+	/* return early if we have already fetched these */
+	if (adev->mes.sched_version && adev->mes.kiq_version)
+		return;
+
 	/* get MES scheduler/KIQ versions */
 	mutex_lock(&adev->srbm_mutex);
 

Original file line number	Diff line number	Diff line change
`@@ -894,6 +894,10 @@ static void mes_v11_0_get_fw_version(struct amdgpu_device *adev)`
`894`	`894`	`{`
`895`	`895`	`int pipe;`
`896`	`896`
	`897`	`+ /* return early if we have already fetched these */`
	`898`	`+ if (adev->mes.sched_version && adev->mes.kiq_version)`
	`899`	`+ return;`
	`900`	`+`
`897`	`901`	`/* get MES scheduler/KIQ versions */`
`898`	`902`	`mutex_lock(&adev->srbm_mutex);`
`899`	`903`