Hi Jeff,
> -Original Message-
> From: Jeff Guo
> Sent: Wednesday, April 15, 2020 8:11 PM
> To: bernard.iremon...@intel.com; Ori Kam ;
> xiaolong...@intel.com; qi.z.zh...@intel.com
> Cc: dev@dpdk.org; jingjing...@intel.com; yahui@intel.com;
> simei...@intel.com; jia@intel.com
> S
> > > Subject: Re: [dpdk-dev] [PATCH v2 1/2] virtio: one way barrier for
> > > split vring used idx
> > >
> > > On Thu, 2 Apr 2020 10:57:52 +0800
> > > Joyce Kong wrote:
> > >
> > > > -(vq)->vq_used_cons_idx))
> > > > +static inline uint16_t
> > > > +virtqueue_nused(struct virtqueue *vq)
> > >
On 2020-04-15 21:45, David Marchand wrote:
> On Wed, Apr 15, 2020 at 8:15 PM Mattias Rönnblom
> wrote:
>> Add redundant stack variable initialization to work around
>> false-positive warnings in older versions of GCC.
>>
>> Fixes: bba7a1aeef46 ("event/dsw: improve migration mechanism")
> The commi
Hi Luca,
> -Original Message-
> From: Luca Boccassi
> Sent: Wednesday, April 15, 2020 1:45 PM
> To: Juraj Linkeš ; bruce.richard...@intel.com;
> acon...@redhat.com; maicolgabr...@hotmail.com; ruifeng.w...@arm.com
> Cc: dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v3 1/4] build: port arm3
On Thu, Apr 16, 2020 at 4:42 AM Dan Gora wrote:
>
> Instead of choosing to use getentropy() or the rdseed instruction for
> the random number generator entropy source using compilation flags,
> determine the best source at run time.
>
> This is accomplished by defining a weak symbol for getentropy
This patch adds a support of VIRTIO_NET_F_SPEED_DUPLEX feature
for virtio driver.
There are two ways to specify speed of the link:
* 'speed' devarg
* negotiate speed from qemu via VIRTIO_NET_F_SPEED_DUPLEX
The highest priority is devarg. If devarg is not specified,
driver tries to negotiate it
strtoull returns 0 if it fails to parse input string. It's ignored
in get_integer_arg.
This patch handles error cases for strtoull function.
Fixes: ce2eabdd43ec ("net/virtio-user: add virtual device")
Cc: sta...@dpdk.org
Signed-off-by: Ivan Dyukov
---
drivers/net/virtio/virtio_user_ethdev.c | 1
virtio driver already parses speed devarg. virtio-user should add
it to list of valid devargs and call eth_virtio_dev_init function
which init speed value.
eth_virtio_dev_init already is called from virtio_user_pmd_probe
function. The only change is required to enable speed devargs:
adding speed t
refactor vdpa specific devargs parsing to more generic way
Signed-off-by: Ivan Dyukov
---
drivers/net/virtio/virtio_ethdev.c | 34 +-
1 file changed, 24 insertions(+), 10 deletions(-)
diff --git a/drivers/net/virtio/virtio_ethdev.c
b/drivers/net/virtio/virtio_ethdev
doc/guides/nics/virtio.rst | 15 +++
drivers/net/virtio/virtio_ethdev.c | 143
++
drivers/net/virtio/virtio_ethdev.h |
Default value of link speed is 0x (Unknown)
This patch adds speed devarg which allows to configure
link speed of virtio device.
Signed-off-by: Ivan Dyukov
---
doc/guides/nics/virtio.rst | 7 ++
drivers/net/virtio/virtio_ethdev.c | 107 -
drivers/net
This patch set speed to unknown
Signed-off-by: Ivan Dyukov
---
drivers/net/virtio/virtio_ethdev.c | 4 ++--
lib/librte_ethdev/rte_ethdev.h | 27 ++-
2 files changed, 16 insertions(+), 15 deletions(-)
diff --git a/drivers/net/virtio/virtio_ethdev.c
b/drivers/net/vir
Remove its own bit operation APIs and use the common one,
this can reduce the code duplication largely.
Signed-off-by: Joyce Kong
Reviewed-by: Gavin Hu
---
drivers/net/hinic/Makefile| 1 +
drivers/net/hinic/base/hinic_compat.h | 33 +--
drivers/net/hinic/hin
Remove its own bit operation APIs and use the common one,
this can reduce the code duplication largely.
Signed-off-by: Joyce Kong
Reviewed-by: Gavin Hu
---
drivers/net/qede/base/bcm_osal.c | 22 +
drivers/net/qede/base/bcm_osal.h | 14 -
drivers/net/qede/ba
Remove its own bit operation APIs and use the common one,
this can reduce the code duplication largely.
Signed-off-by: Joyce Kong
Reviewed-by: Gavin Hu
---
drivers/net/axgbe/axgbe_common.h | 29 +
drivers/net/axgbe/axgbe_ethdev.c | 14 +++---
drivers/net/axgb
Remove its own bit operation APIs and use the common one,
this can reduce the code duplication largely.
Signed-off-by: Joyce Kong
Reviewed-by: Gavin Hu
---
drivers/net/bnx2x/bnx2x.c| 271 +--
drivers/net/bnx2x/bnx2x.h| 10 +-
drivers/net/bnx2x/ecore_sp.c
Add test cases for setting bit, clearing bit, testing
and setting bit, testing and clearing bit operation.
Signed-off-by: Joyce Kong
Reviewed-by: Gavin Hu
Reviewed-by: Phil Yang
---
MAINTAINERS | 1 +
app/test/Makefile | 1 +
app/test/autotest_data.py | 6 ++
app/te
Bitwise operation APIs are defined and used in a lot of PMDs,
which caused a huge code duplication. To reduce duplication,
this patch consolidates them into a common API family and use
it for all the PMDs(we started with a few selectively).
v8:
1. Change 'rte_pmd_bitops.h' to 'rte_bitops.h' allo
Bitwise operation APIs are defined and used in a lot of PMDs,
which caused a huge code duplication. To reduce duplication,
this patch consolidates them into a common API family.
Signed-off-by: Joyce Kong
Reviewed-by: Gavin Hu
Reviewed-by: Phil Yang
Acked-by: Morten Brørup
---
MAINTAINERS
On Thu, 16 Apr 2020 11:03:57 +0800
wangyunjian wrote:
> From: Yunjian Wang
>
> This series include five fixes patches for tap PMD driver.
>
> --
> v5:
> * fix check fd error
> v4:
> * Update some code suggested by Ferruh Yigit and Stephen Hemminger
> * Update commit log
> * Add fix leak of fds
> > Subject: Re: [dpdk-dev] [PATCH v2 1/2] virtio: one way barrier for
> > split vring used idx
> >
> > On Thu, 2 Apr 2020 10:57:52 +0800
> > Joyce Kong wrote:
> >
> > > -(vq)->vq_used_cons_idx))
> > > +static inline uint16_t
> > > +virtqueue_nused(struct virtqueue *vq)
> > vq is unmodified and
hi, ferruh and bernard
On 4/16/2020 6:13 AM, Ferruh Yigit wrote:
On 4/15/2020 4:38 PM, Iremonger, Bernard wrote:
Hi Jeff,
-Original Message-
From: Guo, Jia
Sent: Wednesday, April 15, 2020 6:11 PM
To: Iremonger, Bernard ;
or...@mellanox.com; Ye, Xiaolong ; Zhang, Qi Z
Cc: dev@dpdk
Align the package file search sequence with PF only for DCF mode. Get
the DSN through the virtual channel firstly to check the package file
accessible.
Signed-off-by: Haiyue Wang
---
drivers/net/ice/ice_dcf_parent.c | 30 ++
1 file changed, 26 insertions(+), 4 deletio
From: Yunjian Wang
When eth_dev_tap_create() is failed, nlsk_fd and ka_fd won't be closed
thus leading fds leak. Zero is a valid fd. Ultimately leads to a valid
fd was closed by mistake.
Fixes: bf7b7f437b49 ("net/tap: create netdevice during probing")
Fixes: cb7e68da630a ("net/tap: fix cleanup o
From: Yunjian Wang
The internal structure is freed and set to NULL in the
rte_eth_dev_release_port() and zero is a valid fd. Ultimately
leads to a valid fd was closed by mistake.
Fixes: 3101191c63ab ("net/tap: fix device removal when no queue exist")
CC: sta...@dpdk.org
Reviewed-by: Ferruh Yigi
From: Yunjian Wang
For the tap PMD, we should release mbufs and iovecs from the Rx queue when
closing device. In order to remove duplicated code, rte_pmd_tap_remove()
calls tap_dev_close().
Fixes: 0781f5762cfe ("net/tap: support segmented mbufs")
CC: sta...@dpdk.org
Reviewed-by: Ferruh Yigit
S
From: Yunjian Wang
Now the rxq->pool is mbuf concatenation, but its nb_segs is 1. When
conducting some sanity checks on the mbuf with debug enabled, it fails.
Fixes: 0781f5762cfe ("net/tap: support segmented mbufs")
CC: sta...@dpdk.org
Signed-off-by: Yunjian Wang
---
drivers/net/tap/rte_eth_t
From: Yunjian Wang
When the tap_write_mbufs() function return with break, mbuf was freed
without increasing num_packets, which could cause applications to free
the mbuf again. And the pmd_tx_burst() function should returns the
number of original packets it actually sent excluding tso mbufs.
Fixe
From: Yunjian Wang
This series include five fixes patches for tap PMD driver.
--
v5:
* fix check fd error
v4:
* Update some code suggested by Ferruh Yigit and Stephen Hemminger
* Update commit log
* Add fix leak of fds on failure
v3:
* Add fix close a vaild fd and netlink socket file descriptor
This commit converts port id action to indexed.
Using the uint32_t index instead of pointer saves 4 bytes memory for the
flow handle. For millions flows, it will save several MBytes of memory.
Signed-off-by: Suanming Mou
Acked-by: Viacheslav Ovsiienko
---
drivers/net/mlx5/mlx5.c | 11 +
This commit convert jump resource to indexed.
The table data struct is allocated from indexed memory. As it is add in
the hash list, the pointer is still used for hash list search. The index
is added to the table struct, and the pointer in flow handle is decrease
to uint32_t type. For flow without
This commit converts hrxq to indexed.
Using the uint32_t index instead of pointer saves 4 bytes memory for the
flow handle. For millions flows, it will save several MBytes of memory.
Signed-off-by: Suanming Mou
Acked-by: Viacheslav Ovsiienko
---
drivers/net/mlx5/mlx5.c| 31
This commit converts the flow encap/decap resource to indexed.
Using the uint32_t index instead of pointer saves 4 bytes memory for the
flow handle. For millions flows, it will save several MBytes of memory.
Signed-off-by: Suanming Mou
Acked-by: Viacheslav Ovsiienko
---
drivers/net/mlx5/mlx5.c
This commit converts flow dev handle to indexed.
Change the mlx5 flow handle from pointer to uint32_t saves memory for
flow. With million flow, it saves several MBytes memory.
Signed-off-by: Suanming Mou
Acked-by: Viacheslav Ovsiienko
---
drivers/net/mlx5/mlx5.c| 30 +++
This commit convert tag resource to indexed.
As tag resources are add in the hash list, to avoid introduce performance
issue and keep the hash list, only the tag resource memory is allocated
from indexed memory. The resources is still added to the hash list. Add
four bytes index in the tag resourc
While entries are fully freed in trunk, it means the trunk is free now.
User may prefer the free trunk memory can be reclaimed.
Add the trunk release memory option for indexed pool in this case.
Signed-off-by: Suanming Mou
Acked-by: Viacheslav Ovsiienko
---
drivers/net/mlx5/mlx5_utils.c | 37 +
Currently, flow may allocate several action resources during creation.
Both allocate the individual resources and save the object handles have
big memory overhead for the flow.
Indexed memory pool is introduced to optimize the memory overhead. The
resources can be allocated from the resource's own
Currently, the memory allocated by rte_malloc() also introduced more
than 64 bytes overhead. It means when allocate 64 bytes memory, the
real cost in memory maybe double. And the libc malloc() overhead is 16
bytes, If users try allocating millions of small memory blocks, the
overhead costing maybe
This commit add trunk dynamic grow for the indexed pool.
In case for pools which are not sure the entry number needed, pools can
be configured in increase progressively mode. It means the trunk size
will be increased dynamically one after one, then reach a stable value.
It saves memory to avoid al
This commit converts the push VLAN resource to indexed.
Using the uint32_t index instead of pointer saves 4 bytes memory for the
flow handle. For millions flows, it will save several MBytes of memory.
Signed-off-by: Suanming Mou
Acked-by: Viacheslav Ovsiienko
---
drivers/net/mlx5/mlx5.c
> -Original Message-
> From: Ferruh Yigit [mailto:ferruh.yi...@intel.com]
> Sent: Wednesday, April 15, 2020 11:13 PM
> To: wangyunjian ; dev@dpdk.org
> Cc: keith.wi...@intel.com; Lilijun (Jerry) ;
> xudingke
> ; sta...@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v4 3/5] net/tap: fix check fo
> -Original Message-
> From: Ferruh Yigit [mailto:ferruh.yi...@intel.com]
> Sent: Wednesday, April 15, 2020 11:15 PM
> To: wangyunjian ; dev@dpdk.org
> Cc: keith.wi...@intel.com; Lilijun (Jerry) ;
> xudingke
> ; sta...@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v4 5/5] net/tap: fix leak of
> -Original Message-
> From: Iremonger, Bernard
> Sent: Wednesday, April 15, 2020 7:14 PM
> To: dev@dpdk.org; Xing, Beilei ; Zhang, Qi Z
> ; Doherty, Declan
> Cc: Ananyev, Konstantin ; Iremonger,
> Bernard
> Subject: [PATCH v4 1/3] librte_ethdev: add RSS offload types for ESP and AH
>
> -Original Message-
> From: Iremonger, Bernard
> Sent: Wednesday, April 15, 2020 7:14 PM
> To: dev@dpdk.org; Xing, Beilei ; Zhang, Qi Z
> ; Doherty, Declan
> Cc: Ananyev, Konstantin ; Iremonger,
> Bernard
> Subject: [PATCH v4 3/3] net/i40e: add ESP patterns
>
> add ESP patterns to i
Hi,
> -Original Message-
> From: Chautru, Nicolas
> Sent: Wednesday, April 15, 2020 23:51
> To: Xu, Rosen ; dev@dpdk.org
> Cc: Richardson, Bruce ; O'Hare, Cathal
> ; akhil.go...@nxp.com
> Subject: RE: [dpdk-dev] [PATCH v2 06/13] baseband/fpga_5gnr_fec: add
> queue configuration
>
> Hi
>
> -Original Message-
> From: Maxime Coquelin
> Sent: Wednesday, April 15, 2020 11:04 PM
> To: Liu, Yong ; Ye, Xiaolong ;
> Wang, Zhihong ; epere...@redhat.com
> Cc: dev@dpdk.org
> Subject: Re: [PATCH] vhost: remove deferred shadow update
>
> Hi Marvin,
>
> On 4/15/20 4:55 PM, Liu, Yon
Instead of choosing to use getentropy() or the rdseed instruction for
the random number generator entropy source using compilation flags,
determine the best source at run time.
This is accomplished by defining a weak symbol for getentropy(),
checking that the compiler can generate the rdseed instr
Hi Lukasz,
> Hi Konstantin,
>
> In this patch I moved the sa_init() before rte_eth_dev_start() in order to
> avoid dropping
> of IPsec pkts when a traffic flows and the ipsec-secgw application is started.
>
> However I remember that during review of event mode patches you mentioned that
> mo
Answering the questions. All snipped comments will be fixed in v4.
> [...]
> > +/**
> > + * Memory reservation flags.
> > + */
> > +enum eal_mem_reserve_flags {
> > + /**< Reserve hugepages (support may be limited or missing). */
> > + EAL_RESERVE_HUGEPAGES = 1 << 0,
> > + /**< Fail if reque
HWRM API allows drivers to query stats per PCI function.
These stats can provide some useful information in certain
circumstances.
Signed-off-by: Ajit Khaparde
Reviewed-by: Somnath Kotur
---
drivers/net/bnxt/bnxt_hwrm.c| 10 +++-
drivers/net/bnxt/bnxt_hwrm.h| 4 +++-
drivers/net/bn
Instead of choosing to use getentropy() or the rdseed instruction for
the random number generator entropy source using compilation flags,
determine the best source at run time.
This is accomplished by defining a weak symbol for getentropy(),
checking that the compiler can generate the rdseed instr
16/04/2020 00:19, Doherty, Declan:
> On 14/04/2020 3:44 PM, Thomas Monjalon wrote:
> > 14/04/2020 16:02, Trahe, Fiona:
> >> From: Thomas Monjalon
> >>> 14/04/2020 15:04, Trahe, Fiona:
> > 14/04/2020 12:21, Ferruh Yigit:
> >
> >>> http://inbox.dpdk.org/dev/mn2pr11mb35507d4b96677a41e66440c5e
On 14/04/2020 3:44 PM, Thomas Monjalon wrote:
14/04/2020 16:02, Trahe, Fiona:
Hi Thomas,
From: Thomas Monjalon
14/04/2020 15:04, Trahe, Fiona:
14/04/2020 12:21, Ferruh Yigit:
http://inbox.dpdk.org/dev/mn2pr11mb35507d4b96677a41e66440c5e3...@mn2pr11mb3550.na
mprd11.prod.outlook.com/
I am n
14/04/2020 21:44, Dmitry Kozlyuk:
> All supported OS create memory segment lists (MSL) and reserve VA space
> for them in a nearly identical way. Move common code into EAL private
> functions to reduce duplication.
>
> Signed-off-by: Dmitry Kozlyuk
> ---
> lib/librte_eal/common/eal_common_memory
14/04/2020 21:44, Dmitry Kozlyuk:
> System meory management is implemented differently for POSIX and
meory -> memory
> Windows. Introduce wrapper functions for operations used across DPDK:
>
> * rte_mem_map()
> Create memory mapping for a regular file or a page file (swap).
> This supports m
On 4/15/2020 4:38 PM, Iremonger, Bernard wrote:
> Hi Jeff,
>
>
>> -Original Message-
>> From: Guo, Jia
>> Sent: Wednesday, April 15, 2020 6:11 PM
>> To: Iremonger, Bernard ;
>> or...@mellanox.com; Ye, Xiaolong ; Zhang, Qi Z
>>
>> Cc: dev@dpdk.org; Wu, Jingjing ; Cao, Yahui
>> ; Su, Sim
Thanks for starting the new directory for Unix common implementation.
14/04/2020 21:44, Dmitry Kozlyuk:
> --- /dev/null
> +++ b/lib/librte_eal/unix/eal.c
Please take care of not creating a new file without adding
SPDX tag and copyright owner.
> @@ -0,0 +1,47 @@
> +#include
> +#include
> +#incl
From: Harini Ramakrishnan
Attendees: Thomas Monjalon, Ranjit Menon, Tal Shnaiderman, Pallavi Kadam,
Ranjit Menon, Narcisa Ana Maria Vasile, Harini Ramakrishnan, Dmitry Kozliuk,
William Tu, Tasnim Bashar
1/ UIO Driver
* New Intel-Microsoft DA needed to unblock licensing issues.
* Naty
Hi Honnappa,
> > +
> > +/**
> > + * @internal This function updates the producer head for enqueue
> > + *
> > + * @param r
> > + * A pointer to the ring structure
> > + * @param is_sp
> > + * Indicates whether multi-producer path is needed or not
> > + * @param n
> > + * The number of elemen
Default value of link speed is 0x (Unknown)
This patch adds speed devarg which allows to configure
link speed of virtio device.
Signed-off-by: Ivan Dyukov
---
doc/guides/nics/virtio.rst | 7 ++
drivers/net/virtio/virtio_ethdev.c | 107 -
drivers/net
This patch adds a support of VIRTIO_NET_F_SPEED_DUPLEX feature
for virtio driver.
There are two ways to specify speed of the link:
* 'speed' devarg
* negotiate speed from qemu via VIRTIO_NET_F_SPEED_DUPLEX
The highest priority is devarg. If devarg is not specified,
driver tries to negotiate it
virtio driver already parses speed devarg. virtio-user should add
it to list of valid devargs and call eth_virtio_dev_init function
which init speed value.
eth_virtio_dev_init already is called from virtio_user_pmd_probe
function. The only change is required to enable speed devargs:
adding speed t
refactor vdpa specific devargs parsing to more generic way
Signed-off-by: Ivan Dyukov
---
drivers/net/virtio/virtio_ethdev.c | 34 +-
1 file changed, 24 insertions(+), 10 deletions(-)
diff --git a/drivers/net/virtio/virtio_ethdev.c
b/drivers/net/virtio/virtio_ethdev
strtoull returns 0 if it fails to parse input string. It's ignored
in get_integer_arg.
This patch handles error cases for strtoull function.
Fixes: ce2eabdd43ec ("net/virtio-user: add virtual device")
Cc: sta...@dpdk.org
Signed-off-by: Ivan Dyukov
---
drivers/net/virtio/virtio_user_ethdev.c | 1
This patch set speed to unknown
Signed-off-by: Ivan Dyukov
---
drivers/net/virtio/virtio_ethdev.c | 4 ++--
lib/librte_ethdev/rte_ethdev.h | 27 ++-
2 files changed, 16 insertions(+), 15 deletions(-)
diff --git a/drivers/net/virtio/virtio_ethdev.c
b/drivers/net/vir
doc/guides/nics/virtio.rst | 15 +++
drivers/net/virtio/virtio_ethdev.c | 143
++
drivers/net/virtio/virtio_ethdev.h |
On Wed, Apr 15, 2020 at 8:15 PM Mattias Rönnblom
wrote:
>
> Add redundant stack variable initialization to work around
> false-positive warnings in older versions of GCC.
>
> Fixes: bba7a1aeef46 ("event/dsw: improve migration mechanism")
The commitid in master is 1f2b99e8d.
> Cc: sta...@dpdk.org
>
> This patch adds support for plain SHA-1, SHA-224, SHA-256,
> SHA-384 and SHA-512 hashes to QAT PMD.
>
> Signed-off-by: Adam Dybkowski
> ---
App/test changes may not be required.
Rebase this patch over Pablo's crypto test refactoring.
Hi Pablo,
> When authenticating after encrypting, if the operation
> is out-of-place, the destination buffer is the one
> that will get authenticated.
> If the cipher offset is higher than the authentication
> offset, it means that part of the text to authenticate
> will be plaintext, so this need
Hi Vladimir,
>
> Get rid of hardcoded limit of cryptodev sessions.
>
> Fixes: e1143d7dbbf4 ("examples/ipsec-secgw: get rid of maximum SA limitation")
> Cc: vladimir.medved...@intel.com
>
> Signed-off-by: Vladimir Medvedkin
> ---
This will create an issue in case of rte_security sessions.
Rte_s
On 2020-04-15 20:17, Vladimir Medvedkin wrote:
> K32V64 hash is a hash table that supports 32 bit keys and 64 bit values.
> This table is hash function agnostic so user must provide
> precalculated hash signature for add/delete/lookup operations.
>
> Signed-off-by: Vladimir Medvedkin
> ---
> lib
Hi Praveen,
>
> Support load distribution in security gateway application using
> NIC load distribution feature(Flow Director).
> Flow Director is used to redirect the specified inbound ipsec flow
> to a specified queue.This is achieved by extending the SA rule syntax
> to support specification b
>Initial IP fragmentation unit test.
>
>Signed-off-by: Aaron Conole
>---
> MAINTAINERS| 1 +
> app/test/meson.build | 2 +
> app/test/test_ipfrag.c | 276
>+
> 3 files changed, 279 insertions(+)
> create mode 100644 app/test/test_ipfrag.c
>
Hi all,
Here is a new LTS release:
https://fast.dpdk.org/rel/dpdk-18.11.7.tar.xz
The git tree is at:
https://dpdk.org/browse/dpdk-stable/?h=18.11
It has about 120 bugfixes since the previous release.
Thanks to the authors who helped with backports and to
the following who helped
On 2020-04-15 20:17, Vladimir Medvedkin wrote:
> Currently DPDK has a special implementation of a hash table for
> 4 byte keys which is called FBK hash. Unfortunately its main drawback
> is that it only supports 2 byte values.
> The new implementation called K32V64 hash
> supports 4 byte keys and 8
Hi Fiona/Adam,
> This patch adds a new test suite for verification of the "internal
> QAT IM buffer too small" case handling. These unit tests are
> specific to the QAT PMD only - that's why they are contained in
> a separate test suite.
>
> Signed-off-by: Adam Dybkowski
> ---
Why do we need to
Add redundant stack variable initialization to work around
false-positive warnings in older versions of GCC.
Fixes: bba7a1aeef46 ("event/dsw: improve migration mechanism")
Cc: sta...@dpdk.org
Signed-off-by: Mattias Rönnblom
---
drivers/event/dsw/dsw_event.c | 6 +++---
1 file changed, 3 inserti
On 2020-04-15 20:07, Mattias Rönnblom wrote:
> Add redundant stack variable initialization to work around
> false-positive warnings in older versions of GCC.
>
> Fixes: bba7a1aeef46 ("event/dsw: improve migration mechanism")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Mattias Rönnblom
> ---
> drive
> >
> > This patch adds ability to run unit tests in cpu crypto mode for AESNI MB
> > cryptodev.
> >
> > Signed-off-by: Konstantin Ananyev
> > ---
> > app/test/test_cryptodev.c | 146 ++
>
> Acked-by: Pablo de Lara
Acked-by: Akhil Goyal
On 2020-04-15 19:45, Thomas Monjalon wrote:
> 15/04/2020 19:39, Mattias Rönnblom:
>> On 2020-04-15 18:37, David Marchand wrote:
>>> On Sat, Apr 4, 2020 at 4:35 PM Jerin Jacob Kollanukkaran
>>> wrote:
From: Mattias Rönnblom
> Performance and statistics improvements for the distributed sof
Add redundant stack variable initialization to work around
false-positive warnings in older versions of GCC.
Fixes: bba7a1aeef46 ("event/dsw: improve migration mechanism")
Cc: sta...@dpdk.org
Signed-off-by: Mattias Rönnblom
---
drivers/event/dsw/dsw_event.c | 4 ++--
1 file changed, 2 insertion
Add performance tests for rte_k32v64_hash
Signed-off-by: Vladimir Medvedkin
---
app/test/test_hash_perf.c | 130 ++
1 file changed, 130 insertions(+)
diff --git a/app/test/test_hash_perf.c b/app/test/test_hash_perf.c
index a438eae..f45a8d9 100644
---
Add programmers guide and doxygen API for k32v64 hash library
Signed-off-by: Vladimir Medvedkin
---
doc/api/doxy-api-index.md | 1 +
doc/guides/prog_guide/index.rst | 1 +
doc/guides/prog_guide/k32v64_hash_lib.rst | 66 +++
3 files changed,
Add autotests for rte_k32v64_hash library
Signed-off-by: Vladimir Medvedkin
---
app/test/Makefile | 1 +
app/test/autotest_data.py | 12 +++
app/test/meson.build| 3 +
app/test/test_k32v64_hash.c | 229
4 files changed, 245 in
K32V64 hash is a hash table that supports 32 bit keys and 64 bit values.
This table is hash function agnostic so user must provide
precalculated hash signature for add/delete/lookup operations.
Signed-off-by: Vladimir Medvedkin
---
lib/Makefile | 2 +-
lib/librte_hash
Currently DPDK has a special implementation of a hash table for
4 byte keys which is called FBK hash. Unfortunately its main drawback
is that it only supports 2 byte values.
The new implementation called K32V64 hash
supports 4 byte keys and 8 byte associated values,
which is enough to store a point
>
> From: Yunjian Wang
>
> This NULL check is unnecessary, container_of is never NULL.
>
> Signed-off-by: Yunjian Wang
> ---
Acked-by: Akhil Goyal
>
> Adding support for the following,
> 1. AES-192-GCM
> 2. AES-256-GCM
> 3. AES-192-CBC
>
> Signed-off-by: Anoob Joseph
> Signed-off-by: Tejasree Kondoj
> ---
>
Acked-by: Akhil Goyal
15/04/2020 19:39, Mattias Rönnblom:
> On 2020-04-15 18:37, David Marchand wrote:
> > On Sat, Apr 4, 2020 at 4:35 PM Jerin Jacob Kollanukkaran
> > wrote:
> >> From: Mattias Rönnblom
> >>> Performance and statistics improvements for the distributed software
> >>> (DSW) event device.
> >>>
> >>> Mat
On 2020-04-15 18:37, David Marchand wrote:
> On Sat, Apr 4, 2020 at 4:35 PM Jerin Jacob Kollanukkaran
> wrote:
>>> -Original Message-
>>> From: Mattias Rönnblom
>>> Sent: Monday, March 9, 2020 12:21 PM
>>> To: Jerin Jacob Kollanukkaran
>>> Cc: dev@dpdk.org; stefan.sundkv...@ericsson.com;
15/04/2020 16:52, jer...@marvell.com:
> From: Jerin Jacob
>
> Vamsi is no longer associated with Marvell.
> Updating Marvell OCTEON TX2 PMDs MAINTAINERS file.
>
> Signed-off-by: Jerin Jacob
Applied
Do a formal parameter check of mtu length, as well as
checking the the various inputs for validity. If any
aren't acceptable, we bail.
Signed-off-by: Aaron Conole
---
lib/librte_ip_frag/rte_ipv4_fragmentation.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/lib/librte_ip_frag/rte_
Initial IP fragmentation unit test.
Signed-off-by: Aaron Conole
---
MAINTAINERS| 1 +
app/test/meson.build | 2 +
app/test/test_ipfrag.c | 276 +
3 files changed, 279 insertions(+)
create mode 100644 app/test/test_ipfrag.c
diff --git a/
In addition, do a formal parameter check.
Signed-off-by: Aaron Conole
---
lib/librte_ip_frag/rte_ipv6_fragmentation.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/lib/librte_ip_frag/rte_ipv6_fragmentation.c
b/lib/librte_ip_frag/rte_ipv6_fragmentation.c
index 43449970e5..ee984aed
This adds a simple unit test for the ip fragmentation library
and covers fragmenting ipv4 and ipv6. Additionally, some fixes
are introduced, which are really just sanity rather than real
issues in the field.
v3->v4:
- Drop the frag header detection from ipv6 code
- Full parameter sanity checking
Hi Ray/Bruce/Thomas,
We've been trying to make sense of the versioning scheme - but as the docs
describe and give examples
of a 2-number scheme and the code has a 3-number scheme it's pretty confusing.
If you can help me fill out the following table this may help:
VERSION | ABI_VERSION | LIBA
On 4/15/2020 2:06 AM, John Daley wrote:
> From: Hyong Youb Kim
>
> The current implementation produces wrong ordering for several cases
> like these:
>
> 1. mark, decap, steer
> Current: steer, mark, decap
> Correct: mark, steer, decap
>
> 2. decap, steer, steer
> Current: steer, steer, decap
>
On 24/03/2020 12:57, Kevin Traynor wrote:
> On 24/03/2020 07:04, Muhammad Bilal wrote:
>> On Fri, Mar 20, 2020 at 3:02 PM Kevin Traynor wrote:
>>>
>>> On 20/03/2020 09:53, Muhammad Bilal wrote:
On Fri, Mar 20, 2020 at 2:39 PM Luca Boccassi wrote:
>
> On Fri, 2020-03-20 at 13:02 +0500
On Sat, Apr 4, 2020 at 4:35 PM Jerin Jacob Kollanukkaran
wrote:
>
> > -Original Message-
> > From: Mattias Rönnblom
> > Sent: Monday, March 9, 2020 12:21 PM
> > To: Jerin Jacob Kollanukkaran
> > Cc: dev@dpdk.org; stefan.sundkv...@ericsson.com; ola.liljed...@arm.com;
> > Mattias Rönnblom
On Tue, 14 Apr 2020 09:48:31 +
Ori Kam wrote:
> > @@ -2299,6 +2299,22 @@ cmd_config_rss_parsed(void *parsed_result,
> > rss_conf.rss_hf = ETH_RSS_L4_SRC_ONLY;
> > else if (!strcmp(res->value, "l4-dst-only"))
> > rss_conf.rss_hf = ETH_RSS_L4_DST_ONLY;
> > + else i
1 - 100 of 284 matches
Mail list logo
