ice: Add infrastructure for mqprio support via ndo_setup_tc
Add infrastructure required for "ndo_setup_tc:qdisc_mqprio". ice_vsi_setup is modified to configure traffic classes based on mqprio data received from the stack. This includes low-level functions to configure min, max rate-limit parameters in hardware for traffic classes. Each traffic class gets mapped to a hardware channel (VSI) which can be individually configured with different bandwidth parameters. Co-developed-by: Tarun Singh <tarun.k.singh@intel.com> Signed-off-by: Tarun Singh <tarun.k.singh@intel.com> Signed-off-by: Kiran Patil <kiran.patil@intel.com> Signed-off-by: Amritha Nambiar <amritha.nambiar@intel.com> Signed-off-by: Sudheer Mogilappagari <sudheer.mogilappagari@intel.com> Tested-by: Bharathi Sreenivas <bharathi.sreenivas@intel.com> Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
This commit is contained in:
@@ -4,53 +4,11 @@
|
||||
#include "ice_dcb_lib.h"
|
||||
#include "ice_dcb_nl.h"
|
||||
|
||||
/**
|
||||
* ice_vsi_cfg_netdev_tc - Setup the netdev TC configuration
|
||||
* @vsi: the VSI being configured
|
||||
* @ena_tc: TC map to be enabled
|
||||
*/
|
||||
void ice_vsi_cfg_netdev_tc(struct ice_vsi *vsi, u8 ena_tc)
|
||||
{
|
||||
struct net_device *netdev = vsi->netdev;
|
||||
struct ice_pf *pf = vsi->back;
|
||||
struct ice_dcbx_cfg *dcbcfg;
|
||||
u8 netdev_tc;
|
||||
int i;
|
||||
|
||||
if (!netdev)
|
||||
return;
|
||||
|
||||
if (!ena_tc) {
|
||||
netdev_reset_tc(netdev);
|
||||
return;
|
||||
}
|
||||
|
||||
if (netdev_set_num_tc(netdev, vsi->tc_cfg.numtc))
|
||||
return;
|
||||
|
||||
dcbcfg = &pf->hw.port_info->qos_cfg.local_dcbx_cfg;
|
||||
|
||||
ice_for_each_traffic_class(i)
|
||||
if (vsi->tc_cfg.ena_tc & BIT(i))
|
||||
netdev_set_tc_queue(netdev,
|
||||
vsi->tc_cfg.tc_info[i].netdev_tc,
|
||||
vsi->tc_cfg.tc_info[i].qcount_tx,
|
||||
vsi->tc_cfg.tc_info[i].qoffset);
|
||||
|
||||
for (i = 0; i < ICE_MAX_USER_PRIORITY; i++) {
|
||||
u8 ets_tc = dcbcfg->etscfg.prio_table[i];
|
||||
|
||||
/* Get the mapped netdev TC# for the UP */
|
||||
netdev_tc = vsi->tc_cfg.tc_info[ets_tc].netdev_tc;
|
||||
netdev_set_prio_tc_map(netdev, i, netdev_tc);
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* ice_dcb_get_ena_tc - return bitmap of enabled TCs
|
||||
* @dcbcfg: DCB config to evaluate for enabled TCs
|
||||
*/
|
||||
u8 ice_dcb_get_ena_tc(struct ice_dcbx_cfg *dcbcfg)
|
||||
static u8 ice_dcb_get_ena_tc(struct ice_dcbx_cfg *dcbcfg)
|
||||
{
|
||||
u8 i, num_tc, ena_tc = 1;
|
||||
|
||||
@@ -178,6 +136,67 @@ u8 ice_dcb_get_num_tc(struct ice_dcbx_cfg *dcbcfg)
|
||||
return ret;
|
||||
}
|
||||
|
||||
/**
|
||||
* ice_get_first_droptc - returns number of first droptc
|
||||
* @vsi: used to find the first droptc
|
||||
*
|
||||
* This function returns the value of first_droptc.
|
||||
* When DCB is enabled, first droptc information is derived from enabled_tc
|
||||
* and PFC enabled bits. otherwise this function returns 0 as there is one
|
||||
* TC without DCB (tc0)
|
||||
*/
|
||||
static u8 ice_get_first_droptc(struct ice_vsi *vsi)
|
||||
{
|
||||
struct ice_dcbx_cfg *cfg = &vsi->port_info->qos_cfg.local_dcbx_cfg;
|
||||
struct device *dev = ice_pf_to_dev(vsi->back);
|
||||
u8 num_tc, ena_tc_map, pfc_ena_map;
|
||||
u8 i;
|
||||
|
||||
num_tc = ice_dcb_get_num_tc(cfg);
|
||||
|
||||
/* get bitmap of enabled TCs */
|
||||
ena_tc_map = ice_dcb_get_ena_tc(cfg);
|
||||
|
||||
/* get bitmap of PFC enabled TCs */
|
||||
pfc_ena_map = cfg->pfc.pfcena;
|
||||
|
||||
/* get first TC that is not PFC enabled */
|
||||
for (i = 0; i < num_tc; i++) {
|
||||
if ((ena_tc_map & BIT(i)) && (!(pfc_ena_map & BIT(i)))) {
|
||||
dev_dbg(dev, "first drop tc = %d\n", i);
|
||||
return i;
|
||||
}
|
||||
}
|
||||
|
||||
dev_dbg(dev, "first drop tc = 0\n");
|
||||
return 0;
|
||||
}
|
||||
|
||||
/**
|
||||
* ice_vsi_set_dcb_tc_cfg - Set VSI's TC based on DCB configuration
|
||||
* @vsi: pointer to the VSI instance
|
||||
*/
|
||||
void ice_vsi_set_dcb_tc_cfg(struct ice_vsi *vsi)
|
||||
{
|
||||
struct ice_dcbx_cfg *cfg = &vsi->port_info->qos_cfg.local_dcbx_cfg;
|
||||
|
||||
switch (vsi->type) {
|
||||
case ICE_VSI_PF:
|
||||
vsi->tc_cfg.ena_tc = ice_dcb_get_ena_tc(cfg);
|
||||
vsi->tc_cfg.numtc = ice_dcb_get_num_tc(cfg);
|
||||
break;
|
||||
case ICE_VSI_CHNL:
|
||||
vsi->tc_cfg.ena_tc = BIT(ice_get_first_droptc(vsi));
|
||||
vsi->tc_cfg.numtc = 1;
|
||||
break;
|
||||
case ICE_VSI_CTRL:
|
||||
case ICE_VSI_LB:
|
||||
default:
|
||||
vsi->tc_cfg.ena_tc = ICE_DFLT_TRAFFIC_CLASS;
|
||||
vsi->tc_cfg.numtc = 1;
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* ice_dcb_get_tc - Get the TC associated with the queue
|
||||
* @vsi: ptr to the VSI
|
||||
@@ -218,11 +237,68 @@ void ice_vsi_cfg_dcb_rings(struct ice_vsi *vsi)
|
||||
|
||||
qoffset = vsi->tc_cfg.tc_info[n].qoffset;
|
||||
qcount = vsi->tc_cfg.tc_info[n].qcount_tx;
|
||||
for (i = qoffset; i < (qoffset + qcount); i++) {
|
||||
tx_ring = vsi->tx_rings[i];
|
||||
rx_ring = vsi->rx_rings[i];
|
||||
tx_ring->dcb_tc = n;
|
||||
rx_ring->dcb_tc = n;
|
||||
for (i = qoffset; i < (qoffset + qcount); i++)
|
||||
vsi->tx_rings[i]->dcb_tc = n;
|
||||
|
||||
qcount = vsi->tc_cfg.tc_info[n].qcount_rx;
|
||||
for (i = qoffset; i < (qoffset + qcount); i++)
|
||||
vsi->rx_rings[i]->dcb_tc = n;
|
||||
}
|
||||
/* applicable only if "all_enatc" is set, which will be set from
|
||||
* setup_tc method as part of configuring channels
|
||||
*/
|
||||
if (vsi->all_enatc) {
|
||||
u8 first_droptc = ice_get_first_droptc(vsi);
|
||||
|
||||
/* When DCB is configured, TC for ADQ queues (which are really
|
||||
* PF queues) should be the first drop TC of the main VSI
|
||||
*/
|
||||
ice_for_each_chnl_tc(n) {
|
||||
if (!(vsi->all_enatc & BIT(n)))
|
||||
break;
|
||||
|
||||
qoffset = vsi->mqprio_qopt.qopt.offset[n];
|
||||
qcount = vsi->mqprio_qopt.qopt.count[n];
|
||||
for (i = qoffset; i < (qoffset + qcount); i++) {
|
||||
vsi->tx_rings[i]->dcb_tc = first_droptc;
|
||||
vsi->rx_rings[i]->dcb_tc = first_droptc;
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* ice_dcb_ena_dis_vsi - disable certain VSIs for DCB config/reconfig
|
||||
* @pf: pointer to the PF instance
|
||||
* @ena: true to enable VSIs, false to disable
|
||||
* @locked: true if caller holds RTNL lock, false otherwise
|
||||
*
|
||||
* Before a new DCB configuration can be applied, VSIs of type PF, SWITCHDEV
|
||||
* and CHNL need to be brought down. Following completion of DCB configuration
|
||||
* the VSIs that were downed need to be brought up again. This helper function
|
||||
* does both.
|
||||
*/
|
||||
static void ice_dcb_ena_dis_vsi(struct ice_pf *pf, bool ena, bool locked)
|
||||
{
|
||||
int i;
|
||||
|
||||
ice_for_each_vsi(pf, i) {
|
||||
struct ice_vsi *vsi = pf->vsi[i];
|
||||
|
||||
if (!vsi)
|
||||
continue;
|
||||
|
||||
switch (vsi->type) {
|
||||
case ICE_VSI_CHNL:
|
||||
case ICE_VSI_SWITCHDEV_CTRL:
|
||||
case ICE_VSI_PF:
|
||||
if (ena)
|
||||
ice_ena_vsi(vsi, locked);
|
||||
else
|
||||
ice_dis_vsi(vsi, locked);
|
||||
break;
|
||||
default:
|
||||
continue;
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -331,7 +407,9 @@ int ice_pf_dcb_cfg(struct ice_pf *pf, struct ice_dcbx_cfg *new_cfg, bool locked)
|
||||
*/
|
||||
if (!locked)
|
||||
rtnl_lock();
|
||||
ice_dis_vsi(pf_vsi, true);
|
||||
|
||||
/* disable VSIs affected by DCB changes */
|
||||
ice_dcb_ena_dis_vsi(pf, false, true);
|
||||
|
||||
memcpy(curr_cfg, new_cfg, sizeof(*curr_cfg));
|
||||
memcpy(&curr_cfg->etsrec, &curr_cfg->etscfg, sizeof(curr_cfg->etsrec));
|
||||
@@ -359,7 +437,8 @@ int ice_pf_dcb_cfg(struct ice_pf *pf, struct ice_dcbx_cfg *new_cfg, bool locked)
|
||||
ice_pf_dcb_recfg(pf);
|
||||
|
||||
out:
|
||||
ice_ena_vsi(pf_vsi, true);
|
||||
/* enable previously downed VSIs */
|
||||
ice_dcb_ena_dis_vsi(pf, true, true);
|
||||
if (!locked)
|
||||
rtnl_unlock();
|
||||
free_cfg:
|
||||
@@ -674,6 +753,8 @@ void ice_pf_dcb_recfg(struct ice_pf *pf)
|
||||
tc_map = ICE_DFLT_TRAFFIC_CLASS;
|
||||
ice_dcb_noncontig_cfg(pf);
|
||||
}
|
||||
} else if (vsi->type == ICE_VSI_CHNL) {
|
||||
tc_map = BIT(ice_get_first_droptc(vsi));
|
||||
} else {
|
||||
tc_map = ICE_DFLT_TRAFFIC_CLASS;
|
||||
}
|
||||
@@ -684,10 +765,11 @@ void ice_pf_dcb_recfg(struct ice_pf *pf)
|
||||
vsi->idx);
|
||||
continue;
|
||||
}
|
||||
/* no need to proceed with remaining cfg if it is switchdev
|
||||
* VSI
|
||||
/* no need to proceed with remaining cfg if it is CHNL
|
||||
* or switchdev VSI
|
||||
*/
|
||||
if (vsi->type == ICE_VSI_SWITCHDEV_CTRL)
|
||||
if (vsi->type == ICE_VSI_CHNL ||
|
||||
vsi->type == ICE_VSI_SWITCHDEV_CTRL)
|
||||
continue;
|
||||
|
||||
ice_vsi_map_rings_to_vectors(vsi);
|
||||
@@ -862,7 +944,6 @@ ice_dcb_process_lldp_set_mib_change(struct ice_pf *pf,
|
||||
struct ice_dcbx_cfg tmp_dcbx_cfg;
|
||||
bool need_reconfig = false;
|
||||
struct ice_port_info *pi;
|
||||
struct ice_vsi *pf_vsi;
|
||||
u8 mib_type;
|
||||
int ret;
|
||||
|
||||
@@ -938,14 +1019,9 @@ ice_dcb_process_lldp_set_mib_change(struct ice_pf *pf,
|
||||
clear_bit(ICE_FLAG_DCB_ENA, pf->flags);
|
||||
}
|
||||
|
||||
pf_vsi = ice_get_main_vsi(pf);
|
||||
if (!pf_vsi) {
|
||||
dev_dbg(dev, "PF VSI doesn't exist\n");
|
||||
goto out;
|
||||
}
|
||||
|
||||
rtnl_lock();
|
||||
ice_dis_vsi(pf_vsi, true);
|
||||
/* disable VSIs affected by DCB changes */
|
||||
ice_dcb_ena_dis_vsi(pf, false, true);
|
||||
|
||||
ret = ice_query_port_ets(pf->hw.port_info, &buf, sizeof(buf), NULL);
|
||||
if (ret) {
|
||||
@@ -956,7 +1032,8 @@ ice_dcb_process_lldp_set_mib_change(struct ice_pf *pf,
|
||||
/* changes in configuration update VSI */
|
||||
ice_pf_dcb_recfg(pf);
|
||||
|
||||
ice_ena_vsi(pf_vsi, true);
|
||||
/* enable previously downed VSIs */
|
||||
ice_dcb_ena_dis_vsi(pf, true, true);
|
||||
unlock_rtnl:
|
||||
rtnl_unlock();
|
||||
out:
|
||||
|
||||
Reference in New Issue
Block a user