drm/amdgpu: fix cleaning up reserved VMID on release

ChristianKoenigAMD · alexdeucher · commit 4463b1eea28a · 2023-01-18T23:13:14.000-05:00
We need to reset this or otherwise run into list corruption later on. Fixes: e44a0fe ("drm/amdgpu: rework reserved VMID handling") Signed-off-by: Christian König <christian.koenig@amd.com> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> Tested-by: Candice Li <candice.li@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ids.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ids.c
@@ -497,6 +497,7 @@ void amdgpu_vmid_free_reserved(struct amdgpu_device *adev,
 	    !--id_mgr->reserved_use_count) {
 		/* give the reserved ID back to normal round robin */
 		list_add(&id_mgr->reserved->list, &id_mgr->ids_lru);
+		id_mgr->reserved = NULL;
 	}
 	vm->reserved_vmid[vmhub] = false;
 	mutex_unlock(&id_mgr->lock);

Original file line number	Diff line number	Diff line change
`@@ -497,6 +497,7 @@ void amdgpu_vmid_free_reserved(struct amdgpu_device *adev,`
`497`	`497`	`!--id_mgr->reserved_use_count) {`
`498`	`498`	`/* give the reserved ID back to normal round robin */`
`499`	`499`	`list_add(&id_mgr->reserved->list, &id_mgr->ids_lru);`
	`500`	`+ id_mgr->reserved = NULL;`
`500`	`501`	`}`
`501`	`502`	`vm->reserved_vmid[vmhub] = false;`
`502`	`503`	`mutex_unlock(&id_mgr->lock);`