[dpdk-dev] [v2] app/testpmd: fix PPPOES flow API

2020-03-31 Thread Xiao Zhang
te 0 pattern pppoe_proto_id is Fixes: 226c6e60c35b ("ethdev: add PPPoE to flow API") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v2 change the command line according to review comments. --- app/test-pmd/cmdline_flow.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletion

[dpdk-dev] app/testpmd: fix PPPOES flow API

2020-03-27 Thread Xiao Zhang
The command line to create RTE flow for specific proto_id of PPPOES is not correct. This patch is to fix this issue. Fixes: 226c6e60c35b ("ethdev: add PPPoE to flow API") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- app/test-pmd/cmdline_flow.c | 13 +++-- 1 file

[dpdk-dev] [v2] ethdev: add PFCP header to flow API

2020-03-05 Thread Xiao Zhang
, when create rules for PFCP Session header with certain SEID the S field need be set 1. Signed-off-by: Xiao Zhang --- v2 - change type of msg_len and seid to BE format. --- app/test-pmd/cmdline_flow.c | 37 + doc/guides/prog_guide/rte_flow.rst

[dpdk-dev] net/i40e: add promiscuous configure unsupported check

2020-02-27 Thread Xiao Zhang
Return ENOTSUP error code when configuring i40evf promiscuous mode to fix port start hang issue on platforms which are unsupported to configure promiscuous mode. Fixes: ddc7cb0d9453 ("net/i40e: re-program promiscuous mode on VF interface") Cc: sta...@dpdk.org Signed-off-by:

[dpdk-dev] ethdev: add PFCP header to flow API

2020-02-19 Thread Xiao Zhang
, when create rules for PFCP Session header with certain SEID the S field need be set 1. Signed-off-by: Xiao Zhang --- app/test-pmd/cmdline_flow.c | 37 + doc/guides/prog_guide/rte_flow.rst | 11 + doc/guides/testpmd_app_ug

[dpdk-dev] [v2] net/ixgbe: check illegal packets

2020-02-17 Thread Xiao Zhang
Some illegal packets will lead to TX hang. This patch checks those illegal packets and protects TX from hanging. Fixes: 7829b8d52be0 ("net/ixgbe: add Tx preparation") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v2 change the minimum packet length according to the datasheet. --

[dpdk-dev] net/ixgbe: check illegal packets

2020-02-13 Thread Xiao Zhang
Some illegal packets will lead to TX hang. This patch checks those illegal packets and protects TX from hanging. Fixes: 7829b8d52be0 ("net/ixgbe: add Tx preparation") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- drivers/net/ixgbe/ixgbe_rxtx.c | 6 ++ drivers/net/ixgbe/ixgbe_

[dpdk-dev] [v2] app/testpmd: parse flow command line for AH

2020-02-13 Thread Xiao Zhang
Add AH and AH SPI to testpmd rte flow command line. Add note and sample AH rules in testpmd guide. Signed-off-by: Xiao Zhang --- v2 Add sample rules in testpmd guide. --- app/test-pmd/cmdline_flow.c | 29 + doc/guides/testpmd_app_ug/testpmd_funcs.rst

[dpdk-dev] app/testpmd: parse flow command line for AH

2020-02-12 Thread Xiao Zhang
add ITEM_AH add ITEM_AH_SPI Signed-off-by: Xiao Zhang --- app/test-pmd/cmdline_flow.c | 29 + 1 file changed, 29 insertions(+) diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c index c2cc4c5..680cb1c 100644 --- a/app/test-pmd/cmdline_flow.c

[dpdk-dev] [v2] net/ixgbe:fix wrong link status

2019-10-28 Thread Xiao Zhang
t;net/ixgbe: improve link state check on VF") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang Reviewed-by: Wei Zhao --- v2 Rebase the patch to latest master code. --- drivers/net/ixgbe/ixgbe_ethdev.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/d

[dpdk-dev] [v3] net/i40e: fix vlan packets drop

2019-10-28 Thread Xiao Zhang
ned-off-by: Xiao Zhang --- v3 Checking more places using max packet len. v2 Add fix for i40evf and correct the checking when using the max_pkt_len. --- drivers/net/i40e/i40e_ethdev.c| 2 +- drivers/net/i40e/i40e_ethdev_vf.c | 11 +++ drivers/net/i40e/i40e_fdir.c | 2 +- driver

[dpdk-dev] [v2] net/i40e: fix vlan packets drop

2019-10-20 Thread Xiao Zhang
ned-off-by: Xiao Zhang --- v2 add fix for i40evf and correct the checking when using the max_pkt_len. --- drivers/net/i40e/i40e_ethdev.c| 2 +- drivers/net/i40e/i40e_ethdev_vf.c | 8 +--- drivers/net/i40e/i40e_rxtx.c | 6 -- 3 files changed, 10 insertions(+), 6 deletions(-) diff --git

[dpdk-dev] [v2] net/i40e: fix integer overflow

2019-10-14 Thread Xiao Zhang
poll mode driver") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v2 Correct the fixline and remove temporary variable. --- drivers/net/i40e/i40e_rxtx.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/net/i40e/i40e_rxtx.c b/drivers/net/i40e/i40e_rx

[dpdk-dev] net/i40e: fix integer overflow

2019-10-14 Thread Xiao Zhang
ivate size") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- drivers/net/i40e/i40e_rxtx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/i40e/i40e_rxtx.c b/drivers/net/i40e/i40e_rxtx.c index bfe161f..ca15b12 100644 --- a/drivers/net/i40e/i40e_rxtx.c +++ b/d

[dpdk-dev] net/i40e: fix vlan packets drop

2019-10-14 Thread Xiao Zhang
vlan packets with ip length bigger then 1496 will not be received by i40e due to wrong packets size checking. This patch fixes the issue by correcting the maximum frame size during checking. Fixes: 35b2d13fd6fd ("net: add rte prefix to ether defines") Cc: sta...@dpdk.org Signed-of

[dpdk-dev] net/e1000: correct mb function

2019-09-10 Thread Xiao Zhang
") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- drivers/net/e1000/em_rxtx.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/net/e1000/em_rxtx.c b/drivers/net/e1000/em_rxtx.c index 5925e49..49c5371 100644 --- a/drivers/net/e1000/em_rxtx.c +++ b/drivers/net/

[dpdk-dev] net/e1000: fix wrong mac type checking

2019-09-10 Thread Xiao Zhang
The mac types of i219 are e1000_pch_spt and e1000_pch_cnp, correct the checking code of mac type when flushing i219 descriptor rings. Fixes: 1fc9701238ed ("net/e1000: fix i219 hang on reset/close") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- drivers/net/e1000/em_ethdev.c | 2

[dpdk-dev] examples/l3fwd-power: fix RX interrupt disable

2019-09-10 Thread Xiao Zhang
Interrupt will not be received when disabling RX interrupt without synchronization mechanism sometimes which leads to wake up issue, add spinlock to fix it. Fixes: b736d64787fc ("mples/l3fwd-power: disable Rx interrupt when waking up") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang ---

[dpdk-dev] [v6] net/i40e: fix vf runtime queues rss config

2019-08-13 Thread Xiao Zhang
not work for X722 vf, use aq command to setup the LUT to make it work properly. Fixes: cea7a51c1750 ("i40evf: support RSS") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v6 update commit message v5 fix compile issue v4 move local variable definition to the begin of the function v

[dpdk-dev] [v5] net/i40e: fix vf runtime queues rss config

2019-08-12 Thread Xiao Zhang
. Fixes: cea7a51c1750 ("i40evf: support RSS") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v5 fix compile issue v4 move local variable definition to the begin of the function v3 move LUT configuration in to i40evf_configure_rss v2 change for loop format to avoid build patch issue --

[dpdk-dev] [v4] net/i40e: fix vf runtime queues rss config

2019-08-12 Thread Xiao Zhang
. Fixes: cea7a51c1750 ("i40evf: support RSS") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v4 move local variable definition to the begin of the function v3 move LUT configuration in to i40evf_configure_rss v2 change for loop format to avoid build patch issue --- driver

[dpdk-dev] [v3] net/i40e: fix vf runtime queues rss config

2019-08-11 Thread Xiao Zhang
. Fixes: cea7a51c1750 ("i40evf: support RSS") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v3 move LUT configuration in to i40evf_configure_rss v2 change for loop format to avoid build patch issue --- drivers/net/i40e/i40e_ethdev_vf.c | 32 ++-- 1 file c

[dpdk-dev] [v2] net/i40e: fix vf runtime queues rss config

2019-08-08 Thread Xiao Zhang
. Fixes: cea7a51c1750 ("i40evf: support RSS") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v2 change for loop format to avoid build patch issue. --- drivers/net/i40e/i40e_ethdev_vf.c | 36 +--- 1 file changed, 29 insertions(+), 7 deletions(-) di

[dpdk-dev] net/i40e: fix vf runtime queues rss config

2019-08-06 Thread Xiao Zhang
properly. Fixes: cea7a51c1750 ("i40evf: support RSS") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- drivers/net/i40e/i40e_ethdev_vf.c | 35 --- 1 file changed, 28 insertions(+), 7 deletions(-) diff --git a/drivers/net/i40e/i40e_ethdev_vf.c b/driver

[dpdk-dev] [PATCH v2 5/5] net/fm10k: fix dereference after null check coverity

2019-08-05 Thread Xiao Zhang
This patch fixes (dereference after null check) coverity issue. The address of first segmented packets was not set correctly during reassembling packets which led to this issue. Coverity issue: 343416 Fixes: fe65e1e1 ("fm10k: add vector scatter Rx") Cc: sta...@dpdk.org Signed-off-by:

[dpdk-dev] [PATCH v2 1/5] net/ixgbe: fix dereference after null check coverity

2019-08-05 Thread Xiao Zhang
) Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c b/drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c index c9ba482..599ba30 100644 --- a/drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c +++

[dpdk-dev] [PATCH v2 2/5] net/ice: fix dereference after null check coverity

2019-08-05 Thread Xiao Zhang
.@dpdk.org Signed-off-by: Xiao Zhang --- drivers/net/ice/ice_rxtx_vec_avx2.c | 1 + drivers/net/ice/ice_rxtx_vec_sse.c | 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/net/ice/ice_rxtx_vec_avx2.c b/drivers/net/ice/ice_rxtx_vec_avx2.c index fac869a..5ce29c2 100644 --- a/drive

[dpdk-dev] [PATCH v2 4/5] net/iavf: fix dereference after null check coverity

2019-08-05 Thread Xiao Zhang
This patch fixes (dereference after null check) coverity issue. The address of first segmented packets was not set correctly during reassembling packets which led to this issue. Coverity issue: 343447 Fixes: 319c421f ("net/avf: enable SSE Rx Tx") Cc: sta...@dpdk.org Signed-off-by:

[dpdk-dev] [PATCH v2 3/5] net/i40e: fix dereference after null check coverity

2019-08-05 Thread Xiao Zhang
ector driver") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- drivers/net/i40e/i40e_rxtx_vec_avx2.c | 1 + drivers/net/i40e/i40e_rxtx_vec_sse.c | 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/net/i40e/i40e_rxtx_vec_avx2.c b/drivers/net/i40e/i40e_rxtx_vec_avx2.c index c4dc99

[dpdk-dev] [v2 0/5] fix dereference after null check coverity

2019-08-05 Thread Xiao Zhang
k: add vector scatter Rx") Coverity issue: 13245 Fixes: 8a44c15a ("net/ixgbe: extract non-x86 specific code from vector driver") Cc: sta...@dpdk.org v2 Add cover letter and move the fix after invalidation check Xiao Zhang (5): net/ixgbe: fix dereference after null check coverity

[dpdk-dev] [PATCH] net/e1000: fix unchecked return value coverity issue

2019-08-04 Thread Xiao Zhang
Add return value checking when reading configure information from PCI register to avoid coverity issue. Fixes: 1fc97012 ("net/e1000: fix i219 hang on reset/close") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- drivers/net/e1000/em_rxtx.c | 18 +++--- 1 file c

[dpdk-dev] [PATCH 4/5] net/ice: fix dereference after null check coverity

2019-08-04 Thread Xiao Zhang
The address of receive queue start segment was not updated when found by iterated checking, update the address to fix coverity issue. Coverity issue: 343452 Coverity issue: 343407 Fixes: c68a52b8 ("net/ice: support vector SSE in Rx") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --

[dpdk-dev] [PATCH 5/5] net/ixgbe: fix dereference after null check coverity

2019-08-04 Thread Xiao Zhang
The address of receive queue start segment was not updated when found by iterated checking, update the address to fix coverity issue. Coverity issue: 13245 Fixes: 8a44c15a ("net/ixgbe: extract non-x86 specific code from vector driver") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --

[dpdk-dev] [PATCH 2/5] net/iavf: fix dereference after null check coverity

2019-08-04 Thread Xiao Zhang
The address of receive queue start segment was not updated when found by iterated checking, update the address to fix coverity issue. Coverity issue: 343447 Fixes: 319c421f ("net/avf: enable SSE Rx Tx") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- drivers/net/iavf/iavf_rxtx_vec

[dpdk-dev] [PATCH 3/5] net/i40e: fix dereference after null check coverity

2019-08-04 Thread Xiao Zhang
ned-off-by: Xiao Zhang --- drivers/net/i40e/i40e_rxtx_vec_avx2.c | 1 + drivers/net/i40e/i40e_rxtx_vec_sse.c | 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/net/i40e/i40e_rxtx_vec_avx2.c b/drivers/net/i40e/i40e_rxtx_vec_avx2.c index c4dc990..0cbb2c2 100644 --- a/driver

[dpdk-dev] [PATCH 1/5] net/fm10k: fix dereference after null check coverity

2019-08-04 Thread Xiao Zhang
The address of receive queue start segment was not updated when found by iterated checking, update the address to fix coverity issue. Coverity issue: 343416 Fixes: fe65e1e1 ("fm10k: add vector scatter Rx") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- drivers/net/fm10k/fm10k_

[dpdk-dev] [v3] net/ice: enable multi-process support

2019-07-24 Thread Xiao Zhang
Add multiple processes support for ice, secondary processes will share memory and configuration with primary process, do not need further initialization for secondary processes. Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v3 Update the title and commit message. v2 Remove limitation for

[dpdk-dev] [v2] net/ice: CVL multi-process support

2019-07-24 Thread Xiao Zhang
Add multiple processes support for CVL, secondary processes will share memory and configuration with primary process, do not need further initialization for secondary processes. Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v2 Remove limitation for secondary processes control path

[dpdk-dev] [v8] net/e1000: fix i219 hang on reset/close

2019-07-21 Thread Xiao Zhang
("e1000: support EM devices (also known as e1000/e1000e)") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v8 Modify to follow code style of dpdk community. v7 Add fix line. v6 Change the fix on em driver instead of igb driver and update the register address according to C-Spec. v5

[dpdk-dev] [v7] net/e1000: fix i219 hang on reset/close

2019-07-21 Thread Xiao Zhang
("e1000: support EM devices (also known as e1000/e1000e)") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v7 Add fix line. v6 Change the fix on em driver instead of igb driver and update the register address according to C-Spec. v5 Change the subject. v4 Correct the tail descriptor of t

[dpdk-dev] [v6] net/e1000: fix i219 hang on reset/close

2019-07-21 Thread Xiao Zhang
Signed-off-by: Xiao Zhang --- v6 Change the fix on em driver instead of igb driver and update the register address according to C-Spec. v5 Change the subject. v4 Correct the tail descriptor of tx ring. v3 Add loop to handle all tx and rx queues. v2 Use configuration register instead of NVM7 to

[dpdk-dev] [v2] net/e1000: fix buffer overrun while i219 processing DMA transactions

2019-07-20 Thread Xiao Zhang
: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v2 Change the fix from igb_rxtx to em_rxtx, since i219 use em PMD. v1 Apply kernel fix to igb PMD driver. --- drivers/net/e1000/base/e1000_ich8lan.h | 1 + drivers/net/e1000/em_rxtx.c| 16 2 files changed, 17 insertions

[dpdk-dev] net/ice: CVL multi-process support

2019-07-19 Thread Xiao Zhang
be allowed to do configuration but share from primary process. Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- drivers/net/ice/ice_ethdev.c | 85 drivers/net/ice/ice_rxtx.c | 24 + 2 files changed, 109 insertions(+) diff --git a

[dpdk-dev] [DPDK] net/ice: CVL multi-process support

2019-07-19 Thread Xiao Zhang
be allowed to do configuration but share from primary process. Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- drivers/net/ice/ice_ethdev.c | 85 drivers/net/ice/ice_rxtx.c | 24 + 2 files changed, 109 insertions(+) diff --git a

[dpdk-dev] [v4] net/i40e: fix SFP X722 not work with FW4.16

2019-07-18 Thread Xiao Zhang
ch AQ instead of register setting") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v4 Add more detailed information in the commit log. v3 Modify the commit log to make it not confused and add fixed line. v2 Unset adminq flag instead of only checking device id when set TPID. v1 Check device id wh

[dpdk-dev] [v3] net/i40e: fix SFP X722 not work with FW4.16

2019-07-18 Thread Xiao Zhang
witch AQ instead of register setting") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v3 Modify the commit log to make it not confused and add fixed line. v2 Unset adminq flag instead of only checking device id when set TPID. v1 Check device id when setting TPID, use adminq if not SFP

[dpdk-dev] [v2] net/i40e: fix SFP X722 not work with FW4.16

2019-07-18 Thread Xiao Zhang
FW4.16 of SFP X722 does not support setting TPID by set_switch_config adminq operation, for firmware version less than 4.16 TPID is set by register instead. For device SFP X722, disable adminq flag to follow the same path with old firmware version to make it work. Signed-off-by: Xiao Zhang

[dpdk-dev] [DPDK] net/i40e: fix SFP X722 not work with FW4.16

2019-07-18 Thread Xiao Zhang
FW4.16 of SFP X722 does not support setting TPID by set_switch_config adminq operation, for firmware version less than 4.16 TPID is set by register instead. Follow the same path with new firmware version for SFP X722 to make it work. Signed-off-by: Xiao Zhang --- drivers/net/i40e/i40e_ethdev.c

[dpdk-dev] [v2] drivers/net: fix dereference after null check coverity

2019-07-14 Thread Xiao Zhang
k: add vector scatter Rx") Coverity issue: 13245 Fixes: 8a44c15a ("net/ixgbe: extract non-x86 specific code from vector driver") Cc: sta...@dpdk.org Signed-off-by: Xiao Zhang --- v2 changes: * update commit log with fixes information. drivers/net/fm10k/fm10k_rxtx_vec.c| 3

[dpdk-dev] [DPDK] drivers/net: fix dereference after null check coverity

2019-07-12 Thread Xiao Zhang
This patch tries to fix the coverity issues of dereference after null check. Coverity issue: 343452 Coverity issue: 343447 Coverity issue: 343422 Coverity issue: 343416 Coverity issue: 343407 Coverity issue: 343403 Coverity issue: 13245 Signed-off-by: Xiao Zhang --- drivers/net/fm10k

[dpdk-dev] [v5] net/e1000: fix i219 hang on reset/close

2019-07-10 Thread Xiao Zhang
Unit hang may occur if multiple descriptors are available in the rings during reset or close. This state can be detected by configure status by bit 8 in register. If the bit is set and there are pending descriptors in one of the rings, we must flush them before reset or close. Signed-off-by: Xiao

[dpdk-dev] [v4] net/e1000: i219 unit hang issue fix on reset/close

2019-07-09 Thread Xiao Zhang
Unit hang may occur if multiple descriptors are available in the rings during reset or close. This state can be detected by configure status by bit 8 in register. If the bit is set and there are pending descriptors in one of the rings, we must flush them before reset or close. Signed-off-by: Xiao

[dpdk-dev] [v3] net/e1000: i219 unit hang issue fix on reset/close

2019-07-08 Thread Xiao Zhang
Unit hang may occur if multiple descriptors are available in the rings during reset or close. This state can be detected by configure status by bit 8 in register. If the bit is set and there are pending descriptors in one of the rings, we must flush them before reset or close. Signed-off-by: Xiao

[dpdk-dev] [DPDK] net/e1000: fix buffer overrun while i219 processing DMA transactions

2019-07-07 Thread Xiao Zhang
-off-by: Xiao Zhang --- drivers/net/e1000/base/e1000_ich8lan.h | 1 + drivers/net/e1000/igb_rxtx.c | 16 2 files changed, 17 insertions(+) diff --git a/drivers/net/e1000/base/e1000_ich8lan.h b/drivers/net/e1000/base/e1000_ich8lan.h index 1f2a3f8..084eb9c 100644 --- a

[dpdk-dev] [v2] net/e1000: i219 unit hang issue fix on reset/close

2019-07-07 Thread Xiao Zhang
Unit hang may occur if multiple descriptors are available in the rings during reset or close. This state can be detected by configure status by bit 8 in register. If the bit is set and there are pending descriptors in one of the rings, we must flush them before reset or close. Signed-off-by: Xiao

[dpdk-dev] [DPDK] net/e1000: Intel i219 reset hang issue fix.

2019-07-03 Thread Xiao Zhang
Unit hang may occur if multiple descriptors are available in the rings during reset or close. Flush them before reset or close to avoid the issue. Change-Id: Ia5e8dd06ad890d19f5093f1696faf68f7f52b491 Signed-off-by: Xiao Zhang --- drivers/net/e1000/base/e1000_ich8lan.h | 1 + drivers/net/e1000