mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-06 02:50:49 +09:00
mlxsw: spectrum_acl_tcam: Add missing mutex_destroy()
[ Upstream commit 65823e07b1 ]
Pair mutex_init() with a mutex_destroy() in the error path. Found during
code review. No functional changes.
Signed-off-by: Ido Schimmel <idosch@nvidia.com>
Reviewed-by: Jiri Pirko <jiri@nvidia.com>
Signed-off-by: Petr Machata <petrm@nvidia.com>
Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
174ac6b53a
commit
5dbedec7e5
@@ -52,8 +52,10 @@ int mlxsw_sp_acl_tcam_init(struct mlxsw_sp *mlxsw_sp,
|
|||||||
max_regions = max_tcam_regions;
|
max_regions = max_tcam_regions;
|
||||||
|
|
||||||
tcam->used_regions = bitmap_zalloc(max_regions, GFP_KERNEL);
|
tcam->used_regions = bitmap_zalloc(max_regions, GFP_KERNEL);
|
||||||
if (!tcam->used_regions)
|
if (!tcam->used_regions) {
|
||||||
return -ENOMEM;
|
err = -ENOMEM;
|
||||||
|
goto err_alloc_used_regions;
|
||||||
|
}
|
||||||
tcam->max_regions = max_regions;
|
tcam->max_regions = max_regions;
|
||||||
|
|
||||||
max_groups = MLXSW_CORE_RES_GET(mlxsw_sp->core, ACL_MAX_GROUPS);
|
max_groups = MLXSW_CORE_RES_GET(mlxsw_sp->core, ACL_MAX_GROUPS);
|
||||||
@@ -78,6 +80,8 @@ err_tcam_init:
|
|||||||
bitmap_free(tcam->used_groups);
|
bitmap_free(tcam->used_groups);
|
||||||
err_alloc_used_groups:
|
err_alloc_used_groups:
|
||||||
bitmap_free(tcam->used_regions);
|
bitmap_free(tcam->used_regions);
|
||||||
|
err_alloc_used_regions:
|
||||||
|
mutex_destroy(&tcam->lock);
|
||||||
return err;
|
return err;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user