From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from shards.monkeyblade.net (shards.monkeyblade.net [184.105.139.130]) by lists.ozlabs.org (Postfix) with ESMTP id 3tHcyY52tHzDvhF for ; Tue, 15 Nov 2016 04:31:49 +1100 (AEDT) Date: Mon, 14 Nov 2016 12:31:41 -0500 (EST) Message-Id: <20161114.123141.575224690412855747.davem@davemloft.net> To: madalin.bucur@nxp.com Cc: netdev@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org, oss@buserror.net, ppc@mindchasers.com, joe@perches.com, pebolle@tiscali.nl, joakim.tjernlund@infinera.com Subject: Re: [PATCH net-next v7 03/10] dpaa_eth: add option to use one buffer pool set From: David Miller In-Reply-To: References: <1478852407-27420-4-git-send-email-madalin.bucur@nxp.com> <20161113.124617.2176429700446266337.davem@davemloft.net> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Madalin-Cristian Bucur Date: Mon, 14 Nov 2016 10:25:13 +0000 > I've introduced this Kconfig option as a backwards compatible option, to > be able to run comparative tests between the independent buffer pool setup > and the previous common buffer pool setup. There are not so many reasons > to use the same buffer pool besides "having the old setup", the memory > saving is marginal, in all other aspects the separate buffer pools setup > fares better. > > I'll remove this patch from the next submission. Should anyone care for > this I can add an entry to the feature backlog to add runtime support but > it will be quite low in priority. If it's a debugging feature then that's certainly how this should be handled.