During SA creation, if the required algorithm is not supported,
drivers can return ENOTSUP. But in most of the IPsec test cases,
if the SA creation does not success, it just returns
TEST_FAILED.
This patch fixes this issue by returning the actual return values
from the driver to the application, s
When key length is 0, zmalloc will return NULL pointor
and in that case it should not return NOMEM.
So in this patch, adding a check on key length.
Fixes: 8d1f3a5d751b ("crypto/dpaa2_sec: support crypto operation")
Cc: akhil.go...@nxp.com
---
drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c | 2 +-
1
Hi,
> -Original Message-
> From: Suanming Mou
> Sent: Sunday, January 24, 2021 1:02 PM
> To: Slava Ovsiienko ; Matan Azrad
>
> Cc: Raslan Darawsheh ; dev@dpdk.org;
> sta...@dpdk.org
> Subject: [PATCH 3/4] net/mlx5: fix secondary process attach port Tx queue
>
> Currently, the secondary
Hi,
> -Original Message-
> From: Suanming Mou
> Sent: Sunday, January 24, 2021 1:02 PM
> To: Slava Ovsiienko ; Matan Azrad
>
> Cc: Raslan Darawsheh ; dev@dpdk.org
> Subject: [PATCH 1/4] net/mlx5: fix invalid multi-process ID
>
> The device port_id is used for inter-process communication
Here fixed 3 issues for 'max-pkt-len':
1. When cmdline option '--max-pkt-len' set the value less then
'1500 + overhead', the app/testpmd will force to resize the 'max-pkt-len'
to '1500 + overhead'. However, the user really want to configure
'max-pkt-len' to a specified value (< 1500 + over
The MTU value should be updated to 'max_rx_pkt_len - overhead'
no matter if the JUMBO FRAME offload enabled. If not update this MTU,
use will get the wrong MTU info via some command.
E.g.: 'show port info all' in testpmd tool.
Actually, the 'max_rx_pkt_len' has been used for other purposes in many
Moved the setting of 'DEV_RX_OFFLOAD_JUMBO_FRAME' from
'cmd_config_max_pkt_len_parsed()' to 'init_config()' caused fail the case
where 'max_rx_pkt_len' is changed from the command line via
"port config all max-pkt-len".
The 'init_config()' function is only called when testpmd is started,
but the D
On 1/22/21 8:58 PM, Thomas Monjalon wrote:
> If a failure happens when closing a port,
> it was unnecessarily failing again in the function eth_err(),
> because of a check against HW removal cause.
> Indeed there is a big chance the port is released at this point.
> Given the port is in the middle
On Fri, Jan 22, 2021 at 1:19 PM Dumitrescu, Cristian
wrote:
> > This python script provides a shebang that only makes sense if the
> > script has the executable bit set.
> >
> > Fixes: fde7a772701a ("examples/pipeline: add VXLAN encapsulation
> > example")
> > Cc: sta...@dpdk.org
> >
> > Reported-
On 23/01/2021 11:38, Thomas Monjalon wrote:
> 22/01/2021 23:24, Dmitry Kozlyuk:
>> On Fri, 22 Jan 2021 21:57:15 +0100, Thomas Monjalon wrote:
>>> 22/01/2021 21:31, Dmitry Kozlyuk:
On Wed, 20 Jan 2021 11:24:21 +0100, Thomas Monjalon wrote:
> 20/01/2021 08:23, Dmitry Kozlyuk:
>> O
On 1/22/2021 2:18 AM, Huang, Wei wrote:
-Original Message-
From: Ferruh Yigit
Sent: Friday, January 22, 2021 00:30
To: Huang, Wei ; dev@dpdk.org; Xu, Rosen ;
Zhang, Qi Z
Cc: sta...@dpdk.org; Zhang, Tianfei ; Ray Kinsella
Subject: Re: [dpdk-stable] [PATCH v11 1/4] raw/ifpga: add f
On 25/01/2021 09:25, Kinsella, Ray wrote:
>
>
> On 23/01/2021 11:38, Thomas Monjalon wrote:
>> 22/01/2021 23:24, Dmitry Kozlyuk:
>>> On Fri, 22 Jan 2021 21:57:15 +0100, Thomas Monjalon wrote:
22/01/2021 21:31, Dmitry Kozlyuk:
> On Wed, 20 Jan 2021 11:24:21 +0100, Thomas Monjalon wrote
On Mon, 25 Jan 2021 09:25:51 +, Kinsella, Ray wrote:
> On 23/01/2021 11:38, Thomas Monjalon wrote:
> > 22/01/2021 23:24, Dmitry Kozlyuk:
> >> On Fri, 22 Jan 2021 21:57:15 +0100, Thomas Monjalon wrote:
> >>> 22/01/2021 21:31, Dmitry Kozlyuk:
> On Wed, 20 Jan 2021 11:24:21 +0100, Thoma
On 1/22/2021 2:43 AM, Huang, Wei wrote:
-Original Message-
From: Ferruh Yigit
Sent: Friday, January 22, 2021 00:30
To: Huang, Wei ; dev@dpdk.org; Xu, Rosen ;
Zhang, Qi Z
Cc: sta...@dpdk.org; Zhang, Tianfei
Subject: Re: [dpdk-dev] [PATCH v11 3/4] raw/ifpga: add OPAE API for OpenStac
Hi,
> -Original Message-
> From: Suanming Mou
> Sent: Sunday, January 24, 2021 1:02 PM
> To: Slava Ovsiienko ; Matan Azrad
>
> Cc: Raslan Darawsheh ; dev@dpdk.org
> Subject: [PATCH 0/4] net/mlx: fix secondary process bugs
>
> This patch series fix several secondary process bugs.
>
> Su
On 25/01/2021 10:05, Dmitry Kozlyuk wrote:
> On Mon, 25 Jan 2021 09:25:51 +, Kinsella, Ray wrote:
>> On 23/01/2021 11:38, Thomas Monjalon wrote:
>>> 22/01/2021 23:24, Dmitry Kozlyuk:
On Fri, 22 Jan 2021 21:57:15 +0100, Thomas Monjalon wrote:
> 22/01/2021 21:31, Dmitry Kozlyuk:
24/01/2021 21:51, Dmitry Kozlyuk:
> --- a/devtools/libabigail.abignore
> +++ b/devtools/libabigail.abignore
> +; Ignore all changes in generated PMD information strings.
> +[suppress_variable]
> +name_regex = _pmd_info$
I will fix this rule as "name_regexp" so it works.
Hi Feifei,
> > > For legacy modes, rename ring_generic/c11 to ring_generic/c11_pvt.
> > > Furthermore, add new file ring_elem_pvt.h which includes
> > > ring_do_eq/deq and ring element copy/delete APIs.
> > >
> > > For other modes, rename xx_c11_mem to xx_elem_pvt. Move all private
> > > APIs in
>
> Hi Feifei,
>
> > > > For legacy modes, rename ring_generic/c11 to ring_generic/c11_pvt.
> > > > Furthermore, add new file ring_elem_pvt.h which includes
> > > > ring_do_eq/deq and ring element copy/delete APIs.
> > > >
> > > > For other modes, rename xx_c11_mem to xx_elem_pvt. Move all priv
On Mon, Jan 25, 2021 at 11:01 AM Kinsella, Ray wrote:
>
>
>
> On 25/01/2021 09:25, Kinsella, Ray wrote:
> >
> >
> > On 23/01/2021 11:38, Thomas Monjalon wrote:
> >> 22/01/2021 23:24, Dmitry Kozlyuk:
> >>> On Fri, 22 Jan 2021 21:57:15 +0100, Thomas Monjalon wrote:
> 22/01/2021 21:31, Dmitry Ko
On Mon, 25 Jan 2021 10:11:07 +, Kinsella, Ray wrote:
> On 25/01/2021 10:05, Dmitry Kozlyuk wrote:
> > On Mon, 25 Jan 2021 09:25:51 +, Kinsella, Ray wrote:
> >> On 23/01/2021 11:38, Thomas Monjalon wrote:
> >>> 22/01/2021 23:24, Dmitry Kozlyuk:
> On Fri, 22 Jan 2021 21:57:15 +01
Hi,
> -Original Message-
> From: Ferruh Yigit
> Sent: Monday, January 25, 2021 18:01
> To: Huang, Wei ; dev@dpdk.org; Xu, Rosen
> ; Zhang, Qi Z
> Cc: sta...@dpdk.org; Zhang, Tianfei ; Ray Kinsella
>
> Subject: Re: [dpdk-stable] [PATCH v11 1/4] raw/ifpga: add fpga rsu function
>
> On 1/
On 25/01/2021 10:29, David Marchand wrote:
> On Mon, Jan 25, 2021 at 11:01 AM Kinsella, Ray wrote:
>>
>>
>>
>> On 25/01/2021 09:25, Kinsella, Ray wrote:
>>>
>>>
>>> On 23/01/2021 11:38, Thomas Monjalon wrote:
22/01/2021 23:24, Dmitry Kozlyuk:
> On Fri, 22 Jan 2021 21:57:15 +0100, Thoma
On 22-Jan-21 5:58 PM, Thomas Monjalon wrote:
If a failure happens when closing a port,
it was unnecessarily failing again in the function eth_err(),
because of a check against HW removal cause.
Indeed there is a big chance the port is released at this point.
Given the port is in the middle (or at
On 1/20/21 1:56 AM, Thomas Monjalon wrote:
> 19/01/2021 22:24, Maxime Coquelin:
>> This patch adds an helper macro to get the rte_vdev_device
>> pointer from a rte_eth_dev pointer.
>>
>> This is similar to RTE_ETH_DEV_TO_PCI().
> [...]
>> --- a/drivers/bus/vdev/rte_bus_vdev.h
>> +++ b/drivers/bu
25/01/2021 11:46, Kinsella, Ray:
> On 25/01/2021 10:29, David Marchand wrote:
> > The symbol itself can be hidden from the ABeyes.
> > It is only a placeholder for the PMD_INFO_STRING= string used by
> > usertools/dpdk-pmdinfo.py and maybe some other parsing tool.
> >
> > I guess a static symbol w
25/01/2021 11:53, Maxime Coquelin:
>
> On 1/20/21 1:56 AM, Thomas Monjalon wrote:
> > 19/01/2021 22:24, Maxime Coquelin:
> >> This patch adds an helper macro to get the rte_vdev_device
> >> pointer from a rte_eth_dev pointer.
> >>
> >> This is similar to RTE_ETH_DEV_TO_PCI().
> > [...]
> >> --- a/
在 2020/10/18 17:25, David Marchand 写道:
On Thu, Oct 8, 2020 at 5:31 PM wrote:
From: Rohit Raj
Certain bus objects may need to be closed and re-acquired
while terminating and rerunning the client application.
Hence a signal handler is required to catch the termination
of the App and hence c
Change the fatal returning to print the error message only, so that the
VF device can continue to be configured.
Fixes: 1c301e8c3cff ("net/iavf: support new VLAN capabilities")
Signed-off-by: Haiyue Wang
---
drivers/net/iavf/iavf_ethdev.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
On 1/21/2021 3:27 PM, Lance Richardson wrote:
On Mon, Jan 18, 2021 at 2:08 AM Steve Yang wrote:
When the max rx packet length is smaller than the sum of mtu size and
ether overhead size, it should be enlarged, otherwise the VLAN packets
will be dropped.
Removed the rx_offloads assignment for
On 1/19/2021 8:46 AM, oulijun wrote:
在 2021/1/18 18:42, Ferruh Yigit 写道:
On 1/15/2021 10:44 AM, oulijun wrote:
Hi Steve
This is a very good job! But I have some question and suggestions.
Please check it.
在 2021/1/14 17:45, Steve Yang 写道:
Ethdev is using default Ethernet overhead to decide i
On 1/21/21 7:47 AM, Xia, Chenbo wrote:
> Hi Maxime,
>
>> -Original Message-
>> From: Maxime Coquelin
>> Sent: Wednesday, January 20, 2021 5:25 AM
>> To: dev@dpdk.org; Xia, Chenbo ; olivier.m...@6wind.com;
>> amore...@redhat.com; david.march...@redhat.com
>> Cc: Maxime Coquelin
>> Subj
25/01/2021 10:13, Andrew Rybchenko:
> On 1/22/21 8:58 PM, Thomas Monjalon wrote:
> > If a failure happens when closing a port,
> > it was unnecessarily failing again in the function eth_err(),
> > because of a check against HW removal cause.
> > Indeed there is a big chance the port is released at
On 1/25/2021 9:49 AM, Yang, SteveX wrote:
Hi Huisong,
Thanks for your review.
The validity of the pair should be
checked from application layer (e.g.: testpmd),
and the RTE layer should keep open enough to adapt the high-layer requirement.
I’m not sure if exists some applications/NICs that
25/01/2021 13:37, Thomas Monjalon:
> 25/01/2021 10:13, Andrew Rybchenko:
> > On 1/22/21 8:58 PM, Thomas Monjalon wrote:
> > > If a failure happens when closing a port,
> > > it was unnecessarily failing again in the function eth_err(),
> > > because of a check against HW removal cause.
> > > Indeed
On 1/21/21 8:12 AM, Xia, Chenbo wrote:
> Hi Maxime,
>
>> -Original Message-
>> From: Maxime Coquelin
>> Sent: Wednesday, January 20, 2021 5:25 AM
>> To: dev@dpdk.org; Xia, Chenbo ; olivier.m...@6wind.com;
>> amore...@redhat.com; david.march...@redhat.com
>> Cc: Maxime Coquelin
>> Subj
On 1/25/2021 8:32 AM, Steve Yang wrote:
The MTU value should be updated to 'max_rx_pkt_len - overhead'
no matter if the JUMBO FRAME offload enabled. If not update this MTU,
use will get the wrong MTU info via some command.
E.g.: 'show port info all' in testpmd tool.
Actually, the 'max_rx_pkt_len
> -Original Message-
> From: Xing, Beilei
> Sent: Friday, January 22, 2021 12:05 PM
> To: Ding, Xuan ; Zhang, Qi Z ;
> Wu, Jingjing
> Cc: dev@dpdk.org; sta...@dpdk.org
> Subject: RE: [PATCH] net/iavf: fix symmetric rule creating
>
>
>
> > -Original Message-
> > From: Ding, X
> -Original Message-
> From: Wang, Haiyue
> Sent: Friday, January 22, 2021 2:12 PM
> To: Guo, Junfeng ; Zhang, Qi Z
> ; Wu, Jingjing ; Xing, Beilei
>
> Cc: dev@dpdk.org; Zhang, Yuying
> Subject: RE: [PATCH] net/iavf: fix port VLAN cfg err for AVF with SVM
>
> > -Original Message-
> Dmitry Kozlyuk (3):
> pmdinfogen: add Python implementation
> build: use Python pmdinfogen
> pmdinfogen: remove C implementation
Applied, thanks.
Reminder: it is adding a NEW DEPENDENCY on pyelftools,
as highlighted already in CI and maintainers meetings:
http://mails.dpdk.org/arc
Include 'rte_branch_prediction.h' to get the likely/unlikely macro
definitions.
Fixes: 2173fb61 ("mcslock: add MCS queued lock implementation")
Cc: sta...@dpdk.org
Signed-off-by: Bruce Richardson
---
lib/librte_eal/include/generic/rte_mcslock.h | 1 +
1 file changed, 1 insertion(+)
diff --
For some libraries, there may be some header files which are not for direct
inclusion, but rather are to be included via other header files. To allow
later checking of these files for missing includes, we separate out the
indirect include files from the direct ones.
Signed-off-by: Bruce Richardson
As a general principle, each header file should include any other
headers it needs to provide data type definitions or macros. For
example, any header using the uintX_t types in structures or function
prototypes should include "stdint.h" to provide those type definitions.
In practice, while many,
To verify that all DPDK headers are ok for inclusion directly in a C file,
and are not missing any other pre-requisite headers, we can auto-generate
for each header an empty C file that includes that header. Compiling these
files will throw errors if any header has unmet dependencies.
To ensure on
The check-includes script allowed checking header files in a given
directory to ensure that each header compiled alone without requiring
any other header inclusions.
With header checking now being done by the chkincs app in the build
system this script can be removed.
Signed-off-by: Bruce Richard
On Thu, Jan 21, 2021 at 6:02 AM Kadam, Pallavi wrote:
> On 1/20/2021 11:21 PM, Ferruh Yigit wrote:
>
> And for the mingw, I have same result with Ali, I can reproduce with (Fedora
> MinGW 9.2.1-6.fc32).
>
> But different from the CI, I am getting the error [1] for all following files:
> rte_rando
On 1/25/2021 8:32 AM, Steve Yang wrote:
Moved the setting of 'DEV_RX_OFFLOAD_JUMBO_FRAME' from
'cmd_config_max_pkt_len_parsed()' to 'init_config()' caused fail the case
where 'max_rx_pkt_len' is changed from the command line via
"port config all max-pkt-len".
The 'init_config()' function is only
On 1/21/21 9:56 AM, Xia, Chenbo wrote:
> Hi Maxime,
>
>> -Original Message-
>> From: Maxime Coquelin
>> Sent: Wednesday, January 20, 2021 5:25 AM
>> To: dev@dpdk.org; Xia, Chenbo ; olivier.m...@6wind.com;
>> amore...@redhat.com; david.march...@redhat.com
>> Cc: Maxime Coquelin
>> Subj
On 1/21/21 9:58 AM, Xia, Chenbo wrote:
> Hi Maxime,
>
>> -Original Message-
>> From: Maxime Coquelin
>> Sent: Wednesday, January 20, 2021 5:24 AM
>> To: dev@dpdk.org; Xia, Chenbo ; olivier.m...@6wind.com;
>> amore...@redhat.com; david.march...@redhat.com
>> Cc: Maxime Coquelin
>> Subj
This series adds SSL/TLS way of cipher-auth operations support i.e. auth
generation followed by encryption and decryption followed by auth verify in
OCTEON TX, OCTEON TX2 PMDs and sample unit test application.
It also adds feature flag for SSL/TLS order in cryptodev library so that
SSL test cases a
Added device feature flag for SSL/TLS way of cipher-auth
operations support i.e. auth generation followed by encryption
and decryption followed by auth verify so that SSL/TLS tests
are skipped if the device doesn't support the feature.
Signed-off-by: Tejasree Kondoj
---
doc/guides/cryptodevs/fea
Adding support for SSL/TLS way of cipher-auth operations order
- auth generation followed by encryption
- decryption followed by auth verify
Signed-off-by: Tejasree Kondoj
---
doc/guides/cryptodevs/features/octeontx.ini | 1 +
doc/guides/cryptodevs/features/octeontx2.ini | 1 +
doc/
Adding support for SSL/TLS way of cipher-auth operations order
- auth generation followed by encryption
- decryption followed by auth verify
Signed-off-by: Tejasree Kondoj
---
app/test/test_cryptodev_aes_test_vectors.h | 589 +
app/test/test_cryptodev_blockcipher.c
On 1/22/21 8:25 AM, Xia, Chenbo wrote:
> Hi Maxime,
>
>> -Original Message-
>> From: Maxime Coquelin
>> Sent: Wednesday, January 20, 2021 5:25 AM
>> To: dev@dpdk.org; Xia, Chenbo ; olivier.m...@6wind.com;
>> amore...@redhat.com; david.march...@redhat.com
>> Cc: Maxime Coquelin
>> Subj
On 1/22/21 8:34 AM, Xia, Chenbo wrote:
> Hi Maxime,
>
>> -Original Message-
>> From: Maxime Coquelin
>> Sent: Wednesday, January 20, 2021 5:25 AM
>> To: dev@dpdk.org; Xia, Chenbo ; olivier.m...@6wind.com;
>> amore...@redhat.com; david.march...@redhat.com
>> Cc: Maxime Coquelin
>> Subj
>
> 22/01/2021 10:07, Jerin Jacob:
> > On Fri, Jan 22, 2021 at 2:28 PM Thomas Monjalon
> wrote:
> > > 22/01/2021 09:39, Juraj Linkeš:
> > > > > > > > disabled drivers, similarly how the command line option
> > > > > > > > works and remove unneeded driver options ported from the
> > > > > > > >
On 1/22/21 9:55 AM, Xia, Chenbo wrote:
> Hi Maxime,
>
>> -Original Message-
>> From: Maxime Coquelin
>> Sent: Wednesday, January 20, 2021 5:25 AM
>> To: dev@dpdk.org; Xia, Chenbo ; olivier.m...@6wind.com;
>> amore...@redhat.com; david.march...@redhat.com
>> Cc: Maxime Coquelin
>> Subj
On 1/22/21 10:06 AM, Xia, Chenbo wrote:
> Hi Maxime,
>
>> -Original Message-
>> From: Maxime Coquelin
>> Sent: Wednesday, January 20, 2021 5:25 AM
>> To: dev@dpdk.org; Xia, Chenbo ; olivier.m...@6wind.com;
>> amore...@redhat.com; david.march...@redhat.com
>> Cc: Maxime Coquelin
>> Sub
On 1/22/21 10:11 AM, Xia, Chenbo wrote:
> Hi Maxime,
>
>> -Original Message-
>> From: Maxime Coquelin
>> Sent: Wednesday, January 20, 2021 5:25 AM
>> To: dev@dpdk.org; Xia, Chenbo ; olivier.m...@6wind.com;
>> amore...@redhat.com; david.march...@redhat.com
>> Cc: Maxime Coquelin
>> Sub
Parts of the unit tests code rely on forked/secondary processes
(expectedly) failing.
A crash in those situations could be missed so add a check on coredumps
presence after unit tests have run.
When unit tests fail, it can also help checking for coredumps as it
could give more insights on what happ
For newer smartNIC NVRAM versions the chip model should be read from the
PluDevice register as it provides the authoritative chip model/revision.
This method of reading the chip model is backwards compatible with
legacy NVRAM versions too.
Since the model number is purely used for reporting purpos
25/01/2021 15:58, Honnappa Nagarahalli:
> > 22/01/2021 10:07, Jerin Jacob:
> > > On Fri, Jan 22, 2021 at 2:28 PM Thomas Monjalon
> > wrote:
> > > > 22/01/2021 09:39, Juraj Linkeš:
> > > > > > > > > disabled drivers, similarly how the command line option
> > > > > > > > > works and remove unneeded
On Mon, Jan 25, 2021 at 3:11 PM Bruce Richardson
wrote:
>
> As a general principle, each header file should include any other
> headers it needs to provide data type definitions or macros. For
> example, any header using the uintX_t types in structures or function
> prototypes should include "stdi
08/01/2021 03:47, Dmitry Kozlyuk:
> --- /dev/null
> +++ b/buildtools/gen-pmdinfo-cfile.py
[...]
> +with tempfile.TemporaryDirectory() as temp:
> +proc = subprocess.run(
> +# Don't use "ar p", because its output is corrupted on Windows.
> +[ar, "xv", os.path.abspath(archive)], ca
08/01/2021 03:47, Dmitry Kozlyuk:
> Dmitry Kozlyuk (4):
> pmdinfogen: support COFF
> pmdinfogen: allow multiple input files
> buildtools: support object file extraction for Windows
> build: enable pmdinfogen for Windows
Applied, thanks.
Please follow-up on temporary file location.
Hi Thomas,
We have updated all UNH-IOL environments to include pyelftools.
Thanks,
Brandon
On Mon, Jan 25, 2021 at 8:13 AM Thomas Monjalon wrote:
>
> > Dmitry Kozlyuk (3):
> > pmdinfogen: add Python implementation
> > build: use Python pmdinfogen
> > pmdinfogen: remove C implementation
>
On Mon, Jan 25, 2021 at 8:28 PM Honnappa Nagarahalli
wrote:
>
>
>
> >
> > 22/01/2021 10:07, Jerin Jacob:
> > > On Fri, Jan 22, 2021 at 2:28 PM Thomas Monjalon
> > wrote:
> > > > 22/01/2021 09:39, Juraj Linkeš:
> > > > > > > > > disabled drivers, similarly how the command line option
> > > > > >
On 1/22/21 10:24 AM, Xia, Chenbo wrote:
> Hi Maxime,
>
>> -Original Message-
>> From: Maxime Coquelin
>> Sent: Wednesday, January 20, 2021 5:25 AM
>> To: dev@dpdk.org; Xia, Chenbo ; olivier.m...@6wind.com;
>> amore...@redhat.com; david.march...@redhat.com
>> Cc: Maxime Coquelin
>> Sub
Hi Tejasree,
> Subject: [PATCH v2 1/3] cryptodev: add feature flag for SSL/TLS order
>
> Added device feature flag for SSL/TLS way of cipher-auth
> operations support i.e. auth generation followed by encryption
> and decryption followed by auth verify so that SSL/TLS tests
> are skipped if the de
libnfb and libsze2 works OK.
The libnfb is provided by netcope-common package, so it is possible to
leave it like this.
Tested-by: Martin Spinler
Hi Tejasree,
Sorry I missed this mail earlier. I believe it is same as Digest encrypted case.
> -Original Message-
> From: Tejasree Kondoj
> Sent: Thursday, January 21, 2021 12:57 PM
> To: Tejasree Kondoj ; Akhil Goyal
> ; Radu Nicolau
> Cc: Anoob Joseph ; Ankur Dwivedi
> ; dev@dpdk.org
Hi,
This seems like a good task for us to do. I will see what it would
take in order to convert the difference into a decimal-formatted
percentage.
I have put this into bugzilla to keep track of this issue:
https://bugs.dpdk.org/show_bug.cgi?id=626
Thanks,
Brandon
On Sat, Jan 23, 2021 at 3:57 AM
Define hardware ID for GTP TEID modification. This value
can be used later in the RTE modify field API.
Signed-off-by: Alexander Kozyrev
---
drivers/common/mlx5/mlx5_prm.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/common/mlx5/mlx5_prm.h b/drivers/common/mlx5/mlx5_prm.h
index fe
Attaching to an NVMe disk on Windows using SPDK requires the
PCI class ID and device.bus fields. Decode the class ID from the PCI
device info strings if it is present and set device.bus.
Signed-off-by: Nick Connolly
---
drivers/bus/pci/windows/pci.c | 10 +-
1 file changed, 9 insertions(
Thanks Chenbo, for reviewing the v2. All the comments you
made should be taken into account.
This series significantly rework Virtio PMD to improve
the Virtio-user PMD and its backends integration.
First part of the series removes the dependency of
Virtio-user ethdev on Virtio PCI, by creating ge
This patch adds an helper macro to get the rte_vdev_device
pointer from a rte_eth_dev pointer.
This is similar to RTE_ETH_DEV_TO_PCI().
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
Reviewed-by: David Marchand
---
drivers/bus/vdev/rte_bus_vdev.h | 2 ++
1 file changed, 2 insertions(+
This patch adds driver flag in vdev bus driver so that
vdev drivers can require VA IOVA mode to be used, which
for example the case of Virtio-user PMD.
The patch implements the .get_iommu_class() callback, that
is called before devices probing to determine the IOVA mode
to be used.
It also adds a
This patch fixes getting reset status from the restarted
vhost-user backend in case of reconnection, instead of the
status at the time of the disconnection.
This issue was not spotted earlier because Vhost-user
protocol status feature was disabled in server mode.
Fixes: 47235f16505f ("net/virtio-
This patch is preliminary work for introducing a bus layer
in Virtio PMD, in order to improve Virtio-user integration.
A new bus type is added to provide a unified way to distinguish
which bus type is used (PCI modern, PCI legacy or Virtio-user).
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenb
This patch moves the virtio_hw structure into the virtio_user_dev
structure, with the goal of making virtio_hw bus-agnostic.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
---
drivers/net/virtio/virtio_ethdev.c| 2 +-
drivers/net/virtio/virtio_pci.h | 1 -
..
This patch initiate refactoring of Virtio PCI, by introducing
a new device structure for PCI-specific metadata.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
---
drivers/net/virtio/virtio_ethdev.c | 2 +-
drivers/net/virtio/virtio_pci.c| 2 ++
drivers/net/virtio/virtio_pci.h| 5
This patch moves the PCI Ethernet device registration bits
in a dedicated patch. In following patches, more code will
be moved there, with the goal of making virtio_ethdev.c
truly bus-agnostic.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
---
drivers/net/virtio/meson.build |
This patch introduces a new callback to notify the bus
driver some interrupt related operation was done.
This is used by Virtio PCI driver to check msix status.
Signed-off-by: Maxime Coquelin
---
drivers/net/virtio/virtio_ethdev.c | 12 +--
drivers/net/virtio/virtio_pci.c| 120
This patch moves the PCI specific initialization from
eth_virtio_dev_init() to eth_virtio_pci_init().
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
Reviewed-by: David Marchand
---
drivers/net/virtio/virtio_ethdev.c | 63 +--
drivers/net/virtio/virtio_pci_ethdev
At least Vhost-user backend of Virtio-user PMD requires
IOVA as VA mode. Until now, it was implemented as a hack
by forcing to use mbuf's buf_addr field instead of buf_iova.
This patch removes all this logic and just fails probing
if IOVA as VA mode is not selected. It simplifies the
code overall,
Going further in making the Virtio ethdev layer bus agnostic,
this patch adds a boolean in the Virtio PCI device metadata.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
Reviewed-by: David Marchand
---
drivers/net/virtio/virtio_pci.c| 18 +++---
drivers/net/virtio/v
This patch introduces a new callback for device closing,
making virtio_dev_close() bus-agnostic.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
Reviewed-by: David Marchand
---
drivers/net/virtio/meson.build | 2 --
drivers/net/virtio/virtio_ethdev.c | 13 +
d
This patch provides a new callback for the bus type
to validate negotiated features are compatible with it.
Only user for now is PCI modern bus type, which implies
that the device supports Virtio 1.0+.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
---
drivers/net/virtio/virtio_ethdev.
Bus type awareness at the generic ethdev level is no
more needed as previous patches have made it bus-agnostic.
This patch removes it from struct virtio_hw.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
Reviewed-by: David Marchand
---
drivers/net/virtio/virtio_ethdev.c | 18
This patch moves the fields from virtio_hw structure that
are PCI-specific to virtio_pci_dev_struct.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
Reviewed-by: David Marchand
---
drivers/net/virtio/virtio_pci.c | 139 ++--
drivers/net/virtio/virtio_pci.h |
This patch improves the virtio_hw struct packing,
going from 88 down to 80 bytes with a 6 bytes hole in
the end of the first cacheline. Fields only used in the
slow path are placed in the end, so that hot path only
uses the first cacheline.
The patch also changes booleans fields to uint8_t type, a
This patch moves Virtio PCI legacy IO handling to
virtio_pci.c. Two functions are created so that
virtio_pci_ethdev does not have to care about it.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
Reviewed-by: David Marchand
---
drivers/net/virtio/virtio_pci.c| 28 +++
This patch moves virtio_hw and virtio callbacks into
a generic virtio header, now that they have been
curated from PCI references.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
---
drivers/net/virtio/virtio.h | 74 ++
drivers/net/virtio/virtio_ethdev.c
This patch moves all the Virtio definition to the generic
header. It also renames some helpers to no more reference
PCI.
Signed-off-by: Maxime Coquelin
---
drivers/net/virtio/meson.build| 3 +-
drivers/net/virtio/virtio.c | 22
drivers/net/virtio/virtio.
This patch moves the virtqueues defines from PCI header
to the generic one.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
---
drivers/net/virtio/virtio.h| 18 ++
drivers/net/virtio/virtio_ethdev.h | 4 +++-
drivers/net/virtio/virtio_pci.
This patch aims at isolating MSIX notion into PCI
layer.
Signed-off-by: Maxime Coquelin
Reviewed-by: David Marchand
---
drivers/net/virtio/virtio.c | 6
drivers/net/virtio/virtio.h | 11 +--
drivers/net/virtio/virtio_ethdev.c | 7 ++---
drivers/net/virtio
This patch moves config and status definitions from the PCI
header to the generic one.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
---
drivers/net/virtio/virtio.c | 43 +++
drivers/net/virtio/virtio.h | 50 ++
drivers/net/vi
This patch finalizes the bus isolation part of this
refactoring.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
Reviewed-by: David Marchand
---
drivers/net/virtio/virtio_ethdev.c | 21 +---
drivers/net/virtio/virtio_rxtx.c | 18 -
d
This patch moves vring alignment define to the generic
Virtio header.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
Reviewed-by: David Marchand
---
drivers/net/virtio/virtio.h | 3 +++
drivers/net/virtio/virtio_ethdev.c | 10 +-
drivers/net/virtio/virtio_pci.
This patch makes vhost_user_write() consistent with
vhost_user_read(), by passing a Vhost-user message
pointer instead of a buffer pointer and its length, which
is now calculated in the function.
Signed-off-by: Maxime Coquelin
Reviewed-by: Chenbo Xia
---
drivers/net/virtio/virtio_user/vhost_use
1 - 100 of 181 matches
Mail list logo
