From mboxrd@z Thu Jan  1 00:00:00 1970
From: Remy Horton <remy.horton@intel.com>
Subject: Re: [RFC PATCH v1 1/4] ethdev: add support for PMD-tuned
 Tx/Rx parameters
Date: Thu, 15 Mar 2018 09:02:36 +0000
Message-ID: <ee367969-ab1e-31f0-a878-c001cadb8627@intel.com>
References: <20180307120851.5822-1-remy.horton@intel.com>
 <20180307120851.5822-2-remy.horton@intel.com>
 <023fbd6c-7cac-6c8b-9a40-7a62e5d47bb7@intel.com>
 <HE1PR0402MB27802A5661A9ABFCD0595D0E90D10@HE1PR0402MB2780.eurprd04.prod.outlook.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Cc: Wenzhuo Lu <wenzhuo.lu@intel.com>, Jingjing Wu <jingjing.wu@intel.com>,
 Qi Zhang <qi.z.zhang@intel.com>, Beilei Xing <beilei.xing@intel.com>,
 Thomas Monjalon <thomas@monjalon.net>
To: Shreyansh Jain <shreyansh.jain@nxp.com>,
 Ferruh Yigit <ferruh.yigit@intel.com>, "dev@dpdk.org" <dev@dpdk.org>
Return-path: <dev-bounces@dpdk.org>
Received: from mga18.intel.com (mga18.intel.com [134.134.136.126])
 by dpdk.org (Postfix) with ESMTP id 3829B7CFC
 for <dev@dpdk.org>; Thu, 15 Mar 2018 10:02:40 +0100 (CET)
In-Reply-To: <HE1PR0402MB27802A5661A9ABFCD0595D0E90D10@HE1PR0402MB2780.eurprd04.prod.outlook.com>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>


On 14/03/2018 15:10, Shreyansh Jain wrote:
[..]
>> What do you think renaming structure and variable name,
>> "preferred_dev_config" perhaps?
>
> I missed this naming while reading this patch. In the deprecation
> notice, 'preferred_size' was the name we came up with precisely on
> this issue of structure having queue length and burst size.
>
> What about using that same name?

Because of namespace issues it will need at least the rte_ prefix. 
Otherwise seems good to me as I cannot think of anything else that is 
relatively short.