[dpdk-dev] [PATCH v4 12/12] examples/l3fwd: add option to parse ptype

2016-02-26 Thread Tan, Jianfeng
Hi Konstantin, On 2/26/2016 9:14 PM, Ananyev, Konstantin wrote: > Hi Jianfeng, > >> +static int ... >> +if (hdr_len == sizeof(struct ipv4_hdr) && >> +(hdr->next_proto_id == 6 || >> + hdr->next_proto_id == 17)) > Use IPPORTO_UDP, I

[dpdk-dev] [PATCH v3 1/3] fm10k: enable FTAG based forwarding

2016-02-26 Thread Thomas Monjalon
2016-02-26 09:24, Wang, Xiao W: > Hi, > > > -Original Message- > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Friday, February 26, 2016 5:06 PM > > To: Wang, Xiao W > > Cc: Richardson, Bruce ; Chen, Jing D > > ; dev at dpdk.org > > Subject: Re: [dpdk-dev] [PATCH

[dpdk-dev] [PATCH] i40e: Fix a typo.

2016-02-26 Thread Rami Rosen
This patch fixes a typo in a comment in the definition of the i40e_pf struct. Signed-off-by: Rami Rosen --- drivers/net/i40e/i40e_ethdev.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/i40e/i40e_ethdev.h b/drivers/net/i40e/i40e_ethdev.h index 6edd7dd..8ebac9d 10

[dpdk-dev] DPDK with vhostuser - Issue in Testing with DPDK-Pktgen

2016-02-26 Thread Abhijeet Karve
Dear Przemek, Sean & Sugesh, It would be great if get your valuable inputs for the below listed issue. We have setup DPDK with vhostuser in AllinOne setup on following software platform. _ Openstack: Kilo Distribution: Ubuntu 14.04 OVS Version: 2.4.0 DPDK 2.0.0 _ In our

[dpdk-dev] Virtio xstats problem

2016-02-26 Thread Igor Ryzhov
Sent a patch: http://dpdk.org/dev/patchwork/patch/10887/ . > 26 . 2016 ?., ? 17:35, Igor Ryzhov ???(?): > > Hello, Harry. > > Understood about size of packets. It's a bit confusing, because in all other > drivers undersized packet is an erro

[dpdk-dev] [PATCH] virtio: don't count broadcast packets in multicast packets counter

2016-02-26 Thread Igor Ryzhov
Signed-off-by: Igor Ryzhov --- drivers/net/virtio/virtio_rxtx.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/drivers/net/virtio/virtio_rxtx.c b/drivers/net/virtio/virtio_rxtx.c index 41a1366..fe18e1d 100644 --- a/drivers/net/virtio/virtio_rxtx.c +++ b/drivers/net/v

[dpdk-dev] [PATCH 0/2] bonding fixes

2016-02-26 Thread Bruce Richardson
On Mon, Feb 22, 2016 at 01:20:18PM +, Doherty, Declan wrote: > > -Original Message- > > From: Iremonger, Bernard > > Sent: Wednesday, February 10, 2016 10:14 AM > > To: dev at dpdk.org > > Cc: Doherty, Declan ; Iremonger, Bernard > > > > Subject: [PATCH 0/2] bonding fixes > > > > Thes

[dpdk-dev] client_server example application crash with virtio

2016-02-26 Thread Kyle Larose
Hrm. This may not be correct. It looks like there is some interaction between the primary and secondary here. vtpci_ops must be in shared memory. When I initialize my secondary, it overwrites the primary's with its address. I just checked, and without my patch, the secondary's vtpci_ops points t

[dpdk-dev] [PATCH v3] driver/net/mpipe: fix the crash/hung issue when testpmd quits

2016-02-26 Thread Bruce Richardson
On Wed, Feb 10, 2016 at 12:15:21AM -0500, Liming Sun wrote: > Fixes: the hung/crash issue when quitting testpmd under high > traffic rate. The following issue were found and fixed. > 1. edesc->size is not initialized properly in mpipe_do_xmit() and could >cause buffer leak or corruption when HW

[dpdk-dev] [PATCH v9 0/2] Add VHOST PMD

2016-02-26 Thread Tetsuya Mukawa
On 2016/02/26 13:29, Tetsuya Mukawa wrote: > On 2016/02/25 16:51, Qiu, Michael wrote: >> On 2/24/2016 1:10 PM, Tetsuya Mukawa wrote: >>> On 2016/02/24 11:45, Qiu, Michael wrote: Hi, Tetsuya When I applied your v6 patch, I could reach 9.5Mpps with 64B packet. But when apply

[dpdk-dev] Virtio xstats problem

2016-02-26 Thread Igor Ryzhov
Hello, Harry. Understood about size of packets. It's a bit confusing, because in all other drivers undersized packet is an error. Maybe we should add another one size bin for virtio - 60 to 63 bytes? I already checked about multicast/broadcast counters - broadcast packets are counted twice:

[dpdk-dev] [PATCH v3 2/2] cryptodev: change burst API to be crypto op oriented

2016-02-26 Thread Declan Doherty
This patch modifies the crypto burst enqueue/dequeue APIs to operate on bursts rte_crypto_op's rather than the current implementation which operates on rte_mbuf bursts, this simplifies the burst processing in the crypto PMDs and the use of crypto operations in general. The changes also continues t

[dpdk-dev] [PATCH v3 1/2] cryptodev: API tidy and changes to support future extensions

2016-02-26 Thread Declan Doherty
From: Fiona Trahe This patch splits symmetric specific definitions and functions away from the common crypto APIs to facilitate the future extension and expansion of the cryptodev framework, in order to allow asymmetric crypto operations to be introduced at a later date, as well as to clean the

[dpdk-dev] [PATCH v3 0/2] cryptodev API changes

2016-02-26 Thread Declan Doherty
This patch set separates the symmetric crypto operations from generic operations and then modifies the cryptodev burst API to accept bursts of rte_crypto_op rather than rte_mbufs. V3: - Addresses V2 comments - Rebased for head Declan Doherty (1): cryptodev: change burst API to be crypto op or

[dpdk-dev] [PATCH v2] e1000: enable promiscuous and allmulticast support for VF

2016-02-26 Thread Bruce Richardson
On Tue, Feb 16, 2016 at 12:43:38AM +, Lu, Wenzhuo wrote: > Hi Yury, > > Acked-by: Wenzhuo Lu > Applied to dpdk-next-net/rel_16_04 /Bruce

[dpdk-dev] [PATCH v4 3/3] null: remove duplicate fields in internal data struct

2016-02-26 Thread Ferruh Yigit
1- remove duplicate nb_rx/tx_queues fields from internals 2- remove duplicate numa_node field from internals Signed-off-by: Ferruh Yigit Tested-by: Tetsuya Mukawa Acked-by: Tetsuya Mukawa Acked-by: Nicolas Pernas Maradei --- drivers/net/null/rte_eth_null.c | 36 ---

[dpdk-dev] [PATCH v4 2/3] ring: variable rename and code cleanup

2016-02-26 Thread Ferruh Yigit
Rename nb_rx/tx_queues fields in internals struct to max_rx/tx_queues Updated fields required to keep max queue numbers configured. For current queue number requirements data->nb_rx/tx_queues fields used. Some checkpatch corrections and code clenaup. Signed-off-by: Ferruh Yigit --- drivers/net/

[dpdk-dev] [PATCH v4 1/3] pcap: remove duplicate fields in internal data struct

2016-02-26 Thread Ferruh Yigit
1- Remove duplicate nb_rx/tx_queues fields from internals 2- Move duplicate code into a common function Signed-off-by: Ferruh Yigit Acked-by: Nicolas Pernas Maradei --- drivers/net/pcap/rte_eth_pcap.c | 130 +++- 1 file changed, 61 insertions(+), 69 deletions

[dpdk-dev] [PATCH v4 0/3] clean-up on virtual PMDs

2016-02-26 Thread Ferruh Yigit
This is a clean-up patch, no defect fixed, no functional difference expected. Patch removes duplicated fields between data->dev_private and data (struct rte_eth_dev_data) for pcap and null PMDs. For ring, renames some variables and does code cleanup. Also for pcap: move a common code into a func

[dpdk-dev] Virtio xstats problem

2016-02-26 Thread Igor Ryzhov
Hello. I found some problem with virtio xstats counters. Example: rx_good_packets: 3 rx_good_bytes: 180 rx_errors: 0 rx_q0_good_packets: 3 rx_q0_good_bytes: 180 rx_q0_errors: 0 rx_q0_multicast_packets: 3 rx_q0_broadcast_packets: 1 rx_q0_undersize_packets: 3 It means that undersize packets are

[dpdk-dev] client_server example application crash with virtio

2016-02-26 Thread Kyle Larose
I just ran into an issue trying to run the client server example application using virito. Whenever the client (running as a secondary process) tried to send packets, it would crash. I traced the issue to an invalid vtpci_ops structure: (gdb) p *vq->hw->vtpci_ops $6 = { read_dev_cfg = 0x756d2

[dpdk-dev] [PATCH v2] fm10k: handle err flags in vector RX func

2016-02-26 Thread Bruce Richardson
On Thu, Feb 18, 2016 at 08:36:55AM +, Liang, Cunming wrote: > Hi, > > > -Original Message- > > From: Chen, Jing D > > Sent: Saturday, February 06, 2016 2:49 PM > > To: Liang, Cunming; He, Shaopeng > > Cc: dev at dpdk.org; Chen, Jing D > > Subject: [PATCH v2] fm10k: handle err flags in

[dpdk-dev] DPDK with vhostuser - Issue in Testing with DPDK-Pktgen

2016-02-26 Thread Wiles, Keith
>Dear Przemek, Sean & Sugesh, > >It would be great if get your valuable inputs for the below listed issue. >We have setup DPDK with vhostuser in AllinOne setup on following software >platform. >_ >Openstack: Kilo >Distribution: Ubuntu 14.04 >OVS Version: 2.4.0 >DPDK 2.0.0 >___

[dpdk-dev] [PATCH v4 09/12] pmd/mlx5: add dev_ptype_info_get implementation

2016-02-26 Thread Tan, Jianfeng
Hi, On 2/26/2016 4:26 PM, Adrien Mazarguil wrote: > Hi Jianfeng, > > On Fri, Feb 26, 2016 at 08:09:28AM +0800, Jianfeng Tan wrote: >> Signed-off-by: Jianfeng Tan >> --- >> drivers/net/mlx4/mlx4.c| 2 +- >> drivers/net/mlx5/mlx5.c| 1 + >> drivers/net/mlx5/mlx5.h| 1

[dpdk-dev] [PATCH v3 2/3] ring: rename fields in internal data struct

2016-02-26 Thread Bruce Richardson
On Fri, Feb 26, 2016 at 03:26:54PM +, Ferruh Yigit wrote: > Rename nb_rx/tx_queues fields in internals struct to max_rx/tx_queues > > This patch just does variable renaming to help understanding code better, > nothing changed in functionality of the code. > > Updated fields required to keep m

[dpdk-dev] [PATCH v4 5/5] ixgbe: support VxLAN & NVGRE TX checksum off-load

2016-02-26 Thread Wenzhuo Lu
The patch add VxLAN & NVGRE TX checksum off-load. When the flag of outer IP header checksum offload is set, we'll set the context descriptor to enable this checksum off-load. Also update release note for VxLAN & NVGRE checksum off-load support. Signed-off-by: Wenzhuo Lu --- doc/guides/rel_notes

[dpdk-dev] [PATCH v4 4/5] ixgbe: support VxLAN & NVGRE RX checksum off-load

2016-02-26 Thread Wenzhuo Lu
X550 will do VxLAN & NVGRE RX checksum off-load automatically. This patch exposes the result of the checksum off-load. Signed-off-by: Wenzhuo Lu --- drivers/net/ixgbe/ixgbe_ethdev.c | 4 drivers/net/ixgbe/ixgbe_rxtx.c | 11 ++- lib/librte_ether/rte_ethdev.h| 1 + lib/librte_

[dpdk-dev] [PATCH v4 3/5] ixgbe: support UDP tunnel port config

2016-02-26 Thread Wenzhuo Lu
Add UDP tunnel port add/del support on ixgbe. Now only support VxLAN port configuration. Although the VxLAN port has a default value 4789, it can be changed. We support VxLAN port configuration to meet the change. Note, the default value of VxLAN port in ixgbe NICs is 0. So please set it when using

[dpdk-dev] [PATCH v4 2/5] i40e: rename the tunnel port config functions

2016-02-26 Thread Wenzhuo Lu
As the names of tunnel port config functions are not accurate, change them from tunnel_add/del to tunnel_port_add/del. And support both the old and new rte ops. Signed-off-by: Wenzhuo Lu --- drivers/net/i40e/i40e_ethdev.c | 22 -- 1 file changed, 12 insertions(+), 10 deletion

[dpdk-dev] [PATCH v4 1/5] lib/librte_ether: change function name of tunnel port config

2016-02-26 Thread Wenzhuo Lu
The names of function for tunnel port configuration are not accurate. They're tunnel_add/del, better change them to tunnel_port_add/del. As it may be an ABI change if change the names directly, the new functions are added but not remove the old ones. The old ones will be removed in the next release

[dpdk-dev] [PATCH v4 0/5] Support VxLAN & NVGRE checksum off-load on X550

2016-02-26 Thread Wenzhuo Lu
This patch set add the VxLAN & NVGRE checksum off-load support. Both RX and TX checksum off-load can be used for VxLAN & NVGRE. And the VxLAN port can be set, it's implemented in this patch set either. V2: * Update release note. V3: * Update RX/TX offload capability. * Reuse PKT_RX_EIP_CKSUM_BAD

[dpdk-dev] [PATCH v3 1/3] fm10k: enable FTAG based forwarding

2016-02-26 Thread Bruce Richardson
On Fri, Feb 26, 2016 at 04:00:49PM +0100, David Marchand wrote: > On Fri, Feb 26, 2016 at 3:48 PM, Bruce Richardson > wrote: > > On Fri, Feb 26, 2016 at 09:24:06AM +, Wang, Xiao W wrote: > >> Hi, > >> > > Thanks for the discussion, Thomas, do you have any suggestions? > >> > > >> > I don't und

[dpdk-dev] [PATCH v2 1/3] cmdline: increase command line buffer

2016-02-26 Thread NĂ©lio Laranjeiro
On Mon, Jan 18, 2016 at 03:38:43PM +0100, Olivier MATZ wrote: > Hi, > > On 01/15/2016 10:00 AM, Panu Matilainen wrote: > diff --git a/lib/librte_cmdline/cmdline_rdline.h > b/lib/librte_cmdline/cmdline_rdline.h > index b9aad9b..72e2dad 100644 > --- a/lib/librte_cmdline/cmdline_r

[dpdk-dev] [PATCH v2 1/3] cmdline: increase command line buffer

2016-02-26 Thread Mcnamara, John
> -Original Message- > From: N?lio Laranjeiro [mailto:nelio.laranjeiro at 6wind.com] > Sent: Friday, February 26, 2016 3:17 PM > To: Mcnamara, John > Cc: Olivier MATZ ; Panu Matilainen > ; dev at dpdk.org; Zhang, Helin intel.com>; > thomas.monjalon at 6wind.com; Lu, Wenzhuo ; > olgas at

[dpdk-dev] [PATCH v3 1/3] fm10k: enable FTAG based forwarding

2016-02-26 Thread David Marchand
On Fri, Feb 26, 2016 at 3:48 PM, Bruce Richardson wrote: > On Fri, Feb 26, 2016 at 09:24:06AM +, Wang, Xiao W wrote: >> Hi, >> > > Thanks for the discussion, Thomas, do you have any suggestions? >> > >> > I don't understand why you say this feature is specific to fm10k. Can we >> > imagine ano

[dpdk-dev] [PATCH v5 11/11] pmd/vmxnet3: add dev_ptype_info_get implementation

2016-02-26 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/vmxnet3/vmxnet3_ethdev.c | 16 1 file changed, 16 insertions(+) diff --git a/drivers/net/vmxnet3/vmxnet3_ethdev.c b/drivers/net/vmxnet3/vmxnet3_ethdev.c index c363bf6..ac120a1 100644 --- a/drivers/net/vmxnet3/vmxnet3_ethdev.c +++ b/dr

[dpdk-dev] [PATCH v5 10/11] pmd/nfp: add dev_ptype_info_get implementation

2016-02-26 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/nfp/nfp_net.c | 19 +++ 1 file changed, 19 insertions(+) diff --git a/drivers/net/nfp/nfp_net.c b/drivers/net/nfp/nfp_net.c index fd4dd39..5894a9d 100644 --- a/drivers/net/nfp/nfp_net.c +++ b/drivers/net/nfp/nfp_net.c @@ -1073,6 +1073,2

[dpdk-dev] [PATCH v5 09/11] pmd/mlx5: add dev_ptype_info_get implementation

2016-02-26 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/mlx5/mlx5.c| 1 + drivers/net/mlx5/mlx5.h| 1 + drivers/net/mlx5/mlx5_ethdev.c | 20 drivers/net/mlx5/mlx5_rxtx.c | 2 ++ 4 files changed, 24 insertions(+) diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx

[dpdk-dev] [PATCH v5 08/11] pmd/mlx4: add dev_ptype_info_get implementation

2016-02-26 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/mlx4/mlx4.c | 21 + 1 file changed, 21 insertions(+) diff --git a/drivers/net/mlx4/mlx4.c b/drivers/net/mlx4/mlx4.c index ee00151..58f4e1a 100644 --- a/drivers/net/mlx4/mlx4.c +++ b/drivers/net/mlx4/mlx4.c @@ -2835,6 +2835,8 @@ rxq_

[dpdk-dev] [PATCH v5 07/11] pmd/ixgbe: add dev_ptype_info_get implementation

2016-02-26 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/ixgbe/ixgbe_ethdev.c | 38 ++ drivers/net/ixgbe/ixgbe_ethdev.h | 2 ++ drivers/net/ixgbe/ixgbe_rxtx.c | 4 +++- 3 files changed, 43 insertions(+), 1 deletion(-) diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/dri

[dpdk-dev] [PATCH v5 06/11] pmd/i40e: add dev_ptype_info_get implementation

2016-02-26 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/i40e/i40e_ethdev.c| 1 + drivers/net/i40e/i40e_ethdev_vf.c | 1 + drivers/net/i40e/i40e_rxtx.c | 46 ++- drivers/net/i40e/i40e_rxtx.h | 1 + 4 files changed, 48 insertions(+), 1 deletion(-) diff --gi

[dpdk-dev] [PATCH v5 05/11] pmd/fm10k: add dev_ptype_info_get implementation

2016-02-26 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/fm10k/fm10k_ethdev.c | 50 ++ drivers/net/fm10k/fm10k_rxtx.c | 3 +++ drivers/net/fm10k/fm10k_rxtx_vec.c | 3 +++ 3 files changed, 56 insertions(+) diff --git a/drivers/net/fm10k/fm10k_ethdev.c b/drivers/net/

[dpdk-dev] [PATCH v5 04/11] pmd/enic: add dev_ptype_info_get implementation

2016-02-26 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/enic/enic_ethdev.c | 17 + 1 file changed, 17 insertions(+) diff --git a/drivers/net/enic/enic_ethdev.c b/drivers/net/enic/enic_ethdev.c index 2a88043..fbeab6f 100644 --- a/drivers/net/enic/enic_ethdev.c +++ b/drivers/net/enic/enic_ethd

[dpdk-dev] [PATCH v5 03/11] pmd/e1000: add dev_ptype_info_get implementation

2016-02-26 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/e1000/igb_ethdev.c | 30 ++ 1 file changed, 30 insertions(+) diff --git a/drivers/net/e1000/igb_ethdev.c b/drivers/net/e1000/igb_ethdev.c index 4ed5e95..b3a3ee6 100644 --- a/drivers/net/e1000/igb_ethdev.c +++ b/drivers/net/e

[dpdk-dev] [PATCH v5 02/11] pmd/cxgbe: add dev_ptype_info_get implementation

2016-02-26 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/cxgbe/cxgbe_ethdev.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/drivers/net/cxgbe/cxgbe_ethdev.c b/drivers/net/cxgbe/cxgbe_ethdev.c index 97ef152..33bd815 100644 --- a/drivers/net/cxgbe/cxgbe_ethdev.c +++ b/drivers/net/cxgbe/cxg

[dpdk-dev] [PATCH v5 01/11] ethdev: add API to query packet type filling info

2016-02-26 Thread Jianfeng Tan
Add a new API rte_eth_dev_get_ptype_info to query whether/what packet type can be filled by given pmd rx burst function. Signed-off-by: Jianfeng Tan --- lib/librte_ether/rte_ethdev.c | 26 ++ lib/librte_ether/rte_ethdev.h | 26 ++ 2 files changed,

[dpdk-dev] [PATCH v5 00/11] Add API to get packet type info

2016-02-26 Thread Jianfeng Tan
To achieve this, a new function pointer, dev_ptype_info_get, is added into struct eth_dev_ops. For those devices who do not implement it, it means it will not provide any ptype info. v5: - Exclude l3fwd change from this series, as a separated one. - Fix malposition of mlx4 code in mlx5 commit

[dpdk-dev] [PATCH v3 3/3] null: remove duplicate fields in internal data struct

2016-02-26 Thread Ferruh Yigit
1- remove duplicate nb_rx/tx_queues fields from internals 2- remove duplicate numa_node field from internals Signed-off-by: Ferruh Yigit Tested-by: Tetsuya Mukawa Acked-by: Tetsuya Mukawa Acked-by: Nicolas Pernas Maradei --- drivers/net/null/rte_eth_null.c | 36 ---

[dpdk-dev] [PATCH v3 2/3] ring: rename fields in internal data struct

2016-02-26 Thread Ferruh Yigit
Rename nb_rx/tx_queues fields in internals struct to max_rx/tx_queues This patch just does variable renaming to help understanding code better, nothing changed in functionality of the code. Updated fields required to keep max queue numbers configured. For current queue number requirements data->n

[dpdk-dev] [PATCH v3 1/3] pcap: remove duplicate fields in internal data struct

2016-02-26 Thread Ferruh Yigit
1- Remove duplicate nb_rx/tx_queues fields from internals 2- Move duplicate code into a common function Signed-off-by: Ferruh Yigit Acked-by: Nicolas Pernas Maradei --- drivers/net/pcap/rte_eth_pcap.c | 130 +++- 1 file changed, 61 insertions(+), 69 deletions

[dpdk-dev] [PATCH v3 0/3] clean-up on virtual PMDs

2016-02-26 Thread Ferruh Yigit
This is a clean-up patch, no defect fixed, no functional difference expected. Patch removes duplicated fields between data->dev_private and data (struct rte_eth_dev_data) for pcap and null PMDs. For ring, renames some fields in private structure. Also for pcap: move a common code into a function

[dpdk-dev] [PATCH] virtio: don't count broadcast packets in multicast packets counter

2016-02-26 Thread Van Haaren, Harry
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Igor Ryzhov > Sent: Friday, February 26, 2016 3:01 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] virtio: don't count broadcast packets in > multicast packets > counter > > Signed-off-by: Igor Ryzhov Acked-by: Harry van Haaren

[dpdk-dev] [PATCH v2] examples/skeleton-cat: PQoS CAT and CDP, example of libpqos usage

2016-02-26 Thread Wojciech Andralojc
Because of the feedback that we have received off the mailing list, that extending EAL commands is not an option due to the Intel Architecture nature of CAT, we have changed the design of PQoS patch. The current V2 patch implements a sample code, based on the DPDK skeleton example app, that links

[dpdk-dev] [PATCH v3 2/2] i40evf: support to report pf reset event

2016-02-26 Thread Jingjing Wu
When Linux PF and DPDK VF are used for i40e PMD, In case of PF reset, interrupt will go via adminq event, VF need be informed the event, a callback mechanism is introduced by VF. This will allow VF to invoke callback when reset happens. Users can register a callback for this interrupt event like:

[dpdk-dev] [PATCH v3 1/2] i40evf: allocate virtchnl cmd buffer for each vf

2016-02-26 Thread Jingjing Wu
Currently, i40evf PMD uses a global static buffer to send virtchnl command to host driver. It is shared by multi VFs. This patch changed to allocate virtchnl cmd buffer for each VF. Signed-off-by: Jingjing Wu --- drivers/net/i40e/i40e_ethdev.h| 2 + drivers/net/i40e/i40e_ethdev_vf.c | 178

[dpdk-dev] [PATCH v3 0/2] i40evf: pf reset event report

2016-02-26 Thread Jingjing Wu
v3 changes: - commit log doc rewording. - rebase on latest dpdk-next-net/rel_16_04 branch. - remove few useless line - adjust interval and increase times for waiting pf msg v2 changes: - remove the change on vf reset status checking - add pf event report support in release note When Linux P

[dpdk-dev] [PATCH 3/3 v2] ixgbe: fix setting of VF MAC address

2016-02-26 Thread Bernard Iremonger
Fix setting of VF MAC address to allow a zero address. Reprogram the RAR[0] with a zero MAC address, to ensure ensure that the VF traffic goes to the PF after stop, close and detach of the VF. Fixes: af75078fece3 ("first public release") Fixes: 00e30184daa0 ("ixgbe: add PF support") ixgbe: don't

[dpdk-dev] [PATCH 2/3 v2] ixgbe: add more information to the error message

2016-02-26 Thread Bernard Iremonger
Add the nb_rx_q and nb_tx_q values to the error message to give details about the error. Fixes: 27b609cbd1c6 ("ethdev: move the multi-queue mode check to specific drivers") Signed-off-by: Bernard Iremonger --- drivers/net/ixgbe/ixgbe_ethdev.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletio

[dpdk-dev] [PATCH 1/3 v2] ixgbe: cleanup eth_ixgbevf_dev_uninit

2016-02-26 Thread Bernard Iremonger
Releasing the rx and tx queues is already done in ixgbe_dev_close() so it does not need to be done in eth_ixgbevf_dev_uninit(). Fixes: 2866c5f1b87e ("ixgbe: support port hotplug") Signed-off-by: Bernard Iremonger --- drivers/net/ixgbe/ixgbe_ethdev.c | 15 +-- 1 file changed, 1 insert

[dpdk-dev] [PATCH 0/3 v2] ixgbe fixes

2016-02-26 Thread Bernard Iremonger
This patch set implements the following: Removes code which was duplicated in eth_ixgbevf_dev_init(). Adds more information to the error message in ixgbe_check_mq_mode(). Allows the MAC address of the VF to be set to zero. Changes in v2: Do not overwrite the VF perm_add with zero. Bernard Iremong

[dpdk-dev] [PATCH v3 1/3] fm10k: enable FTAG based forwarding

2016-02-26 Thread Bruce Richardson
On Fri, Feb 26, 2016 at 09:24:06AM +, Wang, Xiao W wrote: > Hi, > > > -Original Message- > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Friday, February 26, 2016 5:06 PM > > To: Wang, Xiao W > > Cc: Richardson, Bruce ; Chen, Jing D > > ; dev at dpdk.org > >

[dpdk-dev] [PATCH v2] doc: Malicious Driver Detection not supported by ixgbe

2016-02-26 Thread Bruce Richardson
On Fri, Feb 26, 2016 at 12:48:37PM +0800, Wenzhuo Lu wrote: > Announce that Malicious Driver Detection is not supported. > > V2: > *Rework the words. > > Signed-off-by: Wenzhuo Lu Hi Wenzhuo, just for future reference, please put the V2,v3 etc. updates below the cut line "---" so that they ca

[dpdk-dev] [PATCH v2] ixgbe: Fix disable interrupt twice

2016-02-26 Thread Bruce Richardson
On Tue, Feb 23, 2016 at 02:10:57AM +, Zhang, Helin wrote: > > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Michael Qiu > > Sent: Friday, January 29, 2016 1:58 PM > > To: dev at dpdk.org > > Subject: [dpdk-dev] [PATCH v2] ixgbe: Fix disable interru

[dpdk-dev] virtio PMD is not working with master version

2016-02-26 Thread Santosh Shukla
On Fri, Feb 26, 2016 at 2:14 PM, Xie, Huawei wrote: > On 2/26/2016 4:29 PM, David Marchand wrote: >> On Fri, Feb 26, 2016 at 3:23 AM, Yuanhan Liu >> wrote: >>> Mauricio, thanks for the testing and report. >>> >>> On Thu, Feb 25, 2016 at 02:30:18PM +0100, David Marchand wrote: >From the logs,

[dpdk-dev] Virtio xstats problem

2016-02-26 Thread Van Haaren, Harry
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Igor Ryzhov > I found some problem with virtio xstats counters. > > Example: > > rx_good_packets: 3 > rx_good_bytes: 180 > rx_errors: 0 > > rx_q0_good_packets: 3 > rx_q0_good_bytes: 180 > rx_q0_errors: 0 > > rx_q0_multicast_packets: 3 >

[dpdk-dev] [PATCH v4 12/12] examples/l3fwd: add option to parse ptype

2016-02-26 Thread Ananyev, Konstantin
> > Actually it is a good point: > > for EM case should l3fwd process only TCP/UDP packets? > > If yes, then it needs to check not only L3, but also L4 type too > > Which means that for EM and LPM check_packet_type_ok() should also be > > different. > > Or we can leave it as it is - in that case E

[dpdk-dev] [PATCH v1] virtio: Use cpuflag for vector api

2016-02-26 Thread Santosh Shukla
Check cpuflag macro before using vectored api. -virtio_recv_pkts_vec() uses _sse3__ simd instruction for now so added cpuflag. - Also wrap other vectored freind api ie.. 1) virtqueue_enqueue_recv_refill_simple 2) virtio_rxq_vec_setup todo: 1) Move virtio_recv_pkts_vec() implementation to driver

[dpdk-dev] [PATCH v3 4/4] examples/ethtool: add control interface support to the application

2016-02-26 Thread Ferruh Yigit
Control interface APIs added into the sample application. To have the support corresponding kernel module (KCP) needs to be inserted. If kernel module is not there, application will run as it is without kernel control path support. When KCP module inserted, running application creates a virtual L

[dpdk-dev] [PATCH v3 3/4] rte_ctrl_if: add control interface library

2016-02-26 Thread Ferruh Yigit
This library gets control messages form kernelspace and forwards them to librte_ether and returns response back to the kernelspace. Library does: 1) Trigger Linux virtual interface creation 2) Initialize the netlink socket communication 3) Provides process() API to the application that does proces

[dpdk-dev] [PATCH v3 2/4] kcp: add kernel control path kernel module

2016-02-26 Thread Ferruh Yigit
This kernel module is based on KNI module, but this one is stripped version of it and only for control messages, no data transfer functionality provided. This Linux kernel module helps userspace application create virtual interfaces and when a control command issued into that virtual interface, mo

[dpdk-dev] [PATCH v3 1/4] lib/librte_ethtool: move librte_ethtool form examples to lib folder

2016-02-26 Thread Ferruh Yigit
With KCP, examples/ethtool/lib/ethtool has two users, to prevent code dublication, moving library from examples folder into lib/ folder. Signed-off-by: Ferruh Yigit --- config/common_linuxapp | 5 + doc/api/doxy-api-index.md | 3 +- doc/api/doxy-api.conf

[dpdk-dev] [PATCH v3 0/4] Use common Linux tools to control DPDK ports

2016-02-26 Thread Ferruh Yigit
This work is to make DPDK ports more visible and to enable using common Linux tools to configure DPDK ports. Patch is based on KNI but contains only control functionality of it, also this patch does not include any Linux kernel network driver as part of it. Basically with the help of a kernel mod

[dpdk-dev] [PATCH] virtio: fix rx ring descriptor starvation

2016-02-26 Thread Kyle Larose
Virtio has an mbuf descriptor ring containing mbufs to be used for receiving traffic. When the host queues traffic to be sent to the guest, it consumes these descriptors. If none exist, it discards the packet. The virtio pmd allocates mbufs to the descriptor ring every time it succesfully receives

[dpdk-dev] [PATCH v3] doc: add Vector FM10K introductions

2016-02-26 Thread Chen Jing D(Mark)
From: "Chen Jing D(Mark)" Add introductions on how to enable Vector FM10K Rx/Tx functions, the preconditions and assumptions on Rx/Tx configuration parameters. The new content also lists the limitations of vector, so app/customer can do better to select best Rx/Tx functions. Signed-off-by: Chen

[dpdk-dev] [PATCH v9 0/2] Add VHOST PMD

2016-02-26 Thread Tetsuya Mukawa
On 2016/02/25 16:51, Qiu, Michael wrote: > On 2/24/2016 1:10 PM, Tetsuya Mukawa wrote: >> On 2016/02/24 11:45, Qiu, Michael wrote: >>> Hi, Tetsuya >>> >>> When I applied your v6 patch, I could reach 9.5Mpps with 64B packet. >>> >>> But when apply v9 only 8.4 Mpps, could you figure out why has >>>

[dpdk-dev] [PATCH v4 12/12] examples/l3fwd: add option to parse ptype

2016-02-26 Thread Ananyev, Konstantin
Hi Jianfeng, > > +static int > +check_packet_type_ok(int portid) > +{ > + int i, ret; > + int ptype_l3_ipv4 = 0, ptype_l3_ipv6 = 0; > + > + ret = rte_eth_dev_get_ptype_info(portid, RTE_PTYPE_L3_MASK, NULL, 0); > + if (ret <= 0) > + return 0; > + > + uint32_t ptypes

[dpdk-dev] [PATCH v2] doc: Malicious Driver Detection not supported by ixgbe

2016-02-26 Thread Wenzhuo Lu
Announce that Malicious Driver Detection is not supported. V2: *Rework the words. Signed-off-by: Wenzhuo Lu --- doc/guides/nics/ixgbe.rst | 20 doc/guides/rel_notes/release_16_04.rst | 23 +++ 2 files changed, 43 insertions(+) diff --git a

[dpdk-dev] ACL memory allocation failures

2016-02-26 Thread Ananyev, Konstantin
> > Thanks Konstantin. > > Previous allocation error was coming with 1024 huge pages of 2 MB size. > > After increasing the huge pages to 2048, I was able to add another ~140 rules > [IPv4 rule data--> with src, dst IP address & port, next > header ] more, ie., 950 rules were added. That's s

[dpdk-dev] ACL memory allocation failures

2016-02-26 Thread Rapelly, Varun
Thanks Konstantin. Previous allocation error was coming with 1024 huge pages of 2 MB size. After increasing the huge pages to 2048, I was able to add another ~140 rules [IPv4 rule data--> with src, dst IP address & port, next header ] more, ie., 950 rules were added. Logically it did not incr

[dpdk-dev] [PATCH v3] ixgbe: support link speed auto-neg on x550em_x

2016-02-26 Thread Wenzhuo Lu
Normally the auto-negotiation is supported by FW. SW need not care about that. But on x550em_x, FW doesn't support auto-neg. As the ports of x550em_x are 10G, if we connect the port will a peer which is 1G, the link will always be down. We need support auto-neg by SW to avoid this link down issue.

[dpdk-dev] ACL memory allocation failures

2016-02-26 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Rapelly, Varun > Sent: Friday, February 26, 2016 10:28 AM > To: dev at dpdk.org > Subject: Re: [dpdk-dev] ACL memory allocation failures > > Hi All, > > When I'm trying to configure some 5000+ ACL rules with

[dpdk-dev] ACL memory allocation failures

2016-02-26 Thread Rapelly, Varun
Hi All, When I'm trying to configure some 5000+ ACL rules with different source IP addresses, getting ACL memory allocation failure. I'm using DPDK 2.1. [root at ACLISSUE log_2015_10_26_08_19_42]# vim np.log match nodes/bytes used: 816/104448 total: 12940832 bytes ACL: Build phase for ACL "ipv4_

[dpdk-dev] ACL memory allocation failures

2016-02-26 Thread Rapelly, Varun
Hi All, When I'm trying to configure some 5000+ ACL rules with different source IP addresses, getting ACL memory allocation failure.\ [root at ACLISSUE log_2015_10_26_08_19_42]# vim np.log match nodes/bytes used: 816/104448 total: 12940832 bytes ACL: Build phase for ACL "ipv4_acl_table2": memory

[dpdk-dev] virtio PMD is not working with master version

2016-02-26 Thread Yuanhan Liu
Mauricio, thanks for the testing and report. On Thu, Feb 25, 2016 at 02:30:18PM +0100, David Marchand wrote: > On Thu, Feb 25, 2016 at 12:30 PM, Mauricio V?squez > wrote: > > ... > > EAL: PCI device :00:04.0 on NUMA socket -1 > > EAL: probe driver: 1af4:1000 rte_virtio_pmd > > EAL: PCI me

[dpdk-dev] Virtio xstats problem

2016-02-26 Thread Kyle Larose
On Fri, Feb 26, 2016 at 9:35 AM, Igor Ryzhov wrote: > Hello, Harry. > > Understood about size of packets. It's a bit confusing, because in all other > drivers undersized packet is an error. Maybe we should add another one size > bin for virtio - 60 to 63 bytes? > I'm not sure we need to add a n

[dpdk-dev] [PATCH v6 1/2] mbuf: provide rte_pktmbuf_alloc_bulk API

2016-02-26 Thread Olivier MATZ
Hi Huawei, On 02/26/2016 10:07 AM, Xie, Huawei wrote: > On 2/26/2016 4:56 PM, Olivier MATZ wrote: >> test_one_pktmbuf(struct rte_mbuf *m) >> { >> /* same as before without the allocation/free */ >> } >> >> test_pkt_mbuf(void) >> { >> m = rte_pktmbuf_alloc(pool); >> test_one_pktmbuf(

[dpdk-dev] [PATCH v3 1/3] fm10k: enable FTAG based forwarding

2016-02-26 Thread Thomas Monjalon
2016-02-26 04:31, Wang, Xiao W: > From: Richardson, Bruce > > On Thu, Feb 25, 2016 at 03:45:45PM +, Chen, Jing D wrote: > > > From: Richardson, Bruce > > > > On Thu, Feb 25, 2016 at 10:04:02AM +, Chen, Jing D wrote: > > > > > This feature is trying to use FTAG (a unique tech in fm10k) > > >

[dpdk-dev] [PATCH v6 1/2] mbuf: provide rte_pktmbuf_alloc_bulk API

2016-02-26 Thread Olivier MATZ
On 02/23/2016 06:35 AM, Xie, Huawei wrote: >>> Also, it would be nice to have a simple test function in >>> app/test/test_mbuf.c. For instance, you could update >>> test_one_pktmbuf() to take a mbuf pointer as a parameter and remove >>> the mbuf allocation from the function. Then it could be call

[dpdk-dev] [PATCH v4 4/4] virtio: return 1 to tell the upper layer we don't take over this device

2016-02-26 Thread Huawei Xie
v4 changes: Rebase as io port map is moved to eal. Only fall back to PORT IO when there isn't any kernel driver (including VFIO/UIO) managing the device. Before v4, we fall back to PORT IO even if VFIO/UIO fails. Reword the commit message. v3 changes: Change log message to tell user that the v

[dpdk-dev] [PATCH v4 3/4] eal: call pci_ioport_map when kernel driver isn't managing the device

2016-02-26 Thread Huawei Xie
Call rte_eal_pci_ioport_map only if driver type is RTE_KDRV_NONE, which means kernel driver(including UIO/VFIO) isn't managing the device. other minor changes: * use RTE_ARCH_X86 for pci ioport map * rework rte_eal_pci_ioport_map a bit Signed-off-by: Huawei Xie --- lib/librte_eal/linuxapp/eal

[dpdk-dev] [PATCH v4 2/4] eal: set kdrv to RTE_KDRV_NONE if kernel driver isn't managing the device

2016-02-26 Thread Huawei Xie
v4 changes: reword the commit message. When we mention kernel driver, emphasizes that it includes UIO/VFIO. Use RTE_KDRV_NONE to indicate that kernel driver(including UIO/VFIO) isn't manipulating the device. Signed-off-by: Huawei Xie Acked-by: Yuanhan Liu --- lib/librte_eal/linuxapp/eal/eal_p

[dpdk-dev] [PATCH v4 1/4] eal: make the comment more accurate

2016-02-26 Thread Huawei Xie
positive return of rte_eal_pci_probe_one_driver means the driver doesn't support the device. Signed-off-by: Huawei Xie Acked-by: Yuanhan Liu --- lib/librte_eal/common/eal_common_pci.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/librte_eal/common/eal_common_pc

[dpdk-dev] [PATCH v4 0/4] fix the issue that DPDK takes over virtio device blindly

2016-02-26 Thread Huawei Xie
v4 changes: Rebase as IO port map is moved to EAL. Reword some commit messages. Don't fall back to PORT IO if VFIO/UIO fails. v3 changes: Change log message to tell user that the virtio device is skipped due to it is managed by kernel driver, instead of asking user to unbind it from kernel dri

[dpdk-dev] [PATCH v6 1/2] mbuf: provide rte_pktmbuf_alloc_bulk API

2016-02-26 Thread Olivier MATZ
On 02/26/2016 08:39 AM, Xie, Huawei wrote: With 8 mbufs allocated, there is about 6% performance increase using inline. With 16 mbufs allocated, we could still observe obvious performance difference, though only 1%-2% > > On 2/24/2016 9:23 PM, Ananyev, Konstantin wrote: >>

[dpdk-dev] [PATCH v3 01/12] ethdev: add API to query packet type filling info

2016-02-26 Thread Tan, Jianfeng
Hi Konstantin, On 2/26/2016 1:16 AM, Ananyev, Konstantin wrote: > >> -Original Message- >> From: Tan, Jianfeng >> Sent: Thursday, February 25, 2016 4:36 PM >> To: Ananyev, Konstantin; dev at dpdk.org >> Cc: Zhang, Helin; nelio.laranjeiro at 6wind.com; adrien.mazarguil at >> 6wind.com; rah

[dpdk-dev] [PATCH v3 0/4] fix the issue that DPDK takes over virtio device blindly

2016-02-26 Thread David Marchand
On Fri, Feb 26, 2016 at 7:09 AM, Xie, Huawei wrote: > On 2/24/2016 8:45 PM, Thomas Monjalon wrote: >>> Huawei Xie (4): >>> eal: make the comment more accurate >>> eal: set kdrv to RTE_KDRV_NONE if kernel driver isn't manipulating the >>> device. >>> virtio: return 1 to tell the kernel we do

[dpdk-dev] virtio PMD is not working with master version

2016-02-26 Thread David Marchand
On Fri, Feb 26, 2016 at 3:23 AM, Yuanhan Liu wrote: > Mauricio, thanks for the testing and report. > > On Thu, Feb 25, 2016 at 02:30:18PM +0100, David Marchand wrote: >> >From the logs, I would say I broke uio_pci_generic since we are in >> "uio" case, but uio portio sysfs does not exist. >> virti

[dpdk-dev] [PATCH v4 09/12] pmd/mlx5: add dev_ptype_info_get implementation

2016-02-26 Thread Adrien Mazarguil
Hi Jianfeng, On Fri, Feb 26, 2016 at 08:09:28AM +0800, Jianfeng Tan wrote: > Signed-off-by: Jianfeng Tan > --- > drivers/net/mlx4/mlx4.c| 2 +- > drivers/net/mlx5/mlx5.c| 1 + > drivers/net/mlx5/mlx5.h| 1 + > drivers/net/mlx5/mlx5_ethdev.c | 20 >

[dpdk-dev] [PATCH v3 1/3] fm10k: enable FTAG based forwarding

2016-02-26 Thread Wang, Xiao W
Hi, > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Friday, February 26, 2016 5:06 PM > To: Wang, Xiao W > Cc: Richardson, Bruce ; Chen, Jing D > ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3 1/3] fm10k: enable FTAG based forwarding > >

[dpdk-dev] [PATCH v6 1/2] mbuf: provide rte_pktmbuf_alloc_bulk API

2016-02-26 Thread Xie, Huawei
On 2/26/2016 4:56 PM, Olivier MATZ wrote: > > On 02/23/2016 06:35 AM, Xie, Huawei wrote: Also, it would be nice to have a simple test function in app/test/test_mbuf.c. For instance, you could update test_one_pktmbuf() to take a mbuf pointer as a parameter and remove the mbuf all

  1   2   >