In the receive path, the function to set mbuf ol_flags used the
mbuf packet_type before it was set.

Fixes: 947d860c821f ("enic: improve Rx performance")

Signed-off-by: John Daley <johndale at cisco.com>
---
 drivers/net/enic/enic_rx.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/enic/enic_rx.c b/drivers/net/enic/enic_rx.c
index 945a60f..59ebaa4 100644
--- a/drivers/net/enic/enic_rx.c
+++ b/drivers/net/enic/enic_rx.c
@@ -210,7 +210,7 @@ enic_cq_rx_to_pkt_flags(struct cq_desc *cqd, struct 
rte_mbuf *mbuf)
        ciflags = enic_cq_rx_desc_ciflags(cqrd);
        bwflags = enic_cq_rx_desc_bwflags(cqrd);

-       ASSERT(mbuf->ol_flags == 0);
+       mbuf->ol_flags = 0;

        /* flags are meaningless if !EOP */
        if (unlikely(!enic_cq_rx_desc_eop(ciflags)))
@@ -340,10 +340,10 @@ enic_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts,
                rxmb->pkt_len = rx_pkt_len;
                rxmb->data_len = rx_pkt_len;
                rxmb->port = enic->port_id;
+               rxmb->packet_type = enic_cq_rx_flags_to_pkt_type(&cqd);
                rxmb->ol_flags = ol_err_flags;
                if (!ol_err_flags)
                        enic_cq_rx_to_pkt_flags(&cqd, rxmb);
-               rxmb->packet_type = enic_cq_rx_flags_to_pkt_type(&cqd);

                /* prefetch mbuf data for caller */
                rte_packet_prefetch(RTE_PTR_ADD(rxmb->buf_addr,
-- 
2.7.0

Reply via email to