ice: Add Support for XPS
authorBenita Bose <benita.bose@intel.com>
Tue, 2 Mar 2021 18:12:02 +0000 (10:12 -0800)
committerTony Nguyen <anthony.l.nguyen@intel.com>
Wed, 31 Mar 2021 21:21:27 +0000 (14:21 -0700)
Enable and configure XPS. The driver code implemented sets up the Transmit
Packet Steering Map, which in turn will be used by the kernel in queue
selection during Tx.

Signed-off-by: Benita Bose <benita.bose@intel.com>
Tested-by: Tony Brelinski <tonyx.brelinski@intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
drivers/net/ethernet/intel/ice/ice_base.c
drivers/net/ethernet/intel/ice/ice_txrx.h

index 1148d76..be26775 100644 (file)
@@ -215,6 +215,26 @@ static u16 ice_calc_q_handle(struct ice_vsi *vsi, struct ice_ring *ring, u8 tc)
 }
 
 /**
+ * ice_cfg_xps_tx_ring - Configure XPS for a Tx ring
+ * @ring: The Tx ring to configure
+ *
+ * This enables/disables XPS for a given Tx descriptor ring
+ * based on the TCs enabled for the VSI that ring belongs to.
+ */
+static void ice_cfg_xps_tx_ring(struct ice_ring *ring)
+{
+       if (!ring->q_vector || !ring->netdev)
+               return;
+
+       /* We only initialize XPS once, so as not to overwrite user settings */
+       if (test_and_set_bit(ICE_TX_XPS_INIT_DONE, ring->xps_state))
+               return;
+
+       netif_set_xps_queue(ring->netdev, &ring->q_vector->affinity_mask,
+                           ring->q_index);
+}
+
+/**
  * ice_setup_tx_ctx - setup a struct ice_tlan_ctx instance
  * @ring: The Tx ring to configure
  * @tlan_ctx: Pointer to the Tx LAN queue context structure to be initialized
@@ -664,6 +684,9 @@ ice_vsi_cfg_txq(struct ice_vsi *vsi, struct ice_ring *ring,
        u16 pf_q;
        u8 tc;
 
+       /* Configure XPS */
+       ice_cfg_xps_tx_ring(ring);
+
        pf_q = ring->reg_idx;
        ice_setup_tx_ctx(ring, &tlan_ctx, pf_q);
        /* copy context contents into the qg_buf */
index 5dab775..40e34ed 100644 (file)
@@ -195,6 +195,11 @@ struct ice_rxq_stats {
        u64 gro_dropped; /* GRO returned dropped */
 };
 
+enum ice_ring_state_t {
+       ICE_TX_XPS_INIT_DONE,
+       ICE_TX_NBITS,
+};
+
 /* this enum matches hardware bits and is meant to be used by DYN_CTLN
  * registers and QINT registers or more generally anywhere in the manual
  * mentioning ITR_INDX, ITR_NONE cannot be used as an index 'n' into any
@@ -292,6 +297,7 @@ struct ice_ring {
        };
 
        struct rcu_head rcu;            /* to avoid race on free */
+       DECLARE_BITMAP(xps_state, ICE_TX_NBITS);        /* XPS Config State */
        struct bpf_prog *xdp_prog;
        struct xsk_buff_pool *xsk_pool;
        u16 rx_offset;