From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH 0/5] ixgbe: vPMD compiling issue and testpmd default value change Date: Thu, 26 Jun 2014 14:33:06 +0200 Message-ID: <3985757.ddR0XZUxTh@xps13> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: Cunming Liang Return-path: In-Reply-To: List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" 2014-06-26 14:53, Cunming Liang: > This patch is used to fix several issue found in R1.7 RC1. > 1. There's compiling error when change some defalut config in > common_linuxapp. They are > RTE_LIBRTE_IXGBE_RX_ALLOC_BULK_ALLOC=n > RTE_LIBRTE_IEEE1588=y > 2. Renaming RTE_IXGBE_RX_OLFLAGS_DISABLE to RTE_IXGBE_RX_OLFLAGS_ENABLE > This avoid making user confused by defalut setting 'not DISABLE'. > 3. Set testpmd defalut burst size to 32 > As vPMD RX only accept no less than 32, this avoid testpmd automation > pass. > 4. Change testpmd mbuf cache and rx/tx threshold defalut value. > It makes easier to get better performance with vPMD. > > Cunming Liang (5): > ixgbe: avoid compiler error when RTE_IXGBE_INC_VECTOR=y and > RTE_LIBRTE_IXGBE_RX_ALLOC_BULK_ALLOC=n > ixgbe: fix vpmd compiling error when RTE_LIBRTE_IEEE1588=y > ixgbe: not 'DISABLE' may make confuse, change > RTE_IXGBE_RX_OLFLAGS_DISABLE=n to RTE_IXGBE_RX_OLFLAGS_ENABLE=y > ixgbe: set default burst size 32 in testpmd > testpmd: change some of testpmd default parameter to well tuned value Applied for version 1.7.0. Just a note about commit titles for next times: As explained on the website (http://dpdk.org/dev#send), "The title will be clearly visible in the git repository and in the email archives. So it is important to make it short and clear for quick reading and searches." You may have a look at how I've reworded these titles: http://dpdk.org/browse/dpdk/log It's not the most important thing in the patches, so it's just a reminder for contributors reading this email. Thanks -- Thomas