made second cache line access behavior same as IA
Signed-off-by: Jerin Jacob
---
drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c
b/drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c
index 9c1d124..6
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
---
MAINTAINERS | 6 ++
1 file changed, 6 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index be09a98..e41ce13 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -343,6 +343,12 @@ M: Sony Chacko
F: drivers/net/qede/
F: doc/guides/
Updated doc/guides/nics/overview.rst, doc/guides/nics/thunderx.rst
and release notes
Changed "*" to "P" in overview.rst to capture the partially supported
feature as "*" creating alignment issues with Sphinx table
Signed-off-by: Jerin Jacob
Signed-off-by: Slawomir Rosek
Acked-by: John McNamara
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/nicvf_ethdev.c | 467
1 file changed, 467 insertions
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/nicvf_ethdev.c | 59 +
1 file changed, 59 insertions(
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/nicvf_ethdev.c | 167
drivers/net/thunderx/nicvf_rxt
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/nicvf_ethdev.c | 1 +
drivers/net/thunderx/nicvf_rxtx.c | 9 +
drivers/net/thun
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/nicvf_ethdev.c | 40 +
1 file changed, 40 insertions(
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
Reviewed-by: Ferruh Yigit
---
drivers/net/thunderx/nicvf_ethdev.h | 33
drivers/net/thunderx/nicvf_rxtx.c
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/Makefile | 2 +
drivers/net/thunderx/nicvf_ethdev.c | 5 +-
drivers/net/thunder
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
Reviewed-by: Ferruh Yigit
---
drivers/net/thunderx/nicvf_ethdev.c | 66 +
1 f
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/nicvf_ethdev.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/drivers/net/thu
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
Reviewed-by: Ferruh Yigit
---
drivers/net/thunderx/nicvf_ethdev.c | 44 +
dri
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
Reviewed-by: Ferruh Yigit
---
drivers/net/thunderx/nicvf_ethdev.c | 172
1 f
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/nicvf_ethdev.c | 175
1 file changed, 175 insertions
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
Reviewed-by: Ferruh Yigit
---
drivers/net/thunderx/nicvf_ethdev.c | 136
dri
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
Reviewed-by: Ferruh Yigit
---
drivers/net/thunderx/nicvf_ethdev.c | 45 +
dri
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
Reviewed-by: Ferruh Yigit
---
drivers/net/thunderx/nicvf_ethdev.c | 78 +
1 f
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/nicvf_ethdev.c | 27 +++
1 file changed, 27 insertions(+)
diff -
Extended the nicvf_interrupt function to respond
NIC_MBOX_MSG_BGX_LINK_CHANGE mbox message from PF and update
struct rte_eth_link accordingly.
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed
Introduce driver initialization and enable build infrastructure for
nicvf pmd driver.
By default, It is enabled only for defconfig_arm64-thunderx-*
config as it is an inbuilt NIC device.
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/base/nicvf_hw.c | 45
drivers/net/thunderx/base/nicv
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/base/nicvf_hw.c | 129 +++
drivers/net/thunderx/base/nicv
add nicvf hardware specific APIs for initialization and configuration.
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/base/nicvf_hw.c
DPDK nicvf driver doesn't have access to NIC's PF address space.
Introduce a mailbox mechanism to communicate with PF driver through
shared 128bit register interface.
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by
implement DPDK based platform abstraction for base code
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/base/nicvf_plat.h | 129
add HW register definitions of ThunderX inbuilt NIC
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/base/nicvf_hw_defs.h | 668 +
add HW constants of ThunderX inbuilt NIC
Signed-off-by: Jerin Jacob
Signed-off-by: Maciej Czekaj
Signed-off-by: Kamil Rytarowski
Signed-off-by: Zyta Szpak
Signed-off-by: Slawomir Rosek
Signed-off-by: Radoslaw Biernacki
---
drivers/net/thunderx/base/nicvf_hw_defs.h | 551
This patch set provides the initial version of DPDK PMD for the
built-in NIC device in Cavium ThunderX SoC family.
Implemented features and ThunderX nicvf PMD documentation added
in doc/guides/nics/overview.rst and doc/guides/nics/thunderx.rst
respectively in this patch set.
These patches are che
2016-06-17 18:04, David Marchand:
> On Fri, Jun 17, 2016 at 6:02 PM, Thomas Monjalon
> wrote:
> > rte_thread_setname was a macro defined only for Linux.
> > The function rte_thread_setname() can now be used on FreeBSD
> > as well on Linux.
> > It is required to build librte_pdump.
> >
> > The macr
On Fri, Jun 17, 2016 at 6:02 PM, Thomas Monjalon
wrote:
> rte_thread_setname was a macro defined only for Linux.
> The function rte_thread_setname() can now be used on FreeBSD
> as well on Linux.
> It is required to build librte_pdump.
>
> The macro was 0 for old glibc. The function is now returni
rte_thread_setname was a macro defined only for Linux.
The function rte_thread_setname() can now be used on FreeBSD
as well on Linux.
It is required to build librte_pdump.
The macro was 0 for old glibc. The function is now returning -1.
The related logs are decreased from error to debug level beca
On Fri, Jun 17, 2016 at 5:25 PM, Thomas Monjalon
wrote:
> rte_thread_setname was a macro defined only for Linux.
> The function rte_thread_setname() can now be used on FreeBSD
> as well on Linux.
> It is required to build librte_pdump.
>
> The macro was 0 for old glibc. The function is now returni
2016-06-17 17:02, Thomas Monjalon:
> The function rte_keepalive_register_alive_callback do not exist.
> The function rte_keepalive_register_relay_callback was missing for BSD.
>
> Fixes: 90c622f35679 ("keepalive: add liveness callback")
>
> Signed-off-by: Thomas Monjalon
Applied
On 6/14/2016 10:22 AM, Adrien Mazarguil wrote:
> A missing define in the previous patch causes additional compilation
> issues.
>
> Fixes: 3025206f5060 ("mlx: fix compilation with recent Glibc")
>
> Signed-off-by: Adrien Mazarguil
"older Glibc" or "recent Glibc" can be very hard to trace from g
On 6/10/2016 11:18 PM, Chas Williams wrote:
> From: "Charles (Chas) Williams"
>
> If MSIX is available, the vector count given by the table size is one
> less than the actual count. This count also limits the receive and
> transmit queue resources the VF can support.
>
> Fixes: 540a211084a7 ("b
rte_thread_setname was a macro defined only for Linux.
The function rte_thread_setname() can now be used on FreeBSD
as well on Linux.
It is required to build librte_pdump.
The macro was 0 for old glibc. The function is now returning -1.
The related logs are decreased from error to debug level beca
rte_keepalive_mark_sleep is also missing from both.
On 17/06/2016 16:02, Thomas Monjalon wrote:
> The function rte_keepalive_register_alive_callback do not exist.
> The function rte_keepalive_register_relay_callback was missing for BSD.
>
> Fixes: 90c622f35679 ("keepalive: add liveness callback")
On 6/8/2016 10:47 AM, Nelio Laranjeiro wrote:
> Enhance mlx5 with a data path that bypasses Verbs.
>
> The first half of this patchset removes support for functionality completely
> rewritten in the second half (scatter/gather, inline send), while the data
> path is refactored without Verbs.
>
>
The function rte_keepalive_register_alive_callback do not exist.
The function rte_keepalive_register_relay_callback was missing for BSD.
Fixes: 90c622f35679 ("keepalive: add liveness callback")
Signed-off-by: Thomas Monjalon
---
lib/librte_eal/bsdapp/eal/rte_eal_version.map | 2 +-
lib/librte
On Fri, 17 Jun 2016 14:53:38 +0100
David Hunt wrote:
> By default, the mempool ops used for mbuf allocations is a multi
> producer and multi consumer ring. We could imagine a target (maybe some
> network processors?) that provides an hardware-assisted pool
> mechanism. In this case, the default c
On Fri, 17 Jun 2016 14:53:37 +0100
David Hunt wrote:
> Create a minimal custom mempool handler and check that it
> passes basic mempool autotests.
>
> Signed-off-by: Olivier Matz
> Signed-off-by: David Hunt
> Acked-by: Shreyansh Jain
> Acked-by: Olivier Matz
> ---
Reviewed-by: Jan Viktorin
Hi David,
still few nits... Do you like the upstreaming process? :) I hope finish this
patchset soon. The major issues seem to be OK.
[...]
> +
> +/**
> + * @internal Get the mempool ops struct from its index.
> + *
> + * @param ops_index
> + * The index of the ops struct in the ops struct tab
2016-06-17 15:19, Thomas Monjalon:
> 2016-06-17 14:52, Jan Viktorin:
> > Using of the /dev/stdin generates a warning when compiling on FreeBSD:
> >
> > objcopy: Warning: '/dev/stdin' is not an ordinary file
> > app/test/Makefile:78: recipe for target 'test_resource_c.res.o' failed
> > # ls -l /
2016-06-17 15:22, Thomas Monjalon:
> The variables AESNI_MULTI_BUFFER_LIB_PATH and LIBSSO_PATH
> are not required for "make clean".
> It is the same fix as in the commit e277b2397.
Fixes: eec136f3c54f ("aesni_gcm: add driver for AES-GCM crypto operations")
Fixes: 3aafc423cf4d ("snow3g: add driver
On 07/05/16 04:45, Eric Kinzie wrote:
> Instead of a hard-coded maximum receive length, allow the bond interface
> to inherit this limit from the slave interfaces. This allows
> an application that uses jumbo frames to pass realistic values to
> rte_eth_dev_configure without causing an err
The variables AESNI_MULTI_BUFFER_LIB_PATH and LIBSSO_PATH
are not required for "make clean".
It is the same fix as in the commit e277b2397.
Signed-off-by: Thomas Monjalon
---
drivers/crypto/aesni_gcm/Makefile | 2 ++
drivers/crypto/snow3g/Makefile| 2 ++
2 files changed, 4 insertions(+)
dif
2016-06-17 14:52, Jan Viktorin:
> Using of the /dev/stdin generates a warning when compiling on FreeBSD:
>
> objcopy: Warning: '/dev/stdin' is not an ordinary file
> app/test/Makefile:78: recipe for target 'test_resource_c.res.o' failed
> # ls -l /dev/stdin
> lrwxr-xr-x 1 root wheel 4 Jun 1
Hi Olivier,
On 23/5/2016 1:55 PM, Olivier Matz wrote:
> Hi David,
>
> Please find some comments below.
>
> On 05/19/2016 04:48 PM, David Hunt wrote:
>> [...]
>> +++ b/lib/librte_mempool/rte_mempool_stack.c
>> @@ -0,0 +1,145 @@
>> +/*-
>> + * BSD LICENSE
>> + *
>> + * Copyright(c) 2010-2014 Int
On Fri, Jun 17, 2016 at 10:18:30AM +0200, Thomas Monjalon wrote:
> 2016-06-16 16:25, Jain, Deepak K:
> > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> > > 2016-06-16 16:29, Jain, Deepak K:
> > > > Due to addition of CLASS_ID in EAL, class_id is amended into the code.
> > >
> > > Wh
By default, the mempool ops used for mbuf allocations is a multi
producer and multi consumer ring. We could imagine a target (maybe some
network processors?) that provides an hardware-assisted pool
mechanism. In this case, the default configuration for this architecture
would contain a different va
Create a minimal custom mempool handler and check that it
passes basic mempool autotests.
Signed-off-by: Olivier Matz
Signed-off-by: David Hunt
Acked-by: Shreyansh Jain
Acked-by: Olivier Matz
---
app/test/test_mempool.c | 122 +++-
1 file changed, 1
Until now, the objects stored in a mempool were internally stored in a
ring. This patch introduces the possibility to register external handlers
replacing the ring.
The default behavior remains unchanged, but calling the new function
rte_mempool_set_ops_byname() right after rte_mempool_create_empt
Here's the latest version of the Mempool Handler feature (previously
known as the External Mempool Manager.
It's re-based on top of the latest head as of 17/6/2016, including
Olivier's 35-part patch series on mempool re-org [1]
[1] http://dpdk.org/ml/archives/dev/2016-May/039229.html
v14 changes
Using of the /dev/stdin generates a warning when compiling on FreeBSD:
objcopy: Warning: '/dev/stdin' is not an ordinary file
app/test/Makefile:78: recipe for target 'test_resource_c.res.o' failed
# ls -l /dev/stdin
lrwxr-xr-x 1 root wheel 4 Jun 17 12:24 /dev/stdin -> fd/0
Replace /dev/std
Information on pmdinfogen may be useful to 3rd party driver developers.
Include documentation on what it does
Signed-off-by: Neil Horman
Acked-by: Panu Matilainen
Acked-by: John McNamara
CC: Bruce Richardson
CC: Thomas Monjalon
CC: Stephen Hemminger
CC: Panu Matilainen
---
doc/guides/prog_
This tool searches for the primer sting PMD_DRIVER_INFO= in any ELF binary,
and, if found parses the remainder of the string as a json encoded string,
outputting the results in either a human readable or raw, script parseable
format
Note that, in the case of dynamically linked applications, pmdinf
Modify the compilation makefile to identify C files that export PMD
information, and use that to trigger execution of the pmdinfo binary. If
the execution of pmdinfo is successful, compile the output C file to an
object, and use the linker to do relocatable linking on the resultant
object file int
Export a symbol containing the string:
DPDK_PLUGIN_PATH="$(CONFIG_RTE_EAL_PMD_PATH)"
Where the latter half of the string is set at build time to a location from
which autoloaded DSO's will be found. This string is used by pmdinfo in
'plugin' mode, whereby a user can specify a dpdk installation di
Modify the PMD_REGISTER_DRIVER macro, adding a name argument to it. The
addition of a name argument creates a token that can be used for subsequent
macros in the creation of unique symbol names to export additional bits of
information for use by the pmdinfogen tool. For example:
PMD_REGISTER_DRI
pmdinfogen is a tool used to parse object files and build json strings for
use in later determining hardware support in a dso or application binary.
pmdinfo looks for the non-exported symbol names this_pmd_name and
this_pmd_tbl (where n is a integer counter). It records the name of
each of these t
Hey all-
So heres attempt number 2 at a method for exporting PMD hardware support
information. As we discussed previously, the consensus seems to be that pmd
information should be:
1) Able to be interrogated on any ELF binary (application binary or individual
DSO)
2) Equally functional on
On Thu, Jun 16, 2016 at 4:33 AM, Tetsuya Mukawa wrote:
> This patch fixes resource leak of pci_uio_map_secondary().
> If pci_map_resource() succeeds but mapped address is different from an
> address primary process mapped, this should be error.
> Then the addresses secondary process mapped should
On Thu, Jun 16, 2016 at 4:33 AM, Tetsuya Mukawa wrote:
> This patch fixes wrong resource release of pci_uio_unmap().
> The 'path' member of mapped_pci_resource structure is allocated by
> primary process, but currently it will be freed by both primary
> and secondary process.
> The patch fixes to
On Thu, Jun 16, 2016 at 4:33 AM, Tetsuya Mukawa wrote:
> This patch fixes wrong error checking of rte_eal_parse_devargs_str().
> Currently, a return value of strdup() is wrongly checked.
>
> Fixes: 0fe11ec592b2 ("eal: add vdev init and uninit")
> Signed-off-by: Tetsuya Mukawa
Forgot to pass this
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Pablo de Lara
> Sent: Friday, June 17, 2016 12:25 PM
> To: dev at dpdk.org
> Cc: Doherty, Declan ; De Lara Guarch, Pablo
>
> Subject: [dpdk-dev] [PATCH v2 00/11] Snow3G bit-level support
>
> The underlying li
> -Original Message-
> From: De Lara Guarch, Pablo
> Sent: Friday, June 17, 2016 11:33 AM
> To: dev at dpdk.org
> Cc: Doherty, Declan ; De Lara Guarch, Pablo
>
> Subject: [PATCH v2 0/3] Add new KASUMI SW PMD
>
> Added new SW PMD which makes use of the libsso SW library, which
> provides
The function rte_hash_get_key_with_position is added in this patch.
As the position returned when adding a key is frequently used as an
offset into an array of user data, this function performs the operation
of retrieving a key given this offset.
A possible use case would be to delete a key from t
Hi Jerin,
On 06/03/2016 09:02 AM, Jerin Jacob wrote:
> On Thu, Jun 02, 2016 at 11:16:16PM +0200, Olivier MATZ wrote:
> Hi Olivier,
>
>> This is probably more a measure of the pure CPU cost of the mempool
>> function, without considering the memory cache aspect. So, of course,
>> a real use-case t
On 06/16/2016 01:02 PM, Lazaros Koromilas wrote:
> The mempool cache is only available to EAL threads as a per-lcore
> resource. Change this so that the user can create and provide their own
> cache on mempool get and put operations. This works with non-EAL threads
> too. This commit introduces t
On 06/16/2016 01:02 PM, Lazaros Koromilas wrote:
> Re: [PATCH v3 2/3] mempool: use bit flags instead of is_mp and is_mc
There is a script to check the format of title. The underscores are
now forbidden, because it often reference function or variable names,
which is not ideal in titles.
$ ./scrip
Hi Lazaros,
On 06/16/2016 01:02 PM, Lazaros Koromilas wrote:
> Updated version of the user-owned cache patchset. It applies on top of
> the latest external mempool manager patches from David Hunt [1].
>
> [1] http://dpdk.org/ml/archives/dev/2016-June/041479.html
>
> v3 changes:
>
> * Depreca
2016-05-19 17:56, Thomas Monjalon:
> 2016-05-19 21:48, Jianbo Liu:
> > On 13 May 2016 at 23:49, Thomas Monjalon
> > wrote:
> > > 2016-05-10 14:01, Jianbo Liu:
> > >> Other APP may call rte_memcpy by function pointer,
> > >> so change it to an inline function.
> > >
> > > Any example in mind?
> >
SNOW3G and KASUMI unit tests are very similar and
they were using duplicated code, so this commit
refactor and remove some of the duplicated functions.
Signed-off-by: Pablo de Lara
---
app/test/test_cryptodev.c | 440 --
config/common_base| 4
With the new libsso library, buffers can be encrypted/decrypted,
providing an offset in bits, so an extra unit test has been
added to cover this case.
Signed-off-by: Pablo de Lara
---
app/test/test_cryptodev.c | 121 ++
1 file changed, 121 insertions(+
Snow3G PMD supports now buffers that are non byte multiple,
so tests to cover this case have been added.
Signed-off-by: Pablo de Lara
---
app/test/test_cryptodev.c | 50 ++
app/test/test_cryptodev_snow3g_hash_test_vectors.h | 107 +
2 files c
Signed-off-by: Pablo de Lara
---
app/test/test_cryptodev.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
index 7dc5648..803f1f6 100644
--- a/app/test/test_cryptodev.c
+++ b/app/test/test_cryptodev.c
@@ -4057,6 +4057,11 @@ static str
No padding was added in the input buffers for snow3G tests,
due to a wrong calculation of the length (should be multiple
of the block size). This fix takes into account the case
where the length is not byte multiple.
Fixes: 8bdf665fe6c0 ("app/test: add SNOW 3G")
Signed-off-by: Pablo de Lara
---
Instead of modifying the content of the buffers, to compare them
at bit-level, use the new macro TEST_ASSERT_BUFFERS_ARE_EQUAL_BIT,
which does not make any modifications in the buffers.
Signed-off-by: Pablo de Lara
---
app/test/test_cryptodev.c | 68 +-
The underlying libsso library supports different instruction sets
(SSE4.1, AVX), so feature flags of the crypto device must be updated
to reflect this.
Signed-off-by: Pablo de Lara
---
drivers/crypto/snow3g/rte_snow3g_pmd.c | 15 ++-
1 file changed, 14 insertions(+), 1 deletion(-)
d
Underlying libsso_snow3g library now supports bit-level
operations, so PMD has been updated to allow them.
Signed-off-by: Pablo de Lara
---
doc/guides/cryptodevs/snow3g.rst | 5 +-
drivers/crypto/snow3g/rte_snow3g_pmd.c | 193 -
2 files changed, 142 insert
In order to avoid using magic numbers, macros for
the IV and digest lengths for Snow3G have been added.
Signed-off-by: Pablo de Lara
---
drivers/crypto/snow3g/rte_snow3g_pmd.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/crypto/snow3g/rte_snow3g_pmd.c
b/dr
With the library update, the way to compile the library
has changed, so documentation reflects this change.
Also, the patch to fix the compilation issues present with gcc > 5.0
has been removed, as the issues have been fixed in the library.
Signed-off-by: Pablo de Lara
---
doc/guides/cryptodevs/
The underlying libsso library that SNOW3G PMD uses has been updated,
so now it is called libsso_snow3g. Also, the path to the library
has been renamed to reflect this changes (now called LIBSSO_SNOW3G_PATH).
Signed-off-by: Pablo de Lara
---
doc/guides/cryptodevs/snow3g.rst | 6 +++---
drivers/c
The underlying library libsso for Snow3G PMD will be updated,
to support bit-level operations.
This patchset updates the PMD to allow the user to perform
crypto operation at bit-level. It also add some unit tests and
updates the documentation.
This patchset depends on the following patchset:
"Add
2016-06-13 16:51, Remy Horton:
>
> On 12/06/2016 15:51, Zyta Szpak wrote:
> > I would prefer having only one function rte_eth_dev_get_regs()
> > which returns length and width if data is NULL.
> > The first call is a parameter request before buffer allocation,
> > and the second ca
Hi David,
If you plan to do a v14 for this API comment, I'm wondering if the
documentation could be slightly modified too. I think "external mempool
manager" was the legacy name for the feature, but maybe it could be
changed in "alternative mempool handlers" or "changing the me
Hi David,
While testing Lazaros' patch, I found an issue in this series.
I the test application is started with --no-huge, it does not work,
the mempool_autotest does not work. Please find the exaplanation
below:
On 06/16/2016 02:30 PM, David Hunt wrote:
> @@ -386,9 +352,9 @@ rte_mempool_populate
> > The objcopy and tar commands were printed even in quiet mode.
> > They are now replaced by a simple line and still visible in verbose mode.
> >
> > Fixes: ab64f5df8004 ("app/test: support resources externally linked")
> > Fixes: 66819e6c11d8 ("app/test: support resources archived by tar")
> >
On 17/6/2016 11:18 AM, Olivier Matz wrote:
> Hi David,
>
> While testing Lazaros' patch, I found an issue in this series.
> I the test application is started with --no-huge, it does not work,
> the mempool_autotest does not work. Please find the exaplanation
> below:
>
> On 06/16/2016 02:30 PM, Da
Signed-off-by: Pablo de Lara
---
app/test/test_cryptodev.c | 995 +++--
app/test/test_cryptodev.h | 1 +
app/test/test_cryptodev_kasumi_hash_test_vectors.h | 260 ++
app/test/test_cryptodev_kasumi_test_vectors.h | 308 ++
In order to compare buffers with length and offset in bits,
new macros have been created, which use the previous compare function
to compare full bytes and then, compare first and last bytes of
each buffer separately.
Signed-off-by: Pablo de Lara
---
app/test/test.h | 57
Added new SW PMD which makes use of the libsso_kasumi SW library,
which provides wireless algorithms KASUMI F8 and F9
in software.
This PMD supports cipher-only, hash-only and chained operations
("cipher then hash" and "hash then cipher") of the following
algorithms:
- RTE_CRYPTO_SYM_CIPHER_KASUMI
Added new SW PMD which makes use of the libsso SW library,
which provides wireless algorithms KASUMI F8 and F9
in software.
This PMD supports cipher-only, hash-only and chained operations
("cipher then hash" and "hash then cipher") of the following
algorithms:
- RTE_CRYPTO_SYM_CIPHER_KASUMI_F8
- R
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 unix domain socket and
add it to the unix socket list. This is problematic for applications
which would like to set the permissions, or applications which are not
directly allowed
On Tue, Jun 14, 2016 at 05:03:57AM +, Chen, Jing D wrote:
> Hi,
>
> > -Original Message-
> > From: Wang, Xiao W
> > Sent: Monday, June 06, 2016 5:01 PM
> > To: Chen, Jing D
> > Cc: dev at dpdk.org; Wang, Xiao W
> > Subject: [PATCH] fm10k: fix VF cannot receive broadcast traffic
> >
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wenzhuo Lu
> Sent: Thursday, May 05, 2016 10:34 PM
> To: dev at dpdk.org
> Cc: Lu, Wenzhuo
> Subject: [dpdk-dev] [PATCH v3] ethdev: fix DCB config issue on ixgbe
>
> An issue is found that DCB cannot be config
On Mon, Jun 06, 2016 at 05:00:47PM +0800, Wang Xiao W wrote:
> When app tries promisc/allmulti setting, fm10k will check if a valid glort
> is acquired, if not then exit without doing anything. It's a long journey
> for VF to acquire glort info from VF to PF mailbox, PF to switch mailbox.
> It coul
2016-06-17 09:42, Hunt, David:
>
> On 17/6/2016 9:08 AM, Olivier Matz wrote:
> > Hi David,
> >
> > On 06/17/2016 08:58 AM, Hunt, David wrote:
> >> A comment below:
> >>
> >> On 16/6/2016 1:30 PM, David Hunt wrote:
> >>> +/**
> >>> + * Set the ops of a mempool.
> >>> + *
> >>> + * This can only be
On Mon, Mar 28, 2016 at 04:48:17PM +0800, Jianbo Liu wrote:
> Hi Qian,
>
> On 28 March 2016 at 10:30, Xu, Qian Q wrote:
> > Jianbo
> > Could you tell me the case that can reproduce the issue? We can help
> > evaluate the impact of performance on ixgbe, but I'm not sure how to check
> > if your
1 - 100 of 111 matches
Mail list logo
