net/mlx5: Allow 0 for total host VFs

danielj-mellanox · gregkh · commit a824d012ad8f · 2023-08-16T18:27:28.000+02:00
commit 2dc2b39 upstream. When querying eswitch functions 0 is a valid number of host VFs. After introducing ARM SRIOV falling through to getting the max value from PCI results in using the total VFs allowed on the ARM for the host. Fixes: 86eec50 ("net/mlx5: Support querying max VFs from device"); Signed-off-by: Daniel Jurgens <danielj@nvidia.com> Signed-off-by: Saeed Mahameed <saeedm@nvidia.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/sriov.c b/drivers/net/ethernet/mellanox/mlx5/core/sriov.c
@@ -264,8 +264,7 @@ static u16 mlx5_get_max_vfs(struct mlx5_core_dev *dev)
 		host_total_vfs = MLX5_GET(query_esw_functions_out, out,
 					  host_params_context.host_total_vfs);
 		kvfree(out);
-		if (host_total_vfs)
-			return host_total_vfs;
+		return host_total_vfs;
 	}
 
 done:

Original file line number	Diff line number	Diff line change
`@@ -264,8 +264,7 @@ static u16 mlx5_get_max_vfs(struct mlx5_core_dev *dev)`
`264`	`264`	`host_total_vfs = MLX5_GET(query_esw_functions_out, out,`
`265`	`265`	`host_params_context.host_total_vfs);`
`266`	`266`	`kvfree(out);`
`267`		`- if (host_total_vfs)`
`268`		`- return host_total_vfs;`
	`267`	`+ return host_total_vfs;`
`269`	`268`	`}`
`270`	`269`
`271`	`270`	`done:`