drm/amdgpu: optimize list operation in amdgpu_xgmi
simplify the list operation. Signed-off-by: Kevin Wang <kevin1.wang@amd.com> Reviewed-by: Felix Kuehling <Felix.Kuehling@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
@@ -324,7 +324,7 @@ static void amdgpu_xgmi_sysfs_rem_dev_info(struct amdgpu_device *adev,
|
|||||||
|
|
||||||
struct amdgpu_hive_info *amdgpu_get_xgmi_hive(struct amdgpu_device *adev)
|
struct amdgpu_hive_info *amdgpu_get_xgmi_hive(struct amdgpu_device *adev)
|
||||||
{
|
{
|
||||||
struct amdgpu_hive_info *hive = NULL, *tmp = NULL;
|
struct amdgpu_hive_info *hive = NULL;
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
if (!adev->gmc.xgmi.hive_id)
|
if (!adev->gmc.xgmi.hive_id)
|
||||||
@@ -337,11 +337,9 @@ struct amdgpu_hive_info *amdgpu_get_xgmi_hive(struct amdgpu_device *adev)
|
|||||||
|
|
||||||
mutex_lock(&xgmi_mutex);
|
mutex_lock(&xgmi_mutex);
|
||||||
|
|
||||||
if (!list_empty(&xgmi_hive_list)) {
|
list_for_each_entry(hive, &xgmi_hive_list, node) {
|
||||||
list_for_each_entry_safe(hive, tmp, &xgmi_hive_list, node) {
|
if (hive->hive_id == adev->gmc.xgmi.hive_id)
|
||||||
if (hive->hive_id == adev->gmc.xgmi.hive_id)
|
goto pro_end;
|
||||||
goto pro_end;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
hive = kzalloc(sizeof(*hive), GFP_KERNEL);
|
hive = kzalloc(sizeof(*hive), GFP_KERNEL);
|
||||||
|
|||||||
Reference in New Issue
Block a user