Re: [dpdk-dev] [PATCH] net/mlx5: handle expected errno properly

2018-08-23 Thread Jack MIN
On Thu, Aug 23, 2018 at 02:08:09PM -0700, Yongseok Koh wrote: > On Thu, Aug 23, 2018 at 02:38:51PM +0800, Xiaoyu Min wrote: > > rte_errno is a per thread variable and is widely used as an > > error indicator, which means a function could affect > > other functions' results by setting rte_errno care

Re: [dpdk-dev] [PATCH] net/virtio-user: fix memory hotplug support

2018-08-23 Thread Tiwei Bie
On Thu, Aug 23, 2018 at 03:01:30PM +0100, Burakov, Anatoly wrote: > On 23-Aug-18 12:19 PM, Sean Harte wrote: > > On Thu, 23 Aug 2018 at 10:05, Burakov, Anatoly > > wrote: > > > > > > On 23-Aug-18 3:57 AM, Tiwei Bie wrote: > > > > Deadlock can occur when allocating memory if a vhost-kernel > > > >

Re: [dpdk-dev] [PATCH 8/8] mem: support using memfd segments for in-memory mode

2018-08-23 Thread Jerin Jacob
-Original Message- > Date: Thu, 23 Aug 2018 17:59:55 +0100 > From: Anatoly Burakov > To: dev@dpdk.org > CC: tiwei@intel.com, ray.kinse...@intel.com, zhihong.w...@intel.com, > maxime.coque...@redhat.com, kuralamudhan.ramakrish...@intel.com > Subject: [dpdk-dev] [PATCH 8/8] mem: support

Re: [dpdk-dev] [PATCH 2/2] virtio: fix PCI config err handling

2018-08-23 Thread Tiwei Bie
On Thu, Aug 23, 2018 at 01:52:25PM +0100, Luca Boccassi wrote: > On Tue, 2018-08-21 at 10:40 +0800, Tiwei Bie wrote: > > On Mon, Aug 20, 2018 at 05:45:35PM +0100, Luca Boccassi wrote: > > > On Mon, 2018-08-20 at 16:18 +0800, Tiwei Bie wrote: > > > > On Thu, Aug 16, 2018 at 07:49:43PM +0100, Luca Bo

Re: [dpdk-dev] [PATCH v2] hash table: add an iterator over conflicting entries

2018-08-23 Thread Wang, Yipeng1
Thanks for the patch and sorry for the late review. I believe this will be a good addition to the library and please keep me cc'ed for V3 as well. We have the linked list cuckoo hash on the roadmap for 18.11 http://mails.dpdk.org/archives/dev/2018-August/109986.html I think with Honnappa suggeste

Re: [dpdk-dev] [PATCH 03/11] telemetry: add client feature and sockets

2018-08-23 Thread Stephen Hemminger
On Thu, 23 Aug 2018 13:08:05 +0100 Ciara Power wrote: > This patch introduces clients to the telemetry API. > > When a client makes a connection through the initial telemetry > socket, they can send a message through the socket to be > parsed. Register messages are expected through this socket,

Re: [dpdk-dev] [PATCH 01/11] telemetry: initial telemetry infrastructure

2018-08-23 Thread Stephen Hemminger
On Thu, 23 Aug 2018 13:08:03 +0100 Ciara Power wrote: > +/* Logging Macros */ > +extern int telemetry_log_level; > + > +#define TELEMETRY_LOG(level, fmt, args...) \ > + rte_log(RTE_LOG_ ##level, telemetry_log_level, "%s(): "fmt "\n", \ > + __func__, ##args) > + > +#define TELEMETR

Re: [dpdk-dev] [PATCH 01/11] telemetry: initial telemetry infrastructure

2018-08-23 Thread Stephen Hemminger
On Thu, 23 Aug 2018 13:08:03 +0100 Ciara Power wrote: > +/* SPDX-License-Identifier: BSD-3-Clause > + * Copyright(c) 2018 Intel Corporation > + */ > + > +#include > + > +#ifndef _RTE_TELEMETRY_H_ > +#define _RTE_TELEMETRY_H_ > + > +/** > + * Get the telemetry_impl structure device pointer initia

Re: [dpdk-dev] [PATCH 01/11] telemetry: initial telemetry infrastructure

2018-08-23 Thread Stephen Hemminger
On Thu, 23 Aug 2018 13:08:03 +0100 Ciara Power wrote: > + TELEMETRY_LOG_ERR("Error - Calling thread could not be" > + " put to sleep\n"); Don't break strings across lines. It makes it harder for developers who want to use tools to look for log mess

Re: [dpdk-dev] [PATCH 01/11] telemetry: initial telemetry infrastructure

2018-08-23 Thread Stephen Hemminger
On Thu, 23 Aug 2018 13:08:03 +0100 Ciara Power wrote: > +static int32_t > +rte_telemetry_run(void *userdata) > +{ > + struct telemetry_impl *telemetry = (struct telemetry_impl *)userdata; Casting a void * pointer is unnecessary in C. Assignment has an implicit cast.

Re: [dpdk-dev] [PATCH 01/11] telemetry: initial telemetry infrastructure

2018-08-23 Thread Stephen Hemminger
On Thu, 23 Aug 2018 13:08:03 +0100 Ciara Power wrote: > + > +static int > +telemetry_probe(struct rte_vdev_device *vdev) > +{ > + int ret; > + > + RTE_SET_USED(vdev); > + ret = rte_telemetry_init(rte_socket_id()); > + if (ret < 0) { > + printf("Error - Telemetry initia

Re: [dpdk-dev] [PATCH 07/11] telemetry: add tests for telemetry api

2018-08-23 Thread Stephen Hemminger
On Thu, 23 Aug 2018 13:08:09 +0100 Ciara Power wrote: > This patch adds all tests for the Telemetry API. > The tests added include a parser test, selftest, and socket > messaging tests. > > The parser tests pass valid and invalid messages to the parser > to ensure the correct return values are r

[dpdk-dev] [PATCH] net/mlx5: fix interrupt completion queue index wrapping

2018-08-23 Thread Xueming Li
Rxq cq_ci was 16 bits while hardware is expecting to wrap around 24 bits, this caused interrupt failure after burst of packets. Fixes: 43e9d9794cde ("net/mlx5: support upstream rdma-core") Cc: Shahaf Shuler Signed-off-by: Xueming Li --- drivers/net/mlx5/mlx5_rxtx.c | 2 +- drivers/net/mlx5/mlx

Re: [dpdk-dev] [RFC] ethdev: support metadata as flow rule criteria

2018-08-23 Thread Yongseok Koh
On Wed, Aug 22, 2018 at 12:13:19PM +, Ananyev, Konstantin wrote: > Hi Dekel, > > > > > > > > > > > > > > > > -Original Message- > > > > > From: Dekel Peled [mailto:dek...@mellanox.com] > > > > > Sent: Monday, August 13, 2018 10:47 AM > > > > > To: dev@dpdk.org > > > > > Cc: Ori Kam ; S

Re: [dpdk-dev] [RFC] ethdev: support metadata as flow rule criteria

2018-08-23 Thread Yongseok Koh
On Wed, Aug 22, 2018 at 04:31:14PM +0300, Andrew Rybchenko wrote: > On 13.08.2018 10:46, Dekel Peled wrote: > > Current implementation of rte_flow allows match pattern of flow rule, > > based on packet data or header fields. > > This limits the application use of match patterns. > > > > For exampl

Re: [dpdk-dev] [PATCH] net/mlx5: handle expected errno properly

2018-08-23 Thread Yongseok Koh
On Thu, Aug 23, 2018 at 02:38:51PM +0800, Xiaoyu Min wrote: > rte_errno is a per thread variable and is widely used as an > error indicator, which means a function could affect > other functions' results by setting rte_errno carelessly > > During rxq setup, an EINVAL rte_errno is expected since >

Re: [dpdk-dev] [RFC v2] ethdev: add flow metadata

2018-08-23 Thread Yongseok Koh
> On Aug 20, 2018, at 8:45 AM, Ferruh Yigit wrote: > > On 6/26/2018 9:54 AM, Xueming Li wrote: >> Currently, rte_flow pattern only match packet header fields. >> This patch adds additional data to match the packet. >> >> For example, in egress direction, to do an action depending on the VM >> i

Re: [dpdk-dev] [RFC 1/1] eventdev: add distributed software (DSW) event device

2018-08-23 Thread Mattias Rönnblom
On 2018-08-23 15:44, Jerin Jacob wrote: -Original Message- Date: Thu, 23 Aug 2018 15:08:29 +0200 From: Mattias Rönnblom To: Jerin Jacob CC: dev@dpdk.org, bruce.richard...@intel.com Subject: Re: [dpdk-dev] [RFC 1/1] eventdev: add distributed software (DSW) event device User-Agent: Moz

[dpdk-dev] [PATCH 6/8] memalloc: add EAL-internal API to get and set segment fd's

2018-08-23 Thread Anatoly Burakov
Enable setting and retrieving segment fd's internally. For now, retrieving fd's will not be used anywhere until we get an external API, but it will be useful for things like virtio, where we wish to share segment fd's. Setting segment fd's will not be available as a public API at this time, but i

[dpdk-dev] [PATCH 2/8] eal: don't allow legacy mode with in-memory mode

2018-08-23 Thread Anatoly Burakov
In-memory mode was never meant to support legacy mode, because we cannot sort anonymous pages anyway. Fixes: 72b49ff623c4 ("mem: support --in-memory mode") Cc: sta...@dpdk.org Signed-off-by: Anatoly Burakov --- lib/librte_eal/common/eal_common_options.c | 6 ++ 1 file changed, 6 insertions(

[dpdk-dev] [PATCH 3/8] mem: raise maximum fd limit unconditionally

2018-08-23 Thread Anatoly Burakov
Previously, when we allocated hugepages, we closed the fd's corresponding to them after we've done our mappings. Since we did mmap(), we didn't actually lose the reference, but file descriptors used for mmap() do not count against the fd limit. Since we are going to store all of our fd's, we will h

[dpdk-dev] [PATCH 5/8] memalloc: track page fd's in non-single file mode

2018-08-23 Thread Anatoly Burakov
Previously, we were only tracking lock file fd's in single-file segments mode, but did not track fd's in non-single file mode because we didn't need to (mmap() call still kept the lock). Now that we are going to expose these fd's to the world, we need to have access to them, so track them even in n

[dpdk-dev] [PATCH 7/8] mem: add external API to retrieve page fd from EAL

2018-08-23 Thread Anatoly Burakov
Now that we can retrieve page fd's internally, we can expose it as an external API. This will add two flavors of API - thread-safe and non-thread-safe. Signed-off-by: Anatoly Burakov --- lib/librte_eal/common/eal_common_memory.c | 47 ++ lib/librte_eal/common/include/rte_mem

[dpdk-dev] [PATCH 4/8] memalloc: rename lock list to fd list

2018-08-23 Thread Anatoly Burakov
Previously, we were only using lock lists to store per-page lock fd's because we cannot use modern fcntl() file description locks to lock parts of the page in single file segments mode. Now, we will be using this list to store either lock fd's (along with memseg list fd) in single file segments mo

[dpdk-dev] [PATCH 8/8] mem: support using memfd segments for in-memory mode

2018-08-23 Thread Anatoly Burakov
Enable using memfd-created segments if supported by the system. This will allow having real fd's for pages but without hugetlbfs mounts, which will enable in-memory mode to be used with virtio. The implementation is mostly piggy-backing on existing real-fd code, except that we no longer need to u

[dpdk-dev] [PATCH 1/8] fbarray: fix detach in noshconf mode

2018-08-23 Thread Anatoly Burakov
In noshconf mode, no shared files are created, but we're still trying to unlink them, resulting in detach/destroy failure even though it should have succeeded. Fix it by exiting early in noshconf mode. Fixes: 3ee2cde248a7 ("fbarray: support --no-shconf mode") Cc: sta...@dpdk.org Signed-off-by: An

[dpdk-dev] [PATCH 0/8] Improve running DPDK without hugetlbfs mounpoint

2018-08-23 Thread Anatoly Burakov
This patchset further improves DPDK support for running without hugetlbfs mountpoints. First of all, it enables using memfd-created hugepages in in-memory mode. This way, instead of anonymous hugepages, we can get proper fd's for each page (or for the entire segment, if we're using single-file seg

[dpdk-dev] [RFC 10/10] vhost: enable postcopy protocol feature

2018-08-23 Thread Maxime Coquelin
Enable postcopy protocol feature except if dequeue zero-copy is enabled. In this case, guest memory requires to be populated, which is not compatible with userfaultfd. Signed-off-by: Dr. David Alan Gilbert Signed-off-by: Maxime Coquelin --- lib/librte_vhost/vhost_user.c | 7 +++ lib/librte_

[dpdk-dev] [RFC 09/10] vhost: add support to postcopy's end request

2018-08-23 Thread Maxime Coquelin
The master sends this message before stopping handling userfaults, so that the backend closes the userfaultfd. The master waits for the slave to acknowledge the request with an empty 64bits payload for synchronization purpose. Signed-off-by: Dr. David Alan Gilbert Signed-off-by: Maxime Coquelin

[dpdk-dev] [RFC 08/10] vhost: send userfault range addresses back to qemu

2018-08-23 Thread Maxime Coquelin
Signed-off-by: Dr. David Alan Gilbert Signed-off-by: Maxime Coquelin --- lib/librte_vhost/vhost_user.c | 48 --- 1 file changed, 44 insertions(+), 4 deletions(-) diff --git a/lib/librte_vhost/vhost_user.c b/lib/librte_vhost/vhost_user.c index 0861feff1..29e3e2a07

[dpdk-dev] [RFC 07/10] vhost: avoid useless VhostUserMemory copy

2018-08-23 Thread Maxime Coquelin
The VHOST_USER_SET_MEM_TABLE payload is copied when handled, whereas it could directly be referenced. This is not very important, but next, we'll need to update the payload and send it back to Qemu. Signed-off-by: Dr. David Alan Gilbert Signed-off-by: Maxime Coquelin --- lib/librte_vhost/vhost

[dpdk-dev] [RFC 05/10] vhost: add support for postcopy's listen message

2018-08-23 Thread Maxime Coquelin
Signed-off-by: Dr. David Alan Gilbert Signed-off-by: Maxime Coquelin --- lib/librte_vhost/vhost.h | 1 + lib/librte_vhost/vhost_user.c | 18 ++ lib/librte_vhost/vhost_user.h | 4 +++- 3 files changed, 22 insertions(+), 1 deletion(-) diff --git a/lib/librte_vhost/vhost.h b

[dpdk-dev] [RFC 06/10] vhost: register new regions with userfaultfd

2018-08-23 Thread Maxime Coquelin
Signed-off-by: Dr. David Alan Gilbert Signed-off-by: Maxime Coquelin --- lib/librte_vhost/vhost_user.c | 22 ++ 1 file changed, 22 insertions(+) diff --git a/lib/librte_vhost/vhost_user.c b/lib/librte_vhost/vhost_user.c index d93febbc3..1c75d8015 100644 --- a/lib/librte_vhos

[dpdk-dev] [RFC 04/10] vhost: introduce postcopy's advise message

2018-08-23 Thread Maxime Coquelin
This patch opens a userfaultfd and sends it back to Qemu's VHOST_USER_POSTCOPY_ADVISE request. Signed-off-by: Dr. David Alan Gilbert Signed-off-by: Maxime Coquelin --- lib/librte_vhost/vhost.h | 2 ++ lib/librte_vhost/vhost_user.c | 37 +++ lib/librte_vhost

[dpdk-dev] [RFC 03/10] vhost: enable fds passing when sending vhost-user messages

2018-08-23 Thread Maxime Coquelin
Passing userfault fds to Qemu will be required for postcopy live-migration feature. Signed-off-by: Dr. David Alan Gilbert Signed-off-by: Maxime Coquelin --- lib/librte_vhost/vhost_user.c | 26 ++ 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/lib/librte_

[dpdk-dev] [RFC 02/10] vhost: add number of fds to vhost-user messages and use it

2018-08-23 Thread Maxime Coquelin
As soons as some anciliarry datai (fds) are received, it is copied without checking its length. This patch adds adds the number of fds received to the message, which is set in read_vhost_message(). This is preliminary work to support sending fds to Qemu. Signed-off-by: Dr. David Alan Gilbert Si

[dpdk-dev] [RFC 01/10] vhost: define postcopy protocol flag

2018-08-23 Thread Maxime Coquelin
Signed-off-by: Dr. David Alan Gilbert Signed-off-by: Maxime Coquelin --- lib/librte_vhost/rte_vhost.h | 4 1 file changed, 4 insertions(+) diff --git a/lib/librte_vhost/rte_vhost.h b/lib/librte_vhost/rte_vhost.h index b02673d4a..b3cc6990d 100644 --- a/lib/librte_vhost/rte_vhost.h +++ b/lib

[dpdk-dev] [RFC 00/10] vhost: add postcopy live-migration support

2018-08-23 Thread Maxime Coquelin
This RFC adds support for postcopy live-migration. With classic live-migration, the VM runs on source while its content is being migrated to destination. When pages already migrated to destination are dirtied by the source, they get copied until both source and destination memory converge. At that

Re: [dpdk-dev] [dpdk-stable] [PATCH] doc: fix style and syntax errors in flow API guide

2018-08-23 Thread Ferruh Yigit
On 8/23/2018 5:32 PM, Ferruh Yigit wrote: > On 8/23/2018 9:05 AM, Ilya Maximets wrote: >> Fixes: 3e0ceb9f17ff ("doc: add basic howto for flow API") >> Cc: sta...@dpdk.org >> >> Signed-off-by: Ilya Maximets > > Acked-by: Ferruh Yigit Applied to dpdk-next-net/master, thanks.

Re: [dpdk-dev] [dpdk-stable] [PATCH] doc: fix style and syntax errors in flow API guide

2018-08-23 Thread Ferruh Yigit
On 8/23/2018 9:05 AM, Ilya Maximets wrote: > Fixes: 3e0ceb9f17ff ("doc: add basic howto for flow API") > Cc: sta...@dpdk.org > > Signed-off-by: Ilya Maximets Acked-by: Ferruh Yigit

Re: [dpdk-dev] [PATCH] net/octeontx: fix packet corruption on Tx

2018-08-23 Thread Ferruh Yigit
On 8/17/2018 5:06 PM, santosh wrote: > > On Friday 17 August 2018 08:00 PM, Jerin Jacob wrote: >> LMTST does not guarantee packet content get synced with L2C when >> HW access the packet for transmitting. >> >> Adding coherent IO write barrier will make sure HW sees the correct >> packet if its mo

Re: [dpdk-dev] [PATCH v3 0/6] PMD driver for AF_XDP

2018-08-23 Thread William Tu
Hi Zhang Qi, I'm not familiar with DPDK code, but I'm curious about the benefits of using AF_XDP pmd, specifically I have a couple questions: 1) With zero-copy driver support, is AF_XDP pmd expects to have similar performance than other pmd? Since AF_XDP is still using native device driver, isn't

Re: [dpdk-dev] 17.11.4 patches review and test

2018-08-23 Thread Yongseok Koh
> On Aug 22, 2018, at 5:19 PM, Yongseok Koh wrote: > > On Tue, Aug 21, 2018 at 12:07:49PM +0200, Alejandro Lucero wrote: >> Hi Yonngseok, >> >> There is a patchset aimed at 17.11.x: >> >> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatches.dpdk.org%2Fcover%2F42741%2F&da

[dpdk-dev] 17.11.4 will be released on Aug 30

2018-08-23 Thread Yongseok Koh
Hi, all Testings seem to be still ongoing, there was a compilation error fixed and there were missing patches. Therefore, I'll postpone the release by a week. I'll release -rc2 on this Friday (or weekend) and GA will be on Aug 30 if I get testing results by the time. Thanks for your understandi

Re: [dpdk-dev] [PATCH v3] netvsc: resize event buffer as needed

2018-08-23 Thread Ferruh Yigit
On 8/14/2018 5:45 PM, Stephen Hemminger wrote: > The event buffer was changed to be a fixed size value, > had a couple of issues. The big one is that rte_free was still > being called for a pointer that was not setup with rte_malloc(). > > The event buffer was also too small to handle heavy receiv

Re: [dpdk-dev] [dpdk-stable] [PATCH v4] net/bonding: per-slave intermediate rx ring

2018-08-23 Thread Chas Williams
On Thu, Aug 23, 2018 at 3:28 AM Matan Azrad wrote: > Hi > > From: Eric Kinzie > > On Wed Aug 22 11:42:37 + 2018, Matan Azrad wrote: > > > Hi Luca > > > > > > From: Luca Boccassi > > > > On Wed, 2018-08-22 at 07:09 +, Matan Azrad wrote: > > > > > Hi Chas > > > > > > > > > > From: Chas Will

Re: [dpdk-dev] [PATCH 2/4] bus/vmbus: add host latency tuning function

2018-08-23 Thread Stephen Hemminger
On Thu, 23 Aug 2018 15:45:44 +0100 Ferruh Yigit wrote: > On 8/9/2018 6:50 PM, Stephen Hemminger wrote: > > Add vmbus API to allow tuning the scan interval on the host side. > > > > Signed-off-by: Stephen Hemminger > > <...> > > > @@ -20,6 +20,7 @@ DPDK_18.08 { > > rte_vmbus_probe; > >

Re: [dpdk-dev] [PATCH 1/4] netvsc: chimney buffer size error handling

2018-08-23 Thread Stephen Hemminger
On Thu, 23 Aug 2018 15:45:17 +0100 Ferruh Yigit wrote: > On 8/9/2018 6:50 PM, Stephen Hemminger wrote: > > Fix the error handling in setting up transmit buffer. > > If setting up chimney buffer fails, then it is not connected so > > no need to send disconnect. > > > > Allow for some unused area

Re: [dpdk-dev] [RFC] ethdev: support metadata as flow rule criteria

2018-08-23 Thread Ferruh Yigit
On 8/13/2018 9:03 AM, Dekel Peled wrote: > Adding relevant maintainers. > >> -Original Message- >> From: Dekel Peled [mailto:dek...@mellanox.com] >> Sent: Monday, August 13, 2018 10:47 AM >> To: dev@dpdk.org >> Cc: Ori Kam ; Shahaf Shuler >> >> Subject: [RFC] ethdev: support metadata as

Re: [dpdk-dev] [PATCH] net/virtio-user: fix memory hotplug support

2018-08-23 Thread Sean Harte
On Thu, 23 Aug 2018 at 15:01, Burakov, Anatoly wrote: > > On 23-Aug-18 12:19 PM, Sean Harte wrote: > > On Thu, 23 Aug 2018 at 10:05, Burakov, Anatoly > > wrote: > >> > >> On 23-Aug-18 3:57 AM, Tiwei Bie wrote: > >>> Deadlock can occur when allocating memory if a vhost-kernel > >>> based virtio-us

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/bonding: stop and deactivate slaves when bonding port is stopped

2018-08-23 Thread Chas Williams
On Thu, Aug 23, 2018 at 9:15 AM Ferruh Yigit wrote: > On 8/6/2018 4:50 PM, Chas Williams wrote: > > On Sun, Aug 5, 2018 at 5:55 PM Thomas Monjalon > wrote: > > > >> 02/08/2018 15:38, Doherty, Declan: > >>> On 01/08/2018 2:18 PM, Radu Nicolau wrote: > When a bonding port is stopped also stop

Re: [dpdk-dev] [PATCH 0/4] netvsc performance enhancements

2018-08-23 Thread Ferruh Yigit
On 8/9/2018 6:50 PM, Stephen Hemminger wrote: > Some more netvsc performance related changes. > The biggest benefit comes from lowering the hypervisor polling rate > from 100us to 50us. Supporting tx_done_cleanup can also help some > applications get through the send completions faster. > > Steph

Re: [dpdk-dev] [PATCH 4/4] netvsc: implement tx_done_cleanup

2018-08-23 Thread Ferruh Yigit
On 8/9/2018 6:50 PM, Stephen Hemminger wrote: > Add tx_done_cleanup ethdev hook to allow application to > control if/when it wants completions to be handled. > > Signed-off-by: Stephen Hemminger > --- > drivers/net/netvsc/hn_ethdev.c | 1 + > drivers/net/netvsc/hn_rndis.c | 2 +- > drivers/ne

Re: [dpdk-dev] [PATCH 2/4] bus/vmbus: add host latency tuning function

2018-08-23 Thread Ferruh Yigit
On 8/9/2018 6:50 PM, Stephen Hemminger wrote: > Add vmbus API to allow tuning the scan interval on the host side. > > Signed-off-by: Stephen Hemminger <...> > @@ -20,6 +20,7 @@ DPDK_18.08 { > rte_vmbus_probe; > rte_vmbus_register; > rte_vmbus_scan; > + rte_vmbus_set_latenc

Re: [dpdk-dev] [PATCH 1/4] netvsc: chimney buffer size error handling

2018-08-23 Thread Ferruh Yigit
On 8/9/2018 6:50 PM, Stephen Hemminger wrote: > Fix the error handling in setting up transmit buffer. > If setting up chimney buffer fails, then it is not connected so > no need to send disconnect. > > Allow for some unused area if full area is not used. > > Signed-off-by: Stephen Hemminger sub

Re: [dpdk-dev] [PATCH] net/virtio-user: fix memory hotplug support

2018-08-23 Thread Burakov, Anatoly
On 23-Aug-18 12:19 PM, Sean Harte wrote: On Thu, 23 Aug 2018 at 10:05, Burakov, Anatoly wrote: On 23-Aug-18 3:57 AM, Tiwei Bie wrote: Deadlock can occur when allocating memory if a vhost-kernel based virtio-user device is in use. Besides, it's possible to have much more than 64 non-contiguous

Re: [dpdk-dev] [RFC] ethdev: add generic TTL rewrite actions

2018-08-23 Thread Ferruh Yigit
On 8/7/2018 3:20 PM, Jack Min wrote: > There is a need to rewrite TTL by decrease or just set it directly, > and it's not necessary to check if the final result is zero or not. > > This is slightly different from the one defined by openflow and > make the actions more generic. > > > > > Signed

Re: [dpdk-dev] [RFC] ethdev: add generic MAC address rewrite actions

2018-08-23 Thread Ferruh Yigit
On 8/7/2018 3:20 PM, Jack Min wrote: > There is a need to offload rewrite MAC address for both destination and source > from the matched flow > > The proposed actions could make above easily achieved > > > Signed-off-by: Xiaoyu Min mailto:jack...@mellanox.com>> Looks good to me. @Adrien, @Andr

Re: [dpdk-dev] [PATCH v1 1/1] crypto/openssl: replace macros by static inline functions

2018-08-23 Thread Akhil Goyal
On 7/31/2018 7:46 PM, Ashish Gupta wrote: Replace macros by static inline functions in openssl version compatibility layer Signed-off-by: Sunila Sahu Signed-off-by: Shally Verma Signed-off-by: Ashish Gupta --- Applied to dpdk-next-crypto Thanks

Re: [dpdk-dev] [PATCH] examples/ipsec-secgw: add 3des-cbc support

2018-08-23 Thread Akhil Goyal
On 7/25/2018 3:19 PM, Hemant Agrawal wrote: Signed-off-by: Hemant Agrawal --- doc/guides/sample_app_ug/ipsec_secgw.rst | 3 ++- examples/ipsec-secgw/esp.c | 3 +++ examples/ipsec-secgw/sa.c| 11 ++- 3 files changed, 15 insertions(+), 2 deletions(-)

Re: [dpdk-dev] [PATCH] app/testbbdev: fix inputs mbuf creation issue

2018-08-23 Thread Akhil Goyal
On 8/17/2018 12:49 PM, Kamil Chalupnik wrote: Omitting inputs and outputs mbuf creation for BaseBand Null Device as inputs and outputs data do not exist for Null Device Fixes: b2a4654f082b ("mempool: check for zero size creation") Cc: pablo.de.lara.gua...@intel.com Signed-off-by: Kamil Chalu

Re: [dpdk-dev] [PATCH v1 1/1] crypto/openssl: replace macros by static inline functions

2018-08-23 Thread Akhil Goyal
On 7/31/2018 7:46 PM, Ashish Gupta wrote: Replace macros by static inline functions in openssl version compatibility layer Signed-off-by: Sunila Sahu Signed-off-by: Shally Verma Signed-off-by: Ashish Gupta --- drivers/crypto/openssl/compat.h | 265 ++-

Re: [dpdk-dev] [PATCH v2 0/7] ethdev: add flow API object converter

2018-08-23 Thread Ferruh Yigit
On 8/3/2018 2:36 PM, Adrien Mazarguil wrote: > This is a follow up to the "Flow API helpers enhancements" series submitted > almost a year ago [1]. The new title is due to the reduced scope of this > version. > > rte_flow_conv() is a flexible replacement to rte_flow_copy(), itself a > temporary so

Re: [dpdk-dev] [RFC 1/1] eventdev: add distributed software (DSW) event device

2018-08-23 Thread Jerin Jacob
-Original Message- > Date: Thu, 23 Aug 2018 15:08:29 +0200 > From: Mattias Rönnblom > To: Jerin Jacob > CC: dev@dpdk.org, bruce.richard...@intel.com > Subject: Re: [dpdk-dev] [RFC 1/1] eventdev: add distributed software (DSW) > event device > User-Agent: Mozilla/5.0 (X11; Linux x86_64; r

Re: [dpdk-dev] [PATCH] net/bonding: add support to match on 0x88A8 ethertype

2018-08-23 Thread Ferruh Yigit
On 8/16/2018 1:56 PM, Chas Williams wrote: > On Tue, Aug 7, 2018 at 4:11 AM wangyunjian wrote: > >> From: Yunjian Wang >> >> We assume VLAN ethtertype is 0x8100 in get_vlan_offset() function, >> but it could be 0x88A8 if QinQ is supported. >> >> Signed-off-by: Yunjian Wang >> > > Acked-by: Cha

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/bonding: stop and deactivate slaves when bonding port is stopped

2018-08-23 Thread Ferruh Yigit
On 8/6/2018 4:50 PM, Chas Williams wrote: > On Sun, Aug 5, 2018 at 5:55 PM Thomas Monjalon wrote: > >> 02/08/2018 15:38, Doherty, Declan: >>> On 01/08/2018 2:18 PM, Radu Nicolau wrote: When a bonding port is stopped also stop and deactivate all slaves. Otherwise slaves will be still lis

Re: [dpdk-dev] [dpdk-announce] CFP for DPDK Summit Userspace

2018-08-23 Thread O'Driscoll, Tim
The agenda for this event is now available at: https://dpdkuserspace2018.sched.com/. All of the proposals that were submitted were of a very high standard, so it was a difficult job to decide which ones to accept/reject. Anybody who had a proposal that was rejected should consider resubmitting

Re: [dpdk-dev] [RFC 1/1] eventdev: add distributed software (DSW) event device

2018-08-23 Thread Mattias Rönnblom
On 2018-08-19 08:11, Jerin Jacob wrote: > I would like to include this driver for v18.08. Please send the next > the version in order to complete review before giving RC1 pull request. > I've been working on a test bench with real packet I/O and some more reasonable load to verify load balancing

Re: [dpdk-dev] [PATCH 0/5] remove usage of register keyword in C

2018-08-23 Thread Ferruh Yigit
On 8/1/2018 10:03 PM, Stephen Hemminger wrote: > On Wed, 1 Aug 2018 18:03:04 + > Yongseok Koh wrote: > >>> On Jul 31, 2018, at 11:07 AM, Stephen Hemminger >>> wrote: >>> >>> On Tue, 31 Jul 2018 18:48:40 +0200 >>> Adrien Mazarguil wrote: >>> On Tue, Jul 31, 2018 at 09:30:54AM -0700,

Re: [dpdk-dev] [PATCH] examples/ipsec-secgw: add 3des-cbc support

2018-08-23 Thread Akhil Goyal
On 7/25/2018 3:19 PM, Hemant Agrawal wrote: Signed-off-by: Hemant Agrawal --- doc/guides/sample_app_ug/ipsec_secgw.rst | 3 ++- examples/ipsec-secgw/esp.c | 3 +++ examples/ipsec-secgw/sa.c| 11 ++- 3 files changed, 15 insertions(+), 2 deletions(-)

Re: [dpdk-dev] [PATCH] compress/qat: use compression specific driver name

2018-08-23 Thread Akhil Goyal
On 8/13/2018 8:08 PM, Jozwiak, TomaszX wrote: -Original Message- From: Trahe, Fiona Sent: Friday, August 10, 2018 5:18 PM To: dev@dpdk.org; akhil.go...@nxp.com; De Lara Guarch, Pablo ; Jozwiak, TomaszX Cc: Trahe, Fiona Subject: [PATCH] compress/qat: use compression specific driver

Re: [dpdk-dev] [PATCH] crypto/qat: fix typo

2018-08-23 Thread Akhil Goyal
On 8/10/2018 8:50 PM, Trahe, Fiona wrote: Corrected subject - only 1 patch here. -Original Message- From: Trahe, Fiona Sent: Friday, August 10, 2018 4:19 PM To: dev@dpdk.org; akhil.go...@nxp.com; De Lara Guarch, Pablo ; Jozwiak, TomaszX Cc: Trahe, Fiona Subject: [PATCH 1/2] crypto

Re: [dpdk-dev] [PATCH 2/2] virtio: fix PCI config err handling

2018-08-23 Thread Luca Boccassi
On Tue, 2018-08-21 at 10:40 +0800, Tiwei Bie wrote: > On Mon, Aug 20, 2018 at 05:45:35PM +0100, Luca Boccassi wrote: > > On Mon, 2018-08-20 at 16:18 +0800, Tiwei Bie wrote: > > > On Thu, Aug 16, 2018 at 07:49:43PM +0100, Luca Boccassi wrote: > > > > On Thu, 2018-08-16 at 19:47 +0100, Luca Boccassi

Re: [dpdk-dev] [RFC] ethdev: add generic L2/L3 tunnel encapsulation actions

2018-08-23 Thread Ferruh Yigit
On 7/30/2018 5:19 PM, Ori Kam wrote: > Currenlty the encap/decap actions only support encapsulation > of VXLAN and NVGRE L2 packets. > There is a need to add more L2 tunnels and also L3 tunnels. > > One issue with the current approch is the duplication of code. > For example the code for handling

Re: [dpdk-dev] [PATCH v1] net/tap: explain how to compile eBPF C file

2018-08-23 Thread Ferruh Yigit
On 7/5/2018 1:34 PM, Wiles, Keith wrote: > > >> On Jul 4, 2018, at 3:11 PM, Thomas Monjalon wrote: >> >> 04/07/2018 21:47, Ferruh Yigit: >>> On 6/12/2018 3:02 PM, Ophir Munk wrote: Please note that other than cloning iproute2 we also need to install clang and llvm tools versions 3.7 a

[dpdk-dev] [PATCH 10/11] usertools: add client python script for telemetry

2018-08-23 Thread Ciara Power
This patch adds a python script which can be used as a demo client. The script is interactive and will allow the user to register, request statistics, and unregister. To run the script, an argument for the client file path must be passed in: "python telemetry_client.py "file_path". This script is

[dpdk-dev] [PATCH 11/11] telemetry: add collectd plugin patch

2018-08-23 Thread Ciara Power
This patch adds the patch for the collectd plugin developed for use with the DPDK Telemetry library. The patch is included here to allow users to apply the patch to collectd when using DPDK Telemetry. Further details on applying the collectd patch can be found in the collectd patch commit message.

[dpdk-dev] [PATCH 08/11] telemetry: add vdev kvargs for selftest

2018-08-23 Thread Ciara Power
This patch adds functionality to run the selftest by passing in an argument to vdev, "selftest". To run the tests that were added in the previous patch, the argument "selftest=1" must be added to the vdev arguments. This will enable the user to run telemetry with or without tests, as required. Si

[dpdk-dev] [PATCH 09/11] doc: add telemetry documentation

2018-08-23 Thread Ciara Power
This patch adds all documentation for telemetry. A description on how to use the Telemetry API with a DPDK application is given in this document. Signed-off-by: Ciara Power Signed-off-by: Brian Archbold --- doc/guides/howto/index.rst | 1 + doc/guides/howto/telemetry.rst | 86

[dpdk-dev] [PATCH 06/11] telemetry: format json response when sending stats

2018-08-23 Thread Ciara Power
This patch adds functionality to create a JSON message in order to send it to a client socket. When stats are requested by a client, they are retrieved from the metrics library and encoded in JSON format. Signed-off-by: Ciara Power Signed-off-by: Brian Archbold --- lib/librte_telemetry/rte_tel

[dpdk-dev] [PATCH 04/11] telemetry: add parser for client socket messages

2018-08-23 Thread Ciara Power
This patch adds the parser file. This is used to parse any messages that are received on any of the client sockets. Currently, the unregister functionality works using the parser. Functionality relating to getting statistic values for certain ports will be added in a subsequent patch, however the

[dpdk-dev] [PATCH 07/11] telemetry: add tests for telemetry api

2018-08-23 Thread Ciara Power
This patch adds all tests for the Telemetry API. The tests added include a parser test, selftest, and socket messaging tests. The parser tests pass valid and invalid messages to the parser to ensure the correct return values are received. The selftest tests basic functions in the Telemetry API suc

[dpdk-dev] [PATCH 05/11] telemetry: update metrics before sending stats

2018-08-23 Thread Ciara Power
This patch adds functionality to update the statistics in the metrics library with values from the ethdev stats. Values need to be updated before they are encoded into a JSON message and sent to the client that requested them. The JSON encoding will be added in a subsequent patch. Signed-off-by:

[dpdk-dev] [PATCH 02/11] telemetry: add initial connection socket

2018-08-23 Thread Ciara Power
This patch adds the telemetry UNIX socket. It is used to allow connections from external clients. On the initial connection from a client, ethdev stats are registered in the metrics library, to allow for their retrieval at a later stage. Signed-off-by: Ciara Power Signed-off-by: Brian Archbold

[dpdk-dev] [PATCH 03/11] telemetry: add client feature and sockets

2018-08-23 Thread Ciara Power
This patch introduces clients to the telemetry API. When a client makes a connection through the initial telemetry socket, they can send a message through the socket to be parsed. Register messages are expected through this socket, to enable clients to register and have a client socket setup for f

[dpdk-dev] [PATCH 00/11] introduce telemetry library

2018-08-23 Thread Ciara Power
This patchset introduces a Telemetry library for DPDK Service Assurance. This library provides an easy way to query DPDK Ethdev metrics. The telemetry library provides a method for a service assurance component to retrieve metrics from a DPDK packet forwarding application. Communicating from the s

[dpdk-dev] [PATCH 01/11] telemetry: initial telemetry infrastructure

2018-08-23 Thread Ciara Power
This patch adds the infrastructure and initial code for the telemetry library. A virtual device is used for telemetry, which is included in this patch. To use telemetry, a command-line argument must be used - "--vdev=telemetry". Control threads are used to get CPU cycles for telemetry, which are

Re: [dpdk-dev] [PATCH] examples: remove Rx checksum offload

2018-08-23 Thread Ferruh Yigit
On 7/30/2018 4:30 PM, Andrew Rybchenko wrote: > On 30.07.2018 17:40, Jerin Jacob wrote: >> -Original Message- >>> Date: Mon, 30 Jul 2018 14:12:12 + >>> From: "Ananyev, Konstantin" >>> To: Jerin Jacob >>> CC: Thomas Monjalon , "dev@dpdk.org" , >>> "Yigit, Ferruh" , "shah...@mellanox.

Re: [dpdk-dev] [PATCH v3] net/tap: add queues when attaching from secondary process

2018-08-23 Thread Ferruh Yigit
On 7/20/2018 12:15 PM, Thomas Monjalon wrote: > From: Raslan Darawsheh > > In the case the device is created by the primary process, > the secondary must request some file descriptors to attach the queues. > The file descriptors are shared via IPC Unix socket. > > Thanks to the IPC synchronizati

Re: [dpdk-dev] [PATCH v3 0/6] doc/qat: clarify build config options

2018-08-23 Thread Thomas Monjalon
23/08/2018 13:28, Akhil Goyal: > Hi Thomas, > > On 8/23/2018 4:54 PM, Thomas Monjalon wrote: > > > 23/08/2018 13:12, Akhil Goyal: > >> Patchset applied to dpdk-next-crypto. > >> > >> Subject for "doc/qat: fix typos and make cosmetic changes" changed to > >> "doc/qat: correct typo and make cosmet

Re: [dpdk-dev] 17.11.4 patches review and test

2018-08-23 Thread Thomas Monjalon
23/08/2018 02:19, Yongseok Koh: > For the patchset, > - "mem: add function for checking memsegs IOVAs addresses" > This is adding a new API, so I don't expect any API/ABI breakage, but want > to > double-confirm with Thomas. Thomas? Yes, adding a function is not breaking API/ABI.

Re: [dpdk-dev] [PATCH v3 0/6] doc/qat: clarify build config options

2018-08-23 Thread Akhil Goyal
Hi Thomas, On 8/23/2018 4:54 PM, Thomas Monjalon wrote: 23/08/2018 13:12, Akhil Goyal: Patchset applied to dpdk-next-crypto. Subject for "doc/qat: fix typos and make cosmetic changes" changed to "doc/qat: correct typo and make cosmetic changes" to avoid git log error. Which error? Lack of

Re: [dpdk-dev] [PATCH v3 0/6] doc/qat: clarify build config options

2018-08-23 Thread Thomas Monjalon
23/08/2018 13:12, Akhil Goyal: > Patchset applied to dpdk-next-crypto. > > Subject for "doc/qat: fix typos and make cosmetic changes" changed to > "doc/qat: correct typo and make cosmetic changes" > > to avoid git log error. Which error? Lack of Fixes tag? As a committer, you must be sure the F

Re: [dpdk-dev] [PATCH] net/virtio-user: fix memory hotplug support

2018-08-23 Thread Sean Harte
On Thu, 23 Aug 2018 at 10:05, Burakov, Anatoly wrote: > > On 23-Aug-18 3:57 AM, Tiwei Bie wrote: > > Deadlock can occur when allocating memory if a vhost-kernel > > based virtio-user device is in use. Besides, it's possible > > to have much more than 64 non-contiguous hugepage backed > > memory re

Re: [dpdk-dev] [PATCH] app/testpmd: fix duplicate exit

2018-08-23 Thread Iremonger, Bernard
> -Original Message- > From: Archbold, Brian > Sent: Wednesday, August 22, 2018 7:05 AM > To: Lu, Wenzhuo ; Wu, Jingjing > ; Iremonger, Bernard > Cc: dev@dpdk.org; Archbold, Brian ; > sta...@dpdk.org > Subject: [PATCH] app/testpmd: fix duplicate exit > > In interactive mode, when the "qui

Re: [dpdk-dev] [PATCH v3 0/6] doc/qat: clarify build config options

2018-08-23 Thread Akhil Goyal
On 8/10/2018 7:40 PM, Fiona Trahe wrote: Clarified documentation structure between compressedev, cryptodev and common build parts. Clarified build configuration options. Added Testing section. Fixed typos and made some cosmetic improvements. v3 changes - squashed some patches - added mor

Re: [dpdk-dev] [PATCH v2] ethdev: change vtune profiling approach

2018-08-23 Thread Ferruh Yigit
On 8/22/2018 6:13 PM, Ferruh Yigit wrote: > On 7/24/2018 2:17 PM, Wiles, Keith wrote: >> >> >>> On Jul 19, 2018, at 7:21 AM, ilia.kura...@intel.com wrote: >>> >>> From: Ilia Kurakin >>> >>> The patch changes rx_burst profiling approach: >>> 1. VTune's instrumentation is removed >>> 2. empt

Re: [dpdk-dev] [dpdk-stable] [PATCH 0/2] support MAC changes when no live changes allowed

2018-08-23 Thread Alejandro Lucero
On Wed, Aug 22, 2018 at 6:00 PM, Ferruh Yigit wrote: > On 7/18/2018 9:58 AM, Ferruh Yigit wrote: > > On 7/9/2018 6:14 PM, Alejandro Lucero wrote: > >> This is a patched to fix a functionality coming with the first public > >> release: changing/setting MAC address. > >> > >> The original patch ass

Re: [dpdk-dev] [PATCH] net/virtio-user: fix memory hotplug support

2018-08-23 Thread Burakov, Anatoly
On 23-Aug-18 3:57 AM, Tiwei Bie wrote: Deadlock can occur when allocating memory if a vhost-kernel based virtio-user device is in use. Besides, it's possible to have much more than 64 non-contiguous hugepage backed memory regions due to the memory hotplug, which may cause problems when handling V

Re: [dpdk-dev] [PATCH v2] ethdev: fix device info getting

2018-08-23 Thread Andrew Rybchenko
On 22.08.2018 19:55, Ferruh Yigit wrote: On 8/14/2018 1:57 AM, Lu, Wenzhuo wrote: Hi Andrew, -Original Message- From: Andrew Rybchenko [mailto:arybche...@solarflare.com] Sent: Monday, August 13, 2018 4:39 PM To: Lu, Wenzhuo ; Thomas Monjalon ; Yigit, Ferruh Cc: dev@dpdk.org Subject: R

  1   2   >