On SP230 the Rx MSI-X vector is asserted during packet aggregation. When
Rx interrupts are disabled, skip the per-queue MSI-X enable and instead
configure the vectors' coalescing and auto-mask state at init, so those
assertions are masked instead of preempting a core.

Signed-off-by: Jiacheng Ye <[email protected]>
---
 drivers/net/hinic3/hinic3_ethdev.c | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/drivers/net/hinic3/hinic3_ethdev.c 
b/drivers/net/hinic3/hinic3_ethdev.c
index 8502017c86..05762ecc50 100644
--- a/drivers/net/hinic3/hinic3_ethdev.c
+++ b/drivers/net/hinic3/hinic3_ethdev.c
@@ -1498,6 +1498,9 @@ hinic3_dev_rx_queue_intr_enable(struct rte_eth_dev *dev, 
uint16_t queue_id)
        if (!rte_intr_dp_is_en(intr_handle) || !intr_handle->intr_vec)
                return 0;
 
+       if (HINIC3_IS_SP230_NIC(nic_dev) && !dev->data->dev_conf.intr_conf.rxq)
+               return 0;
+
        msix_intr = (uint16_t)intr_handle->intr_vec[queue_id];
        hinic3_set_msix_auto_mask_state(nic_dev->hwdev, msix_intr,
                                        HINIC3_SET_MSIX_AUTO_MASK);
@@ -1802,7 +1805,13 @@ hinic3_init_rxq_intr(struct rte_eth_dev *dev)
 
        intr_handle = dev->intr_handle;
        nic_dev = HINIC3_ETH_DEV_TO_PRIVATE_NIC_DEV(dev);
-       if (!dev->data->dev_conf.intr_conf.rxq)
+
+       /*
+        * SP230 asserts the Rx MSI-X vector during aggregation; configure the
+        * vectors' coalescing/auto-mask at init so those assertions are masked
+        * instead of preempting a core.
+        */
+       if (!HINIC3_IS_SP230_NIC(nic_dev) && !dev->data->dev_conf.intr_conf.rxq)
                return 0;
 
        if (!rte_intr_cap_multiple(intr_handle)) {
-- 
2.33.0

Reply via email to