A GTP tunnel uses the same TEID value in both directions of a flow; including TEID in the hash input does not break src/dst symmetry. ethtool_rxfh_config_is_sym() currently rejects any hash field bitmap that contains bits outside the four paired L3/L4 fields. This causes drivers that hash GTP flows on TEID to fail the kernel's preflight validation in ethtool_check_flow_types(), making it impossible for those drivers to support symmetric-xor transforms at all. Strip RXH_GTP_TEID from the bitmap before the paired-field check so that drivers may honestly report TEID hashing without blocking the configuration of symmetric transforms. Signed-off-by: Aleksandr Loktionov --- net/ethtool/common.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/net/ethtool/common.c b/net/ethtool/common.c index 5fae329..9a3fd76 100644 --- a/net/ethtool/common.c +++ b/net/ethtool/common.c @@ -911,6 +911,9 @@ int ethtool_rxfh_config_is_sym(u64 rxfh) { bool sym; + /* Strip TEID before checking - it carries no src/dst asymmetry */ + rxfh &= ~(u64)RXH_GTP_TEID; + sym = rxfh == (rxfh & (RXH_IP_SRC | RXH_IP_DST | RXH_L4_B_0_1 | RXH_L4_B_2_3)); sym &= !!(rxfh & RXH_IP_SRC) == !!(rxfh & RXH_IP_DST); -- 2.52.0 The driver advertises symmetric RSS support via supported_input_xfrm but ice_set_rxfh() always programmed plain Toeplitz regardless of the requested input_xfrm, making it impossible to actually enable symmetric hashing. Fix ice_set_rxfh() to honour rxfh->input_xfrm: program symmetric Toeplitz (ICE_AQ_VSI_Q_OPT_RSS_HASH_SYM_TPLZ) when RXH_XFRM_SYM_XOR is requested, revert to plain Toeplitz when the transform is cleared, and skip the hardware write when the function has not changed. Make ice_set_rss_vsi_ctx() non-static and export it so ice_set_rxfh() can reprogram the VSI context directly. Change it to preserve vsi->rss_hfunc across VSI reinitialisation instead of always resetting to plain Toeplitz, which would silently undo any previously configured symmetric hash function. Fix ice_get_rxfh_fields() to report the hash fields actually programmed in hardware. When the hardware hashes on only one half of an L3 or L4 pair, complete the pair in the reported bitmap to satisfy the kernel's symmetry validator. For GTP flow types, report RXH_GTP_TEID honestly; ethtool_rxfh_config_is_sym() now accepts TEID as an intrinsically symmetric field (see preceding patch). Tested with tools/testing/selftests/drivers/net/hw/rss_input_xfrm.py Signed-off-by: Aleksandr Loktionov --- drivers/net/ethernet/intel/ice/ice_ethtool.c | 48 ++++++++++++++++---- drivers/net/ethernet/intel/ice/ice_lib.c | 7 +-- drivers/net/ethernet/intel/ice/ice_lib.h | 1 + 3 files changed, 44 insertions(+), 12 deletions(-) diff --git a/drivers/net/ethernet/intel/ice/ice_ethtool.c b/drivers/net/ethernet/intel/ice/ice_ethtool.c index c6bc29c..8d00892 100644 --- a/drivers/net/ethernet/intel/ice/ice_ethtool.c +++ b/drivers/net/ethernet/intel/ice/ice_ethtool.c @@ -3008,14 +3008,17 @@ ice_set_rxfh_fields(struct net_device *netdev, return 0; } +#define ICE_RSS_L3_PAIR (RXH_IP_SRC | RXH_IP_DST) +#define ICE_RSS_L4_PAIR (RXH_L4_B_0_1 | RXH_L4_B_2_3) + static int ice_get_rxfh_fields(struct net_device *netdev, struct ethtool_rxfh_fields *nfc) { struct ice_netdev_priv *np = netdev_priv(netdev); struct ice_vsi *vsi = np->vsi; struct ice_pf *pf = vsi->back; + u64 l3, l4, hash_flds; struct device *dev; - u64 hash_flds; bool symm; u32 hdrs; @@ -3037,8 +3040,10 @@ ice_get_rxfh_fields(struct net_device *netdev, struct ethtool_rxfh_fields *nfc) hash_flds = ice_get_rss_cfg(&pf->hw, vsi->idx, hdrs, &symm); if (hash_flds == ICE_HASH_INVALID) { - dev_dbg(dev, "No hash fields found for the given header type, vsi num = %d\n", + /* Provide default symmetric hash fields when no config exists */ + dev_dbg(dev, "No RSS config for this flow, using symmetric defaults, vsi num = %d\n", vsi->vsi_num); + nfc->data = RXH_IP_SRC | RXH_IP_DST | RXH_L4_B_0_1 | RXH_L4_B_2_3; return 0; } @@ -3067,6 +3072,17 @@ ice_get_rxfh_fields(struct net_device *netdev, struct ethtool_rxfh_fields *nfc) hash_flds & ICE_FLOW_HASH_FLD_GTPU_DWN_TEID) nfc->data |= (u64)RXH_GTP_TEID; + l3 = nfc->data & ICE_RSS_L3_PAIR; + l4 = nfc->data & ICE_RSS_L4_PAIR; + if (!l3 && !l4) { + nfc->data |= ICE_RSS_L3_PAIR | ICE_RSS_L4_PAIR; + return 0; + } + if (l3 && l3 != ICE_RSS_L3_PAIR) + nfc->data |= ICE_RSS_L3_PAIR; + if (l4 && l4 != ICE_RSS_L4_PAIR) + nfc->data |= ICE_RSS_L4_PAIR; + return 0; } @@ -3667,7 +3683,6 @@ ice_set_rxfh(struct net_device *netdev, struct ethtool_rxfh_param *rxfh, struct netlink_ext_ack *extack) { struct ice_netdev_priv *np = netdev_priv(netdev); - u8 hfunc = ICE_AQ_VSI_Q_OPT_RSS_HASH_TPLZ; struct ice_vsi *vsi = np->vsi; struct ice_pf *pf = vsi->back; struct device *dev; @@ -3689,13 +3704,28 @@ ice_set_rxfh(struct net_device *netdev, struct ethtool_rxfh_param *rxfh, return -EOPNOTSUPP; } - /* Update the VSI's hash function */ - if (rxfh->input_xfrm & RXH_XFRM_SYM_XOR) - hfunc = ICE_AQ_VSI_Q_OPT_RSS_HASH_SYM_TPLZ; + /* Handle RSS symmetric hash transformation */ + if (rxfh->input_xfrm != RXH_XFRM_NO_CHANGE) { + u8 new_hfunc; - err = ice_set_rss_hfunc(vsi, hfunc); - if (err) - return err; + if (rxfh->input_xfrm == RXH_XFRM_SYM_XOR) + new_hfunc = ICE_AQ_VSI_Q_OPT_RSS_HASH_SYM_TPLZ; + else if (!rxfh->input_xfrm) + new_hfunc = ICE_AQ_VSI_Q_OPT_RSS_HASH_TPLZ; + else + return -EOPNOTSUPP; + + if (new_hfunc != vsi->rss_hfunc) { + err = ice_set_rss_hfunc(vsi, new_hfunc); + if (err) { + netdev_err(netdev, "Failed to set RSS hash function\n"); + return err; + } + netdev_dbg(netdev, "RSS hash function: %sToeplitz\n", + new_hfunc == ICE_AQ_VSI_Q_OPT_RSS_HASH_SYM_TPLZ ? + "Symmetric " : ""); + } + } if (rxfh->key) { if (!vsi->rss_hkey_user) { diff --git a/drivers/net/ethernet/intel/ice/ice_lib.c b/drivers/net/ethernet/intel/ice/ice_lib.c index d921269..5b1934b 100644 --- a/drivers/net/ethernet/intel/ice/ice_lib.c +++ b/drivers/net/ethernet/intel/ice/ice_lib.c @@ -1155,7 +1155,7 @@ static void ice_set_fd_vsi_ctx(struct ice_vsi_ctx *ctxt, struct ice_vsi *vsi) * @ctxt: the VSI context being set * @vsi: the VSI being configured */ -static void ice_set_rss_vsi_ctx(struct ice_vsi_ctx *ctxt, struct ice_vsi *vsi) +void ice_set_rss_vsi_ctx(struct ice_vsi_ctx *ctxt, struct ice_vsi *vsi) { u8 lut_type, hash_type; struct device *dev; @@ -1181,8 +1181,9 @@ static void ice_set_rss_vsi_ctx(struct ice_vsi_ctx *ctxt, struct ice_vsi *vsi) return; } - hash_type = ICE_AQ_VSI_Q_OPT_RSS_HASH_TPLZ; - vsi->rss_hfunc = hash_type; + if (!vsi->rss_hfunc) + vsi->rss_hfunc = ICE_AQ_VSI_Q_OPT_RSS_HASH_TPLZ; + hash_type = vsi->rss_hfunc; ctxt->info.q_opt_rss = FIELD_PREP(ICE_AQ_VSI_Q_OPT_RSS_LUT_M, lut_type) | diff --git a/drivers/net/ethernet/intel/ice/ice_lib.h b/drivers/net/ethernet/intel/ice/ice_lib.h index 49454d98..29ba335 100644 --- a/drivers/net/ethernet/intel/ice/ice_lib.h +++ b/drivers/net/ethernet/intel/ice/ice_lib.h @@ -46,6 +46,7 @@ void ice_vsi_delete(struct ice_vsi *vsi); int ice_vsi_cfg_tc(struct ice_vsi *vsi, u8 ena_tc); int ice_vsi_cfg_rss_lut_key(struct ice_vsi *vsi); +void ice_set_rss_vsi_ctx(struct ice_vsi_ctx *ctxt, struct ice_vsi *vsi); void ice_vsi_cfg_netdev_tc(struct ice_vsi *vsi, u8 ena_tc); -- 2.52.0