ice: don't set target VSI for subfunction
authorMichal Swiatkowski <michal.swiatkowski@linux.intel.com>
Tue, 20 Aug 2024 06:57:55 +0000 (08:57 +0200)
committerTony Nguyen <anthony.l.nguyen@intel.com>
Fri, 6 Sep 2024 18:01:24 +0000 (11:01 -0700)
Add check for subfunction before setting target VSI. It is needed for PF
in switchdev mode but not for subfunction (even in switchdev mode).

Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: Michal Swiatkowski <michal.swiatkowski@linux.intel.com>
Tested-by: Rafal Romanowski <rafal.romanowski@intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
drivers/net/ethernet/intel/ice/ice_txrx.c

index c9bc3f1add5d37839e00dea9f9fecc04e737b327..8208055d6e7fc59148f8c86d03b454c649234744 100644 (file)
@@ -2368,7 +2368,7 @@ ice_xmit_frame_ring(struct sk_buff *skb, struct ice_tx_ring *tx_ring)
                                        ICE_TXD_CTX_QW1_CMD_S);
 
        ice_tstamp(tx_ring, skb, first, &offload);
-       if (ice_is_switchdev_running(vsi->back))
+       if (ice_is_switchdev_running(vsi->back) && vsi->type != ICE_VSI_SF)
                ice_eswitch_set_target_vsi(skb, &offload);
 
        if (offload.cd_qw1 & ICE_TX_DESC_DTYPE_CTX) {