net/mlx5: DR, Add missing query for vport 0
Currently, vport 0 capabilities are not set. To fix this, we now querying both eswitch manager and vport 0. Eswitch manager has an access to all the vports - for eswitch manager PF, all vports can be referred as other vports. The exception is embedded CPU mode, where there is vport 0 of ECPF and the PF vport 0. Here is how vport are queried: For Connect-X5/6: PF vport (0) and vports 1..n: vport number, other = true esw_manager is vport 0 (PF) For BlueField (in embedded CPU mode): ECPF vport: vport = 0, other = false PF vport (0) and 1..n: vport number, other = true esw_manager = vport 0 (ECPF) Also, note that there's no need for other_vport function parameter in dr_domain_query_vport - this value is now deduced locally in the function. Signed-off-by: Yuval Avnery <yuvalav@mellanox.com> Signed-off-by: Yevgeny Kliteynik <kliteyn@nvidia.com> Reviewed-by: Muhammad Sammar <muhammads@nvidia.com> Signed-off-by: Saeed Mahameed <saeedm@nvidia.com>
This commit is contained in:
parent
7ae8ac9a58
commit
dd4acb2a09
drivers/net/ethernet/mellanox/mlx5/core/steering
@ -195,6 +195,8 @@ int mlx5dr_cmd_query_device(struct mlx5_core_dev *mdev,
|
||||
|
||||
caps->roce_min_src_udp = MLX5_CAP_ROCE(mdev, r_roce_min_src_udp_port);
|
||||
|
||||
caps->is_ecpf = mlx5_core_is_ecpf_esw_manager(mdev);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
@ -125,17 +125,21 @@ static void dr_domain_uninit_resources(struct mlx5dr_domain *dmn)
|
||||
}
|
||||
|
||||
static int dr_domain_query_vport(struct mlx5dr_domain *dmn,
|
||||
bool other_vport,
|
||||
u16 vport_number)
|
||||
u16 vport_number,
|
||||
struct mlx5dr_cmd_vport_cap *vport_caps)
|
||||
{
|
||||
struct mlx5dr_cmd_vport_cap *vport_caps;
|
||||
u16 cmd_vport = vport_number;
|
||||
bool other_vport = true;
|
||||
int ret;
|
||||
|
||||
vport_caps = &dmn->info.caps.vports_caps[vport_number];
|
||||
if (dmn->info.caps.is_ecpf && vport_number == MLX5_VPORT_ECPF) {
|
||||
other_vport = false;
|
||||
cmd_vport = 0;
|
||||
}
|
||||
|
||||
ret = mlx5dr_cmd_query_esw_vport_context(dmn->mdev,
|
||||
other_vport,
|
||||
vport_number,
|
||||
cmd_vport,
|
||||
&vport_caps->icm_address_rx,
|
||||
&vport_caps->icm_address_tx);
|
||||
if (ret)
|
||||
@ -143,7 +147,7 @@ static int dr_domain_query_vport(struct mlx5dr_domain *dmn,
|
||||
|
||||
ret = mlx5dr_cmd_query_gvmi(dmn->mdev,
|
||||
other_vport,
|
||||
vport_number,
|
||||
cmd_vport,
|
||||
&vport_caps->vport_gvmi);
|
||||
if (ret)
|
||||
return ret;
|
||||
@ -154,6 +158,13 @@ static int dr_domain_query_vport(struct mlx5dr_domain *dmn,
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int dr_domain_query_esw_mngr(struct mlx5dr_domain *dmn)
|
||||
{
|
||||
return dr_domain_query_vport(dmn,
|
||||
dmn->info.caps.is_ecpf ? MLX5_VPORT_ECPF : 0,
|
||||
&dmn->info.caps.esw_manager_vport_caps);
|
||||
}
|
||||
|
||||
static int dr_domain_query_vports(struct mlx5dr_domain *dmn)
|
||||
{
|
||||
struct mlx5dr_esw_caps *esw_caps = &dmn->info.caps.esw_caps;
|
||||
@ -161,9 +172,15 @@ static int dr_domain_query_vports(struct mlx5dr_domain *dmn)
|
||||
int vport;
|
||||
int ret;
|
||||
|
||||
ret = dr_domain_query_esw_mngr(dmn);
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
/* Query vports (except wire vport) */
|
||||
for (vport = 0; vport < dmn->info.caps.num_esw_ports - 1; vport++) {
|
||||
ret = dr_domain_query_vport(dmn, !!vport, vport);
|
||||
ret = dr_domain_query_vport(dmn,
|
||||
vport,
|
||||
&dmn->info.caps.vports_caps[vport]);
|
||||
if (ret)
|
||||
return ret;
|
||||
}
|
||||
@ -267,11 +284,7 @@ static int dr_domain_caps_init(struct mlx5_core_dev *mdev,
|
||||
|
||||
dmn->info.rx.type = DR_DOMAIN_NIC_TYPE_RX;
|
||||
dmn->info.tx.type = DR_DOMAIN_NIC_TYPE_TX;
|
||||
vport_cap = mlx5dr_get_vport_cap(&dmn->info.caps, 0);
|
||||
if (!vport_cap) {
|
||||
mlx5dr_err(dmn, "Failed to get esw manager vport\n");
|
||||
return -ENOENT;
|
||||
}
|
||||
vport_cap = &dmn->info.caps.esw_manager_vport_caps;
|
||||
|
||||
dmn->info.supp_sw_steering = true;
|
||||
dmn->info.tx.default_icm_addr = vport_cap->icm_address_tx;
|
||||
|
@ -797,8 +797,10 @@ struct mlx5dr_cmd_caps {
|
||||
u32 num_vports;
|
||||
struct mlx5dr_esw_caps esw_caps;
|
||||
struct mlx5dr_cmd_vport_cap *vports_caps;
|
||||
struct mlx5dr_cmd_vport_cap esw_manager_vport_caps;
|
||||
bool prio_tag_required;
|
||||
struct mlx5dr_roce_cap roce_caps;
|
||||
u8 is_ecpf:1;
|
||||
u8 isolate_vl_tc:1;
|
||||
};
|
||||
|
||||
@ -1104,6 +1106,9 @@ mlx5dr_ste_htbl_may_grow(struct mlx5dr_ste_htbl *htbl)
|
||||
static inline struct mlx5dr_cmd_vport_cap *
|
||||
mlx5dr_get_vport_cap(struct mlx5dr_cmd_caps *caps, u16 vport)
|
||||
{
|
||||
if (caps->is_ecpf && vport == MLX5_VPORT_ECPF)
|
||||
return &caps->esw_manager_vport_caps;
|
||||
|
||||
if (!caps->vports_caps ||
|
||||
(vport >= caps->num_vports && vport != MLX5_VPORT_UPLINK))
|
||||
return NULL;
|
||||
|
Loading…
Reference in New Issue
Block a user