From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 6790A152A for ; Mon, 16 Jan 2017 08:37:35 +0100 (CET) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga101.fm.intel.com with ESMTP; 15 Jan 2017 23:37:34 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,238,1477983600"; d="scan'208";a="809347571" Received: from yliu-dev.sh.intel.com (HELO yliu-dev) ([10.239.67.162]) by FMSMGA003.fm.intel.com with ESMTP; 15 Jan 2017 23:37:33 -0800 Date: Mon, 16 Jan 2017 15:39:47 +0800 From: Yuanhan Liu To: Stephen Hemminger Cc: Ferruh Yigit , "Ananyev, Konstantin" , Sergey Vyazmitinov , "olivier.matz@6wind.com" , "dev@dpdk.org" Message-ID: <20170116073947.GO9770@yliu-dev.sh.intel.com> References: <1483048216-2936-1-git-send-email-s.vyazmitinov@brain4net.com> <20170111081759.7b1ee146@xeon-e3> <2601191342CEEE43887BDE71AB9772583F103F8F@irsmsx105.ger.corp.intel.com> <20170111093559.753a0fc9@xeon-e3> <2601191342CEEE43887BDE71AB9772583F103FCA@irsmsx105.ger.corp.intel.com> <63819ae1-f056-0ad7-b7dd-041fe1fe08fa@intel.com> <2601191342CEEE43887BDE71AB9772583F104048@irsmsx105.ger.corp.intel.com> <20170111105620.00af45a7@xeon-e3> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170111105620.00af45a7@xeon-e3> User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH] kni: use bulk functions to allocate and free mbufs X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 16 Jan 2017 07:37:35 -0000 On Wed, Jan 11, 2017 at 10:56:20AM -0800, Stephen Hemminger wrote: > Please write generic code. Something like the following (untested). Despite there are few compile errors (which are easy to fix), it works fine. More importantly, it saves near 2000 cycles if I apply the bulk free to virtio_xmit_cleanup(). Only one mp is being used. --yliu > > diff --git a/lib/librte_mbuf/rte_mbuf.h b/lib/librte_mbuf/rte_mbuf.h > index 4476d75379fd..b7a743ec5c87 100644 > --- a/lib/librte_mbuf/rte_mbuf.h > +++ b/lib/librte_mbuf/rte_mbuf.h > @@ -306,6 +306,9 @@ extern "C" { > /** Alignment constraint of mbuf private area. */ > #define RTE_MBUF_PRIV_ALIGN 8 > > +/** Maximum number of mbufs freed in bulk. */ > +#define RTE_MBUF_BULK_FREE 64 > + > /** > * Get the name of a RX offload flag > * > @@ -1261,6 +1264,50 @@ static inline void rte_pktmbuf_free(struct rte_mbuf *m) > } > > /** > + * Free n packets mbuf back into its original mempool. > + * > + * Free each mbuf, and all its segments in case of chained buffers. Each > + * segment is added back into its original mempool. > + * > + * @param mbufs > + * The packets mbufs array to be freed. > + * @param n > + * Number of packets. > + */ > +static inline void rte_pktmbuf_free_bulk(struct rte_mbuf **mbufs, > + unsigned n) > +{ > + struct rte_mbuf *tofree[RTE_MBUF_BULK_FREE]; > + struct rte_mempool *mp; > + unsigned i, count = 0; > + > + for (i = 0; i < n; i++) { > + struct rte_mbuf *m, *m_next; > + > + for (m = mbufs[i]; m; m = m_next) { > + m_next = m->next; > + > + if (count > 0 && > + (unlikely(m->pool != mp || count == RTE_MBUF_BULK_FREE))) { > + rte_mempool_put_bulk(mp, tofree, count); > + count = 0; > + } > + > + mp = m->pool; > + > + if (likely(__rte_pktmbuf_prefree_seg(m))) { > + m->next = NULL; > + tofree[count++] = m; > + } > + } > + } > + > + if (likely(count > 0)) > + rte_mempool_put_bulk(mp, tofree, count); > +} > + > + > +/** > * Creates a "clone" of the given packet mbuf. > * > * Walks through all segments of the given packet mbuf, and for each of them: > > > This handles multiple pools and multi-segment and indirect mbufs. >