On Fri, Apr 22, 2022 at 4:20 PM Nithin Dabilpuram
<ndabilpu...@marvell.com> wrote:
>
> Fix multi-seg extraction in vwqe path to avoid updating mbuf[]
> array until it is used via cq0 path.
>
> Fixes: 7fbbc981d54f ("event/cnxk: support vectorized Rx event fast path")
> Cc: pbhagavat...@marvell.com
> Cc: sta...@dpdk.org
>
> Signed-off-by: Nithin Dabilpuram <ndabilpu...@marvell.com>
Please fix following check-gitlog.sh errors
Wrong headline prefix:
net/cnxk: add receive channel backpressure for SDP
Is it candidate for Cc: sta...@dpdk.org backport?
common/cnxk: fix SQ flush sequence
common/cnxk: fix issues in soft expiry disable path
net/cnxk: optimize Rx fast path for security pkts
> ---
> drivers/net/cnxk/cn10k_rx.h | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/net/cnxk/cn10k_rx.h b/drivers/net/cnxk/cn10k_rx.h
> index 00bec01..5ecb20f 100644
> --- a/drivers/net/cnxk/cn10k_rx.h
> +++ b/drivers/net/cnxk/cn10k_rx.h
> @@ -1673,10 +1673,6 @@ cn10k_nix_recv_pkts_vector(void *args, struct rte_mbuf
> **mbufs, uint16_t pkts,
> vst1q_u64((uint64_t *)mbuf2->rearm_data, rearm2);
> vst1q_u64((uint64_t *)mbuf3->rearm_data, rearm3);
>
> - /* Store the mbufs to rx_pkts */
> - vst1q_u64((uint64_t *)&mbufs[packets], mbuf01);
> - vst1q_u64((uint64_t *)&mbufs[packets + 2], mbuf23);
> -
> if (flags & NIX_RX_MULTI_SEG_F) {
> /* Multi segment is enable build mseg list for
> * individual mbufs in scalar mode.
> @@ -1695,6 +1691,10 @@ cn10k_nix_recv_pkts_vector(void *args, struct rte_mbuf
> **mbufs, uint16_t pkts,
> mbuf3, mbuf_initializer, flags);
> }
>
> + /* Store the mbufs to rx_pkts */
> + vst1q_u64((uint64_t *)&mbufs[packets], mbuf01);
> + vst1q_u64((uint64_t *)&mbufs[packets + 2], mbuf23);
> +
> /* Mark mempool obj as "get" as it is alloc'ed by NIX */
> RTE_MEMPOOL_CHECK_COOKIES(mbuf0->pool, (void **)&mbuf0, 1, 1);
> RTE_MEMPOOL_CHECK_COOKIES(mbuf1->pool, (void **)&mbuf1, 1, 1);
> --
> 2.8.4
>
