From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH 1/2] net/sfc: free mbufs in bulks on EF10 native Tx datapath reap Date: Tue, 12 Sep 2017 14:51:57 -0700 Message-ID: <20170912145157.0794e9f7@plumbers-lap.home.lan> References: <1504880151-15394-1-git-send-email-arybchenko@solarflare.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: , Ivan Malov To: Andrew Rybchenko Return-path: Received: from mail-pg0-f52.google.com (mail-pg0-f52.google.com [74.125.83.52]) by dpdk.org (Postfix) with ESMTP id EC8CEDE0 for ; Tue, 12 Sep 2017 23:52:00 +0200 (CEST) Received: by mail-pg0-f52.google.com with SMTP id d8so23679416pgt.4 for ; Tue, 12 Sep 2017 14:52:00 -0700 (PDT) In-Reply-To: <1504880151-15394-1-git-send-email-arybchenko@solarflare.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Fri, 8 Sep 2017 15:15:50 +0100 Andrew Rybchenko wrote: > + txd->mbuf = NULL; > + if (m == NULL) > + continue; > + > + if ((nb == RTE_DIM(bulk)) || > + ((nb != 0) && (m->pool != bulk[0]->pool))) { > + rte_mempool_put_bulk(bulk[0]->pool, > + (void *)bulk, nb); > + nb = 0; > } > + Why not add rte_mbuf_free_bulk (inline) to base code, rather than recoding everywhere?