drm/amdgpu: Fix delete nodes that have been relesed

Stanley.Yang · alexdeucher · commit 8a6566111494 · 2023-10-20T15:11:27.000-04:00
Fix delete nodes that it has been freed. Fixes: 5b1270b ("drm/amdgpu: add ras_err_info to identify RAS error source") Signed-off-by: Stanley.Yang <Stanley.Yang@amd.com> Reviewed-by: Yang Wang <kevinyang.wang@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
@@ -3503,10 +3503,8 @@ void amdgpu_ras_error_data_fini(struct ras_err_data *err_data)
 {
 	struct ras_err_node *err_node, *tmp;
 
-	list_for_each_entry_safe(err_node, tmp, &err_data->err_node_list, node) {
+	list_for_each_entry_safe(err_node, tmp, &err_data->err_node_list, node)
 		amdgpu_ras_error_node_release(err_node);
-		list_del(&err_node->node);
-	}
 }
 
 static struct ras_err_node *amdgpu_ras_error_find_node_by_id(struct ras_err_data *err_data,

Original file line number	Diff line number	Diff line change
`@@ -3503,10 +3503,8 @@ void amdgpu_ras_error_data_fini(struct ras_err_data *err_data)`
`3503`	`3503`	`{`
`3504`	`3504`	`struct ras_err_node err_node, tmp;`
`3505`	`3505`
`3506`		`- list_for_each_entry_safe(err_node, tmp, &err_data->err_node_list, node) {`
	`3506`	`+ list_for_each_entry_safe(err_node, tmp, &err_data->err_node_list, node)`
`3507`	`3507`	`amdgpu_ras_error_node_release(err_node);`
`3508`		`- list_del(&err_node->node);`
`3509`		`- }`
`3510`	`3508`	`}`
`3511`	`3509`
`3512`	`3510`	`static struct ras_err_node amdgpu_ras_error_find_node_by_id(struct ras_err_data err_data,`