ixgbe: vPMD compiling issue and testpmd default value change
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest5 patches 1 replyIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/5 | ixgbe: avoid compiler error when RTE_IXGBE_INC_VECTOR=y and RTE_LIBRTE_IXGBE_RX_ALLOC_BULK_ALLOC=nlib/ | +2 | 0 replies | |
| 2/5 | ixgbe: fix vpmd compiling error when RTE_LIBRTE_IEEE1588=ylib/ | +1 | 0 replies | |
| 3/5 | ixgbe: not 'DISABLE' may make confuse, change RTE_IXGBE_RX_OLFLAGS_DISABLE=n to RTE_IXGBE_RX_OLFLAGS_ENABLE=yconfig/ lib/ | +3 | 0 replies | |
| 4/5 | ixgbe: set default burst size 32 in testpmdapp/ lib/ | +3 | 0 replies | |
| 5/5 | testpmd: change some of testpmd default parameter to well tuned valueapp/ app/ | +5 | 0 replies |
Reviews trailers per patch
All 5 patches have a Reviewed-by or Acked-by.
| Patch | Bruce Richardson | Yong Liu | Zhaochen Zhan |
|---|---|---|---|
| 1/5 | A-b | A-b | T-b |
| 2/5 | A-b | A-b | T-b |
| 3/5 | A-b | A-b | T-b |
| 4/5 | A-b | A-b | T-b |
| 5/5 | A-b | A-b | T-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.
Cover letter [PATCH 0/5] ixgbe: vPMD compiling issue and testpmd default value change
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=y2. 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.
*** BLURB HERE ***
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
app/test-pmd/testpmd.c | 6 +++--- app/test-pmd/testpmd.h | 4 +++- config/common_linuxapp | 2 +- lib/librte_pmd_ixgbe/ixgbe_rxtx.c | 3 ++- lib/librte_pmd_ixgbe/ixgbe_rxtx.h | 2 ++ lib/librte_pmd_ixgbe/ixgbe_rxtx_vec.c | 5 +++-- 6 files changed, 14 insertions(+), 8 deletions(-)
Signature · 1 line
1.8.1.4