[dpdk-dev] [PATCH 0/4] eal/common: introduce rte_memset and related test

2016-12-02 Thread Zhiyong Yang
rtio/vhost non-mergeable loopback testing. Zhiyong Yang (4): eal/common: introduce rte_memset on IA platform app/test: add functional autotest for rte_memset app/test: add performance autotest for rte_memset lib/librte_vhost: improve vhost perf using rte_memset app/test/Make

[dpdk-dev] [PATCH 4/4] lib/librte_vhost: improve vhost perf using rte_memset

2016-12-02 Thread Zhiyong Yang
involves both load (from stack, the virtio_hdr var) and store (to virtio driver memory), while rte_memset just involves store. Signed-off-by: Zhiyong Yang --- doc/guides/rel_notes/release_17_02.rst | 11 +++ lib/librte_vhost/virtio_net.c | 18 +++--- 2 files changed, 22

[dpdk-dev] [PATCH 2/4] app/test: add functional autotest for rte_memset

2016-12-02 Thread Zhiyong Yang
run time. Signed-off-by: Zhiyong Yang --- app/test/Makefile | 2 + app/test/test_memset.c | 158 + 2 files changed, 160 insertions(+) create mode 100644 app/test/test_memset.c diff --git a/app/test/Makefile b/app/test/Makefile index 5be023a

[dpdk-dev] [PATCH 1/4] eal/common: introduce rte_memset on IA platform

2016-12-02 Thread Zhiyong Yang
ment are fully taken into consideration. Signed-off-by: Zhiyong Yang --- .../common/include/arch/x86/rte_memset.h | 376 + lib/librte_eal/common/include/generic/rte_memset.h | 51 +++ 2 files changed, 427 insertions(+) create mode 100644 lib/librte_eal/common/inc

[dpdk-dev] [PATCH 3/4] app/test: add performance autotest for rte_memset

2016-12-02 Thread Zhiyong Yang
in memory. Usage: step 1: run ./x86_64-native-linuxapp-gcc/app/test step 2: run command memset_perf_autotest at the run time. Signed-off-by: Zhiyong Yang --- app/test/Makefile | 1 + app/test/test_memset_perf.c | 348 2 files changed, 349

[dpdk-dev] [RFC] lib/librte_ether: consistent PMD batching behavior

2017-01-20 Thread Zhiyong Yang
ply the above method to rte_eth_rx_burst() as well. In summary, The purpose of the RFC makes the job easier and more simple for driver writers and avoids to write too much duplicate code at the applica- tion level. Signed-off-by: Zhiyong Yang --- lib/librte_ether/rte_ethdev.h | 41

[dpdk-dev] [PATCH v2 0/4] eal/common: introduce rte_memset and related test

2016-12-27 Thread Zhiyong Yang
between rte_memset and memset on haswell. Patch 4: Modify release_17_02.rst description. Zhiyong Yang (4): eal/common: introduce rte_memset on IA platform app/test: add functional autotest for rte_memset app/test: add performance autotest for rte_memset lib/librte_vhost: improve vhost per

[dpdk-dev] [PATCH v2 3/4] app/test: add performance autotest for rte_memset

2016-12-27 Thread Zhiyong Yang
20 271 - 362 C 1024 24 - 29 329 - 377 Signed-off-by: Zhiyong Yang --- Change in V2: Add perf comparation data between rte_memset and memset on haswell. app/test/Makefile | 1 + app/test/test_memset_perf.c | 348 2 f

[dpdk-dev] [PATCH v2 2/4] app/test: add functional autotest for rte_memset

2016-12-27 Thread Zhiyong Yang
run time. Signed-off-by: Zhiyong Yang --- app/test/Makefile | 2 + app/test/test_memset.c | 158 + 2 files changed, 160 insertions(+) create mode 100644 app/test/test_memset.c diff --git a/app/test/Makefile b/app/test/Makefile index 5be023a

[dpdk-dev] [PATCH v2 1/4] eal/common: introduce rte_memset on IA platform

2016-12-27 Thread Zhiyong Yang
mory alignment are fully taken into consideration. Signed-off-by: Zhiyong Yang --- Changes in V2: Rename rte_memset.h -> rte_memset_64.h and create a file rte_memset.h for each arch. .../common/include/arch/arm/rte_memset.h | 36 ++ .../common/include/arch/ppc_64/rte_memset.h

[dpdk-dev] [PATCH v2 4/4] lib/librte_vhost: improve vhost perf using rte_memset

2016-12-27 Thread Zhiyong Yang
involves both load (from stack, the virtio_hdr var) and store (to virtio driver memory), while rte_memset just involves store. Signed-off-by: Zhiyong Yang --- Changes in V2: Modify release_17_02.rst description. doc/guides/rel_notes/release_17_02.rst | 7 +++ lib/librte_vhost/virtio_net.c

[dpdk-dev] [PATCH] vhost: fix vhost-user init failed

2017-07-10 Thread Zhiyong Yang
Exception handling is executed in the normal path and it will cause vhost-user init failure. Fixes: d6983a70e259("vhost: check return of pthread calls") Reported-by: Lei Yao Signed-off-by: Zhiyong Yang --- lib/librte_vhost/socket.c | 2 +- 1 file changed, 1 insertion(+), 1 deletio

[dpdk-dev] [PATCH] doc: announce API and ABI change for ethdev

2017-07-12 Thread Zhiyong Yang
extend from 1 bytes to 2 bytes. All ethdev APIs and use cases related to port_id will be changed at the same time. Signed-off-by: Zhiyong Yang --- doc/guides/rel_notes/deprecation.rst | 4 1 file changed, 4 insertions(+) diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides

[dpdk-dev] [PATCH] lib/librte_ether: remove the repeat code

2017-07-28 Thread Zhiyong Yang
The code eth_dev->intr_handle = &dev->intr_handle; has duplicate code in the function rte_eth_copy_pci_info(), remove it here. Signed-off-by: Zhiyong Yang --- lib/librte_ether/rte_ethdev_pci.h | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/librte_ether/rte_ethdev_pc

[dpdk-dev] [PATCH 2/2] virtio: use macro to replace magic number

2017-08-02 Thread Zhiyong Yang
To use macro instead of magic number in order to enhance code readability. Signed-off-by: Zhiyong Yang --- drivers/net/virtio/virtio_pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/virtio/virtio_pci.c b/drivers/net/virtio/virtio_pci.c index e6da6802b

[dpdk-dev] [PATCH 0/2] virtio: cleanup virtio code

2017-08-02 Thread Zhiyong Yang
This patchset has some trivial changes and make it easier to read code. Zhiyong Yang (2): virtio: fix the wrong indent virtio: use macro to replace magic number drivers/net/virtio/virtio_pci.c | 2 +- drivers/net/virtio/virtqueue.h | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions

[dpdk-dev] [PATCH 1/2] virtio: fix the wrong indent

2017-08-02 Thread Zhiyong Yang
Signed-off-by: Zhiyong Yang --- drivers/net/virtio/virtqueue.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/virtio/virtqueue.h b/drivers/net/virtio/virtqueue.h index 2e1208615..9c4f96d2b 100644 --- a/drivers/net/virtio/virtqueue.h +++ b/drivers/net/virtio

[dpdk-dev] [PATCH v2] doc: announce API and ABI change for ethdev

2017-08-03 Thread Zhiyong Yang
from 1 bytes to 2 bytes. All ethdev APIs and use cases related to port_id will be changed at the same time. cc: jianfeng@intel.com cc: y...@fridaylinux.org cc: jerin.ja...@caviumnetworks.com cc: tho...@monjalon.net Signed-off-by: Zhiyong Yang Acked-by: Jianfeng Tan Acked-by: Yuanhan Liu

[dpdk-dev] [PATCH v3] doc: announce API and ABI change for ethdev

2017-08-07 Thread Zhiyong Yang
bytes. All ethdev APIs and usages related to port_id will be changed at the same time. cc: jianfeng@intel.com cc: y...@fridaylinux.org cc: jerin.ja...@caviumnetworks.com cc: tho...@monjalon.net cc: remy.hor...@intel.com Signed-off-by: Zhiyong Yang Acked-by: Jianfeng Tan Acked-by: Yuanhan Liu

[dpdk-dev] [PATCH v3 0/5] consistent PMD batching behaviour

2017-03-29 Thread Zhiyong Yang
3. fix one checkpatch issue in 2/5. Changes in V2: 1. rename ixgbe, i40e and fm10k vec function XXX_xmit_pkts_vec to new name XXX_xmit_fixed_burst_vec, new wrapper functions use original name XXX_xmit_pkts_vec according to Bruce's suggestion. 2. simplify the code to avoid the if or if/else.

[dpdk-dev] [PATCH v3 1/5] net/fm10k: remove limit of fm10k_xmit_pkts_vec burst size

2017-03-29 Thread Zhiyong Yang
To add a wrapper function to remove the limit of tx burst size. The patch makes fm10k vec function an best effort to transmit pkts in the consistent behavior like fm10k_xmit_pkts does that. Cc: Jing Chen Signed-off-by: Zhiyong Yang Acked-by: Konstantin Ananyev --- drivers/net/fm10k/fm10k.h

[dpdk-dev] [PATCH v3 2/5] net/i40e: remove limit of i40e_xmit_pkts_vec burst size

2017-03-29 Thread Zhiyong Yang
To add a wrapper function to remove the limit of tx burst size. The patch makes i40e vec function an best effort to transmit the pkts in the consistent behavior like i40e_xmit_pkts_simple and i40e_xmit_pkts do that. Cc: Helin Zhang Cc: Jingjing Wu Signed-off-by: Zhiyong Yang Acked-by

[dpdk-dev] [PATCH v3 3/5] net/ixgbe: remove limit of ixgbe_xmit_pkts_vec burst size

2017-03-29 Thread Zhiyong Yang
in Ananyev Signed-off-by: Zhiyong Yang Acked-by: Konstantin Ananyev --- drivers/net/ixgbe/ixgbe_rxtx.c | 29 + drivers/net/ixgbe/ixgbe_rxtx.h | 4 ++-- drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c | 4 ++-- drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c | 4 ++-

[dpdk-dev] [PATCH v3 4/5] net/vhost: remove limit of vhost TX burst size

2017-03-29 Thread Zhiyong Yang
vhost removes limit of TX burst size(32 pkts) and supports to make an best effort to transmit pkts. Cc: yuanhan@linux.intel.com Cc: maxime.coque...@redhat.com Signed-off-by: Zhiyong Yang Acked-by: Konstantin Ananyev --- drivers/net/vhost/rte_eth_vhost.c | 17 +++-- 1 file

[dpdk-dev] [PATCH v3 5/5] net/vhost: remove limit of vhost RX burst size

2017-03-29 Thread Zhiyong Yang
vhost removes limit of RX burst size(32 pkts) and supports to make an best effort to receive pkts. Cc: yuanhan@linux.intel.com Cc: maxime.coque...@redhat.com Signed-off-by: Zhiyong Yang Acked-by: Konstantin Ananyev --- doc/guides/rel_notes/release_17_05.rst | 4 drivers/net/vhost

[dpdk-dev] [PATCH 2/2] net/virtio: support to turn on/off the traffic flow

2017-03-31 Thread Zhiyong Yang
virtio_dev_start/stop support the switch of traffic flow. Signed-off-by: Zhiyong Yang --- drivers/net/virtio/virtio_ethdev.c | 37 +++- drivers/net/virtio/virtio_rxtx.c | 12 + drivers/net/virtio/virtio_rxtx_simple.c | 4 +++ drivers

[dpdk-dev] [PATCH 1/2] net/virtio: add data elements to turn on/off traffic flow

2017-03-31 Thread Zhiyong Yang
Add "rte_atomic32_t started" to turn on/off the RX/TX flow from per port perspective. Add rte_atomic32_t allow_queuing to control traffic flow from per Queue perspective. Signed-off-by: Zhiyong Yang --- drivers/net/virtio/virtio_pci.h | 1 + drivers/net/virtio/virtio_rxtx.h | 6

[dpdk-dev] [PATCH 0/2] net/virtio: support to turn on/off the traffic flow

2017-03-31 Thread Zhiyong Yang
Current dpdk code virtio_dev_stop only disables interrupt and marks link down, When it is invoked, tx/rx traffic flows still work. This is a strange behavior. The patchset supports the switch of flow by calling virtio_dev_start/stop. The implementation refers to vhost pmd. Zhiyong Yang (2

[dpdk-dev] [PATCH] l2fwd_fork: fix missing updating the prev_tsc

2017-02-22 Thread Zhiyong Yang
To miss updating the variable prev_tsc in function l2fwd_main_loop() The patch fixes it. Fixes:e2366e74e029("examples: use buffered Tx") CC: sergio.gonzalez.mon...@intel.com Signed-off-by: Zhiyong Yang --- examples/multi_process/l2fwd_fork/main.c | 2 ++ 1 file changed, 2 insertion

[dpdk-dev] [PATCH] net/virtio: fix remove the redundant computing

2017-02-22 Thread Zhiyong Yang
This is not a bug. The minor change aims to remove the redundant computing and make it easier to understand the code. Fixes:01ad44fd374f("net/virtio: split Rx/Tx queue") Cc: yuanhan@linux.intel.com Cc: maxime.coque...@redhat.com Signed-off-by: Zhiyong Yang --- drivers/

[dpdk-dev] [PATCH v2] net/virtio: remove the redundant computing

2017-02-22 Thread Zhiyong Yang
The minor change aims to remove the redundant computing and make it easier to understand the code. Cc: yuanhan@linux.intel.com Cc: maxime.coque...@redhat.com Signed-off-by: Zhiyong Yang --- v2 changes: remove the prefix "fix" and fixline according to yuanhan's advice. dri

[dpdk-dev] [PATCH 0/5] consistent PMD batching behaviour

2017-02-24 Thread Zhiyong Yang
is unified rte_eth_tx/rx_burst interface already, there is no reason for inconsistent behaviors. This patchset fixes it via adding wrapper function at the PMD level. Zhiyong Yang (5): net/fm10k: remove limit of fm10k_xmit_pkts_vec burst size net/i40e: remove limit of i40e_xmit_pkts_vec burst size

[dpdk-dev] [PATCH 4/5] net/vhost: remove limit of vhost TX burst size

2017-02-24 Thread Zhiyong Yang
vhost removes limit of TX burst size(32 pkts) and supports to make an best effort to transmit pkts. Cc: yuanhan@linux.intel.com Cc: maxime.coque...@redhat.com Signed-off-by: Zhiyong Yang --- drivers/net/vhost/rte_eth_vhost.c | 24 ++-- 1 file changed, 22 insertions

[dpdk-dev] [PATCH 5/5] net/vhost: remove limit of vhost RX burst size

2017-02-24 Thread Zhiyong Yang
vhost removes limit of RX burst size(32 pkts) and supports to make an best effort to receive pkts. Cc: yuanhan@linux.intel.com Cc: maxime.coque...@redhat.com Signed-off-by: Zhiyong Yang --- drivers/net/vhost/rte_eth_vhost.c | 23 +-- 1 file changed, 21 insertions(+), 2

[dpdk-dev] [PATCH 1/5] net/fm10k: remove limit of fm10k_xmit_pkts_vec burst size

2017-02-24 Thread Zhiyong Yang
To add a wrapper function fm10k_xmit_pkts_vec_simple to remove the limit of tx burst size. The patch makes fm10k vec function an best effort to transmit the pkts in the consistent behavior like fm10k_xmit_pkts does that. Cc: Jing Chen Signed-off-by: Zhiyong Yang --- drivers/net/fm10k

[dpdk-dev] [PATCH 3/5] net/ixgbe: remove limit of ixgbe_xmit_pkts_vec burst size

2017-02-24 Thread Zhiyong Yang
elin Zhang Cc: Konstantin Ananyev Signed-off-by: Zhiyong Yang --- drivers/net/ixgbe/ixgbe_rxtx.c | 27 ++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c index 9502432..8b80903 100644 --- a/d

[dpdk-dev] [PATCH 2/5] net/i40e: remove limit of i40e_xmit_pkts_vec burst size

2017-02-24 Thread Zhiyong Yang
: Zhiyong Yang --- drivers/net/i40e/i40e_rxtx.c | 27 ++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/drivers/net/i40e/i40e_rxtx.c b/drivers/net/i40e/i40e_rxtx.c index 48429cc..819d1d4 100644 --- a/drivers/net/i40e/i40e_rxtx.c +++ b/drivers/net/i40e/i40e_rxtx.c

[dpdk-dev] [PATCH v2 0/5] consistent PMD batching behaviour

2017-03-03 Thread Zhiyong Yang
e original name XXX_xmit_pkts_vec according to Bruce's suggestion. 2. simplify the code to avoid the if or if/else. Zhiyong Yang (5): net/fm10k: remove limit of fm10k_xmit_pkts_vec burst size net/i40e: remove limit of i40e_xmit_pkts_vec burst size net/ixgbe: remove limit of ixgbe_xmit

[dpdk-dev] [PATCH v2 1/5] net/fm10k: remove limit of fm10k_xmit_pkts_vec burst size

2017-03-03 Thread Zhiyong Yang
To add a wrapper function to remove the limit of tx burst size. The patch makes fm10k vec function an best effort to transmit pkts in the consistent behavior like fm10k_xmit_pkts does that. Cc: Jing Chen Signed-off-by: Zhiyong Yang --- drivers/net/fm10k/fm10k.h | 4 ++-- drivers/net

[dpdk-dev] [PATCH v2 2/5] net/i40e: remove limit of i40e_xmit_pkts_vec burst size

2017-03-03 Thread Zhiyong Yang
To add a wrapper function to remove the limit of tx burst size. The patch makes i40e vec function an best effort to transmit the pkts in the consistent behavior like i40e_xmit_pkts_simple and i40e_xmit_pkts do that. Cc: Helin Zhang Cc: Jingjing Wu Signed-off-by: Zhiyong Yang --- drivers/net

[dpdk-dev] [PATCH v2 3/5] net/ixgbe: remove limit of ixgbe_xmit_pkts_vec burst size

2017-03-03 Thread Zhiyong Yang
in Ananyev Signed-off-by: Zhiyong Yang --- drivers/net/ixgbe/ixgbe_rxtx.c | 29 + drivers/net/ixgbe/ixgbe_rxtx.h | 4 ++-- drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c | 4 ++-- drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c | 4 ++-- 4 files changed, 35 insert

[dpdk-dev] [PATCH v2 5/5] net/vhost: remove limit of vhost RX burst size

2017-03-03 Thread Zhiyong Yang
vhost removes limit of RX burst size(32 pkts) and supports to make an best effort to receive pkts. Cc: yuanhan@linux.intel.com Cc: maxime.coque...@redhat.com Signed-off-by: Zhiyong Yang --- drivers/net/vhost/rte_eth_vhost.c | 17 +++-- 1 file changed, 15 insertions(+), 2

[dpdk-dev] [PATCH v2 4/5] net/vhost: remove limit of vhost TX burst size

2017-03-03 Thread Zhiyong Yang
vhost removes limit of TX burst size(32 pkts) and supports to make an best effort to transmit pkts. Cc: yuanhan@linux.intel.com Cc: maxime.coque...@redhat.com Signed-off-by: Zhiyong Yang --- drivers/net/vhost/rte_eth_vhost.c | 17 +++-- 1 file changed, 15 insertions(+), 2

[dpdk-dev] [PATCH] virtio: xstats name issue

2016-09-01 Thread Zhiyong Yang
The patch fixes some xstats name issues and make the xstats name conform to code implementation(the function virtio_update_packet_stats). Signed-off-by: Zhiyong Yang --- drivers/net/virtio/virtio_ethdev.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/net

[dpdk-dev] [PATCH v2] net/virtio: fix xstats name issue

2016-09-06 Thread Zhiyong Yang
The patch fixes some xstats name issues and make the xstats name conform to the definition of etherStatsPkts1024to1518Octets in rfc2819 page 23. Fixes: 76d4c652e07d ("virtio: add extended stats") --- Changes in v2: modify commit summary. add the fixline. Signed-off-by: Zh

[dpdk-dev] [PATCH v3] net/virtio: fix xstats name issue

2016-09-07 Thread Zhiyong Yang
r fix it by correcting the "if" check in the code, or fix it by just renaming the stats to conform to the code. The latter solution is taken because that's what the RFC2819 suggests. Fixes: 76d4c652e07d ("virtio: add extended stats") Cc: Signed-off-by: Zhiyong Yang ---

[dpdk-dev] [PATCH v2] net/vhost: add pmd xstats

2016-09-09 Thread Zhiyong Yang
virtio_update_packet_stats) is used as reference when implementing the feature. Signed-off-by: Zhiyong Yang --- Changes in v2: 1. remove the compiling switch. 2. fix two code bugs. drivers/net/vhost/rte_eth_vhost.c | 282 +- 1 file changed, 281 insertions(+), 1 deletion

[dpdk-dev] [PATCH v3 0/2] net/vhost: add pmd xstats support

2016-09-20 Thread Zhiyong Yang
the code readability. 3. remove some unnecessary type casts and fix one format issue. Changes in v2: 1. remove the compiling switch. 2. fix two code bugs. --- Zhiyong Yang (2): net/vhost: add a new stats struct net/vhost: add pmd xstats drivers/net/vhost/rte_eth_vhost.c | 306

[dpdk-dev] [PATCH v3 1/2] net/vhost: add a new stats struct

2016-09-20 Thread Zhiyong Yang
; uint64_t missed_pkts; uint64_t rx_bytes; uint64_t tx_bytes; }; Signed-off-by: Zhiyong Yang --- drivers/net/vhost/rte_eth_vhost.c | 44 +-- 1 file changed, 24 insertions(+), 20 deletions(-) diff --git a/drivers/net/vhost/rte_eth_vhost.c b

[dpdk-dev] [PATCH v3 2/2] net/vhost: add pmd xstats

2016-09-20 Thread Zhiyong Yang
virtio_update_packet_stats) is used as reference when implementing the feature. Signed-off-by: Zhiyong Yang --- Changes in v3: 1. rework the vhost_update_packet_xstats and separate it into two parts. One function deals with the generic packets update, another one deals with increasing the

[dpdk-dev] [PATCH v4 0/2] net/vhost: add pmd xstats support

2016-09-21 Thread Zhiyong Yang
code bugs. --- Zhiyong Yang (2): net/vhost: add a new defined stats struct net/vhost: add pmd xstats drivers/net/vhost/rte_eth_vhost.c | 316 +++--- 1 file changed, 296 insertions(+), 20 deletions(-) -- 2.5.5

[dpdk-dev] [PATCH v4 0/2] net/vhost: add pmd xstats support

2016-09-21 Thread Zhiyong Yang
: Zhiyong Yang --- Changes in v4: A queue can be only used as TX or RX, So, we can use pkts instead of rx_pkts and tx_pkts, the same to rx_bytes and tx_bytes. before modification: struct vhost_stats { uint64_t rx_pkts; uint64_t tx_pkts; uint64_t missed_pkts; uint64_t

[dpdk-dev] [PATCH v4 2/2] net/vhost: add pmd xstats

2016-09-21 Thread Zhiyong Yang
pmd xstats(the function virtio_update_packet_stats) is used as reference when implementing the feature. Signed-off-by: Zhiyong Yang --- Changes in v4: 1. add a member VHOST_XSTATS_MAX in enum vhost_xstats_pkts, So, we can define uint64_t xstats[VHOST_XSTATS_MAX]; instead of xstats[16]. 2. restore

[dpdk-dev] [PATCH v4 1/2] net/vhost: add a new defined stats struct

2016-09-21 Thread Zhiyong Yang
The patch moves all stats counters to a new defined struct vhost_stats as follows, in order to manage all stats counters in a unified way and simplify the subsequent function implementation(vhost_dev_xstats_reset). Signed-off-by: Zhiyong Yang --- Changes in v4: A queue can be only used as TX or

[dpdk-dev] [PATCH v5 0/2] net/vhost: add pmd xstats support

2016-09-22 Thread Zhiyong Yang
magic numbers and enhance the code readability. 3. remove some unnecessary type casts and fix one format issue. Changes in v2: 1. remove the compiling switch. 2. fix two code bugs. --- Zhiyong Yang (2): net/vhost: add a new defined stats struct net/vhost: add pmd xstats drivers/net/vhost

[dpdk-dev] [PATCH v5 1/2] net/vhost: add a new defined stats struct

2016-09-22 Thread Zhiyong Yang
The patch moves all stats counters to a new defined struct vhost_stats as follows, in order to manage all stats counters in a unified way and simplify the subsequent function implementation(vhost_dev_xstats_reset). Signed-off-by: Zhiyong Yang --- Changes in v4: A queue can be only used as TX or

[dpdk-dev] [PATCH v5 2/2] net/vhost: add pmd xstats

2016-09-22 Thread Zhiyong Yang
pmd xstats(the function virtio_update_packet_stats) is used as reference when implementing the feature. Signed-off-by: Zhiyong Yang --- Changes in V5: for vhost_count_multicast_broadcast, passing struct rte_mbuf *buf instead of struct rte_mbuf **buf and remove the 3th parameter uint16_t count

[dpdk-dev] [PATCH v6 0/2] net/vhost: add pmd xstats support

2016-09-28 Thread Zhiyong Yang
. Changes in v2: 1. remove the compiling switch. 2. fix two code bugs. --- Zhiyong Yang (2): net/vhost: add a new defined stats struct net/vhost: add pmd xstats doc/guides/rel_notes/release_16_11.rst | 4 + drivers/net/vhost/rte_eth_vhost.c | 314 ++--- 2 files

[dpdk-dev] [PATCH v6 1/2] net/vhost: add a new defined stats struct

2016-09-28 Thread Zhiyong Yang
The patch moves all stats counters to a new defined struct vhost_stats as follows, in order to manage all stats counters in a unified way and simplify the subsequent function implementation(vhost_dev_xstats_reset). Signed-off-by: Zhiyong Yang --- Changes in v4: A queue can be only used as TX or

[dpdk-dev] [PATCH v6 2/2] net/vhost: add pmd xstats

2016-09-28 Thread Zhiyong Yang
pmd xstats(the function virtio_update_packet_stats) is used as reference when implementing the feature. Signed-off-by: Zhiyong Yang --- Changes in V6: 1. Change xstats from per queue to per port. Keep vhost consistent with physical NIC i40e, ixgbe, etc. 2. added the release note. Changes in V5

[dpdk-dev] [PATCH v7 0/2] net/vhost: add pmd xstats support

2016-09-28 Thread Zhiyong Yang
efine enum vhost_stat_pkts to replace the magic numbers and enhance the code readability. 3. remove some unnecessary type casts and fix one format issue. Changes in v2: 1. remove the compiling switch. 2. fix two code bugs. --- Zhiyong Yang (2): net/vhost: add a new defined stats struct net/vhos

[dpdk-dev] [PATCH v7 1/2] net/vhost: add a new defined stats struct

2016-09-28 Thread Zhiyong Yang
The patch moves all stats counters to a new defined struct vhost_stats as follows, in order to manage all stats counters in a unified way and simplify the subsequent function implementation(vhost_dev_xstats_reset). Signed-off-by: Zhiyong Yang --- Changes in v4: A queue can be only used as TX or

[dpdk-dev] [PATCH v7 2/2] net/vhost: add pmd xstats

2016-09-28 Thread Zhiyong Yang
pmd xstats(the function virtio_update_packet_stats) is used as reference when implementing the feature. Signed-off-by: Zhiyong Yang --- Changes in V7: Removed the "_portX" prepend to the xstat names. Keep vhost xstats name consistent with physical NIC i40e, ixgbe, etc. Changes in V6:

[dpdk-dev] [PATCH v8 0/2] net/vhost: add pmd xstats support

2016-09-29 Thread Zhiyong Yang
code bugs. --- Zhiyong Yang (2): net/vhost: add a new defined stats struct net/vhost: add pmd xstats doc/guides/rel_notes/release_16_11.rst | 3 + drivers/net/vhost/rte_eth_vhost.c | 312 ++--- 2 files changed, 295 insertions(+), 20 deletions(-) -- 2.5.5

[dpdk-dev] [PATCH v8 1/2] net/vhost: add a new defined stats struct

2016-09-29 Thread Zhiyong Yang
The patch moves all stats counters to a new defined struct vhost_stats as follows, in order to manage all stats counters in a unified way and simplify the subsequent function implementation(vhost_dev_xstats_reset). Signed-off-by: Zhiyong Yang --- Changes in v4: A queue can be only used as TX or

[dpdk-dev] [PATCH v8 2/2] net/vhost: add pmd xstats

2016-09-29 Thread Zhiyong Yang
Ciara Loftus Signed-off-by: Zhiyong Yang --- Changes in V7: Removed the "_portX" prepended to the xstat names. Keep vhost xstats name consistent with physical NIC i40e, ixgbe, etc. Changes in V6: 1. Change xstats from per queue to per port. Keep vhost consistent with physical NIC i

[dpdk-dev] [PATCH v2] net/virtio: support to turn on/off traffic flow

2017-04-18 Thread Zhiyong Yang
Current virtio_dev_stop only disables interrupt and marks link down, When it is invoked, tx/rx traffic flows still work. This is a strange behavior. The patch supports the switch of flow. Signed-off-by: Zhiyong Yang --- Changes in V2: Use the flag "started" introduced by yuanhan'

[dpdk-dev] [PATCH] vhost: fix MQ fails to startup

2017-04-26 Thread Zhiyong Yang
t the dpdk side in order to avoid the feature support of DPDK + qemu at the same time. if doing like that, MQ can works well. Once Qemu bugs have been fixed and upstreamed, we can enable it. Fixes: 73c8f9f69c6c("vhost: introduce reply ack feature") Reported-by: Loftus, Ciara Signed-

[dpdk-dev] [PATCH v2] vhost: workaround MQ fails to startup

2017-04-27 Thread Zhiyong Yang
by default at the dpdk side in order to avoid the feature support of DPDK + qemu at the same time. if doing like that, MQ can works well. Cc: sta...@dpdk.org Reported-by: Ciara Loftus Signed-off-by: Zhiyong Yang Tested-by: Ciara Loftus --- changes in V2 1. modify "workaround" ins

[dpdk-dev] [PATCH] doc: update virtio doc

2017-05-09 Thread Zhiyong Yang
Add more explanations about vring size changes and virtio_header size. Signed-off-by: Zhiyong Yang --- doc/guides/nics/virtio.rst | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/doc/guides/nics/virtio.rst b/doc/guides/nics/virtio.rst index 91bedea..36ddd08

[dpdk-dev] [PATCH v2] doc: update virtio vring size and virtio_header size

2017-05-10 Thread Zhiyong Yang
Add more explanations about vring size changes and different virtio_header size. Signed-off-by: Zhiyong Yang --- Changes in v2: 1. Remove two lines and add "" to constants and struct names. 2. Modify the title. doc/guides/nics/virtio.rst | 13 - 1 file changed, 8 insert

[dpdk-dev] [PATCH] net/virtio-user: fix multiple queues fail in server mode

2018-05-09 Thread zhiyong . yang
This patch fixes multiple queues failure when virtio-user works in server mode. Fixes: bd8f50a45d0f ("net/virtio-user: support server mode") Cc: sta...@dpdk.org Signed-off-by: Zhiyong Yang --- drivers/net/virtio/virtio_user/vhost_user.c | 3 +++ drivers/net/virtio/v

[dpdk-dev] [PATCH v2] net/virtio-user: fix multiple queues fail in server mode

2018-05-10 Thread zhiyong . yang
This patch fixes multiple queues failure when virtio-user works in server mode. This patch adds feature negotiation in the processing of virtio-user reccnnection. Fixes: bd8f50a45d0f ("net/virtio-user: support server mode") Signed-off-by: Zhiyong Yang --- Changes in V2: 1. fix a co

[dpdk-dev] [PATCH v3] net/virtio-user: fix multiple queues fail in server mode

2018-05-10 Thread zhiyong . yang
ned-off-by: Zhiyong Yang --- Changes in V3: 1. virtio_user_handle_mq is redefined as extern instead of static. 2. add to call virtio_user_handle_mq in virtio_user_server_reconnect. Changes in V2: 1. fix a comment typo. 2. add feature negotiation in the processing of reconnection. drivers/net/virtio/v

[dpdk-dev] [PATCH v4] net/virtio-user: fix multiple queues fail in server mode

2018-05-10 Thread zhiyong . yang
Cc: tiwei@intel.com Cc: ferruh.yi...@intel.com Cc: maxime.coque...@redhat.com Signed-off-by: Zhiyong Yang Reviewed-by: Tiwei Bie --- Changes in V4: 1. fix two typoes. Changes in V3: 1. virtio_user_handle_mq is redefined as extern instead of static. 2. add to call virtio_user_ha

[dpdk-dev] [PATCH] app/testpmd: fix print unused parameter name

2018-05-13 Thread zhiyong . yang
The second parameter "name" in the function rte_eth_dev_detach has been already redefined as "char *name __rte_unused", "port_id" is printed instead of "name" in testpmd. Fixes: b65ecf199324 ("devargs: rename legacy API") Cc: sta...@dpdk.

[dpdk-dev] [PATCH] app/testpmd: add to call detach for vdev when quitting

2018-05-16 Thread zhiyong . yang
calls the rte_eth_dev_detach() for vdev when quitting testpmd. Cc: maxime.coque...@redhat.com Cc: ferruh.yi...@intel.com Cc: tiwei@intel.com Cc: lei.a@intel.com Signed-off-by: Zhiyong Yang --- app/test-pmd/testpmd.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/app/tes

[dpdk-dev] [PATCH v2] app/testpmd: fix pmd_test_exit function for vdevs

2018-05-17 Thread zhiyong . yang
quot;net/virtio-user: support server mode") Signed-off-by: Zhiyong Yang --- changes in V2: 1. change the pache title and add a fixes line. app/test-pmd/testpmd.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index 134401603..1d308f

[dpdk-dev] [PATCH] ethdev: fix eth_dev_last_created_port storage type

2018-05-17 Thread zhiyong . yang
eth_dev_last_created_port is used to store port id type and should be extended to 16bits corresponding to ethdev port id range. Cc: sta...@dpdk.org Cc: ferruh.yi...@intel.com Fixes: f8244c6399d9 ("ethdev: increase port id range") Signed-off-by: Zhiyong Yang --- lib/librte_ethdev/rt

[dpdk-dev] [PATCH] net/virtio-user: fix add pointer checking for rxvq

2018-05-23 Thread zhiyong . yang
@intel.com Signed-off-by: Zhiyong Yang --- drivers/net/virtio/virtio_ethdev.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c index 5833dad73..df50a571a 100644 --- a/drivers/net/virtio/virtio_ethdev.c +++ b/drivers/

[dpdk-dev] [PATCH v2] net/virtio: fix add pointer checking for rxvq

2018-05-24 Thread zhiyong . yang
@intel.com Signed-off-by: Zhiyong Yang Reviewed-by: Maxime Coquelin --- Change in V2: change title prefix from net/virtio-user to net/virtio. drivers/net/virtio/virtio_ethdev.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethd

[dpdk-dev] [PATCH v4 0/1] server mode virtio-user

2018-04-03 Thread zhiyong . yang
e_vhost exposed as new APIs. 3. release note is added in the patch 5/5. 4. squash data structure change patch into 4/5 according to Maxime's suggestion. Zhiyong Yang (1): net/virtio-user: add support for server mode doc/guides/rel_notes/release_18_05.rst | 6 ++ drivers/net/v

[dpdk-dev] [PATCH v4 1/1] net/virtio-user: add support for server mode

2018-04-03 Thread zhiyong . yang
-user supports many times' vhost-user reconnections with the same parameter configurations. Release note is updated in the patch. Signed-off-by: Zhiyong Yang --- doc/guides/rel_notes/release_18_05.rst | 6 ++ drivers/net/virtio/virtio_user/vhost_user.c

[dpdk-dev] [PATCH v5] net/virtio-user: add support for server mode

2018-04-04 Thread zhiyong . yang
tarts, it can reconnect to virtio-user and continue communications. With current implementation, LSC is enabled at virtio-user side to support to accept the coming connection. Release note is updated in this patch. Signed-off-by: Zhiyong Yang --- Changes in V5: 1. Support server mode virtio

[dpdk-dev] [PATCH v6] net/virtio-user: add support for server mode

2018-04-05 Thread zhiyong . yang
tarts, it can reconnect to virtio-user and continue communications. With current implementation, LSC is enabled at virtio-user side to support to accept the coming connection. Release note is updated in this patch. Signed-off-by: Zhiyong Yang --- Changes in V6: 1. fix report wrong link stau

[dpdk-dev] [PATCH v7] net/virtio-user: add support for server mode

2018-04-06 Thread zhiyong . yang
tarts, it can reconnect to virtio-user and continue communications. With current implementation, LSC is enabled at virtio-user side to support to accept the coming connection. Server mode virtio-user only supports to work with vhost-user. Release note is updated in this patch. Signed-off-by: Zh

[dpdk-dev] [PATCH v3] bus/pci: fix wrong intr_handle.type with uio_pci_generic

2018-01-09 Thread Zhiyong Yang
ndary() in order to keep consistencies and avoid future questions. Fixes: 756ce64b1ecd ("eal: introduce PCI ioport API") Cc: sta...@dpdk.org Signed-off-by: Zhiyong Yang --- Changes in V3: 1. Such assignments are removed in order to keep consistencies and avoid future questions. Changes

[dpdk-dev] [PATCH v2] examples/vhost: fix remove dev_info.max_rx_queues checking to solve startup failure

2018-01-09 Thread Zhiyong Yang
val don't affect any functionality (have already validated ixgbe and i40e). The removal can avoid similar issue when introduing new physical NIC. Fixes: 8bd6c395a568("examples/vhost: increase maximum queue number") Cc: sta...@dpdk.org Signed-off-by: Zhiyong Yang --- Changes in

[dpdk-dev] [PATCH] lib/librte_vhost: mov enum definition from PMD to lib

2018-01-12 Thread Zhiyong Yang
The enum definition is placed in librte_vhost in order to avoid many duplication definitions in PMD and example code everywhere. Signed-off-by: Zhiyong Yang --- drivers/net/vhost/rte_eth_vhost.c | 2 -- examples/tep_termination/main.h | 2 -- examples/vhost/main.h | 2 -- lib

[dpdk-dev] [PATCH v2] doc: fix link bonding PMD typo in prog guide

2018-01-12 Thread Zhiyong Yang
fix one typo and a grammatical mistake. Fixes: b0152b1b40fe ("doc: update bonding") Cc: sta...@dpdk.org Signed-off-by: Zhiyong Yang Acked-by: Marko Kovacevic --- Changes in V2: 1. Fix the title and fixline format issue in commit log. doc/guides/prog_guide/link_bonding_poll_mode_d

[dpdk-dev] [PATCH 0/5] remove unnecessary void * pointer cast

2018-01-15 Thread Zhiyong Yang
void * pointer can be assigned to any data type pointer. Unnecessary cast can be removed in order to keep code clearer. Zhiyong Yang (5): lib/librte_lpm: remove unnecessary void * pointer cast lib/librte_efd: remove unnecessary void * pointer cast using rte_zmalloc_socket lib

[dpdk-dev] [PATCH 1/5] lib/librte_lpm: remove unnecessary void * pointer cast

2018-01-15 Thread Zhiyong Yang
void * pointer can be assigned to any data type pointer. Unnecessary cast can be removed in order to keep code clearer. Signed-off-by: Zhiyong Yang --- lib/librte_lpm/rte_lpm.c | 24 lib/librte_lpm/rte_lpm6.c | 4 ++-- 2 files changed, 14 insertions(+), 14 deletions

[dpdk-dev] [PATCH 2/5] lib/librte_efd: remove unnecessary void * pointer cast using rte_zmalloc_socket

2018-01-15 Thread Zhiyong Yang
void * pointer can be assigned to any data type pointer. Unnecessary cast can be removed in order to keep code clearer. Signed-off-by: Zhiyong Yang --- lib/librte_efd/rte_efd.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/librte_efd/rte_efd.c b/lib/librte_efd

[dpdk-dev] [PATCH 3/5] lib/librte_hash: remove unnecessary void * pointer cast using rte_zmalloc_socket

2018-01-15 Thread Zhiyong Yang
void * pointer can be assigned to any data type pointer. Unnecessary cast can be removed in order to keep code clearer. Signed-off-by: Zhiyong Yang --- lib/librte_hash/rte_fbk_hash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/librte_hash/rte_fbk_hash.c b/lib

[dpdk-dev] [PATCH 5/5] lib/librte_pipeline: remove unnecessary void * pointer cast using rte_zmalloc_socket

2018-01-15 Thread Zhiyong Yang
void * pointer can be assigned to any data type pointer. Unnecessary cast can be removed in order to keep code clearer. Signed-off-by: Zhiyong Yang --- lib/librte_pipeline/rte_pipeline.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/librte_pipeline/rte_pipeline.c b

[dpdk-dev] [PATCH 4/5] lib/librte_member: remove unnecessary void * pointer cast

2018-01-15 Thread Zhiyong Yang
void * pointer can be assigned to any data type pointer. Unnecessary cast can be removed in order to keep code clearer. Signed-off-by: Zhiyong Yang --- lib/librte_member/rte_member.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/librte_member/rte_member.c b/lib

[dpdk-dev] [PATCH v3] doc: fix link bonding PMD typo in prog guide

2018-01-18 Thread Zhiyong Yang
fix one typo and a grammatical mistake. Fixes: b0152b1b40fe ("doc: update bonding") Cc: sta...@dpdk.org Signed-off-by: Zhiyong Yang Acked-by: Marko Kovacevic --- Changes in V3: use "similar capabilities" instead of "the similar capabilities" Changes in V2: Fi

[dpdk-dev] [PATCH] lib/librte_mbuf: remove void * pointer cast

2018-01-19 Thread Zhiyong Yang
It is unnecessary to cast from void * to struct rte_mbuf *, the change can make code more simple. Signed-off-by: Zhiyong Yang --- lib/librte_mbuf/rte_mbuf.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/lib/librte_mbuf/rte_mbuf.h b/lib/librte_mbuf/rte_mbuf.h index

[dpdk-dev] [PATCH] mbuf: remove void pointer cast

2018-01-19 Thread Zhiyong Yang
It is unnecessary to cast from void * to struct rte_mbuf *, the change can make code clearer. Signed-off-by: Zhiyong Yang --- lib/librte_mbuf/rte_mbuf.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/lib/librte_mbuf/rte_mbuf.h b/lib/librte_mbuf/rte_mbuf.h index a594e4772

[dpdk-dev] [PATCH] lib: fix wrong cast

2018-01-23 Thread Zhiyong Yang
m Cc: akhil.go...@nxp.com Signed-off-by: Zhiyong Yang --- lib/librte_cryptodev/rte_cryptodev.c | 2 +- lib/librte_security/rte_security.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/librte_cryptodev/rte_cryptodev.c b/lib/librte_cryptodev/rte_cryptodev.c index 77

  1   2   3   >