DPDK patches and discussions
 help / color / mirror / Atom feed
From: Thomas Monjalon <thomas.monjalon@6wind.com>
To: Cunming Liang <cunming.liang@intel.com>
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [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> (raw)
In-Reply-To: <cover.1403759017.git.cunming.liang@intel.com>

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

      parent reply	other threads:[~2014-06-26 12:32 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-26  6:53 Cunming Liang
2014-06-26  6:53 ` [dpdk-dev] [PATCH 1/5] ixgbe: avoid compiler error when RTE_IXGBE_INC_VECTOR=y and RTE_LIBRTE_IXGBE_RX_ALLOC_BULK_ALLOC=n Cunming Liang
2014-06-26  6:53 ` [dpdk-dev] [PATCH 2/5] ixgbe: fix vpmd compiling error when RTE_LIBRTE_IEEE1588=y Cunming Liang
2014-06-26  6:53 ` [dpdk-dev] [PATCH 3/5] ixgbe: not 'DISABLE' may make confuse, change RTE_IXGBE_RX_OLFLAGS_DISABLE=n to RTE_IXGBE_RX_OLFLAGS_ENABLE=y Cunming Liang
2014-06-26  6:53 ` [dpdk-dev] [PATCH 4/5] ixgbe: set default burst size 32 in testpmd Cunming Liang
2014-06-26  6:53 ` [dpdk-dev] [PATCH 5/5] testpmd: change some of testpmd default parameter to well tuned value Cunming Liang
2014-06-26 12:33 ` Thomas Monjalon [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3985757.ddR0XZUxTh@xps13 \
    --to=thomas.monjalon@6wind.com \
    --cc=cunming.liang@intel.com \
    --cc=dev@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git