The macro ASSIGN_UNLESS_EQUAL doesn't assign value to the
variable, so the virtio_net_hdr may not be cleared correctly and the
tx packet may be droped due to wrong virtio_net_hdr.
Fixes: 57f90f894588 ("net/virtio: reuse packed ring functions")
Cc: yong@intel.com
Signed-off-by: javin
---
driv
Hi Stephen,
snipped
>
> 15/07/2020 23:22, Stephen Hemminger:
> > Printing extra borders does not improve readability, and is just
> > unnecessary. Putting TSC hz in header also makes no sense here.
>
> The CPU frequency on headers!
> OK to remove :)
>
The rational of having Time Stamp Counter
The vf index in rte_flow command need to check in order
that not larger than actual vf number. Also
change some error log for wrong action.
Fixes: 829c3106812d ("net/ice: enable switch flow on DCF")
Cc: sta...@dpdk.org
Signed-off-by: Wei Zhao
Tested-by: Nannan Lu
---
v2: change error log
v3:
Hello All,
I know not everyone has the meeting calendar invite, so I'm going to send
out a couple of reminders for the next couple of meetings.
Thursday, 1pm UTC, August 13, 2020
https://zoom.us/j/95952696590?pwd=b0JjVnJaWDZ3TW1XYjZKQkxmZUZuQT09
If you'd like to be added to the calendar evite, p
Run a simple script to remove trailing white space and blank
lines at end of file across all documents.
Signed-off-by: Stephen Hemminger
---
doc/guides/nics/fail_safe.rst| 4 ++--
doc/guides/prog_guide/graph_lib.rst | 1 -
doc/guides/prog_guide/regexdev.rst | 1
07/08/2020 15:28, Bruce Richardson:
>
> > 535 files changed, 256 insertions(+), 18894 deletions(-)
>
> Thanks for taking on this work.
>
> I like those diff-stat numbers. Nearly 19000 lines fewer in DPDK!
Me too. I expect even more than 19000 in the future last version :-)
07/08/2020 14:30, Ciara Power:
> Make is no longer supported for compiling DPDK, references are now
> removed in the documentation.
>
> Signed-off-by: Ciara Power
[...]
> ~/DPDK$ cd examples/helloworld/
> -~/DPDK/examples/helloworld$ export RTE_SDK=/home/user/DPDK
> -~/DPDK/examples/
07/08/2020 14:30, Ciara Power:
> Make is no longer supported for compiling DPDK, references are now
> removed in the documentation.
>
> Signed-off-by: Ciara Power
[...]
> -make build
> -^^
> -
> -.. code-block:: console
> -
> -make config T=arm64-bluefield-linux-gcc
> -mak
07/08/2020 14:30, Ciara Power:
> Make is no longer supported for compiling DPDK, references are now
> removed in the documentation.
>
> Signed-off-by: Ciara Power
> ---
> doc/guides/howto/packet_capture_framework.rst | 17 +---
> doc/guides/howto/pvp_reference_benchmark.rst | 26 ---
07/08/2020 14:29, Ciara Power:
> Make is no longer supported for compiling DPDK, references are now
> removed in the documentation.
You won't get appropriate review if you don't Cc the right maintainers.
Please use --cc-cmd devtools/get-maintainer.sh with git send-email
so the maintainers are auto
07/08/2020 14:29, Ciara Power:
> Make is no longer supported for compiling DPDK, scripts used with make
> are no longer needed.
>
> Signed-off-by: Ciara Power
> ---
> buildtools/auto-config-h.sh | 108
> buildtools/gen-build-mk.sh | 23
> buildtool
07/08/2020 14:29, Ciara Power:
> As previously decided [1] and communicated via deprecation notice [2], the
> make build system is to be removed in the 20.11 release.
Thanks for working on it.
Please, remove the deprecation notice with this patch.
> 535 files changed, 256 insertions(+), 18894 de
Replace the arguments array by one argument.
All objects in the args array have the same values, so there is no need
to use an array, only one struct is enough.
The args object is a lot smaller, and the allocation can be replaced
with a global variable.
The allocation of obj_table isn't needed eit
Remove the part that checks if there is enough room in the stack, it's
always true as long as size of stack >= MAX_BULK*rte_lcore_count().
This check used an atomic cmpset, and read / write to a shared size
variable. These operations result in some form of synchronization
that might get in the way
The current multithread DPDK stack test is using atomics operations to
share information between threads.
The lockfree stack implementation also uses atomic operations.
This is an issue for testing. The atomics operations for the test may
add some extra synchronization to the stack implementation,
Use rte_eal_wait_lcore to wait and get the return value for all cores.
This is used to propagate any error to the main core.
Signed-off-by: Steven Lariau
Reviewed-by: Dharmik Thakkar
Reviewed-by: Phil Yang
Reviewed-by: Ruifeng Wang
Acked-by: Gage Eads
---
app/test/test_stack.c | 10 -
All the cores use the same argument object, so there is no need to use
a loop to launch the test on every core one by one.
Replace loop with one call to rte_eal_mp_remote_launch
Signed-off-by: Steven Lariau
Reviewed-by: Dharmik Thakkar
Reviewed-by: Phil Yang
Reviewed-by: Ruifeng Wang
Acked-by:
From: Ori Kam:
> This patch updates Mellanox maintainers mails from the Mellanox domain to
> Nvidia domain.
>
> Signed-off-by: Ori Kam
Acked-by: Matan Azrad
Looks Like we need here also
Cc: sta...@dpdk.org
Thanks Ori.
On Wed, Aug 12, 2020 at 09:42:30PM +0530, Jerin Jacob wrote:
> On Wed, Aug 12, 2020 at 9:20 PM Bruce Richardson
> wrote:
> >
> > On Fri, Aug 07, 2020 at 05:05:53PM +0200, Gaėtan Rivet wrote:
> > > On 07/08/20 19:31 +0530, Jerin Jacob wrote:
> > > > On Fri, Aug 7, 2020 at 7:04 PM Ferruh Yigit
> >
On Fri, Aug 07, 2020 at 01:29:51PM +0100, Ciara Power wrote:
> The example app makefiles contained sections using the legacy method of
> compiling with make. These are no longer needed, and are removed,
> leaving only the section that uses pkg-config for the make build.
>
> Signed-off-by: Ciara Po
On Wed, Aug 12, 2020 at 9:20 PM Bruce Richardson
wrote:
>
> On Fri, Aug 07, 2020 at 05:05:53PM +0200, Gaėtan Rivet wrote:
> > On 07/08/20 19:31 +0530, Jerin Jacob wrote:
> > > On Fri, Aug 7, 2020 at 7:04 PM Ferruh Yigit
> > > wrote:
> > > >
> > > > On 8/7/2020 2:23 PM, Bruce Richardson wrote:
>
This patch updates Mellanox maintainers mails from
the Mellanox domain to Nvidia domain.
Signed-off-by: Ori Kam
---
MAINTAINERS | 28 ++--
1 file changed, 14 insertions(+), 14 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 23e7ec9..6ac470b 100644
--- a/MAINTAI
On Fri, Aug 07, 2020 at 05:05:53PM +0200, Gaëtan Rivet wrote:
> On 07/08/20 19:31 +0530, Jerin Jacob wrote:
> > On Fri, Aug 7, 2020 at 7:04 PM Ferruh Yigit wrote:
> > >
> > > On 8/7/2020 2:23 PM, Bruce Richardson wrote:
> > > > On Fri, Aug 07, 2020 at 06:49:47PM +0530, Jerin Jacob wrote:
> > > >>
> -Original Message-
> From: Thomas Monjalon
> Sent: Wednesday, August 12, 2020 2:25 PM
> To: Dumitrescu, Cristian
> Cc: dev@dpdk.org; david.march...@redhat.com;
> arybche...@solarflare.com; Yigit, Ferruh ;
> Richardson, Bruce
> Subject: Re: [dpdk-dev] [PATCH] maintainers: remove QoS
On Wed, 12 Aug 2020 14:23:51 +0200
Thomas Monjalon wrote:
> 12/08/2020 12:57, Hemant Agrawal:
> > > -Original Message-
> > > From: techboard On Behalf Of Maxime
> > > Coquelin
> > >
> > > On 8/12/20 12:22 PM, Thomas Monjalon wrote:
> > > > 12/08/2020 11:52, Bruce Richardson:
> > > >
On Tue, Aug 04, 2020 at 03:07:34PM +0100, Louise Kilheeney wrote:
> Changed script to explicitly use python3 only to avoid
> maintaining python 2 and removed deprecation notice.
>
> Cc: Olivier Matz
>
> Signed-off-by: Louise Kilheeney
> ---
With this patch applied I'm seeing a failure with the
On Tue, Aug 04, 2020 at 03:07:33PM +0100, Louise Kilheeney wrote:
> Changed script to explicitly use python3 only to avoid
> maintaining python 2 and removed deprecation notice.
>
> Signed-off-by: Louise Kilheeney
> ---
> usertools/cpu_layout.py | 13 ++---
> 1 file changed, 2 insertions
On Tue, Aug 04, 2020 at 03:07:32PM +0100, Louise Kilheeney wrote:
> Changed script to explicitly use python3 only to avoid
> maintaining python 2 and removed deprecation notice.
>
> Cc: Neil Horman
>
> Signed-off-by: Louise Kilheeney
> ---
> usertools/dpdk-pmdinfo.py | 7 +--
> 1 file chan
On Tue, Aug 04, 2020 at 03:07:31PM +0100, Louise Kilheeney wrote:
> Changed script to explicitly use python3 only to avoid
> maintaining python 2 and removed deprecation notice.
>
> Signed-off-by: Louise Kilheeney
> ---
> usertools/dpdk-devbind.py | 22 --
> 1 file changed, 4
On 12/08/2020 14:22, Thomas Monjalon wrote:
> 12/08/2020 13:26, Ray Kinsella:
>> Added a section describing new abi versions, this provides pointers to
>> the relevant amended rules that apply during the abi breakage window.
>> Also remove the large note at the head of the abi policy describing
> -Original Message-
> From: dev On Behalf Of Vikas Gupta
> Sent: Wednesday, 12 August, 2020 08:31
> To: dev@dpdk.org; akhil.go...@nxp.com
> Cc: ajit.khapa...@broadcom.com; vikram.prak...@broadcom.com; Vikas
> Gupta ; Raveendra Padasalagi
>
> Subject: [dpdk-dev] [PATCH v1 8/8] crypto/bcmf
12/08/2020 14:47, Bruce Richardson:
> On Wed, Aug 12, 2020 at 11:29:15AM +0200, Thomas Monjalon wrote:
> > The default git branch of the main DPDK repository has been renamed
> > from master to main.
> >
> > Signed-off-by: Thomas Monjalon
[...]
> > -The backup maintainer for the master tree shoul
12/08/2020 14:36, Bruce Richardson:
> On Wed, Aug 12, 2020 at 11:07:23AM +0200, Thomas Monjalon wrote:
> > The git trees dpdk-next-qos and dpdk-next-pipeline were created
> > to share the load of patches merging.
> > It has been decided in the Technical Board that the load is not big
> > enough to
12/08/2020 13:26, Ray Kinsella:
> Added a section describing new abi versions, this provides pointers to
> the relevant amended rules that apply during the abi breakage window.
> Also remove the large note at the head of the abi policy describing the
> abi stability process that has taken place ove
On Wed, Aug 12, 2020 at 11:29:15AM +0200, Thomas Monjalon wrote:
> The default git branch of the main DPDK repository has been renamed
> from master to main.
>
> Signed-off-by: Thomas Monjalon
> ---
> devtools/check-git-log.sh | 4 ++--
> devtools/checkpatches.sh| 4 ++--
>
On Wed, Aug 12, 2020 at 11:07:23AM +0200, Thomas Monjalon wrote:
> The git trees dpdk-next-qos and dpdk-next-pipeline were created
> to share the load of patches merging.
> It has been decided in the Technical Board that the load is not big
> enough to justify keeping these repositories.
>
> The p
12/08/2020 12:57, Hemant Agrawal:
> > -Original Message-
> > From: techboard On Behalf Of Maxime
> > Coquelin
> >
> > On 8/12/20 12:22 PM, Thomas Monjalon wrote:
> > > 12/08/2020 11:52, Bruce Richardson:
> > >> On Mon, Aug 10, 2020 at 02:50:57PM +0200, Thomas Monjalon wrote:
> > >>> 10/08
12/08/2020 13:19, Kinsella, Ray:
> On 12/08/2020 11:40, Thomas Monjalon wrote:
> >> --- a/doc/guides/contributing/abi_versioning.rst
> >> +++ b/doc/guides/contributing/abi_versioning.rst
> >> @@ -673,9 +673,9 @@ symbols.
> >> -BIND_DEFAULT_SYMBOL(rte_acl_create, _v20, 20);
> >> +BIND_DEFAULT_SY
Hi Vipul,
> -Original Message-
> From: Vipul Ashri
> Sent: Wednesday, August 12, 2020 5:23 PM
> To: dev@dpdk.org
> Cc: Xia, Chenbo ; vipul.as...@oracle.com;
> edwin.le...@oracle.com
> Subject: [PATCH v2] Fixes: 57f90f894588("net/virtio: reuse packed ring
> functions")
>
> Problem - Stand
Hi Honnappa,
>-Original Message-
>From: Honnappa Nagarahalli
>Sent: Tuesday 11 August 2020 06:37
>To: Power, Ciara ; dev@dpdk.org
>Cc: Richardson, Bruce ; nd ;
>Honnappa Nagarahalli ; nd
>
>Subject: RE: [dpdk-dev] [PATCH 20.11 00/12] add max SIMD bitwidth to EAL
>
>Hi Ciara,
> I h
Hi David,
>-Original Message-
>From: David Christensen
>Sent: Friday 7 August 2020 17:32
>To: Power, Ciara ; dev@dpdk.org
>Cc: Richardson, Bruce ; Ruifeng Wang
>; Jerin Jacob ; Honnappa
>Nagarahalli
>Subject: Re: [PATCH 20.11 02/12] eal: add default SIMD bitwidth values
>
>On 8/7/20 8:5
Added a section describing new abi versions, this provides pointers to
the relevant amended rules that apply during the abi breakage window.
Also remove the large note at the head of the abi policy describing the
abi stability process that has taken place over the previous year.
Signed-off-by: Ray
v4 on the way, other notes are below.
On 12/08/2020 11:40, Thomas Monjalon wrote:
[SNIP]
>> Experimental
>>
>>
>> +Major ABI versions are usually but not always declared aligned with a
>> +:ref:`LTS release `.
>
> Why adding this sentence here?
Will remove.
>
>
>> +
>> APIs
>
> -Original Message-
> From: techboard On Behalf Of Maxime
> Coquelin
>
> On 8/12/20 12:22 PM, Thomas Monjalon wrote:
> > 12/08/2020 11:52, Bruce Richardson:
> >> On Mon, Aug 10, 2020 at 02:50:57PM +0200, Thomas Monjalon wrote:
> >>> 10/08/2020 13:49, Trahe, Fiona:
> Sorry about that
On 8/12/20 12:22 PM, Thomas Monjalon wrote:
> 12/08/2020 11:52, Bruce Richardson:
>> On Mon, Aug 10, 2020 at 02:50:57PM +0200, Thomas Monjalon wrote:
>>> 10/08/2020 13:49, Trahe, Fiona:
Sorry about that Thomas, Akhil, we should have chased up acks sooner.
Given the proposed conte
In the title and below, s/abi/ABI/
11/08/2020 18:39, Ray Kinsella:
> Added a section describing new abi versions, this provides pointers to
> the relevant amended rules that apply during the abi breakage window.
> Also remove the large note a the head of the abi policy describing the
s/a/at/
> a
Add test suites for supported algorithms by bcmfs crypto pmd
Signed-off-by: Vikas Gupta
Signed-off-by: Raveendra Padasalagi
Reviewed-by: Ajit Khaparde
---
app/test/test_cryptodev.c | 261 ++
app/test/test_cryptodev.h | 1 +
2 files changed, 262 insertions(
Add crypto h/w module to process crypto op. Crypto op is processed via
sym_engine module before submitting the crypto request to h/w queues.
Signed-off-by: Vikas Gupta
Signed-off-by: Raveendra Padasalagi
Reviewed-by: Ajit Khaparde
---
drivers/crypto/bcmfs/bcmfs_sym.c| 316
dri
Add queue pair operations exported by supported devices.
Signed-off-by: Vikas Gupta
Signed-off-by: Raveendra Padasalagi
Reviewed-by: Ajit Khaparde
---
drivers/crypto/bcmfs/bcmfs_dev_msg.h | 29 +
drivers/crypto/bcmfs/bcmfs_device.c | 51 ++
drivers/crypto/bcmfs/bcmfs_device.h
Add session handling and capabilities supported by crypto h/w
accelerator.
Signed-off-by: Vikas Gupta
Signed-off-by: Raveendra Padasalagi
Reviewed-by: Ajit Khaparde
---
doc/guides/cryptodevs/bcmfs.rst | 46 ++
doc/guides/cryptodevs/features/bcmfs.ini | 56 ++
drivers/crypt
Add queue pair management APIs which will be used by Crypto device to
manage h/w queues. A bcmfs device structure owns multiple queue-pairs
based on the mapped address allocated to it.
Signed-off-by: Vikas Gupta
Signed-off-by: Raveendra Padasalagi
Reviewed-by: Ajit Khaparde
---
drivers/crypto/
Create a symmetric crypto device and supported cryptodev ops.
Signed-off-by: Vikas Gupta
Signed-off-by: Raveendra Padasalagi
Reviewed-by: Ajit Khaparde
---
drivers/crypto/bcmfs/bcmfs_device.c | 15 ++
drivers/crypto/bcmfs/bcmfs_device.h | 9 +
drivers/crypto/bcmfs/bcmfs_qp.c | 37 ++
Add vfio support for device.
Signed-off-by: Vikas Gupta
Signed-off-by: Raveendra Padasalagi
Reviewed-by: Ajit Khaparde
---
drivers/crypto/bcmfs/Makefile | 1 +
drivers/crypto/bcmfs/bcmfs_device.c | 5 ++
drivers/crypto/bcmfs/bcmfs_device.h | 6 ++
drivers/crypto/bcmfs/bcmfs_vfio.c |
Add Broadcom FlexSparc(FS) device creation driver which registers to a
vdev and create a device. Add APIs for logs, supportive documention and
maintainers file.
Signed-off-by: Vikas Gupta
Signed-off-by: Raveendra Padasalagi
Reviewed-by: Ajit Khaparde
---
MAINTAINERS
Hi,
This patchset contains support for Crypto offload on Broadcom’s
Stingray/Stingray2 SoCs having FlexSparc unit.
BCMFS is an acronym for Broadcom FlexSparc device used in the patchest.
The patchset progressively adds major modules as below.
a) Detection of platform-device based on the known reg
12/08/2020 11:52, Bruce Richardson:
> On Mon, Aug 10, 2020 at 02:50:57PM +0200, Thomas Monjalon wrote:
> > 10/08/2020 13:49, Trahe, Fiona:
> > > Sorry about that Thomas, Akhil, we should have chased up acks sooner.
> > >
> > > Given the proposed content of the patch, and the similar changes that
On Mon, Aug 10, 2020 at 02:50:57PM +0200, Thomas Monjalon wrote:
> 10/08/2020 13:49, Trahe, Fiona:
> > Sorry about that Thomas, Akhil, we should have chased up acks sooner.
> >
> > Given the proposed content of the patch, and the similar changes that are
> > recommended and agreed
> > throughout
From: Pawel Wodkowski
trace_mem is declared as 'void *' which triggers following error:
'...invalid conversion from ‘void*’ to ‘__rte_trace_header*’
[-fpermissive]...'
Fix this by adding proper typecast to 'struct __rte_trace_header *'.
Fixes: ebaee6409702 ("trace: simplify trace point headers"
12/08/2020 10:58, Kinsella, Ray:
> On 11/08/2020 15:03, Thomas Monjalon wrote:
> > Start a new release cycle with empty release notes.
> >
> > The ABI version becomes 21.0.
> > The ABI major is back to normal, having only one number (21 vs 20.0).
> > The map files are updated to the new ABI major
The default git branch of the main DPDK repository has been renamed
from master to main.
Signed-off-by: Thomas Monjalon
---
devtools/check-git-log.sh | 4 ++--
devtools/checkpatches.sh| 4 ++--
doc/guides/contributing/patches.rst | 5 +++--
doc/guides/contributing/stable.rs
The git trees dpdk-next-qos and dpdk-next-pipeline were created
to share the load of patches merging.
It has been decided in the Technical Board that the load is not big
enough to justify keeping these repositories.
The patches for ethdev TM and MTR will be managed in dpdk-next-net.
The sched and
Hi Thomas,
Regex has a similar set of issues,
Warning: symbol rte_regexdev_unregister (Base) should be annotated as ABI
version DPDK_21 / DPDK_22, EXPERIMENTAL, or INTERNAL.
Warning: symbol rte_regexdev_register (Base) should be annotated as ABI
version DPDK_21 / DPDK_22, EXPERIMENTAL, or INT
On 11/08/2020 15:03, Thomas Monjalon wrote:
> Start a new release cycle with empty release notes.
>
> The ABI version becomes 21.0.
> The ABI major is back to normal, having only one number (21 vs 20.0).
> The map files are updated to the new ABI major number (21).
> The ABI exceptions are drop
63 matches
Mail list logo