ice: Enable SW interrupt from FW for LL TS
authorKarol Kolacinski <karol.kolacinski@intel.com>
Wed, 29 Nov 2023 12:40:23 +0000 (13:40 +0100)
committerTony Nguyen <anthony.l.nguyen@intel.com>
Tue, 2 Jan 2024 19:18:32 +0000 (11:18 -0800)
Introduce new capability - Low Latency Timestamping with Interrupt.
On supported devices, driver can request a single timestamp from FW
without polling the register afterwards. Instead, FW can issue
a dedicated interrupt when the timestamp was read from the PHY register
and its value is available to read from the register.
This eliminates the need of bottom half scheduling, which results in
minimal delay for timestamping.

For this mode, allocate TS indices sequentially, so that timestamps are
always completed in FIFO manner.

Co-developed-by: Yochai Hagvi <yochai.hagvi@intel.com>
Signed-off-by: Yochai Hagvi <yochai.hagvi@intel.com>
Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@intel.com>
Signed-off-by: Karol Kolacinski <karol.kolacinski@intel.com>
Tested-by: Pucha Himasekhar Reddy <himasekharx.reddy.pucha@intel.com> (A Contingent worker at Intel)
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
drivers/net/ethernet/intel/ice/ice.h
drivers/net/ethernet/intel/ice/ice_common.c
drivers/net/ethernet/intel/ice/ice_hw_autogen.h
drivers/net/ethernet/intel/ice/ice_main.c
drivers/net/ethernet/intel/ice/ice_ptp.c
drivers/net/ethernet/intel/ice/ice_ptp.h
drivers/net/ethernet/intel/ice/ice_ptp_hw.h
drivers/net/ethernet/intel/ice/ice_type.h

index 30816dce76cf5dfb1acd2c3c8b87ff0b8ab29408..367b613d92c05d2dc73b0373d9d59c1bff34bc05 100644 (file)
@@ -608,6 +608,7 @@ struct ice_pf {
        u32 hw_csum_rx_error;
        u32 oicr_err_reg;
        struct msi_map oicr_irq;        /* Other interrupt cause MSIX vector */
+       struct msi_map ll_ts_irq;       /* LL_TS interrupt MSIX vector */
        u16 max_pf_txqs;        /* Total Tx queues PF wide */
        u16 max_pf_rxqs;        /* Total Rx queues PF wide */
        u16 num_lan_msix;       /* Total MSIX vectors for base driver */
@@ -632,6 +633,7 @@ struct ice_pf {
        unsigned long tx_timeout_last_recovery;
        u32 tx_timeout_recovery_level;
        char int_name[ICE_INT_NAME_STR_LEN];
+       char int_name_ll_ts[ICE_INT_NAME_STR_LEN];
        struct auxiliary_device *adev;
        int aux_idx;
        u32 sw_int_count;
index 6fb0c1e8ae7c2276e8267c6473c803f8d8f64ca5..d3c7e519d29ba12104cb5b6fdd555344743c14df 100644 (file)
@@ -2451,6 +2451,7 @@ ice_parse_1588_dev_caps(struct ice_hw *hw, struct ice_hw_dev_caps *dev_p,
        info->tmr1_ena = ((number & ICE_TS_TMR1_ENA_M) != 0);
 
        info->ts_ll_read = ((number & ICE_TS_LL_TX_TS_READ_M) != 0);
+       info->ts_ll_int_read = ((number & ICE_TS_LL_TX_TS_INT_READ_M) != 0);
 
        info->ena_ports = logical_id;
        info->tmr_own_map = phys_id;
@@ -2471,6 +2472,8 @@ ice_parse_1588_dev_caps(struct ice_hw *hw, struct ice_hw_dev_caps *dev_p,
                  info->tmr1_ena);
        ice_debug(hw, ICE_DBG_INIT, "dev caps: ts_ll_read = %u\n",
                  info->ts_ll_read);
+       ice_debug(hw, ICE_DBG_INIT, "dev caps: ts_ll_int_read = %u\n",
+                 info->ts_ll_int_read);
        ice_debug(hw, ICE_DBG_INIT, "dev caps: ieee_1588 ena_ports = %u\n",
                  info->ena_ports);
        ice_debug(hw, ICE_DBG_INIT, "dev caps: tmr_own_map = %u\n",
index e126dbbd7b2c21cf440dedf4a762589a9d684ca9..cfac1d432c155069b47cf18c2293cc168d7064c4 100644 (file)
 #define GLINT_VECT2FUNC_PF_NUM_M               ICE_M(0x7, 12)
 #define GLINT_VECT2FUNC_IS_PF_S                        16
 #define GLINT_VECT2FUNC_IS_PF_M                        BIT(16)
+#define PFINT_ALLOC                            0x001D2600
+#define PFINT_ALLOC_FIRST                      ICE_M(0x7FF, 0)
 #define PFINT_FW_CTL                           0x0016C800
 #define PFINT_FW_CTL_MSIX_INDX_M               ICE_M(0x7FF, 0)
 #define PFINT_FW_CTL_ITR_INDX_S                        11
index 85004bb2dfe3385b090fc67addea9044943a5baa..2fa46bacf5ba88c529e4e9600c05d3f90f4090a3 100644 (file)
@@ -3046,6 +3046,7 @@ static int ice_xdp(struct net_device *dev, struct netdev_bpf *xdp)
 static void ice_ena_misc_vector(struct ice_pf *pf)
 {
        struct ice_hw *hw = &pf->hw;
+       u32 pf_intr_start_offset;
        u32 val;
 
        /* Disable anti-spoof detection interrupt to prevent spurious event
@@ -3074,6 +3075,47 @@ static void ice_ena_misc_vector(struct ice_pf *pf)
        /* SW_ITR_IDX = 0, but don't change INTENA */
        wr32(hw, GLINT_DYN_CTL(pf->oicr_irq.index),
             GLINT_DYN_CTL_SW_ITR_INDX_M | GLINT_DYN_CTL_INTENA_MSK_M);
+
+       if (!pf->hw.dev_caps.ts_dev_info.ts_ll_int_read)
+               return;
+       pf_intr_start_offset = rd32(hw, PFINT_ALLOC) & PFINT_ALLOC_FIRST;
+       wr32(hw, GLINT_DYN_CTL(pf->ll_ts_irq.index + pf_intr_start_offset),
+            GLINT_DYN_CTL_SW_ITR_INDX_M | GLINT_DYN_CTL_INTENA_MSK_M);
+}
+
+/**
+ * ice_ll_ts_intr - ll_ts interrupt handler
+ * @irq: interrupt number
+ * @data: pointer to a q_vector
+ */
+static irqreturn_t ice_ll_ts_intr(int __always_unused irq, void *data)
+{
+       struct ice_pf *pf = data;
+       u32 pf_intr_start_offset;
+       struct ice_ptp_tx *tx;
+       unsigned long flags;
+       struct ice_hw *hw;
+       u32 val;
+       u8 idx;
+
+       hw = &pf->hw;
+       tx = &pf->ptp.port.tx;
+       spin_lock_irqsave(&tx->lock, flags);
+       ice_ptp_complete_tx_single_tstamp(tx);
+
+       idx = find_next_bit_wrap(tx->in_use, tx->len,
+                                tx->last_ll_ts_idx_read + 1);
+       if (idx != tx->len)
+               ice_ptp_req_tx_single_tstamp(tx, idx);
+       spin_unlock_irqrestore(&tx->lock, flags);
+
+       val = GLINT_DYN_CTL_INTENA_M | GLINT_DYN_CTL_CLEARPBA_M |
+             (ICE_ITR_NONE << GLINT_DYN_CTL_ITR_INDX_S);
+       pf_intr_start_offset = rd32(hw, PFINT_ALLOC) & PFINT_ALLOC_FIRST;
+       wr32(hw, GLINT_DYN_CTL(pf->ll_ts_irq.index + pf_intr_start_offset),
+            val);
+
+       return IRQ_HANDLED;
 }
 
 /**
@@ -3166,7 +3208,19 @@ static irqreturn_t ice_misc_intr(int __always_unused irq, void *data)
 
        if (oicr & PFINT_OICR_TSYN_TX_M) {
                ena_mask &= ~PFINT_OICR_TSYN_TX_M;
-               if (ice_ptp_pf_handles_tx_interrupt(pf)) {
+               if (ice_pf_state_is_nominal(pf) &&
+                   pf->hw.dev_caps.ts_dev_info.ts_ll_int_read) {
+                       struct ice_ptp_tx *tx = &pf->ptp.port.tx;
+                       unsigned long flags;
+                       u8 idx;
+
+                       spin_lock_irqsave(&tx->lock, flags);
+                       idx = find_next_bit_wrap(tx->in_use, tx->len,
+                                                tx->last_ll_ts_idx_read + 1);
+                       if (idx != tx->len)
+                               ice_ptp_req_tx_single_tstamp(tx, idx);
+                       spin_unlock_irqrestore(&tx->lock, flags);
+               } else if (ice_ptp_pf_handles_tx_interrupt(pf)) {
                        set_bit(ICE_MISC_THREAD_TX_TSTAMP, pf->misc_thread);
                        ret = IRQ_WAKE_THREAD;
                }
@@ -3270,6 +3324,20 @@ static void ice_dis_ctrlq_interrupts(struct ice_hw *hw)
        ice_flush(hw);
 }
 
+/**
+ * ice_free_irq_msix_ll_ts- Unroll ll_ts vector setup
+ * @pf: board private structure
+ */
+static void ice_free_irq_msix_ll_ts(struct ice_pf *pf)
+{
+       int irq_num = pf->ll_ts_irq.virq;
+
+       synchronize_irq(irq_num);
+       devm_free_irq(ice_pf_to_dev(pf), irq_num, pf);
+
+       ice_free_irq(pf, pf->ll_ts_irq);
+}
+
 /**
  * ice_free_irq_msix_misc - Unroll misc vector setup
  * @pf: board private structure
@@ -3289,6 +3357,8 @@ static void ice_free_irq_msix_misc(struct ice_pf *pf)
        devm_free_irq(ice_pf_to_dev(pf), misc_irq_num, pf);
 
        ice_free_irq(pf, pf->oicr_irq);
+       if (pf->hw.dev_caps.ts_dev_info.ts_ll_int_read)
+               ice_free_irq_msix_ll_ts(pf);
 }
 
 /**
@@ -3314,10 +3384,12 @@ static void ice_ena_ctrlq_interrupts(struct ice_hw *hw, u16 reg_idx)
               PFINT_MBX_CTL_CAUSE_ENA_M);
        wr32(hw, PFINT_MBX_CTL, val);
 
-       /* This enables Sideband queue Interrupt causes */
-       val = ((reg_idx & PFINT_SB_CTL_MSIX_INDX_M) |
-              PFINT_SB_CTL_CAUSE_ENA_M);
-       wr32(hw, PFINT_SB_CTL, val);
+       if (!hw->dev_caps.ts_dev_info.ts_ll_int_read) {
+               /* enable Sideband queue Interrupt causes */
+               val = ((reg_idx & PFINT_SB_CTL_MSIX_INDX_M) |
+                      PFINT_SB_CTL_CAUSE_ENA_M);
+               wr32(hw, PFINT_SB_CTL, val);
+       }
 
        ice_flush(hw);
 }
@@ -3334,13 +3406,17 @@ static int ice_req_irq_msix_misc(struct ice_pf *pf)
 {
        struct device *dev = ice_pf_to_dev(pf);
        struct ice_hw *hw = &pf->hw;
-       struct msi_map oicr_irq;
+       u32 pf_intr_start_offset;
+       struct msi_map irq;
        int err = 0;
 
        if (!pf->int_name[0])
                snprintf(pf->int_name, sizeof(pf->int_name) - 1, "%s-%s:misc",
                         dev_driver_string(dev), dev_name(dev));
 
+       if (!pf->int_name_ll_ts[0])
+               snprintf(pf->int_name_ll_ts, sizeof(pf->int_name_ll_ts) - 1,
+                        "%s-%s:ll_ts", dev_driver_string(dev), dev_name(dev));
        /* Do not request IRQ but do enable OICR interrupt since settings are
         * lost during reset. Note that this function is called only during
         * rebuild path and not while reset is in progress.
@@ -3349,11 +3425,11 @@ static int ice_req_irq_msix_misc(struct ice_pf *pf)
                goto skip_req_irq;
 
        /* reserve one vector in irq_tracker for misc interrupts */
-       oicr_irq = ice_alloc_irq(pf, false);
-       if (oicr_irq.index < 0)
-               return oicr_irq.index;
+       irq = ice_alloc_irq(pf, false);
+       if (irq.index < 0)
+               return irq.index;
 
-       pf->oicr_irq = oicr_irq;
+       pf->oicr_irq = irq;
        err = devm_request_threaded_irq(dev, pf->oicr_irq.virq, ice_misc_intr,
                                        ice_misc_intr_thread_fn, 0,
                                        pf->int_name, pf);
@@ -3364,10 +3440,34 @@ static int ice_req_irq_msix_misc(struct ice_pf *pf)
                return err;
        }
 
+       /* reserve one vector in irq_tracker for ll_ts interrupt */
+       if (!pf->hw.dev_caps.ts_dev_info.ts_ll_int_read)
+               goto skip_req_irq;
+
+       irq = ice_alloc_irq(pf, false);
+       if (irq.index < 0)
+               return irq.index;
+
+       pf->ll_ts_irq = irq;
+       err = devm_request_irq(dev, pf->ll_ts_irq.virq, ice_ll_ts_intr, 0,
+                              pf->int_name_ll_ts, pf);
+       if (err) {
+               dev_err(dev, "devm_request_irq for %s failed: %d\n",
+                       pf->int_name_ll_ts, err);
+               ice_free_irq(pf, pf->ll_ts_irq);
+               return err;
+       }
+
 skip_req_irq:
        ice_ena_misc_vector(pf);
 
        ice_ena_ctrlq_interrupts(hw, pf->oicr_irq.index);
+       /* This enables LL TS interrupt */
+       pf_intr_start_offset = rd32(hw, PFINT_ALLOC) & PFINT_ALLOC_FIRST;
+       if (pf->hw.dev_caps.ts_dev_info.ts_ll_int_read)
+               wr32(hw, PFINT_SB_CTL,
+                    ((pf->ll_ts_irq.index + pf_intr_start_offset) &
+                     PFINT_SB_CTL_MSIX_INDX_M) | PFINT_SB_CTL_CAUSE_ENA_M);
        wr32(hw, GLINT_ITR(ICE_RX_ITR, pf->oicr_irq.index),
             ITR_REG_ALIGN(ICE_ITR_8K) >> ICE_ITR_GRAN_S);
 
index 95d4c4ed4a62253857a48f2f39fd62d0e7e9e5cb..c4fe28017b8daf02d66f4c48cfec1f5354a696a0 100644 (file)
@@ -524,6 +524,119 @@ ice_ptp_is_tx_tracker_up(struct ice_ptp_tx *tx)
        return tx->init && !tx->calibrating;
 }
 
+/**
+ * ice_ptp_req_tx_single_tstamp - Request Tx timestamp for a port from FW
+ * @tx: the PTP Tx timestamp tracker
+ * @idx: index of the timestamp to request
+ */
+void ice_ptp_req_tx_single_tstamp(struct ice_ptp_tx *tx, u8 idx)
+{
+       struct ice_ptp_port *ptp_port;
+       struct sk_buff *skb;
+       struct ice_pf *pf;
+
+       if (!tx->init)
+               return;
+
+       ptp_port = container_of(tx, struct ice_ptp_port, tx);
+       pf = ptp_port_to_pf(ptp_port);
+
+       /* Drop packets which have waited for more than 2 seconds */
+       if (time_is_before_jiffies(tx->tstamps[idx].start + 2 * HZ)) {
+               /* Count the number of Tx timestamps that timed out */
+               pf->ptp.tx_hwtstamp_timeouts++;
+
+               skb = tx->tstamps[idx].skb;
+               tx->tstamps[idx].skb = NULL;
+               clear_bit(idx, tx->in_use);
+
+               dev_kfree_skb_any(skb);
+               return;
+       }
+
+       ice_trace(tx_tstamp_fw_req, tx->tstamps[idx].skb, idx);
+
+       /* Write TS index to read to the PF register so the FW can read it */
+       wr32(&pf->hw, PF_SB_ATQBAL,
+            TS_LL_READ_TS_INTR | FIELD_PREP(TS_LL_READ_TS_IDX, idx) |
+            TS_LL_READ_TS);
+       tx->last_ll_ts_idx_read = idx;
+}
+
+/**
+ * ice_ptp_complete_tx_single_tstamp - Complete Tx timestamp for a port
+ * @tx: the PTP Tx timestamp tracker
+ */
+void ice_ptp_complete_tx_single_tstamp(struct ice_ptp_tx *tx)
+{
+       struct skb_shared_hwtstamps shhwtstamps = {};
+       u8 idx = tx->last_ll_ts_idx_read;
+       struct ice_ptp_port *ptp_port;
+       u64 raw_tstamp, tstamp;
+       bool drop_ts = false;
+       struct sk_buff *skb;
+       struct ice_pf *pf;
+       u32 val;
+
+       if (!tx->init || tx->last_ll_ts_idx_read < 0)
+               return;
+
+       ptp_port = container_of(tx, struct ice_ptp_port, tx);
+       pf = ptp_port_to_pf(ptp_port);
+
+       ice_trace(tx_tstamp_fw_done, tx->tstamps[idx].skb, idx);
+
+       val = rd32(&pf->hw, PF_SB_ATQBAL);
+
+       /* When the bit is cleared, the TS is ready in the register */
+       if (val & TS_LL_READ_TS) {
+               dev_err(ice_pf_to_dev(pf), "Failed to get the Tx tstamp - FW not ready");
+               return;
+       }
+
+       /* High 8 bit value of the TS is on the bits 16:23 */
+       raw_tstamp = FIELD_GET(TS_LL_READ_TS_HIGH, val);
+       raw_tstamp <<= 32;
+
+       /* Read the low 32 bit value */
+       raw_tstamp |= (u64)rd32(&pf->hw, PF_SB_ATQBAH);
+
+       /* For PHYs which don't implement a proper timestamp ready bitmap,
+        * verify that the timestamp value is different from the last cached
+        * timestamp. If it is not, skip this for now assuming it hasn't yet
+        * been captured by hardware.
+        */
+       if (!drop_ts && tx->verify_cached &&
+           raw_tstamp == tx->tstamps[idx].cached_tstamp)
+               return;
+
+       if (tx->verify_cached && raw_tstamp)
+               tx->tstamps[idx].cached_tstamp = raw_tstamp;
+       clear_bit(idx, tx->in_use);
+       skb = tx->tstamps[idx].skb;
+       tx->tstamps[idx].skb = NULL;
+       if (test_and_clear_bit(idx, tx->stale))
+               drop_ts = true;
+
+       if (!skb)
+               return;
+
+       if (drop_ts) {
+               dev_kfree_skb_any(skb);
+               return;
+       }
+
+       /* Extend the timestamp using cached PHC time */
+       tstamp = ice_ptp_extend_40b_ts(pf, raw_tstamp);
+       if (tstamp) {
+               shhwtstamps.hwtstamp = ns_to_ktime(tstamp);
+               ice_trace(tx_tstamp_complete, skb, idx);
+       }
+
+       skb_tstamp_tx(skb, &shhwtstamps);
+       dev_kfree_skb_any(skb);
+}
+
 /**
  * ice_ptp_process_tx_tstamp - Process Tx timestamps for a port
  * @tx: the PTP Tx timestamp tracker
@@ -575,6 +688,7 @@ ice_ptp_is_tx_tracker_up(struct ice_ptp_tx *tx)
 static void ice_ptp_process_tx_tstamp(struct ice_ptp_tx *tx)
 {
        struct ice_ptp_port *ptp_port;
+       unsigned long flags;
        struct ice_pf *pf;
        struct ice_hw *hw;
        u64 tstamp_ready;
@@ -646,7 +760,7 @@ static void ice_ptp_process_tx_tstamp(struct ice_ptp_tx *tx)
                        drop_ts = true;
 
 skip_ts_read:
-               spin_lock(&tx->lock);
+               spin_lock_irqsave(&tx->lock, flags);
                if (tx->verify_cached && raw_tstamp)
                        tx->tstamps[idx].cached_tstamp = raw_tstamp;
                clear_bit(idx, tx->in_use);
@@ -654,7 +768,7 @@ skip_ts_read:
                tx->tstamps[idx].skb = NULL;
                if (test_and_clear_bit(idx, tx->stale))
                        drop_ts = true;
-               spin_unlock(&tx->lock);
+               spin_unlock_irqrestore(&tx->lock, flags);
 
                /* It is unlikely but possible that the SKB will have been
                 * flushed at this point due to link change or teardown.
@@ -724,6 +838,7 @@ static enum ice_tx_tstamp_work ice_ptp_tx_tstamp_owner(struct ice_pf *pf)
 static enum ice_tx_tstamp_work ice_ptp_tx_tstamp(struct ice_ptp_tx *tx)
 {
        bool more_timestamps;
+       unsigned long flags;
 
        if (!tx->init)
                return ICE_TX_TSTAMP_WORK_DONE;
@@ -732,9 +847,9 @@ static enum ice_tx_tstamp_work ice_ptp_tx_tstamp(struct ice_ptp_tx *tx)
        ice_ptp_process_tx_tstamp(tx);
 
        /* Check if there are outstanding Tx timestamps */
-       spin_lock(&tx->lock);
+       spin_lock_irqsave(&tx->lock, flags);
        more_timestamps = tx->init && !bitmap_empty(tx->in_use, tx->len);
-       spin_unlock(&tx->lock);
+       spin_unlock_irqrestore(&tx->lock, flags);
 
        if (more_timestamps)
                return ICE_TX_TSTAMP_WORK_PENDING;
@@ -771,6 +886,7 @@ ice_ptp_alloc_tx_tracker(struct ice_ptp_tx *tx)
        tx->in_use = in_use;
        tx->stale = stale;
        tx->init = 1;
+       tx->last_ll_ts_idx_read = -1;
 
        spin_lock_init(&tx->lock);
 
@@ -788,6 +904,7 @@ static void
 ice_ptp_flush_tx_tracker(struct ice_pf *pf, struct ice_ptp_tx *tx)
 {
        struct ice_hw *hw = &pf->hw;
+       unsigned long flags;
        u64 tstamp_ready;
        int err;
        u8 idx;
@@ -811,12 +928,12 @@ ice_ptp_flush_tx_tracker(struct ice_pf *pf, struct ice_ptp_tx *tx)
                if (!hw->reset_ongoing && (tstamp_ready & BIT_ULL(phy_idx)))
                        ice_clear_phy_tstamp(hw, tx->block, phy_idx);
 
-               spin_lock(&tx->lock);
+               spin_lock_irqsave(&tx->lock, flags);
                skb = tx->tstamps[idx].skb;
                tx->tstamps[idx].skb = NULL;
                clear_bit(idx, tx->in_use);
                clear_bit(idx, tx->stale);
-               spin_unlock(&tx->lock);
+               spin_unlock_irqrestore(&tx->lock, flags);
 
                /* Count the number of Tx timestamps flushed */
                pf->ptp.tx_hwtstamp_flushed++;
@@ -840,9 +957,11 @@ ice_ptp_flush_tx_tracker(struct ice_pf *pf, struct ice_ptp_tx *tx)
 static void
 ice_ptp_mark_tx_tracker_stale(struct ice_ptp_tx *tx)
 {
-       spin_lock(&tx->lock);
+       unsigned long flags;
+
+       spin_lock_irqsave(&tx->lock, flags);
        bitmap_or(tx->stale, tx->stale, tx->in_use, tx->len);
-       spin_unlock(&tx->lock);
+       spin_unlock_irqrestore(&tx->lock, flags);
 }
 
 /**
@@ -855,9 +974,11 @@ ice_ptp_mark_tx_tracker_stale(struct ice_ptp_tx *tx)
 static void
 ice_ptp_release_tx_tracker(struct ice_pf *pf, struct ice_ptp_tx *tx)
 {
-       spin_lock(&tx->lock);
+       unsigned long flags;
+
+       spin_lock_irqsave(&tx->lock, flags);
        tx->init = 0;
-       spin_unlock(&tx->lock);
+       spin_unlock_irqrestore(&tx->lock, flags);
 
        /* wait for potentially outstanding interrupt to complete */
        synchronize_irq(pf->oicr_irq.virq);
@@ -1257,6 +1378,7 @@ ice_ptp_port_phy_restart(struct ice_ptp_port *ptp_port)
        struct ice_pf *pf = ptp_port_to_pf(ptp_port);
        u8 port = ptp_port->port_num;
        struct ice_hw *hw = &pf->hw;
+       unsigned long flags;
        int err;
 
        if (ice_is_e810(hw))
@@ -1270,9 +1392,9 @@ ice_ptp_port_phy_restart(struct ice_ptp_port *ptp_port)
        kthread_cancel_delayed_work_sync(&ptp_port->ov_work);
 
        /* temporarily disable Tx timestamps while calibrating PHY offset */
-       spin_lock(&ptp_port->tx.lock);
+       spin_lock_irqsave(&ptp_port->tx.lock, flags);
        ptp_port->tx.calibrating = true;
-       spin_unlock(&ptp_port->tx.lock);
+       spin_unlock_irqrestore(&ptp_port->tx.lock, flags);
        ptp_port->tx_fifo_busy_cnt = 0;
 
        /* Start the PHY timer in Vernier mode */
@@ -1281,9 +1403,9 @@ ice_ptp_port_phy_restart(struct ice_ptp_port *ptp_port)
                goto out_unlock;
 
        /* Enable Tx timestamps right away */
-       spin_lock(&ptp_port->tx.lock);
+       spin_lock_irqsave(&ptp_port->tx.lock, flags);
        ptp_port->tx.calibrating = false;
-       spin_unlock(&ptp_port->tx.lock);
+       spin_unlock_irqrestore(&ptp_port->tx.lock, flags);
 
        kthread_queue_delayed_work(pf->ptp.kworker, &ptp_port->ov_work, 0);
 
@@ -2373,18 +2495,23 @@ static long ice_ptp_create_clock(struct ice_pf *pf)
  */
 s8 ice_ptp_request_ts(struct ice_ptp_tx *tx, struct sk_buff *skb)
 {
+       unsigned long flags;
        u8 idx;
 
-       spin_lock(&tx->lock);
+       spin_lock_irqsave(&tx->lock, flags);
 
        /* Check that this tracker is accepting new timestamp requests */
        if (!ice_ptp_is_tx_tracker_up(tx)) {
-               spin_unlock(&tx->lock);
+               spin_unlock_irqrestore(&tx->lock, flags);
                return -1;
        }
 
        /* Find and set the first available index */
-       idx = find_first_zero_bit(tx->in_use, tx->len);
+       idx = find_next_zero_bit(tx->in_use, tx->len,
+                                tx->last_ll_ts_idx_read + 1);
+       if (idx == tx->len)
+               idx = find_first_zero_bit(tx->in_use, tx->len);
+
        if (idx < tx->len) {
                /* We got a valid index that no other thread could have set. Store
                 * a reference to the skb and the start time to allow discarding old
@@ -2398,7 +2525,7 @@ s8 ice_ptp_request_ts(struct ice_ptp_tx *tx, struct sk_buff *skb)
                ice_trace(tx_tstamp_request, skb, idx);
        }
 
-       spin_unlock(&tx->lock);
+       spin_unlock_irqrestore(&tx->lock, flags);
 
        /* return the appropriate PHY timestamp register index, -1 if no
         * indexes were available.
index 032653a7a133cde3d716579f9cd9eac326521ae5..087dd32d876234565e368ece0b959a0fddb0583c 100644 (file)
@@ -131,6 +131,7 @@ enum ice_tx_tstamp_work {
  * @calibrating: if true, the PHY is calibrating the Tx offset. During this
  *               window, timestamps are temporarily disabled.
  * @verify_cached: if true, verify new timestamp differs from last read value
+ * @last_ll_ts_idx_read: index of the last LL TS read by the FW
  */
 struct ice_ptp_tx {
        spinlock_t lock; /* lock protecting in_use bitmap */
@@ -143,6 +144,7 @@ struct ice_ptp_tx {
        u8 init : 1;
        u8 calibrating : 1;
        u8 verify_cached : 1;
+       s8 last_ll_ts_idx_read;
 };
 
 /* Quad and port information for initializing timestamp blocks */
@@ -296,6 +298,8 @@ void ice_ptp_restore_timestamp_mode(struct ice_pf *pf);
 
 void ice_ptp_extts_event(struct ice_pf *pf);
 s8 ice_ptp_request_ts(struct ice_ptp_tx *tx, struct sk_buff *skb);
+void ice_ptp_req_tx_single_tstamp(struct ice_ptp_tx *tx, u8 idx);
+void ice_ptp_complete_tx_single_tstamp(struct ice_ptp_tx *tx);
 enum ice_tx_tstamp_work ice_ptp_process_ts(struct ice_pf *pf);
 
 u64 ice_ptp_get_rx_hwts(const union ice_32b_rx_flex_desc *rx_desc,
@@ -324,6 +328,11 @@ ice_ptp_request_ts(struct ice_ptp_tx *tx, struct sk_buff *skb)
        return -1;
 }
 
+static inline void ice_ptp_req_tx_single_tstamp(struct ice_ptp_tx *tx, u8 idx)
+{ }
+
+static inline void ice_ptp_complete_tx_single_tstamp(struct ice_ptp_tx *tx) { }
+
 static inline bool ice_ptp_process_ts(struct ice_pf *pf)
 {
        return true;
index e976138e934aa19b91fff92d727b12a7ee19726d..1f3e0312443080177399d3851c97229e1b3b4895 100644 (file)
@@ -509,6 +509,7 @@ int ice_cgu_get_output_pin_state_caps(struct ice_hw *hw, u8 pin_id,
 #define TS_LL_READ_RETRIES             200
 #define TS_LL_READ_TS_HIGH             GENMASK(23, 16)
 #define TS_LL_READ_TS_IDX              GENMASK(29, 24)
+#define TS_LL_READ_TS_INTR             BIT(30)
 #define TS_LL_READ_TS                  BIT(31)
 
 /* Internal PHY timestamp address */
index 6df7c4487ad0fc693241707e2461c3afe8789d64..8ed1edd3dea56580d9ad873a9f54aaef945ab00a 100644 (file)
@@ -353,6 +353,7 @@ struct ice_ts_func_info {
 #define ICE_TS_TMR0_ENA_M              BIT(25)
 #define ICE_TS_TMR1_ENA_M              BIT(26)
 #define ICE_TS_LL_TX_TS_READ_M         BIT(28)
+#define ICE_TS_LL_TX_TS_INT_READ_M     BIT(29)
 
 struct ice_ts_dev_info {
        /* Device specific info */
@@ -366,6 +367,7 @@ struct ice_ts_dev_info {
        u8 tmr0_ena;
        u8 tmr1_ena;
        u8 ts_ll_read;
+       u8 ts_ll_int_read;
 };
 
 /* Function specific capabilities */