Re: [dpdk-dev] [PATCH v6 2/2] bus/pci: support MMIO in PCI ioport accessors

2021-02-17 Thread David Marchand
On Fri, Jan 29, 2021 at 4:19 AM 谢华伟(此时此刻) wrote: > @@ -517,6 +525,60 @@ > } > #endif > > +static inline uint8_t ioread8(void *addr) > +{ > + uint8_t val; > + > + val = (uint64_t)(uintptr_t)addr >= PIO_MAX ? > + *(volatile uint8_t *)addr : > + inb((unsigned

[dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits

2021-02-17 Thread Juraj Linkeš
Since Travis added credits we need to make some changes to the Travis CI. Arm has a credit agreement with Travis for arm jobs, so remove x86 jobs. Also, migrate all arm jobs to the latest environment. Juraj Linkeš (2): ci: remove x86 Travis jobs ci: update arm64 Travis jobs to Graviton2 .tra

[dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs

2021-02-17 Thread Juraj Linkeš
Remove x86 jobs for which we don't have Travis credits. Leave arm64 jobs for which we do have the credits. Signed-off-by: Juraj Linkeš --- .travis.yml | 87 - 1 file changed, 87 deletions(-) diff --git a/.travis.yml b/.travis.yml index 6a21812

[dpdk-dev] [RFC PATCH v1 2/2] ci: update arm64 Travis jobs to Graviton2

2021-02-17 Thread Juraj Linkeš
Instead of using the older platform, use only the newer Graviton2 environment in Travis CI. Signed-off-by: Juraj Linkeš --- .travis.yml | 26 -- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/.travis.yml b/.travis.yml index 899e0cd605..fb3913ed20 100644 -

Re: [dpdk-dev] [PATCH v4] doc: add new tables for rte flow items and actions support

2021-02-17 Thread Ferruh Yigit
On 2/17/2021 5:57 AM, Asaf Penso wrote: Hello Ferruh, thanks for the reply and please see my below comments. Regards, Asaf Penso -Original Message- From: Ferruh Yigit Sent: Tuesday, February 16, 2021 3:14 PM To: Asaf Penso ; dev@dpdk.org Cc: NBU-Contact-Thomas Monjalon ; Andrew Rybche

Re: [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs

2021-02-17 Thread Thomas Monjalon
17/02/2021 10:42, Juraj Linkeš: > Remove x86 jobs for which we don't have Travis credits. Leave arm64 jobs > for which we do have the credits. We are missing some references.

Re: [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits

2021-02-17 Thread Ferruh Yigit
On 2/17/2021 9:42 AM, Juraj Linkeš wrote: Since Travis added credits we need to make some changes to the Travis CI. Arm has a credit agreement with Travis for arm jobs, so remove x86 jobs. Also, migrate all arm jobs to the latest environment. So this is becoming supporting a CI infrastructure

Re: [dpdk-dev] [PATCH v4] doc: add new tables for rte flow items and actions support

2021-02-17 Thread Thomas Monjalon
17/02/2021 11:37, Ferruh Yigit: > On 2/17/2021 5:57 AM, Asaf Penso wrote: > > From: Ferruh Yigit > >> On 2/7/2021 10:52 AM, Asaf Penso wrote: > >>> In http://doc.dpdk.org/guides/nics/overview.html, table 1.1 lists all > >>> supported features. > >>> It has a single line for "Flow API" that refers

Re: [dpdk-dev] [PATCH] eal: support using 0 as coremask for no-affinitization

2021-02-17 Thread Burakov, Anatoly
On 16-Feb-21 5:44 PM, Van Haaren, Harry wrote: -Original Message- From: Bruce Richardson Sent: Tuesday, February 16, 2021 5:31 PM To: Van Haaren, Harry Cc: Burakov, Anatoly ; dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH] eal: support using 0 as coremask for no- affinitization On Tue, Fe

Re: [dpdk-dev] [PATCH] eal: support using 0 as coremask for no-affinitization

2021-02-17 Thread Van Haaren, Harry
> -Original Message- > From: Burakov, Anatoly > Sent: Wednesday, February 17, 2021 12:09 PM > To: Van Haaren, Harry ; Richardson, Bruce > > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH] eal: support using 0 as coremask for no- > affinitization > > On 16-Feb-21 5:44 PM, Van Haaren, H

Re: [dpdk-dev] [PATCH] eal: support using 0 as coremask for no-affinitization

2021-02-17 Thread Bruce Richardson
On Wed, Feb 17, 2021 at 12:14:36PM +, Van Haaren, Harry wrote: > > -Original Message- > > From: Burakov, Anatoly > > Sent: Wednesday, February 17, 2021 12:09 PM > > To: Van Haaren, Harry ; Richardson, Bruce > > > > Cc: dev@dpdk.org > > Subject: Re: [dpdk-dev] [PATCH] eal: support usin

Re: [dpdk-dev] [PATCH] eal: support using 0 as coremask for no-affinitization

2021-02-17 Thread Burakov, Anatoly
On 17-Feb-21 1:26 PM, Bruce Richardson wrote: On Wed, Feb 17, 2021 at 12:14:36PM +, Van Haaren, Harry wrote: -Original Message- From: Burakov, Anatoly Sent: Wednesday, February 17, 2021 12:09 PM To: Van Haaren, Harry ; Richardson, Bruce Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PAT

Re: [dpdk-dev] [PATCH 1/2] ethdev: add flow action type update as an offload

2021-02-17 Thread Ferruh Yigit
On 7/3/2020 3:34 PM, Ferruh Yigit wrote: On 11/19/2019 11:09 AM, Thomas Monjalon wrote: 19/11/2019 11:59, Andrew Rybchenko: On 11/19/19 12:50 PM, Thomas Monjalon wrote: 19/11/2019 10:24, Andrew Rybchenko: On 11/8/19 4:30 PM, Thomas Monjalon wrote: 08/11/2019 14:27, Andrew Rybchenko: On 11/8

Re: [dpdk-dev] [PATCH v2] usertools/dpdk-telemetry: print name of app when connected

2021-02-17 Thread Kevin Laatz
On 16/02/2021 11:39, Bruce Richardson wrote: When the dpdk-telemetry client connects to a DPDK instance, we can use the PID provided in the initial connection message to query from /proc the name of the process we are connected to, and display that to the user. We use the "cmdline" procfs entry f

Re: [dpdk-dev] [PATCH 1/2] ethdev: add flow action type update as an offload

2021-02-17 Thread Thomas Monjalon
17/02/2021 14:45, Ferruh Yigit: > On 7/3/2020 3:34 PM, Ferruh Yigit wrote: > > On 11/19/2019 11:09 AM, Thomas Monjalon wrote: > >> 19/11/2019 11:59, Andrew Rybchenko: > >>> On 11/19/19 12:50 PM, Thomas Monjalon wrote: > 19/11/2019 10:24, Andrew Rybchenko: > > On 11/8/19 4:30 PM, Thomas Mon

Re: [dpdk-dev] [PATCH v6 2/2] bus/pci: support MMIO in PCI ioport accessors

2021-02-17 Thread 谢华伟(此时此刻)
On 2021/2/17 17:06, David Marchand wrote: On Fri, Jan 29, 2021 at 4:19 AM 谢华伟(此时此刻) wrote: @@ -517,6 +525,60 @@ } #endif +static inline uint8_t ioread8(void *addr) +{ + uint8_t val; + + val = (uint64_t)(uintptr_t)addr >= PIO_MAX ? + *(volatile uint8_t *)addr : +

[dpdk-dev] [Bug 634] i40e: rte_flow doesn't allow RSS action on IPV4 only

2021-02-17 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=634 Frédéric Coiffier (frederic.coiff...@6cure.com) changed: What|Removed |Added Resolution|--- |INVALID

Re: [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs

2021-02-17 Thread Aaron Conole
Thomas Monjalon writes: > 17/02/2021 10:42, Juraj Linkeš: >> Remove x86 jobs for which we don't have Travis credits. Leave arm64 jobs >> for which we do have the credits. > > We are missing some references. I do see that the travis job is still running, and does include the x86 builds as well...

Re: [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits

2021-02-17 Thread Aaron Conole
Ferruh Yigit writes: > On 2/17/2021 9:42 AM, Juraj Linkeš wrote: >> Since Travis added credits we need to make some changes to the Travis >> CI. Arm has a credit agreement with Travis for arm jobs, so remove x86 >> jobs. Also, migrate all arm jobs to the latest environment. >> > > So this is beco

Re: [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs

2021-02-17 Thread Aaron Conole
Juraj Linkeš writes: > Remove x86 jobs for which we don't have Travis credits. Leave arm64 jobs > for which we do have the credits. > > Signed-off-by: Juraj Linkeš > --- Individual developers will still derive benefit from those jobs, yes? I thought it was possible to use travis still for indi

Re: [dpdk-dev] [dpdklab] Re: [dpdk-ci] [CI] SPDK compilation failures @ DPDK community lab

2021-02-17 Thread Zawadzki, Tomasz
Hi folks, Patch in question is now part of v21.01.x branch and tests in UNH lab can make use of it. https://github.com/spdk/spdk/commit/14a669010247fa8d6c7ce53b2e69fcfaa858 If possible, removing '--without-vhost' would be best going forward. Then SPDK compilation tests would cover rte_vhost

Re: [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits

2021-02-17 Thread Honnappa Nagarahalli
> Subject: Re: [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after > credits > > Ferruh Yigit writes: > > > On 2/17/2021 9:42 AM, Juraj Linkeš wrote: > >> Since Travis added credits we need to make some changes to the Travis > >> CI. Arm has a credit agreement with Travis for arm jobs, s

[dpdk-dev] [PATCH v3] net/bonding: fix LACP system address check

2021-02-17 Thread Ferruh Yigit
From: Vadim Podovinnikov In bond (LACP) we have several NICs (ports), when we have negotiation with peer about what port we prefer, we send information about what system we preferred in partner system name field. Peer also sends us what partner system name it prefer. When we receive a message fr

Re: [dpdk-dev] [PATCH v3] net/bonding: fix LACP system address check

2021-02-17 Thread Ferruh Yigit
On 2/17/2021 4:26 PM, Ferruh Yigit wrote: From: Vadim Podovinnikov In bond (LACP) we have several NICs (ports), when we have negotiation with peer about what port we prefer, we send information about what system we preferred in partner system name field. Peer also sends us what partner system n

Re: [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs

2021-02-17 Thread Honnappa Nagarahalli
> > Thomas Monjalon writes: > > > 17/02/2021 10:42, Juraj Linkeš: > >> Remove x86 jobs for which we don't have Travis credits. Leave arm64 > >> jobs for which we do have the credits. > > > > We are missing some references. > > I do see that the travis job is still running, and does include th

Re: [dpdk-dev] [RFC PATCH v1 1/2] ci: remove x86 Travis jobs

2021-02-17 Thread Honnappa Nagarahalli
> > Juraj Linkeš writes: > > > Remove x86 jobs for which we don't have Travis credits. Leave arm64 > > jobs for which we do have the credits. > > > > Signed-off-by: Juraj Linkeš > > --- > > Individual developers will still derive benefit from those jobs, yes? I > thought > it was possible

Re: [dpdk-dev] [PATCH] ethdev: add security flow item

2021-02-17 Thread Ferruh Yigit
On 9/24/2020 11:07 AM, Tejasree Kondoj wrote: Hi Ori, Please see inline. Thanks, Tejasree -Original Message- From: Ori Kam Sent: Thursday, September 24, 2020 3:22 PM To: Tejasree Kondoj ; Asaf Penso ; Stephen Hemminger Cc: Akhil Goyal ; Radu Nicolau ; Declan Doherty ; NBU-Contact-Th

Re: [dpdk-dev] [PATCH] net/ring: support secondary process

2021-02-17 Thread Ferruh Yigit
On 9/30/2020 12:02 PM, Ferruh Yigit wrote: Also add missing .ini feature list documentation and document some existing features including new "Multiprocess aware" feature. Signed-off-by: Ferruh Yigit --- Cc: Kevin Laatz Hi Bruce, Reminder of this patch waiting for review. Thanks, ferruh

[dpdk-dev] Coverity Defect in DPDK

2021-02-17 Thread Mcnamara, John
Hi, There are a large number of open static code analysis defects in our DPDK Coverity Scan database, see the list below. Please review them. If the defect identified isn't a real issue you can edit the defect online and change the defect "Classification" to "False Positive" or "Intentional" and

[dpdk-dev] [PATCH 19.11 v2] net/mlx5: fix storing the synched MAC to internal table

2021-02-17 Thread Dey, Souvik
From: Souvik Dey [ upstream commit 493f0bb51c1144eedcff2bba199cab1b64ff9fd0 ] As the internal MAC table is divided into Unicast and Multicast address sections, we should check the type of synched MAC address before storing it to the internal table. Currently the check is not done, and the synche

Re: [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits

2021-02-17 Thread Honnappa Nagarahalli
> Subject: [RFC PATCH v1 0/2] Travis CI jobs update after credits > > Since Travis added credits we need to make some changes to the Travis CI. Arm > has a credit agreement with Travis for arm jobs, so remove x86 jobs. Also, I think there is a mis-understanding here. The credits are for the ovsr

[dpdk-dev] [PATCH] app/testpmd: remove unnecessary tunnel UDP cmd check

2021-02-17 Thread Xiaoyun Li
cmd_tunnel_udp_config checked 'cmd' to set prot_type but this cmd is only for rx_vxlan_port. The unnecessary cmd check will cause uninit coverity issue. So remove it and rename 'cmd' to 'rx_vxlan_port'. Coverity issue: 366155 Fixes: bd948f20d609 ("app/testpmd: VXLAN packet identification") Cc: sta

[dpdk-dev] [PATCH] bus/pci/windows: support for PCI scan allowed and blocked lists

2021-02-17 Thread Khoa To
From: Khoa To EAL -a and -b options are used to specify which PCI devices are explicitly allowed or blocked during PCI bus scan. This evaluation is missing in the Windows implementation of rte_pci_scan. This patch provides this missing functionality, so that apps can specify which NetUIO devi

[dpdk-dev] [PATCH v4] log: support custom log function

2021-02-17 Thread Li Feng
By default, the dpdk log is out to stdout/stderr and syslog. The rte_openlog_stream could set an external FILE* stream, but it asks the consumer to give it a FILE* pointer. For C++ or other languages, it's hard to get a libc FILE*. Support to set a hook is another choice for this scenario. Signed

Re: [dpdk-dev] [RFC PATCH v1 0/2] Travis CI jobs update after credits

2021-02-17 Thread Juraj Linkeš
> -Original Message- > From: Honnappa Nagarahalli > Sent: Thursday, February 18, 2021 3:40 AM > To: Juraj Linkeš ; tho...@monjalon.net; > david.march...@redhat.com; acon...@redhat.com; > maicolgabr...@hotmail.com > Cc: dev@dpdk.org; Ruifeng Wang ; nd > ; Honnappa Nagarahalli ; nd > > Su