[dpdk-dev] Issue with non-scattered rx in ixgbe and i40e when mbuf private area size is odd
Thomas Monjalon
thomas.monjalon at 6wind.com
Mon Aug 3 00:32:02 CEST 2015
2015-07-29 18:12, Ananyev, Konstantin:
> As we don't support split header feature anyway, I think we can fix it just by always setting HBA in the RXD to zero.
> Could you try the fix for ixgbe below?
Please Konstantin, could you send a proper patch for this fix?
Thanks
> --- a/drivers/net/ixgbe/ixgbe_rxtx.c
> +++ b/drivers/net/ixgbe/ixgbe_rxtx.c
> @@ -1183,7 +1183,7 @@ ixgbe_rx_alloc_bufs(struct ixgbe_rx_queue *rxq, bool reset_mbuf)
>
> /* populate the descriptors */
> dma_addr = rte_cpu_to_le_64(RTE_MBUF_DATA_DMA_ADDR_DEFAULT(mb));
> - rxdp[i].read.hdr_addr = dma_addr;
> + rxdp[i].read.hdr_addr = 0;
> rxdp[i].read.pkt_addr = dma_addr;
> }
>
> @@ -1414,7 +1414,7 @@ ixgbe_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts,
> rxe->mbuf = nmb;
> dma_addr =
> rte_cpu_to_le_64(RTE_MBUF_DATA_DMA_ADDR_DEFAULT(nmb));
> - rxdp->read.hdr_addr = dma_addr;
> + rxdp->read.hdr_addr = 0;
> rxdp->read.pkt_addr = dma_addr;
>
> /*
> @@ -1741,7 +1741,7 @@ next_desc:
> rxe->mbuf = nmb;
>
> rxm->data_off = RTE_PKTMBUF_HEADROOM;
> - rxdp->read.hdr_addr = dma;
> + rxdp->read.hdr_addr = 0;
> rxdp->read.pkt_addr = dma;
> } else
> rxe->mbuf = NULL;
> @@ -3633,7 +3633,7 @@ ixgbe_alloc_rx_queue_mbufs(struct ixgbe_rx_queue *rxq)
> dma_addr =
> rte_cpu_to_le_64(RTE_MBUF_DATA_DMA_ADDR_DEFAULT(mbuf));
> rxd = &rxq->rx_ring[i];
> - rxd->read.hdr_addr = dma_addr;
> + rxd->read.hdr_addr = 0;
> rxd->read.pkt_addr = dma_addr;
> rxe[i].mbuf = mbuf;
> }
> diff --git a/drivers/net/ixgbe/ixgbe_rxtx_vec.c b/drivers/net/ixgbe/ixgbe_rxtx_vec.c
> index 6c1647e..16a9c64 100644
> --- a/drivers/net/ixgbe/ixgbe_rxtx_vec.c
> +++ b/drivers/net/ixgbe/ixgbe_rxtx_vec.c
> @@ -56,6 +56,8 @@ ixgbe_rxq_rearm(struct ixgbe_rx_queue *rxq)
> RTE_PKTMBUF_HEADROOM);
> __m128i dma_addr0, dma_addr1;
>
> + const __m128i hba_msk = _mm_set_epi64x(0, UINT64_MAX);
> +
> rxdp = rxq->rx_ring + rxq->rxrearm_start;
>
> /* Pull 'n' more MBUFs into the software ring */
> @@ -108,6 +110,9 @@ ixgbe_rxq_rearm(struct ixgbe_rx_queue *rxq)
> dma_addr0 = _mm_add_epi64(dma_addr0, hdr_room);
> dma_addr1 = _mm_add_epi64(dma_addr1, hdr_room);
>
> + dma_addr0 = _mm_and_si128(dma_addr0, hba_msk);
> + dma_addr1 = _mm_and_si128(dma_addr1, hba_msk);
> +
> /* flush desc with pa dma_addr */
> _mm_store_si128((__m128i *)&rxdp++->read, dma_addr0);
> _mm_store_si128((__m128i *)&rxdp++->read, dma_addr1);
> bash-4.2$ cat patch1
> diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
> index a0c8847..94967c5 100644
> --- a/drivers/net/ixgbe/ixgbe_rxtx.c
> +++ b/drivers/net/ixgbe/ixgbe_rxtx.c
> @@ -1183,7 +1183,7 @@ ixgbe_rx_alloc_bufs(struct ixgbe_rx_queue *rxq, bool reset_mbuf)
>
> /* populate the descriptors */
> dma_addr = rte_cpu_to_le_64(RTE_MBUF_DATA_DMA_ADDR_DEFAULT(mb));
> - rxdp[i].read.hdr_addr = dma_addr;
> + rxdp[i].read.hdr_addr = 0;
> rxdp[i].read.pkt_addr = dma_addr;
> }
>
> @@ -1414,7 +1414,7 @@ ixgbe_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts,
> rxe->mbuf = nmb;
> dma_addr =
> rte_cpu_to_le_64(RTE_MBUF_DATA_DMA_ADDR_DEFAULT(nmb));
> - rxdp->read.hdr_addr = dma_addr;
> + rxdp->read.hdr_addr = 0;
> rxdp->read.pkt_addr = dma_addr;
>
> /*
> @@ -1741,7 +1741,7 @@ next_desc:
> rxe->mbuf = nmb;
>
> rxm->data_off = RTE_PKTMBUF_HEADROOM;
> - rxdp->read.hdr_addr = dma;
> + rxdp->read.hdr_addr = 0;
> rxdp->read.pkt_addr = dma;
> } else
> rxe->mbuf = NULL;
> @@ -3633,7 +3633,7 @@ ixgbe_alloc_rx_queue_mbufs(struct ixgbe_rx_queue *rxq)
> dma_addr =
> rte_cpu_to_le_64(RTE_MBUF_DATA_DMA_ADDR_DEFAULT(mbuf));
> rxd = &rxq->rx_ring[i];
> - rxd->read.hdr_addr = dma_addr;
> + rxd->read.hdr_addr = 0;
> rxd->read.pkt_addr = dma_addr;
> rxe[i].mbuf = mbuf;
> }
> diff --git a/drivers/net/ixgbe/ixgbe_rxtx_vec.c b/drivers/net/ixgbe/ixgbe_rxtx_vec.c
> index 6c1647e..16a9c64 100644
> --- a/drivers/net/ixgbe/ixgbe_rxtx_vec.c
> +++ b/drivers/net/ixgbe/ixgbe_rxtx_vec.c
> @@ -56,6 +56,8 @@ ixgbe_rxq_rearm(struct ixgbe_rx_queue *rxq)
> RTE_PKTMBUF_HEADROOM);
> __m128i dma_addr0, dma_addr1;
>
> + const __m128i hba_msk = _mm_set_epi64x(0, UINT64_MAX);
> +
> rxdp = rxq->rx_ring + rxq->rxrearm_start;
>
> /* Pull 'n' more MBUFs into the software ring */
> @@ -108,6 +110,9 @@ ixgbe_rxq_rearm(struct ixgbe_rx_queue *rxq)
> dma_addr0 = _mm_add_epi64(dma_addr0, hdr_room);
> dma_addr1 = _mm_add_epi64(dma_addr1, hdr_room);
>
> + dma_addr0 = _mm_and_si128(dma_addr0, hba_msk);
> + dma_addr1 = _mm_and_si128(dma_addr1, hba_msk);
> +
> /* flush desc with pa dma_addr */
> _mm_store_si128((__m128i *)&rxdp++->read, dma_addr0);
> _mm_store_si128((__m128i *)&rxdp++->read, dma_addr1);
More information about the dev
mailing list