From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752197AbcKNN6O (ORCPT ); Mon, 14 Nov 2016 08:58:14 -0500 Received: from mail-db5eur01on0087.outbound.protection.outlook.com ([104.47.2.87]:60934 "EHLO EUR01-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751134AbcKNN6M (ORCPT ); Mon, 14 Nov 2016 08:58:12 -0500 From: Madalin-Cristian Bucur To: David Miller 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 Thread-Topic: [PATCH net-next v7 03/10] dpaa_eth: add option to use one buffer pool set Thread-Index: AQHSO/TJrkxYddb5ek+K3lY3w+Ps3KDXM8GAgAD5RKA= Date: Mon, 14 Nov 2016 10:25:13 +0000 Message-ID: References: <1478852407-27420-1-git-send-email-madalin.bucur@nxp.com> <1478852407-27420-4-git-send-email-madalin.bucur@nxp.com> <20161113.124617.2176429700446266337.davem@davemloft.net> In-Reply-To: <20161113.124617.2176429700446266337.davem@davemloft.net> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=madalin.bucur@nxp.com; x-originating-ip: [192.88.146.1] x-microsoft-exchange-diagnostics: 1;AM4PR04MB1604;7:YuRAPyZr32Oq8wS1V7cXYAbSv4xQBOAVGMnsNN1BVEEo+La2FKgs+nc9rwQGXv1YTxphVxnQP4qv30nnekYNEYHw0w0mhAdkb10vZjUJJlUZZV+ndwHryXqUXfXFJRmmxi7HbHGzgjUzK11UAGsnhWBtEJZiIvVBbgSphGGtNaDWfEd6PpJnJin9bEi2s4AqIIEcDPB4cU8s6USXdbQ3R0CA4vokhxWfAYH9M/F6ouhlSH/vB/WOfetMal4+5+ElMQ3qs7tUhYL07Lo7kEQi/rcMJUElKN4oE/k+i+wOukpqX9aoWSEdpuDQU8A4V37Zkp1np8Q0dpzrk9D/leZb6o25NsYT9ujnln25BdxgJ0w= x-ms-office365-filtering-correlation-id: dcb48e34-4fc9-4bfe-77fa-08d40c78852e x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:AM4PR04MB1604; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(185117386973197); x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(6045074)(6060326)(601004)(2401047)(8121501046)(5005006)(3002001)(10201501046)(6055026)(6061321)(6046074);SRVR:AM4PR04MB1604;BCL:0;PCL:0;RULEID:;SRVR:AM4PR04MB1604; x-forefront-prvs: 0126A32F74 x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(6009001)(7916002)(336003)(189002)(199003)(377454003)(66066001)(76576001)(3846002)(87936001)(7696004)(101416001)(122556002)(229853002)(6116002)(4326007)(2906002)(33656002)(586003)(102836003)(74316002)(2950100002)(6916009)(81166006)(345774005)(8936002)(9686002)(5660300001)(189998001)(106356001)(7736002)(2900100001)(305945005)(7846002)(77096005)(81156014)(76176999)(50986999)(54356999)(110136003)(97736004)(106116001)(105586002)(3280700002)(86362001)(3660700001)(92566002)(8676002)(68736007);DIR:OUT;SFP:1101;SCL:1;SRVR:AM4PR04MB1604;H:AM4PR04MB1604.eurprd04.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Nov 2016 10:25:13.3034 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM4PR04MB1604 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id uAEDwRJ3019848 > From: David Miller [mailto:davem@davemloft.net] > Sent: Sunday, November 13, 2016 7:46 PM > > From: Madalin Bucur > Date: Fri, 11 Nov 2016 10:20:00 +0200 > > > @@ -8,3 +8,12 @@ menuconfig FSL_DPAA_ETH > > supporting the Freescale QorIQ chips. > > Depends on Freescale Buffer Manager and Queue Manager > > driver and Frame Manager Driver. > > + > > +if FSL_DPAA_ETH > > +config FSL_DPAA_ETH_COMMON_BPOOL > > + bool "Use a common buffer pool set for all the interfaces" > > + ---help--- > > + The DPAA Ethernet netdevices require buffer pools for storing the > buffers > > + used by the FMan hardware for reception. One can use a single > buffer pool > > + set for all interfaces or a dedicated buffer pool set for each > interface. > > +endif # FSL_DPAA_ETH > > This in no way belongs in Kconfig. If you want to support this, > support it wit a run time configuration choice via ethtool flags > or similar. Do not use debugfs, do not use sysfs, do not use > module options. > > If you put it in Kconfig, distributions will have to pick one way or > another which means that users who want the other choice lose. This > never works. 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. Thank you for your review. Madalin