This patchset includes some improvements in the
Crypto performance application, including app fixes
and new parameter additions.
The last patch, in particular, introduces performance improvements.
Currently, crypto operations are allocated in a mempool and mbufs
in a different one. Then mbufs are
Instead of parsing number of segments, from the command line,
parse segment size, as it is a more usual case to have
the segment size fixed and then different packet sizes
will require different number of segments.
Signed-off-by: Pablo de Lara
---
app/test-crypto-perf/cperf_ops.c | 2
Instead of prepending the AAD (Additional Authenticated Data)
in the mbuf, it is easier to set after the crypto operation,
as it is a read-only value, like the IV, and then it is not
restricted to the size of the mbuf headroom.
Signed-off-by: Pablo de Lara
---
app/test-crypto-perf/cperf_ops.c
Since DPDK 17.08, there is specific parameters
for AEAD algorithm, like AES-GCM. When verifying
crypto operations with test vectors, the parser
was not reading AEAD data (such as IV or key).
Fixes: 8a5b494a7f99 ("app/test-crypto-perf: add AEAD parameters")
Cc: sta...@dpdk.org
Signed-off-by: Pablo
When running the verify test, mbufs in the pool were
populated with the test vector loaded from a file.
To avoid limiting the number of operations to the pool size,
mbufs will be rewritten with the test vector, before
linking them to the crypto operations.
Signed-off-by: Pablo de Lara
---
app/te
Add parameter "qps" in crypto performance app,
to create multiple queue pairs per device.
This new parameter is useful to have multiple logical
cores using a single crypto device, without needing
to initialize a crypto device per core.
Signed-off-by: Pablo de Lara
---
app/test-crypto-perf/cperf
For throughput and latency tests, it is not required
to populate the mbufs with any test vector.
For verify test, there is already a function that rewrites
the mbufs every time they are going to be used with
crypto operations.
Signed-off-by: Pablo de Lara
---
app/test-crypto-perf/cperf_test_late
In order to improve memory utilization, a single mempool
is created, containing the crypto operation and mbufs
(one if operation is in-place, two if out-of-place).
This way, a single object is allocated and freed
per operation, reducing the amount of memory in cache,
which improves scalability.
Si
13/09/2017 16:35, alan somers:
> On Wed, Sep 13, 2017 at 3:37 AM, Thomas Monjalon wrote:
> > 27/07/2017 22:12, asom...@gmail.com:
> >> From: Alan Somers
> >>
> >> "/bin/bash" is a Linuxism. "/usr/bin/env bash" is portable.
> >
> > Why is it an issue?
> >
> > Can you run dpdk-setup.sh on a non-Li
On Wed, Sep 13, 2017 at 9:39 AM, Thomas Monjalon wrote:
> 13/09/2017 16:35, alan somers:
>> On Wed, Sep 13, 2017 at 3:37 AM, Thomas Monjalon wrote:
>> > 27/07/2017 22:12, asom...@gmail.com:
>> >> From: Alan Somers
>> >>
>> >> "/bin/bash" is a Linuxism. "/usr/bin/env bash" is portable.
>> >
>> >
> -Original Message-
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Andriy Berestovskyy
> Sent: Friday, September 1, 2017 5:15 PM
> To: Evgeny Agronsky
> Cc: DPDK
> Subject: Re: [dpdk-dev] Why cuckoo based hashing in DPDK library?
>
> Hey Evgeny,
> Please see inline.
>
> On Th
13/09/2017 17:55, alan somers:
> On Wed, Sep 13, 2017 at 9:39 AM, Thomas Monjalon wrote:
> > 13/09/2017 16:35, alan somers:
> >> On Wed, Sep 13, 2017 at 3:37 AM, Thomas Monjalon
> >> wrote:
> >> > 27/07/2017 22:12, asom...@gmail.com:
> >> >> From: Alan Somers
> >> >>
> >> >> "/bin/bash" is a Li
On Wed, 2017-09-13 at 15:12 +0100, Bruce Richardson wrote:
> V3 changes:
> * used two-digit version numbers when globally versioning
> libs+drivers
> * removed symlinks with single-digit version numbers in same case
> * moved driver install location from $prefix/dpdk to
> $prefix/share/dpdk
>
> V2
On Wed, 2017-09-13 at 14:11 +0100, Bruce Richardson wrote:
> On Wed, Sep 13, 2017 at 12:32:24PM +0100, Luca Boccassi wrote:
> > On Tue, 2017-09-12 at 11:38 +0100, Bruce Richardson wrote:
> > > Normally, each library has it's own version number based on the
> > > ABI.
> > > Add an option to have all
On 9/13/2017 3:25 PM, Hu, Xuekun wrote:
> I met the same issue too, only with i40e 2.1.26 PF kernel driver.
>
> -Original Message-
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ferruh Yigit
> Sent: Wednesday, September 13, 2017 7:04 PM
> To: Shijith Thotton ; Yang, Qiming
> ; Gr
Signed-off-by: Ravi Kumar
---
config/common_base | 6 ++
mk/rte.app.mk | 1 +
2 files changed, 7 insertions(+)
diff --git a/config/common_base b/config/common_base
index 5e97a08..fab46b5 100644
--- a/config/common_base
+++ b/config/common_base
@@ -405,6 +405,12 @@ CONFIG_RTE_LIBRTE_AVP_
Signed-off-by: Ravi Kumar
---
MAINTAINERS | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index a0cd75e..0110d54 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -308,6 +308,12 @@ F: drivers/net/ena/
F: doc/guides/nics/ena.rst
F: doc/guides/nics/
Signed-off-by: Ravi Kumar
---
doc/guides/nics/axgbe.rst | 214 +
doc/guides/nics/features/axgbe.ini | 18
doc/guides/nics/index.rst | 1 +
3 files changed, 233 insertions(+)
create mode 100644 doc/guides/nics/axgbe.rst
create mode 10
> On Sep 13, 2017, at 3:50 AM, Shahaf Shuler wrote:
>
> The current Tx error counter counts, according to its description,
> the total number of packets not sent when TX ring full. It is reported
> to application as part of oerrors field.
>
> The drop due to full ring is not the statistic that
> On Sep 13, 2017, at 3:50 AM, Shahaf Shuler wrote:
>
> Tx error counter lacks the logic of incrementation, making it useless for
> applications.
>
> Fixes: 87011737b715 ("mlx5: add software counters")
> Cc: sta...@dpdk.org
> Cc: adrien.mazarg...@6wind.com
>
> Signed-off-by: Shahaf Shuler
> -
-Original Message-
From: dev on behalf of Ferruh Yigit
Date: Wednesday, September 13, 2017 at 10:06 AM
To: "Hu, Xuekun" , "Thotton, Shijith"
, "Yang, Qiming" ,
Gregory Etelson
Cc: "dev@dpdk.org" , "Tan, Jianfeng"
Subject: Re: [dpdk-dev] vf init issue with patch igb_uio: issue FLR duri
On Wed, Sep 13, 2017 at 01:50:39PM +0300, Shahaf Shuler wrote:
> Mellanox NICs has a limitation on the number of mbuf segments a multi
> segment mbuf can have. The max number depends on the Tx offloads requested.
>
> The current code not enforce such limitation, which might cause
> malformed work
23/07/2017 11:15, Wei Dai:
> --- a/lib/librte_ether/rte_ether_version.map
> +++ b/lib/librte_ether/rte_ether_version.map
This file is now lib/librte_ether/rte_ethdev_version.map
> @@ -155,6 +155,7 @@ DPDK_17.08 {
> rte_eth_dev_adjust_nb_rx_tx_desc;
> rte_flow_copy;
> rte_f
23/07/2017 11:15, Wei Dai:
> +/*
> + * Reest PF device.
> + */
Small typo here. I will fix it.
VFs rely on config BAR for getting the MAC address, although a
random one is created a valid address is not found.
A PF port has a fixed MAC which is currently acquired using the NSPU
interface. Some NFP firmwares require the MAC being writing back
to the config BAR for doing proper MAC filtering.
> Wei Dai (5):
> ethdev: add support of NIC reset
> net/ixgbe: add support of reset
> net/i40e: add support of reset
> app/testpmd: enhance command to test NIC reset
> doc: add description of the NIC reset API
Sorry for being so late to get back to it.
Applied, thanks for the improvemen
> -Original Message-
> From: Thomas Monjalon [mailto:tho...@monjalon.net]
> Sent: Wednesday, September 13, 2017 2:21 PM
> To: Ananyev, Konstantin ;
> step...@networkplumber.org
> Cc: dev@dpdk.org; Shahaf Shuler
> Subject: Re: [dpdk-dev] [PATCH 4/4] ethdev: add helpers to move to the new
This commit adds support for enabling multiple pending timer lists in
each lcore's priv_timer struct with a new function; a single list is still
used by default. In the case that multiple lcores repeatedly install
timers on the same target lcore, this option reduces lock contention for
the target l
This commit adds support for timers being created from
non-EAL threads; it maps timers from all such threads to
lcore id RTE_MAX_LCORE, and puts them all in a corresponding
skiplist.
Signed-off-by: Erik Gabriel Carrillo
---
V3:
* Rebased patch on reworked parent commit
v2:
* Address checkpatch
In the current implementation of the DPDK timer library, timers can be
created and set to be handled by a target lcore by adding it to a
skiplist that corresponds to that lcore. However, if an application
enables multiple lcores, and each of these lcores repeatedly attempts
to install timers on th
This change updates the timer library documentation to
reflect a change to the organization of the skiplists
in the implementation.
Signed-off-by: Erik Gabriel Carrillo
---
v3
* Updated implementation details section of timer_lib.rst to reflect the
addition of the option to use multiple pendin
Hi Jiayu,
> >
> >
> > > -Original Message-
> > > From: Ananyev, Konstantin
> > > Sent: Tuesday, September 12, 2017 12:18 PM
> > > To: Hu, Jiayu ; dev@dpdk.org
> > > Cc: Kavanagh, Mark B ; Tan, Jianfeng
> > >
> > > Subject: RE: [PATCH v3 2/5] gso: add TCP/IPv4 GSO support
> > >
> > > > r
rx/tx_queue_setup functions are shared between PF and VF
drivers. So the var 'pf' should not be assigned at the beginning.
This patch fixes the issue, and also corrects the return err code.
Cc: sta...@dpdk.org
Fixes: b6583ee40265 ("i40e: full VMDQ pools support")
Signed-off-by: Jingjing Wu
---
Hi Konstantin,
> -Original Message-
> From: Ananyev, Konstantin
> Sent: Wednesday, September 13, 2017 11:13 PM
> To: Kavanagh, Mark B ; Hu, Jiayu
>
> Cc: dev@dpdk.org; Tan, Jianfeng
> Subject: RE: [PATCH v3 2/5] gso: add TCP/IPv4 GSO support
>
> Hi Mark,
>
> > -Original Message
> -Original Message-
> From: Shijith Thotton [mailto:shijith.thot...@caviumnetworks.com]
> Sent: Wednesday, September 13, 2017 6:49 PM
> To: Yang, Qiming ; Gregory Etelson
>
> Cc: dev@dpdk.org; Tan, Jianfeng ; Yigit, Ferruh
>
> Subject: Re: vf init issue with patch igb_uio: issue FLR duri
Hi, Thomas
Thanks for your great support !
> -Original Message-
> From: Thomas Monjalon [mailto:tho...@monjalon.net]
> Sent: Thursday, September 14, 2017 5:02 AM
> To: Dai, Wei
> Cc: dev@dpdk.org; Lu, Wenzhuo ; Ananyev,
> Konstantin ; Wu, Jingjing
> ; Xing, Beilei
> Subject: Re: [dpdk-d
> -Original Message-
> From: Wu, Jingjing
> Sent: Wednesday, September 13, 2017 7:54 AM
> To: Xing, Beilei
> Cc: dev@dpdk.org; Wu, Jingjing ; sta...@dpdk.org
> Subject: [PATCH v3] net/i40e: fix memory leak if VF init fails
>
> Cc: sta...@dpdk.org
> Fixes: 4861cde46116 ("i40e: new poll m
On Sep 13, 2017 7:54 AM, "Adrien Mazarguil"
wrote:
On Wed, Sep 13, 2017 at 02:21:00PM +0100, Ferruh Yigit wrote:
> On 9/13/2017 1:25 PM, Adrien Mazarguil wrote:
> > On Wed, Sep 13, 2017 at 12:38:37PM +0100, Ferruh Yigit wrote:
> >> On 9/13/2017 8:58 AM, Adrien Mazarguil wrote:
> >>> Hi,
> >>>
> >
Signed-off-by: Jianfeng Tan
Acked-by: Bruce Richardson
---
MAINTAINERS | 2 -
app/test-pmd/Makefile | 4 -
app/test-pmd/testpmd.c | 51 +-
config/common_base | 5 -
config/d
Signed-off-by: Jianfeng Tan
Acked-by: Bruce Richardson
---
MAINTAINERS |1 -
examples/Makefile |1 -
examples/vhost_xen/Makefile | 52 --
examples/vhost_xen/main.c | 1522 ---
examples/vhost_xen
v2:
- Address Bruce's comment on testpmd's memory init for xenvirt in patch 2.
- Update rte_eal_version.map in bsd and eal.
- Switch patch 5 and patch 6 so that we bump library version just once.
Following the calls on the mailing list:
http://dpdk.org/ml/archives/dev/2017-June/068151.ht
Previously, to get MFN address in dom0, this API is a wrapper to
obtain the "physical address".
As we will removed xen dom0 support, this API is not necessary.
Signed-off-by: Jianfeng Tan
Acked-by: Bruce Richardson
---
doc/guides/rel_notes/release_17_11.rst | 2 ++
drivers/net/e1000/em_rx
Signed-off-by: Jianfeng Tan
Acked-by: Bruce Richardson
---
examples/ip_pipeline/app.h | 4 --
examples/ip_pipeline/config_parse.c | 19 -
examples/ip_pipeline/init.c | 5 ---
examples/kni/main.c | 3 --
test/test/process.h | 10 -
t
Signed-off-by: Jianfeng Tan
Acked-by: Bruce Richardson
---
drivers/crypto/qat/qat_qp.c | 7 +--
drivers/net/i40e/i40e_rxtx.c | 8 ++--
lib/librte_ether/rte_ethdev.c| 7 +--
lib/librte_mempool/rte_mempool.c | 8 ++--
4 files changed, 6 insertions(+), 24 deletions(-)
The kernel patch was merged to support pci resource mapping.
https://patchwork.kernel.org/patch/9677441/
So enable igu_uio in the default arm64 configuration.
Signed-off-by: Jianbo Liu
---
config/common_armv8a_linuxapp | 2 --
1 file changed, 2 deletions(-)
diff --git a/config/common_armv8a_li
On 9/13/2017 7:06 PM, Nipun Gupta wrote:
-Original Message-
From: Nikhil Rao [mailto:nikhil@intel.com]
---
lib/librte_eventdev/rte_eventdev_pmd.h | 6 ++
drivers/event/sw/sw_evdev.c| 13 +
2 files changed, 19 insertions(+)
diff --git a/lib/librte_e
> -Original Message-
> From: Yigit, Ferruh
> Sent: Thursday, September 14, 2017 1:06 AM
> To: Hu, Xuekun ; Shijith Thotton
> ; Yang, Qiming
> ; Gregory Etelson
> Cc: dev@dpdk.org; Tan, Jianfeng
> Subject: Re: [dpdk-dev] vf init issue with patch igb_uio: issue FLR during
> open
> and rele
-Original Message-
> Date: Thu, 14 Sep 2017 11:00:03 +0800
> From: Jianbo Liu
> To: dev@dpdk.org
> CC: jerin.ja...@caviumnetworks.com, Jianbo Liu
> Subject: [PATCH] config: enable igb_uio on arm64 platform
> X-Mailer: git-send-email 1.9.1
>
> The kernel patch was merged to support pci re
The kernel patch was merged to support pci resource mapping.
https://patchwork.kernel.org/patch/9677441/
So enable igu_uio in the default arm64 configuration.
v2:
- keep headline format
Signed-off-by: Jianbo Liu
---
config/common_armv8a_linuxapp | 2 --
1 file changed, 2 deletions(-)
diff -
Hi All,
I want to know if the community has implemented the "enqueue_zero_copy" ?
Thanks all,
Best regards
> -Original Message-
> From: Rao, Nikhil [mailto:nikhil@intel.com]
> Sent: Thursday, September 14, 2017 8:34
> To: Nipun Gupta ; jerin.ja...@caviumnetworks.com;
> bruce.richard...@intel.com
> Cc: gage.e...@intel.com; dev@dpdk.org; tho...@monjalon.net;
> harry.van.haa...@intel.com; Hem
Hi Yongseok,
Wednesday, September 13, 2017 10:52 PM, Yongseok Koh:
> >
> > +/* Maximum number of DS in WQE. */
> > +#define MLX5_MAX_DS 63
> How about make it consistent with MLX5_MPW_DSEG_MAX by naming
> MLX5_DSEG_MAX?
>
It doesn't have the same meaning.
The MLX5_MPW_DSEG_MAX is to limit the
On Thursday 14 September 2017 09:23 AM, Jianbo Liu wrote:
> The kernel patch was merged to support pci resource mapping.
> https://patchwork.kernel.org/patch/9677441/
>
> So enable igu_uio in the default arm64 configuration.
>
> v2:
> - keep headline format
>
> Signed-off-by: Jianbo Liu
> ---
Hi Konstantin,
On Thu, Sep 14, 2017 at 06:10:37AM +0800, Ananyev, Konstantin wrote:
>
> Hi Jiayu,
>
> > >
> > >
> > > > -Original Message-
> > > > From: Ananyev, Konstantin
> > > > Sent: Tuesday, September 12, 2017 12:18 PM
> > > > To: Hu, Jiayu ; dev@dpdk.org
> > > > Cc: Kavanagh, Mark
101 - 154 of 154 matches
Mail list logo
