From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v6 0/4] ethdev: add per-PMD tuning of RxTx parmeters Date: Tue, 10 Apr 2018 19:56:22 +0100 Message-ID: References: <20180410094319.24083-1-remy.horton@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Cc: John McNamara , Wenzhuo Lu , Jingjing Wu , Qi Zhang , Beilei Xing , Shreyansh Jain , Thomas Monjalon To: Remy Horton , dev@dpdk.org Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id BA6C11B8E2 for ; Tue, 10 Apr 2018 20:56:29 +0200 (CEST) In-Reply-To: <20180410094319.24083-1-remy.horton@intel.com> Content-Language: en-US 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 4/10/2018 10:43 AM, Remy Horton wrote: > The optimal values of several transmission & reception related parameters, > such as burst sizes, descriptor ring sizes, and number of queues, varies > between different network interface devices. This patchset allows individual > PMDs to specify their preferred parameter values, and if so indicated by an > application, for them to be used automatically by the ethdev layer. > > rte_eth_dev_configure() has been changed so that specifying zero for both > nb_rx_q AND nb_tx_q causes it to use driver preferred values, and if these > are not available, falls back to EAL defaults. Setting one (but not both) > to zero does not cause the use of defaults, as having one of them zeroed is > a valid setup. > > This patchset includes per-PMD values for e1000 and i40e but it is expected > that subsequent patchsets will cover other PMDs. A deprecation notice > covering the API/ABI change is in place. > > Changes in v6: > * Updated/corrected testpmd documentation > * Carried forward acks/review > * Rebased to d218a4d060de > > Changes in v5: > * uint_16_t corrected to uint16_t > > Changes in v4: > * Added API/ABI change documentation > * Rebased to 78f5a2e93d74 > > Changes in v3: > * Changed formatting around new rte_eth_dev_info fields > * Added Doxygen documentation to struct rte_eth_dev_portconf > * Testpmd "port config all burst 0" and --burst=0 uses PMD > Rx burst recommendations. > * Added to release notes > * Rebased to 8ea081f38161 > > Changes in v2: > * Rebased to master > * Removed fallback values from rte_eth_dev_info_get() > * Added fallback values to rte_rte_[rt]x_queue_setup() > * Added fallback values to rte_eth_dev_configure() > * Corrected comment > * Removed deprecation notice > * Split RX and Tx into seperate structures > * Changed parameter names > > > Remy Horton (4): > ethdev: add support for PMD-tuned Tx/Rx parameters > net/e1000: add TxRx tuning parameters > net/i40e: add TxRx tuning parameters > testpmd: make use of per-PMD TxRx parameters Series applied to dpdk-next-net/master, thanks. (Thomas' ack added into ethdev patch)