Skip to content

ixgbe: vPMD compiling issue and testpmd default value change

Cunming Liang dpdk-dev 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest5 patches 1 replyIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5ixgbe: avoid compiler error when RTE_IXGBE_INC_VECTOR=y and RTE_LIBRTE_IXGBE_RX_ALLOC_BULK_ALLOC=nlib/librte_pmd_ixgbe/ixgbe_rxtx.h+2 -00 replies
2/5ixgbe: fix vpmd compiling error when RTE_LIBRTE_IEEE1588=ylib/librte_pmd_ixgbe/ixgbe_rxtx_vec.c+1 -00 replies
3/5ixgbe: not 'DISABLE' may make confuse, change RTE_IXGBE_RX_OLFLAGS_DISABLE=n to RTE_IXGBE_RX_OLFLAGS_ENABLE=yconfig/common_linuxapp lib/librte_pmd_ixgbe/ixgbe_rxtx_vec.c+3 -30 replies
4/5ixgbe: set default burst size 32 in testpmdapp/test-pmd/testpmd.h lib/librte_pmd_ixgbe/ixgbe_rxtx.c+3 -20 replies
5/5testpmd: change some of testpmd default parameter to well tuned valueapp/test-pmd/testpmd.c app/test-pmd/testpmd.h+5 -30 replies

Reviews trailers per patch

All 5 patches have a Reviewed-by or Acked-by.

PatchBruce RichardsonYong LiuZhaochen Zhan
1/5A-b A-b T-b
2/5A-b A-b T-b
3/5A-b A-b T-b
4/5A-b A-b T-b
5/5A-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.

Read the v1 thread

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=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.

*** 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