net/enic: remove a couple unnecessary statements
authorHyong Youb Kim <hyonkim@cisco.com>
Wed, 10 Jan 2018 09:17:07 +0000 (01:17 -0800)
committerFerruh Yigit <ferruh.yigit@intel.com>
Tue, 16 Jan 2018 17:47:49 +0000 (18:47 +0100)
No need to zero ol_flags as it is overwritten at the end of the
function. No need to check for EOP as the caller (enic_recv_pkts) has
already checked it.

Signed-off-by: Hyong Youb Kim <hyonkim@cisco.com>
Reviewed-by: John Daley <johndale@cisco.com>
drivers/net/enic/enic_rxtx.c

index a10d9bd..a28834e 100644 (file)
@@ -230,17 +230,10 @@ static inline void
 enic_cq_rx_to_pkt_flags(struct cq_desc *cqd, struct rte_mbuf *mbuf)
 {
        struct cq_enet_rq_desc *cqrd = (struct cq_enet_rq_desc *)cqd;
-       uint16_t ciflags, bwflags, pkt_flags = 0, vlan_tci;
-       ciflags = enic_cq_rx_desc_ciflags(cqrd);
+       uint16_t bwflags, pkt_flags = 0, vlan_tci;
        bwflags = enic_cq_rx_desc_bwflags(cqrd);
        vlan_tci = enic_cq_rx_desc_vlan(cqrd);
 
-       mbuf->ol_flags = 0;
-
-       /* flags are meaningless if !EOP */
-       if (unlikely(!enic_cq_rx_desc_eop(ciflags)))
-               goto mbuf_flags_done;
-
        /* VLAN STRIPPED flag. The L2 packet type updated here also */
        if (bwflags & CQ_ENET_RQ_DESC_FLAGS_VLAN_STRIPPED) {
                pkt_flags |= PKT_RX_VLAN | PKT_RX_VLAN_STRIPPED;
@@ -292,7 +285,6 @@ enic_cq_rx_to_pkt_flags(struct cq_desc *cqd, struct rte_mbuf *mbuf)
                }
        }
 
- mbuf_flags_done:
        mbuf->ol_flags = pkt_flags;
 }