Commit dd4acb2a authored by Yevgeny Kliteynik's avatar Yevgeny Kliteynik Committed by Saeed Mahameed

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: default avatarYuval Avnery <yuvalav@mellanox.com>
Signed-off-by: default avatarYevgeny Kliteynik <kliteyn@nvidia.com>
Reviewed-by: default avatarMuhammad Sammar <muhammads@nvidia.com>
Signed-off-by: default avatarSaeed Mahameed <saeedm@nvidia.com>
parent 7ae8ac9a
...@@ -195,6 +195,8 @@ int mlx5dr_cmd_query_device(struct mlx5_core_dev *mdev, ...@@ -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->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; return 0;
} }
......
...@@ -125,17 +125,21 @@ static void dr_domain_uninit_resources(struct mlx5dr_domain *dmn) ...@@ -125,17 +125,21 @@ static void dr_domain_uninit_resources(struct mlx5dr_domain *dmn)
} }
static int dr_domain_query_vport(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; 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, ret = mlx5dr_cmd_query_esw_vport_context(dmn->mdev,
other_vport, other_vport,
vport_number, cmd_vport,
&vport_caps->icm_address_rx, &vport_caps->icm_address_rx,
&vport_caps->icm_address_tx); &vport_caps->icm_address_tx);
if (ret) if (ret)
...@@ -143,7 +147,7 @@ static int dr_domain_query_vport(struct mlx5dr_domain *dmn, ...@@ -143,7 +147,7 @@ static int dr_domain_query_vport(struct mlx5dr_domain *dmn,
ret = mlx5dr_cmd_query_gvmi(dmn->mdev, ret = mlx5dr_cmd_query_gvmi(dmn->mdev,
other_vport, other_vport,
vport_number, cmd_vport,
&vport_caps->vport_gvmi); &vport_caps->vport_gvmi);
if (ret) if (ret)
return ret; return ret;
...@@ -154,6 +158,13 @@ static int dr_domain_query_vport(struct mlx5dr_domain *dmn, ...@@ -154,6 +158,13 @@ static int dr_domain_query_vport(struct mlx5dr_domain *dmn,
return 0; 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) static int dr_domain_query_vports(struct mlx5dr_domain *dmn)
{ {
struct mlx5dr_esw_caps *esw_caps = &dmn->info.caps.esw_caps; 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) ...@@ -161,9 +172,15 @@ static int dr_domain_query_vports(struct mlx5dr_domain *dmn)
int vport; int vport;
int ret; int ret;
ret = dr_domain_query_esw_mngr(dmn);
if (ret)
return ret;
/* Query vports (except wire vport) */ /* Query vports (except wire vport) */
for (vport = 0; vport < dmn->info.caps.num_esw_ports - 1; 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) if (ret)
return ret; return ret;
} }
...@@ -267,11 +284,7 @@ static int dr_domain_caps_init(struct mlx5_core_dev *mdev, ...@@ -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.rx.type = DR_DOMAIN_NIC_TYPE_RX;
dmn->info.tx.type = DR_DOMAIN_NIC_TYPE_TX; dmn->info.tx.type = DR_DOMAIN_NIC_TYPE_TX;
vport_cap = mlx5dr_get_vport_cap(&dmn->info.caps, 0); vport_cap = &dmn->info.caps.esw_manager_vport_caps;
if (!vport_cap) {
mlx5dr_err(dmn, "Failed to get esw manager vport\n");
return -ENOENT;
}
dmn->info.supp_sw_steering = true; dmn->info.supp_sw_steering = true;
dmn->info.tx.default_icm_addr = vport_cap->icm_address_tx; dmn->info.tx.default_icm_addr = vport_cap->icm_address_tx;
......
...@@ -797,8 +797,10 @@ struct mlx5dr_cmd_caps { ...@@ -797,8 +797,10 @@ struct mlx5dr_cmd_caps {
u32 num_vports; u32 num_vports;
struct mlx5dr_esw_caps esw_caps; struct mlx5dr_esw_caps esw_caps;
struct mlx5dr_cmd_vport_cap *vports_caps; struct mlx5dr_cmd_vport_cap *vports_caps;
struct mlx5dr_cmd_vport_cap esw_manager_vport_caps;
bool prio_tag_required; bool prio_tag_required;
struct mlx5dr_roce_cap roce_caps; struct mlx5dr_roce_cap roce_caps;
u8 is_ecpf:1;
u8 isolate_vl_tc:1; u8 isolate_vl_tc:1;
}; };
...@@ -1104,6 +1106,9 @@ mlx5dr_ste_htbl_may_grow(struct mlx5dr_ste_htbl *htbl) ...@@ -1104,6 +1106,9 @@ mlx5dr_ste_htbl_may_grow(struct mlx5dr_ste_htbl *htbl)
static inline struct mlx5dr_cmd_vport_cap * static inline struct mlx5dr_cmd_vport_cap *
mlx5dr_get_vport_cap(struct mlx5dr_cmd_caps *caps, u16 vport) 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 || if (!caps->vports_caps ||
(vport >= caps->num_vports && vport != MLX5_VPORT_UPLINK)) (vport >= caps->num_vports && vport != MLX5_VPORT_UPLINK))
return NULL; return NULL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment