net/ngbe: fix packet statistics
[dpdk.git] / drivers / net / bnxt / bnxt_rxr.c
index e2f09ad..b60c247 100644 (file)
@@ -824,6 +824,9 @@ void bnxt_set_mark_in_mbuf(struct bnxt *bp,
 {
        uint32_t cfa_code = 0;
 
+       if (unlikely(bp->mark_table == NULL))
+               return;
+
        cfa_code = rte_le_to_cpu_16(rxcmp1->cfa_code);
        if (!cfa_code)
                return;
@@ -1332,9 +1335,9 @@ int bnxt_init_one_rx_ring(struct bnxt_rx_queue *rxq)
                if (unlikely(!rxr->rx_buf_ring[i])) {
                        if (bnxt_alloc_rx_data(rxq, rxr, raw_prod) != 0) {
                                PMD_DRV_LOG(WARNING,
-                                           "init'ed rx ring %d with %d/%d mbufs only\n",
+                                           "RxQ %d allocated %d of %d mbufs\n",
                                            rxq->queue_id, i, ring->ring_size);
-                               break;
+                               return -ENOMEM;
                        }
                }
                rxr->rx_raw_prod = raw_prod;
@@ -1362,9 +1365,9 @@ int bnxt_init_one_rx_ring(struct bnxt_rx_queue *rxq)
                if (unlikely(!rxr->ag_buf_ring[i])) {
                        if (bnxt_alloc_ag_data(rxq, rxr, raw_prod) != 0) {
                                PMD_DRV_LOG(WARNING,
-                                           "init'ed AG ring %d with %d/%d mbufs only\n",
+                                           "RxQ %d allocated %d of %d mbufs\n",
                                            rxq->queue_id, i, ring->ring_size);
-                               break;
+                               return -ENOMEM;
                        }
                }
                rxr->ag_raw_prod = raw_prod;
@@ -1408,6 +1411,9 @@ int bnxt_flush_rx_cmp(struct bnxt_cp_ring_info *cpr)
                cons = RING_CMP(cpr->cp_ring_struct, raw_cons);
                rxcmp = (struct rx_pkt_cmpl *)&cpr->cp_desc_ring[cons];
 
+               if (!bnxt_cpr_cmp_valid(rxcmp, raw_cons, ring_mask + 1))
+                       break;
+
                if (CMP_TYPE(rxcmp) == CMPL_BASE_TYPE_HWRM_DONE)
                        return 1;