Re: [dpdk-dev] [dpdk-users] Traffic doesn't forward on virtual devices

2018-07-10 Thread Loftus, Ciara
> > Bala Sankaran writes: > > > Perfect! > > > > Thanks for the help. > > > > - Original Message - > >> From: "Keith Wiles" > >> To: "Bala Sankaran" > >> Cc: us...@dpdk.org, "Aaron Conole" > >> Sent: Thursday, July 5, 2018 11:41:46 AM > >> Subject: Re: [dpdk-users] Traffic doesn't for

Re: [dpdk-dev] [dpdk-users] Traffic doesn't forward on virtual devices

2018-07-11 Thread Loftus, Ciara
> > > > > > Bala Sankaran writes: > > > > > > > Perfect! > > > > > > > > Thanks for the help. > > > > > > > > - Original Message - > > > >> From: "Keith Wiles" > > > >> To: "Bala Sankaran" > > > >> Cc: us...@dpdk.org, "Aaron Conole" > > > >> Sent: Thursday, July 5, 2018 11:41:46 AM > >

Re: [dpdk-dev] [PATCH] net/vhost: Initialise vid to -1

2018-04-30 Thread Loftus, Ciara
> > On 04/27/2018 04:19 PM, Ciara Loftus wrote: > > rte_eth_vhost_get_vid_from_port_id returns a value of 0 if called before > > the first call to the new_device callback. A vid value >=0 suggests the > > device is active which is not the case in this instance. Initialise vid > > to a negative val

Re: [dpdk-dev] [PATCH] net/vhost: Initialise vid to -1

2018-05-03 Thread Loftus, Ciara
> > > > > On 04/27/2018 04:19 PM, Ciara Loftus wrote: > > > rte_eth_vhost_get_vid_from_port_id returns a value of 0 if called before > > > the first call to the new_device callback. A vid value >=0 suggests the > > > device is active which is not the case in this instance. Initialise vid > > > to

[dpdk-dev] [ovs-dev] OVS with DPDK Meetup notes

2015-12-02 Thread Loftus, Ciara
> > > > On Thu, Nov 26, 2015 at 05:56:08PM +, Traynor, Kevin wrote: > > > Hi All, > > > > > > Just wanted to post some summary notes on the recent OVS with DPDK > Meetup > > we > > > had after the OVS conference. Thanks to everyone for the often lively > > discussion. > > > I've collated and co

[dpdk-dev] [PATCH] vhost: add interface name to virtio-net struct

2014-12-18 Thread Loftus, Ciara
-Original Message- From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] Sent: Thursday, December 18, 2014 3:33 PM To: Loftus, Ciara Cc: dev at dpdk.org; Anthony Fee Subject: Re: [dpdk-dev] [PATCH] vhost: add interface name to virtio-net struct 2014-12-18 14:55, ciara.loftus at intel.com

[dpdk-dev] [PATCH] vhost: add interface name to virtio-net struct

2014-12-18 Thread Loftus, Ciara
SIZ in v2 of the patch. Thanks, Ciara -Original Message- From: Ananyev, Konstantin Sent: Thursday, December 18, 2014 5:03 PM To: Loftus, Ciara; dev at dpdk.org Cc: Anthony Fee Subject: RE: [dpdk-dev] [PATCH] vhost: add interface name to virtio-net struct > -Original Message

RE: [PATCH v2] net/af_xdp: use libxdp if available

2022-01-27 Thread Loftus, Ciara
> > On Tue, Jan 25, 2022 at 07:20:43AM +, Ciara Loftus wrote: > > AF_XDP support is deprecated in libbpf since v0.7.0 [1]. The > > libxdp library now provides the functionality which once was in > > libbpf and which the AF_XDP PMD relies on. This commit updates the > > AF_XDP meson build to us

RE: [PATCH v2] net/af_xdp: re-enable secondary process support

2022-02-06 Thread Loftus, Ciara
> > On 2/4/2022 12:54 PM, Ciara Loftus wrote: > > Secondary process support had been disabled for the AF_XDP PMD > > because there was no logic in place to share the AF_XDP socket > > file descriptors between the processes. This commit introduces > > this logic using the IPC APIs. > > > > Since AF

RE: [PATCH v2] net/af_xdp: re-enable secondary process support

2022-02-07 Thread Loftus, Ciara
> >> > >> On 2/4/2022 12:54 PM, Ciara Loftus wrote: > >>> Secondary process support had been disabled for the AF_XDP PMD > >>> because there was no logic in place to share the AF_XDP socket > >>> file descriptors between the processes. This commit introduces > >>> this logic using the IPC APIs. > >

RE: [PATCH v3] net/af_xdp: re-enable secondary process support

2022-02-08 Thread Loftus, Ciara
> > On 2/8/2022 6:42 PM, Stephen Hemminger wrote: > > On Tue, 8 Feb 2022 18:00:27 + > > Ferruh Yigit wrote: > > > >> On 2/8/2022 5:45 PM, Stephen Hemminger wrote: > >>> On Tue, 8 Feb 2022 13:48:00 + > >>> Ciara Loftus wrote: > >>> > +- **Secondary Processes** > + > + Rx

RE: [PATCH] ethdev: introduce generic dummy packet burst function

2022-02-09 Thread Loftus, Ciara
> Subject: [PATCH] ethdev: introduce generic dummy packet burst function > > Multiple PMDs have dummy/noop Rx/Tx packet burst functions. > > These dummy functions are very simple, introduce a common function in > the ethdev and update drivers to use it instead of each driver having > its own func

RE: [PATCH v1] net/af_xdp: make umem configure code more readable

2022-02-10 Thread Loftus, Ciara
> Subject: [PATCH v1] net/af_xdp: make umem configure code more readable > > The below compile time defined style make the code not so readable, the > first function end block is after "#endif" segment. > > #if defined(XDP_UMEM_UNALIGNED_CHUNK_FLAG) > > xdp_umem_configure() > {

RE: [PATCH v4] net/af_xdp: re-enable secondary process support

2022-02-10 Thread Loftus, Ciara
> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support > > On 2/9/2022 9:48 AM, Ciara Loftus wrote: > > Secondary process support had been disabled for the AF_XDP PMD > because > > there was no logic in place to share the AF_XDP socket file descriptors > > between the processes.

RE: [PATCH v4] net/af_xdp: re-enable secondary process support

2022-02-10 Thread Loftus, Ciara
> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support > > On 2/10/2022 3:40 PM, Loftus, Ciara wrote: > >> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support > >> > >> On 2/9/2022 9:48 AM, Ciara Loftus wrote: >

RE: [PATCH v4] net/af_xdp: re-enable secondary process support

2022-02-10 Thread Loftus, Ciara
> > On 2/10/2022 5:47 PM, Loftus, Ciara wrote: > >> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support > >> > >> On 2/10/2022 3:40 PM, Loftus, Ciara wrote: > >>>> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary pro

RE: [PATCH v4] net/af_xdp: re-enable secondary process support

2022-02-11 Thread Loftus, Ciara
> > > > On 2/10/2022 5:47 PM, Loftus, Ciara wrote: > > >> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process > support > > >> > > >> On 2/10/2022 3:40 PM, Loftus, Ciara wrote: > > >>>> Subject: Re: [PATCH v4] net/af_

RE: [PATCH v4] net/af_xdp: re-enable secondary process support

2022-02-11 Thread Loftus, Ciara
> > On 2/11/2022 9:26 AM, Loftus, Ciara wrote: > >>> > >>> On 2/10/2022 5:47 PM, Loftus, Ciara wrote: > >>>>> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process > >> support > >>>>> > >>>>>

RE: [PATCH v4] net/af_xdp: re-enable secondary process support

2022-02-11 Thread Loftus, Ciara
> > On 2/11/2022 1:01 PM, Loftus, Ciara wrote: > >> > >> On 2/11/2022 9:26 AM, Loftus, Ciara wrote: > >>>>> > >>>>> On 2/10/2022 5:47 PM, Loftus, Ciara wrote: > >>>>>>> Subject: Re: [PATCH v4] net/af_xdp: re-enab

RE: [PATCH v4] net/af_xdp: re-enable secondary process support

2022-02-16 Thread Loftus, Ciara
> Subject: RE: [PATCH v4] net/af_xdp: re-enable secondary process support > > > > > On 2/11/2022 1:01 PM, Loftus, Ciara wrote: > > >> > > >> On 2/11/2022 9:26 AM, Loftus, Ciara wrote: > > >>>>> > > >>>>> On 2/10/2

RE: [PATCH v2] net/af_xdp: allow operation when multiprocess is disabled

2022-02-17 Thread Loftus, Ciara
> Subject: [PATCH v2] net/af_xdp: allow operation when multiprocess is > disabled > > If EAL multiprocess feature has been disabled via rte_mp_disable() > function, AF_XDP driver may not be able to register its IPC callback. > Previously this leads to probe failure. > This commit adds a check for

RE: [PATCH] net/af_xdp: add missing trailing newline in logs

2022-02-17 Thread Loftus, Ciara
> Subject: [PATCH] net/af_xdp: add missing trailing newline in logs > > Caught while trying --in-memory mode, some log messages in this driver > are not terminated with a newline: > rte_pmd_af_xdp_probe(): net_af_xdp: Failed to register multi-process IPC > callback: Operation not supportedvdev_pro

RE: [PATCH] net/af_xdp: make the PMD compatible with libbpf >= v0.7.0

2022-02-17 Thread Loftus, Ciara
> Subject: Re: [PATCH] net/af_xdp: make the PMD compatible with libbpf >= > v0.7.0 > > On Thu, Feb 17, 2022 at 12:14:30PM +, Ciara Loftus wrote: > > libbpf v0.7.0 deprecates the bpf_prog_load function. Use meson to detect > > if libbpf >= v0.7.0 is linked and if so, use the recommended replace

RE: af_xdp + libbpf 0.8

2022-06-23 Thread Loftus, Ciara
> > 24/06/2022 00:18, Ferruh Yigit: > > On 6/23/2022 10:58 PM, Thomas Monjalon wrote: > > > Hi, > > > > > > It seems DPDK is not compatible with libbpf 0.8: > > > > > > drivers/net/af_xdp/rte_eth_af_xdp.c:871:6: error: 'bpf_get_link_xdp_id' > is deprecated: libbpf v0.8+: use bpf_xdp_query_id() ins

RE: [PATCH] net/af_xdp: limit libbpf version to <= v0.7.0

2022-06-24 Thread Loftus, Ciara
> > 24/06/2022 08:06, Ciara Loftus: > > Linking with libbpf v0.8.0 causes deprication warnings. As a temporary > > measure, prevent linking with libbpf versions v0.8.0 and greater. This > > limitation should be removed in the future when appropriate > > compatibility measures are introduced. > > >

RE: [PATCH] net/af_xdp: make compatible with libbpf v0.8.0

2022-06-27 Thread Loftus, Ciara
> > On 6/24/22 13:23, Ciara Loftus wrote: > > libbpf v0.8.0 deprecates the bpf_get_link_xdp_id and bpf_set_link_xdp_fd > > functions. Use meson to detect if libbpf >= v0.7.0 is linked and if so, use > > the recommended replacement functions bpf_xdp_query_id, > bpf_xdp_attach > > and bpf_xdp_detach

RE: [PATCH] net/af_xdp: make compatible with libbpf v0.8.0

2022-06-27 Thread Loftus, Ciara
> > On 6/27/22 17:17, Loftus, Ciara wrote: > >> > >> On 6/24/22 13:23, Ciara Loftus wrote: > >>> libbpf v0.8.0 deprecates the bpf_get_link_xdp_id and > bpf_set_link_xdp_fd > >>> functions. Use meson to detect if libbpf >= v0.7.0 is linked a

RE: [PATCH] net/af_xdp: make compatible with libbpf v0.8.0

2022-06-28 Thread Loftus, Ciara
> > On 6/24/22 13:23, Ciara Loftus wrote: > > libbpf v0.8.0 deprecates the bpf_get_link_xdp_id and > >> bpf_set_link_xdp_fd > > functions. Use meson to detect if libbpf >= v0.7.0 is linked and if so, > use > > the recommended replacement functions bpf_xdp_query_id, > bpf_

RE: [PATCH] net/af_xdp: make compatible with libbpf v0.8.0

2022-07-21 Thread Loftus, Ciara
> > > > > On 6/24/22 13:23, Ciara Loftus wrote: > > > libbpf v0.8.0 deprecates the bpf_get_link_xdp_id and > > >> bpf_set_link_xdp_fd > > > functions. Use meson to detect if libbpf >= v0.7.0 is linked and if > > > so, > > use > > > the recommended replacement functions b

RE: [PATCH] net/af_xdp: fix custom program loading with multiple queues

2022-03-10 Thread Loftus, Ciara
> When the PMD is configured to load a custom XDP program, it sets > XSK_LIBBPF_FLAGS__INHIBIT_PROG_LOAD flag to prevent libbpf from > loading its default XDP program. However, when queue_count is set to > greater than 1, this flag is only set for the first XSK socket but not > for subsequent XSK s

RE: [PATCH] net/af_xdp: fix custom program loading with multiple queues

2022-03-11 Thread Loftus, Ciara
> > 10/03/2022 09:49, Loftus, Ciara: > > > When the PMD is configured to load a custom XDP program, it sets > > > XSK_LIBBPF_FLAGS__INHIBIT_PROG_LOAD flag to prevent libbpf from > > > loading its default XDP program. However, when queue_count is set to > >

Re: [dpdk-dev] About : Enable optional dequeue zero copy for vHost User

2018-01-23 Thread Loftus, Ciara
...@corp.netease.com] Sent: Wednesday, January 17, 2018 10:41 AM To: Loftus, Ciara Cc: dev@dpdk.org Subject: About : Enable optional dequeue zero copy for vHost User Hi Ciara, I am tesing the function of "vHost dequeue zero copy" for vm2vm on a host, and I have some problems: 1. The net

Re: [dpdk-dev] [PATCH] vhost: fix dequeue zero copy not work with virtio1

2017-12-15 Thread Loftus, Ciara
> > Hi Junjie, > > On 12/13/2017 05:50 PM, Junjie Chen wrote: > > This fix dequeue zero copy can not work with Qemu > > version >= 2.7. Since from Qemu 2.7 virtio device > > use virtio-1 protocol, the zero copy code path > > forget to add offset to buffer address. > > > > Signed-off-by: Junjie Ch

Re: [dpdk-dev] [PATCH] vhost: support rx_queue_count

2017-05-23 Thread Loftus, Ciara
> > This patch implements the ops rx_queue_count for vhost PMD by adding > a helper function rte_vhost_rx_queue_count in vhost lib. > > The ops ops rx_queue_count gets vhost RX queue avail count and helps > to understand the queue fill level. > > Signed-off-by: Zhihong Wang > --- > drivers/net

Re: [dpdk-dev] [PATCH] vhost: fix crash on NUMA

2017-06-02 Thread Loftus, Ciara
> The queue allocation was changed, from allocating one queue-pair at a > time to one queue at a time. Most of the changes have been done, but > just with one being missed: the size of coping the old queue is still > based on queue-pair at numa_realloc(), which leads to overwritten issue. > As a re

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

2017-04-27 Thread Loftus, Ciara
SER_PROTOCOL_F_REPLY_ACK has to be disabled 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. Once Qemu bugs > have been fixed and upstreamed, we can enable it. > > Fixes: 73c8f9f69c6c(&quo

[dpdk-dev] [PATCH] net/vhost: Add function to retreive the 'vid' for a given port id

2016-09-23 Thread Loftus, Ciara
> On Fri, Sep 23, 2016 at 10:43:20AM +0200, Thomas Monjalon wrote: > > 2016-09-23 12:26, Yuanhan Liu: > > > On Thu, Sep 22, 2016 at 06:43:55PM +0200, Thomas Monjalon wrote: > > > > > > > > > > There could be a similar need in other PMD. > > > > > > > > > > If we can get an opaque identifier of the

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

2016-09-29 Thread Loftus, Ciara
> > This feature adds vhost pmd extended statistics from per port perspective > in order to meet the requirements of the applications such as OVS etc. > > The statistics counters are based on RFC 2819 and RFC 2863 as follows: > > rx/tx_good_packets > rx/tx_total_bytes > rx/tx_missed_pkts > rx/tx

RE: [v7 1/1] net/af_xdp: fix multi interface support for K8s

2024-02-09 Thread Loftus, Ciara
> > On 1/11/2024 2:21 PM, Ferruh Yigit wrote: > > On 1/11/2024 12:21 PM, Maryam Tahhan wrote: > >> On 11/01/2024 11:35, Ferruh Yigit wrote: > >>> Devarg is user interface, changing it impacts the user. > >>> > >>> Assume that user of '22.11.3' using 'use_cni' dev_arg, it will be broken > >>> when

RE: [PATCH] net/af_xdp: fix resources leak when xsk configure fails

2024-02-22 Thread Loftus, Ciara
> > On 22/02/2024 03:07, Yunjian Wang wrote: > In xdp_umem_configure() allocated some resources for the > xsk umem, we should delete them when xsk configure fails, > otherwise it will lead to resources leak. > > Fixes: f1debd77efaf ("net/af_xdp: introduce AF_XDP PMD") > Cc: mailto:sta...@dpdk.org

RE: [v9 2/3] net/af_xdp: fix multi interface support for K8s

2024-02-22 Thread Loftus, Ciara
> Subject: [v9 2/3] net/af_xdp: fix multi interface support for K8s > > The original 'use_cni' implementation, was added > to enable support for the AF_XDP PMD in a K8s env > without any escalated privileges. > However 'use_cni' used a hardcoded socket rather > than a configurable one. If a DPDK p

RE: [PATCH v2] net/af_xdp: fix resources leak when xsk configure fails

2024-02-22 Thread Loftus, Ciara
> Subject: [PATCH v2] net/af_xdp: fix resources leak when xsk configure fails > > In xdp_umem_configure() allocated some resources for the > xsk umem, we should delete them when xsk configure fails, > otherwise it will lead to resources leak. > > Fixes: f1debd77efaf ("net/af_xdp: introduce AF_XDP

RE: [PATCH] tap: fix build of tap_bpf_program

2023-07-26 Thread Loftus, Ciara
> On 7/20/2023 8:45 AM, Ferruh Yigit wrote: > > On 7/19/2023 5:12 PM, Stephen Hemminger wrote: > >> On Wed, 19 Jul 2023 11:03:36 +0100 > >> Ferruh Yigit wrote: > >> > >>> On 7/19/2023 11:00 AM, Ferruh Yigit wrote: > On 7/17/2023 8:15 PM, Stephen Hemminger wrote: > > The tap_bpf_program.c

RE: [PATCH v2] net/af_xdp: fix memzone leak in error path

2023-12-05 Thread Loftus, Ciara
> > In xdp_umem_configure() allocated memzone for the 'umem', we should > free it when xsk_umem__create() call fails, otherwise it will lead to > memory zone leak. To fix it move 'umem->mz = mz;' assignment after > 'mz == NULL' check. > > Fixes: f1debd77efaf ("net/af_xdp: introduce AF_XDP PMD") >

RE: [v2] net/af_xdp: enable a sock path alongside use_cni

2023-12-05 Thread Loftus, Ciara
> -Original Message- > From: Maryam Tahhan > Sent: Monday, December 4, 2023 10:31 AM > To: ferruh.yi...@amd.com; step...@networkplumber.org; > lihuis...@huawei.com; fengcheng...@huawei.com; > liuyongl...@huawei.com; Koikkara Reeny, Shibin > > Cc: dev@dpdk.org; Tahhan, Maryam > Subject

RE: [v4] net/af_xdp: enable uds_path instead of use_cni

2023-12-15 Thread Loftus, Ciara
Thanks for the latest patch Maryam. I have one minor suggestion inline. Also, there are still some references to "use_cni" in af_xdp.rst which should be removed/replaced with uds_path. Once that's done I think the patch should be good to go. Perhaps also consider adding a note to the release note

RE: [v5] net/af_xdp: enable uds_path instead of use_cni

2023-12-15 Thread Loftus, Ciara
> > With the original 'use_cni' implementation, (using a > hardcoded socket rather than a configurable one), > if a DPDK pod is requesting multiple net devices > and these devices are from different pools, then > the container attempts to mount all the netdev UDSes > in the pod as /tmp/afxdp.sock.

Re: [dpdk-dev] [PATCH] net/af_xdp: use bpf link for XDP programs

2021-10-22 Thread Loftus, Ciara
> > On 10/14/2021 10:50 AM, Ciara Loftus wrote: > > --- a/drivers/net/af_xdp/compat.h > > +++ b/drivers/net/af_xdp/compat.h > > @@ -2,9 +2,11 @@ > >* Copyright(c) 2020 Intel Corporation. > >*/ > > > > +#include > > #include > > #include > > #include > > +#include > > Hi Ciara,

Re: [dpdk-dev] [PATCH v1 2/7] net/af_xdp: add power monitor support

2021-06-02 Thread Loftus, Ciara
> Subject: [PATCH v1 2/7] net/af_xdp: add power monitor support > > Implement support for .get_monitor_addr in AF_XDP driver. > > Signed-off-by: Anatoly Burakov Thanks Anatoly. LGTM. Acked-by: Ciara Loftus > --- > drivers/net/af_xdp/rte_eth_af_xdp.c | 25 + > 1 file

Re: [dpdk-dev] dpdk-stable-20.11.1 compile with all examples fails on openSUSE Leap 15.3

2021-06-23 Thread Loftus, Ciara
> > Hi, > > I can't compile dpdk-stable-20.11.1 on openSUSE Leap 15.3. > > kernel: 5.3.18-59.5-default > > Configuration options: > Message: > = > Libraries Enabled > = > > libs: > kvargs, telemetry, eal, ring, rcu, mempool, mbuf, net, > meter, e

Re: [dpdk-dev] [PATCH] net/af_xdp: fix support of secondary process

2021-09-20 Thread Loftus, Ciara
> -Original Message- > From: dev On Behalf Of Stephen Hemminger > Sent: Friday 3 September 2021 17:15 > To: dev@dpdk.org > Cc: Stephen Hemminger ; > sta...@dpdk.org; xiaolong...@intel.com > Subject: [dpdk-dev] [PATCH] net/af_xdp: fix support of secondary process > > Doing basic operati

Re: [dpdk-dev] [PATCH] net/af_xdp: fix support of secondary process

2021-09-20 Thread Loftus, Ciara
> > On Mon, 20 Sep 2021 13:23:57 +0000 > "Loftus, Ciara" wrote: > > > > -Original Message- > > > From: dev On Behalf Of Stephen Hemminger > > > Sent: Friday 3 September 2021 17:15 > > > To: dev@dpdk.org > > > C

Re: [dpdk-dev] [PATCH] net/af_xdp: fix support of secondary process

2021-09-23 Thread Loftus, Ciara
> > > > > > > > > > Doing basic operations like info_get or get_stats was broken > > > > > in af_xdp PMD. The info_get would crash because dev->device > > > > > was NULL in secondary process. Fix this by doing same initialization > > > > > as af_packet and tap devices. > > > > > > > > > > The get_s

Re: [dpdk-dev] [21.08 PATCH v1 2/2] net/af_xdp: add power monitor support

2021-05-12 Thread Loftus, Ciara
> > Implement support for .get_monitor_addr in AF_XDP driver. > > Signed-off-by: Anatoly Burakov > --- > drivers/net/af_xdp/rte_eth_af_xdp.c | 52 - > 1 file changed, 37 insertions(+), 15 deletions(-) > > diff --git a/drivers/net/af_xdp/rte_eth_af_xdp.c > b/drivers/

Re: [dpdk-dev] [PATCH] net/af_xdp: fix zero copy Tx queue drain

2021-08-25 Thread Loftus, Ciara
> > Call xsk_ring_prod__submit() before kick_tx() so that the kernel > consumer sees the updated state of Tx ring. Otherwise, Tx packets are > stuck in the ring until the next call to af_xdp_tx_zc(). > > Fixes: d8a210774e1d ("net/af_xdp: support unaligned umem chunks") > Cc: sta...@dpdk.org > >

Re: [dpdk-dev] [PATCH v2] net/af_xdp: fix zero copy Tx queue drain

2021-08-25 Thread Loftus, Ciara
> > Call xsk_ring_prod__submit() before kick_tx() so that the kernel > consumer sees the updated state of Tx ring. Otherwise, Tx packets are > stuck in the ring until the next call to af_xdp_tx_zc(). > > Fixes: d8a210774e1d ("net/af_xdp: support unaligned umem chunks") > Cc: sta...@dpdk.org > >

[dpdk-dev] [RFC PATCH v2] vhost: Add VHOST PMD

2015-10-19 Thread Loftus, Ciara
> On 2015/10/16 21:52, Bruce Richardson wrote: > > On Mon, Aug 31, 2015 at 12:55:26PM +0900, Tetsuya Mukawa wrote: > >> The patch introduces a new PMD. This PMD is implemented as thin > wrapper > >> of librte_vhost. It means librte_vhost is also needed to compile the PMD. > >> The PMD can have 'ifa

[dpdk-dev] [RFC PATCH v2] vhost: Add VHOST PMD

2015-10-20 Thread Loftus, Ciara
> > On 2015/09/24 2:47, Loftus, Ciara wrote: > >> The patch introduces a new PMD. This PMD is implemented as thin > wrapper > >> of librte_vhost. It means librte_vhost is also needed to compile the PMD. > >> The PMD can have 'iface' parameter like

[dpdk-dev] [PATCH 2/3] vhost: Add callback and private data for vhost PMD

2015-10-30 Thread Loftus, Ciara
> > These variables are needed to be able to manage one of virtio devices > using both vhost library APIs and vhost PMD. > For example, if vhost PMD uses current callback handler and private data > provided by vhost library, A DPDK application that links vhost library > cannot use some of vhost li

[dpdk-dev] [RFC] librte_vhost: Add unix domain socket fd registration

2016-06-24 Thread Loftus, Ciara
> > On Tue, Jun 21, 2016 at 09:15:03AM -0400, Aaron Conole wrote: > > Yuanhan Liu writes: > > > > > On Fri, Jun 17, 2016 at 11:32:36AM -0400, Aaron Conole wrote: > > >> Prior to this commit, the only way to add a vhost-user socket to the > > >> system is by relying on librte_vhost to open the uni

[dpdk-dev] [PATCH] vhost: fix missing flag reset on stop

2016-06-29 Thread Loftus, Ciara
> > Commit 550c9d27d143 ("vhost: set/reset device flags internally") moves > the VIRTIO_DEV_RUNNING set/reset to vhost lib. But I missed one reset > on stop; here fixes it. > > Fixes: 550c9d27d143 ("vhost: set/reset device flags internally") > >

[dpdk-dev] [PATCH v2 3/3] vhost: fix vq realloc at numa_realloc

2016-03-07 Thread Loftus, Ciara
> > vq is allocated on pairs, hence we should do pair reallocation > at numa_realloc() as well, otherwise an error like following > occurs while do numa reallocation: > > VHOST_CONFIG: reallocate vq from 0 to 1 node > PANIC in rte_free(): > Fatal error: Invalid memory > > The reason

[dpdk-dev] [PATCH] doc: deprecate vhost-cuse

2016-07-21 Thread Loftus, Ciara
> Subject: [dpdk-dev] [PATCH] doc: deprecate vhost-cuse > > Vhost-cuse was invented before vhost-user exist. The both are actually > doing the same thing: a vhost-net implementation in user space. But they > are not exactly the same thing. > > Firstly, vhost-cuse is harder for use; no one seems t

[dpdk-dev] [PATCH v13 2/2] vhost: Add VHOST PMD

2016-03-21 Thread Loftus, Ciara
Hi Tetsuya, Thanks for the patches. Just one query below re max queue numbers. Thanks, Ciara > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Tetsuya Mukawa > Sent: Monday, March 21, 2016 5:45 AM > To: dev at dpdk.org > Cc: Richardson, Bruce ; > ann.zhuangya

[dpdk-dev] [PATCH v13 2/2] vhost: Add VHOST PMD

2016-03-22 Thread Loftus, Ciara
> > On 2016/03/22 10:55, Tetsuya Mukawa wrote: > > On 2016/03/22 0:40, Loftus, Ciara wrote: > >>> + > >>> +static void > >>> +eth_dev_info(struct rte_eth_dev *dev, > >>> + struct rte_eth_dev_info *dev_info) > >>> +{ >

[dpdk-dev] [PATCH] vhost PMD: Fix wrong handling of maximum value of rx/tx queues

2016-03-22 Thread Loftus, Ciara
> > Currently, the maximum value of rx/tx queueus are kept by EAL. But, > the value are used like below different meanings in vhost PMD. > - The maximum value of current enabled queues. > - The maximum value of current supported queues. > > This wrong double meaning will cause an issue like bel

[dpdk-dev] [PATCH 3/6] vhost: add reconnect ability

2016-05-10 Thread Loftus, Ciara
> On Tue, May 10, 2016 at 09:00:45AM +, Xie, Huawei wrote: > > On 5/10/2016 4:42 PM, Michael S. Tsirkin wrote: > > > On Tue, May 10, 2016 at 08:07:00AM +, Xie, Huawei wrote: > > >> On 5/10/2016 3:56 PM, Michael S. Tsirkin wrote: > > >>> On Tue, May 10, 2016 at 07:24:10AM +, Xie, Huawei

[dpdk-dev] [RFC PATCH] vhost: Add VHOST PMD

2015-09-15 Thread Loftus, Ciara
> + > +static int > +rte_pmd_vhost_devinit(const char *name, const char *params) > +{ > + struct rte_kvargs *kvlist = NULL; > + int ret = 0; > + int index; > + char *iface_name; > + > + RTE_LOG(INFO, PMD, "Initializing pmd_vhost for %s\n", name); > + > + kvlist = rte_kvargs_

[dpdk-dev] [RFC PATCH] vhost: Add VHOST PMD

2015-09-17 Thread Loftus, Ciara
> > On 2015/09/16 1:27, Loftus, Ciara wrote: > >> + > >> +static int > >> +rte_pmd_vhost_devinit(const char *name, const char *params) > >> +{ > >> + struct rte_kvargs *kvlist = NULL; > >> + int ret = 0; > >> +

[dpdk-dev] [RFC PATCH v2] vhost: Add VHOST PMD

2015-09-23 Thread Loftus, Ciara
> The patch introduces a new PMD. This PMD is implemented as thin wrapper > of librte_vhost. It means librte_vhost is also needed to compile the PMD. > The PMD can have 'iface' parameter like below to specify a path to connect > to a virtio-net device. > > $ ./testpmd -c f -n 4 --vdev 'eth_vhost0,

[dpdk-dev] [PATCH] vhost: Fix retrieval of numa information in PMD

2016-04-06 Thread Loftus, Ciara
> > On Wed, Apr 06, 2016 at 03:49:25PM +0900, Tetsuya Mukawa wrote: > > On 2016/04/06 1:09, Ciara Loftus wrote: > > > After some testing, it was found that retrieving numa information > > > about a vhost device via a call to get_mempolicy is more > > > accurate when performed during the new_device

[dpdk-dev] [PATCH] vhost: call rte_vhost_enable_guest_notification only on enabled queues

2016-04-07 Thread Loftus, Ciara
> > If the vhost PMD were configured with more queues than the guest, the old > code would segfault in rte_vhost_enable_guest_notification due to a NULL > virtqueue pointer. > > Fixes: ee584e9710b9 ("vhost: add driver on top of the library") > Signed-off-by: Rich Lane > --- > drivers/net/vhost/

[dpdk-dev] [PATCH] vhost: call rte_vhost_enable_guest_notification only on enabled queues

2016-04-07 Thread Loftus, Ciara
> On 4/7/2016 8:29 AM, Rich Lane wrote: > > If the vhost PMD were configured with more queues than the guest, the > old > > code would segfault in rte_vhost_enable_guest_notification due to a NULL > > virtqueue pointer. > > > > Fixes: ee584e9710b9 ("vhost: add driver on top of the library") > > Sig

[dpdk-dev] [PATCH] vhost: provide vhost API to unregister vhost unix domain socket

2015-06-03 Thread Loftus, Ciara
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Huawei Xie > Sent: Tuesday, June 02, 2015 2:50 AM > To: dev at dpdk.org > Cc: Sun, Peng A > Subject: [dpdk-dev] [PATCH] vhost: provide vhost API to unregister vhost > unix domain socket > > rte_vhost_driver_unre

[dpdk-dev] [PATCH v2] vhost: provide vhost API to unregister vhost unix domain socket

2015-06-05 Thread Loftus, Ciara
> -Original Message- > From: Xie, Huawei > Sent: Friday, June 05, 2015 4:26 AM > To: dev at dpdk.org > Cc: Loftus, Ciara; Xie, Huawei; Sun, Peng A > Subject: [PATCH v2] vhost: provide vhost API to unregister vhost unix domain > socket > > rte_vhost_driver

Re: [dpdk-dev] [PATCH v3] net/i40e: fix TSO pkt exceeds allowed buf size issue

2020-01-02 Thread Loftus, Ciara
> > -Original Message- > > From: Li, Xiaoyun > > Sent: Thursday, December 26, 2019 2:46 PM > > To: Zhang, Qi Z ; Xing, Beilei > ; > > Loftus, Ciara ; dev@dpdk.org > > Cc: Li, Xiaoyun ; sta...@dpdk.org > > Subject: [PATCH v3] net/i40

Re: [dpdk-dev] [PATCH] net/af_xdp: fix redundant check for NEED WAKEUP

2020-01-08 Thread Loftus, Ciara
> > Function kick_tx() has built-in detection on NEED_WAKEUP flag, so just > call it directly, like elsewhere in the driver. > > Fixes: d8a210774e1d ("net/af_xdp: support unaligned umem chunks") > Cc: sta...@dpdk.org > > Signed-off-by: Xiao Wang > --- > drivers/net/af_xdp/rte_eth_af_xdp.c | 5

Re: [dpdk-dev] [PATCH] net/af_xdp: use single-prod-and-cons ring

2020-01-13 Thread Loftus, Ciara
> > The ring is used only by af_xdp pmd itself, so no need to support > multi-producer and multi-consumer mode. This patch changes the ring > to single-producer and single-consumer mode, which could yield better > performance for addr enqueue and dequeue. > > Signed-off-by: Xiao Wang LGTM. I r

Re: [dpdk-dev] [PATCH v2 2/3] net/af_xdp: support pinning of IRQs

2019-10-21 Thread Loftus, Ciara
> > On Mon, 30 Sep 2019 16:42:04 + > > Ciara Loftus wrote: > > > > > +/* drivers supported for the queue_irq option */ > > > +enum supported_drivers { > > > + I40E_DRIVER, > > > + IXGBE_DRIVER, > > > + MLX5_DRIVER, > > > + NUM_DRIVERS > > > +}; > > > > Anything device specific like this raises

Re: [dpdk-dev] [PATCH] net/af_xdp: fix Tx halt when no recv packets

2019-09-17 Thread Loftus, Ciara
> > The kernel only consumes Tx packets if we have some Rx traffic on specified > queue or we have called send(). So we need to issue a send() even when the > allocation fails so that kernel will start to consume packets again. > > Commit 45bba02c95b0 ("net/af_xdp: support need wakeup feature") b

Re: [dpdk-dev] [PATCH 2/3] net/af_xdp: support pinning of IRQs

2019-09-27 Thread Loftus, Ciara
[snip] > >+ > >+static void > >+configure_irqs(struct pmd_internals *internals, uint16_t rx_queue_id) > >+{ > >+int coreid = internals->queue_irqs[rx_queue_id]; > >+char driver[NAME_MAX]; > >+uint16_t netdev_qid = rx_queue_id + internals->start_queue_idx; > >+regex_t r; > >+int

Re: [dpdk-dev] [PATCH v2 2/3] net/af_xdp: support pinning of IRQs

2019-10-03 Thread Loftus, Ciara
> -Original Message- > From: Stephen Hemminger > Sent: Monday 30 September 2019 18:12 > To: Loftus, Ciara > Cc: dev@dpdk.org; Ye, Xiaolong ; Laatz, Kevin > ; Richardson, Bruce > Subject: Re: [dpdk-dev] [PATCH v2 2/3] net/af_xdp: support pinning of IRQs > >

Re: [dpdk-dev] [PATCH] net/af_xdp: enable support for unaligned umem chunks

2019-08-30 Thread Loftus, Ciara
> > This patch enables the unaligned chunks feature for AF_XDP which allows > chunks to be placed at arbitrary places in the umem, as opposed to them > being required to be aligned to 2k. This allows for DPDK application > mempools to be mapped directly into the umem and in turn enable zero > copy

Re: [dpdk-dev] [PATCH] net/af_xdp: enable support for unaligned umem chunks

2019-09-02 Thread Loftus, Ciara
> Hi Ciara, > > I haven't tried this patch but have a question. > > On Thu, Aug 29, 2019 at 8:04 AM Ciara Loftus wrote: > > > > This patch enables the unaligned chunks feature for AF_XDP which > > allows chunks to be placed at arbitrary places in the umem, as opposed > > to them being required t

Re: [dpdk-dev] [PATCH] net/af_xdp: enable support for unaligned umem chunks

2019-09-02 Thread Loftus, Ciara
> > Hi Ciara, > > > > I haven't tried this patch but have a question. > > > > On Thu, Aug 29, 2019 at 8:04 AM Ciara Loftus > wrote: > > > > > > This patch enables the unaligned chunks feature for AF_XDP which > > > allows chunks to be placed at arbitrary places in the umem, as > > > opposed to the

Re: [dpdk-dev] [PATCH] net/af_xdp: enable support for unaligned umem chunks

2019-09-04 Thread Loftus, Ciara
> > Hi, Ciara > > Thanks for the patch, the performance number is quite impressive. > > On 08/29, Ciara Loftus wrote: > >This patch enables the unaligned chunks feature for AF_XDP which allows > >chunks to be placed at arbitrary places in the umem, as opposed to them > >being required to be alig

RE: [v1 1/1] MAINTAINERS: add another AF_XDP maintainer

2024-04-25 Thread Loftus, Ciara
> Subject: [v1 1/1] MAINTAINERS: add another AF_XDP maintainer > > Add Maryam Tahhan as an additional maintainer for AF_XDP > PMD and it's documentation. > > Signed-off-by: Maryam Tahhan Thanks Maryam. Acked-by: Ciara Loftus > --- > MAINTAINERS | 1 + > 1 file changed, 1 insertion(+) > > d

RE: [v1 1/1] docs: af_xdp device plugin repo update

2024-04-26 Thread Loftus, Ciara
> Subject: [v1 1/1] docs: af_xdp device plugin repo update > > Fixup the references to the AF_XDP Device Plugin repo. > > Fixes: 66a2aca4f512 ("docs: fix AF_XDP device plugin howto") > Cc: sta...@dpdk.org > > Signed-off-by: Maryam Tahhan Acked-by: Ciara Loftus > --- > doc/guides/howto/af_xd

RE: [PATCH] net/af_xdp: fix umem map size for zero copy

2024-04-26 Thread Loftus, Ciara
> Subject: [PATCH] net/af_xdp: fix umem map size for zero copy > > The current calculation assumes that the mbufs are contiguous. However, > this assumption is incorrect when the memory spans across a huge page. > Correct to directly read the size from the mempool memory chunks. > > Signed-off-by

RE: [PATCH] net/af_xdp: fix umem map size for zero copy

2024-04-30 Thread Loftus, Ciara
> > > > > Subject: [PATCH] net/af_xdp: fix umem map size for zero copy > > > > > > The current calculation assumes that the mbufs are contiguous. > > > However, this assumption is incorrect when the memory spans across a > huge > > page. > > > Correct to directly read the size from the mempool memo

RE: [PATCH 2/3] net/af_xdp: Fix mbuf alloc failed statistic

2024-05-14 Thread Loftus, Ciara
> > On Fri, 10 May 2024 10:03:57 + > Ciara Loftus wrote: > > > diff --git a/drivers/net/af_xdp/rte_eth_af_xdp.c > b/drivers/net/af_xdp/rte_eth_af_xdp.c > > index fee0d5d5f3..968bbf6d45 100644 > > --- a/drivers/net/af_xdp/rte_eth_af_xdp.c > > +++ b/drivers/net/af_xdp/rte_eth_af_xdp.c > > @@ -

RE: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-17 Thread Loftus, Ciara
> > The current calculation assumes that the mbufs are contiguous. However, > this assumption is incorrect when the memory spans across a huge page. > Correct to directly read the size from the mempool memory chunks. > > Signed-off-by: Frank Du Hi Frank, Thanks for the patch. Before your patc

RE: [PATCH v6 4/9] net/af_xdp: use generic SW stats

2024-05-17 Thread Loftus, Ciara
> Subject: [PATCH v6 4/9] net/af_xdp: use generic SW stats > > Use common code for all SW stats. > > Signed-off-by: Stephen Hemminger > --- > drivers/net/af_xdp/rte_eth_af_xdp.c | 98 - > 1 file changed, 25 insertions(+), 73 deletions(-) > > diff --git a/drivers/net

RE: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-29 Thread Loftus, Ciara
> -Original Message- > From: Du, Frank > Sent: Thursday, May 23, 2024 8:56 AM > To: Morten Brørup ; Ferruh Yigit > ; dev@dpdk.org; Andrew Rybchenko > ; Burakov, Anatoly > > Cc: Loftus, Ciara > Subject: RE: [PATCH v2] net/af_xdp: fix umem map size for zer

RE: [v11 2/3] net/af_xdp: fix multi interface support for K8s

2024-03-01 Thread Loftus, Ciara
snip > @@ -1695,17 +1699,16 @@ xsk_configure(struct pmd_internals *internals, > struct pkt_rx_queue *rxq, > } > > if (internals->use_cni) { > - int err, fd, map_fd; > + int err, map_fd; > > - /* get socket fd from CNI plugin */ > - map_

RE: [v13 2/3] net/af_xdp: fix multi interface support for K8s

2024-04-08 Thread Loftus, Ciara
> > +#ifdef ETH_AF_XDP_UPDATE_XSKMAP > +static __rte_always_inline int > +update_xskmap(struct xsk_socket *xsk, int map_fd, int xsk_queue_idx > __rte_unused) > +{ > + return xsk_socket__update_xskmap(xsk, map_fd); > +} > +#else > +static __rte_always_inline int > +update_xskmap(struct xsk_so

RE: [v14 0/3] net/af_xdp: fix multi interface support for K8s

2024-04-08 Thread Loftus, Ciara
> > The original `use_cni` implementation was limited to > supporting only a single netdev in a DPDK pod. This patchset > aims to fix this limitation transparently to the end user. > It will also enable compatibility with the latest AF_XDP > Device Plugin. > > Signed-off-by: Maryam Tahhan Thank

Re: [dpdk-dev] [PATCH] net/af_xdp: optimisations to improve packet loss

2020-06-23 Thread Loftus, Ciara
> > On Fri, 12 Jun 2020 14:17:46 + > Ciara Loftus wrote: > > > This commit makes some changes to the AF_XDP PMD in an effort to > improve > > its packet loss characteristics. > > > > 1. In the case of failed transmission due to inability to reserve a tx > > descriptor, the PMD now pulls from

Re: [dpdk-dev] [PATCH] net/af_xdp: custom XDP program loading

2020-09-14 Thread Loftus, Ciara
> > > > The new 'xdp_prog=' vdev arg allows the user to specify the path > to > > a custom XDP program to be set on the device, instead of the default libbpf > > one. The program must have an XSK_MAP of name 'xsks_map' which will > > allow for the redirection of some packets to userspace and thus t

Re: [dpdk-dev] [PATCH 1/2] af_xdp: not return a negative value in af_xdp_rx_zc

2020-09-17 Thread Loftus, Ciara
> > af_xdp_rx_zc should always return the number of received packets, > and negative value will be as number of received packets, and > confuse the caller > > Fixes: d8a210774e1d ("net/af_xdp: support unaligned umem chunks") > Cc: sta...@dpdk.org > Signed-off-by: Li RongQing > --- > drivers/net

  1   2   >