diff options
author | Christopher N Bednarz <christopher.n.bednarz@intel.com> | 2019-02-26 19:35:16 -0500 |
---|---|---|
committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2019-03-22 11:19:17 -0400 |
commit | b58dafbc6f1089942c1e74b8ab9c616fe06dbfac (patch) | |
tree | c27b4710456b3d12e30eb75ed46c57e25ce0ff88 /drivers/net/ethernet/intel/ice/ice_switch.c | |
parent | 277b3a4547b8afbbecdfc52fe7217f018de26c21 (diff) |
ice: Do not set LB_EN for prune switch rules
LB_EN for prune switch rules was causing all TX traffic
to loopback to the internal switch and dropped. When
running bi-directional stress workloads with RDMA
the RDPU would hang blocking tx and rx traffic.
Signed-off-by: Christopher N Bednarz <christopher.n.bednarz@intel.com>
Reviewed-by: Bruce Allan <bruce.w.allan@intel.com>
Signed-off-by: Anirudh Venkataramanan <anirudh.venkataramanan@intel.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Diffstat (limited to 'drivers/net/ethernet/intel/ice/ice_switch.c')
-rw-r--r-- | drivers/net/ethernet/intel/ice/ice_switch.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/net/ethernet/intel/ice/ice_switch.c b/drivers/net/ethernet/intel/ice/ice_switch.c index 48864b59036a..22a14e2da94f 100644 --- a/drivers/net/ethernet/intel/ice/ice_switch.c +++ b/drivers/net/ethernet/intel/ice/ice_switch.c | |||
@@ -643,7 +643,12 @@ static void ice_fill_sw_info(struct ice_hw *hw, struct ice_fltr_info *fi) | |||
643 | fi->fltr_act == ICE_FWD_TO_VSI_LIST || | 643 | fi->fltr_act == ICE_FWD_TO_VSI_LIST || |
644 | fi->fltr_act == ICE_FWD_TO_Q || | 644 | fi->fltr_act == ICE_FWD_TO_Q || |
645 | fi->fltr_act == ICE_FWD_TO_QGRP)) { | 645 | fi->fltr_act == ICE_FWD_TO_QGRP)) { |
646 | fi->lb_en = true; | 646 | /* Setting LB for prune actions will result in replicated |
647 | * packets to the internal switch that will be dropped. | ||
648 | */ | ||
649 | if (fi->lkup_type != ICE_SW_LKUP_VLAN) | ||
650 | fi->lb_en = true; | ||
651 | |||
647 | /* Set lan_en to TRUE if | 652 | /* Set lan_en to TRUE if |
648 | * 1. The switch is a VEB AND | 653 | * 1. The switch is a VEB AND |
649 | * 2 | 654 | * 2 |