From mboxrd@z Thu Jan 1 00:00:00 1970 From: Xueming Li Subject: [PktGen] remove mbuf refcnt verification Date: Wed, 18 Oct 2017 15:19:22 +0800 Message-ID: <20171018071922.178848-1-xuemingl@mellanox.com> Cc: dev@dpdk.org, Xueming Li To: Keith Wiles Return-path: Received: from mellanox.co.il (mail-il-dmz.mellanox.com [193.47.165.129]) by dpdk.org (Postfix) with ESMTP id D13DC1B5F6 for ; Wed, 18 Oct 2017 09:19:58 +0200 (CEST) List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" To improve performance, dpdk drivers are using raw mbuf apis w/o refcnt reset when returning packet to pool. This patch remove refcnt validation when allocating mbufs from pool. Signed-off-by: Xueming Li --- lib/common/mbuf.h | 20 -------------------- 1 file changed, 20 deletions(-) diff --git a/lib/common/mbuf.h b/lib/common/mbuf.h index 706ff3f..4c86502 100644 --- a/lib/common/mbuf.h +++ b/lib/common/mbuf.h @@ -81,41 +81,21 @@ pg_pktmbuf_alloc_bulk(struct rte_mempool *pool, switch (count % 4) { case 0: while (idx != count) { -#ifdef RTE_ASSERT - RTE_ASSERT(rte_mbuf_refcnt_read(mbufs[idx]) == 0); -#else - RTE_VERIFY(rte_mbuf_refcnt_read(mbufs[idx]) == 0); -#endif rte_mbuf_refcnt_set(mbufs[idx], 1); pktmbuf_reset(mbufs[idx]); idx++; /* fall-through */ case 3: -#ifdef RTE_ASSERT - RTE_ASSERT(rte_mbuf_refcnt_read(mbufs[idx]) == 0); -#else - RTE_VERIFY(rte_mbuf_refcnt_read(mbufs[idx]) == 0); -#endif rte_mbuf_refcnt_set(mbufs[idx], 1); pktmbuf_reset(mbufs[idx]); idx++; /* fall-through */ case 2: -#ifdef RTE_ASSERT - RTE_ASSERT(rte_mbuf_refcnt_read(mbufs[idx]) == 0); -#else - RTE_VERIFY(rte_mbuf_refcnt_read(mbufs[idx]) == 0); -#endif rte_mbuf_refcnt_set(mbufs[idx], 1); pktmbuf_reset(mbufs[idx]); idx++; /* fall-through */ case 1: -#ifdef RTE_ASSERT - RTE_ASSERT(rte_mbuf_refcnt_read(mbufs[idx]) == 0); -#else - RTE_VERIFY(rte_mbuf_refcnt_read(mbufs[idx]) == 0); -#endif rte_mbuf_refcnt_set(mbufs[idx], 1); pktmbuf_reset(mbufs[idx]); idx++; -- 2.13.3