在 2021/4/9 15:08, Li, Xiaoyun 写道:
-Original Message-
From: oulijun
Sent: Friday, April 9, 2021 14:10
To: linux...@openeuler.org; dev ; Li, Xiaoyun
Subject: Re: [Linuxarm] Re: [PATCH 2/3] app/testpmd: remove forwarding
config from parsing Rx and Tx
在 2021/4/2 10:33, Li, Xiaoyun
在 2021/4/10 8:56, Ferruh Yigit 写道:
On 3/25/2021 2:21 AM, Li, Xiaoyun wrote:
-Original Message-
From: oulijun
Sent: Wednesday, March 24, 2021 21:40
To: linux...@openeuler.org; Li, Xiaoyun ; dev
Subject: Re: [Linuxarm] Re: [PATCH 1/3] app/testpmd: fix forwarding
configuration when
在 2021/4/10 9:12, Ferruh Yigit 写道:
On 3/23/2021 2:17 PM, oulijun wrote:
在 2021/3/23 16:55, Li, Xiaoyun 写道:
Hi
-Original Message-
From: Lijun Ou
Sent: Friday, March 5, 2021 18:22
To: Yigit, Ferruh
Cc: Li, Xiaoyun ; dev@dpdk.org;
linux...@openeuler.org
Subject: [PATCH 1/3] app/te
On 3/23/2021 2:17 PM, oulijun wrote:
在 2021/3/23 16:55, Li, Xiaoyun 写道:
Hi
-Original Message-
From: Lijun Ou
Sent: Friday, March 5, 2021 18:22
To: Yigit, Ferruh
Cc: Li, Xiaoyun ; dev@dpdk.org;
linux...@openeuler.org
Subject: [PATCH 1/3] app/testpmd: fix forwarding configuration whe
From: Hongbo Zheng
Currently, if RAS interrupt and FLR occurred at the same time, FLR will
be detected and corresponding schedule state will be set during RAS
interrupt processing. However, the schedule state value will be
overridden in subsequent RAS processing, resulting in FLR processing
failu
From: Chengchang Tang
Currently, after the reset is complete, the PMD restores the FEC according
to the FEC configuration reserved in the driver. If there is a concurrency
between the FEC setup operation and the restore operation after a reset,
the FEC status of the last hardware may be unknown.
From: Chengwen Feng
Currently, there are four control variables (rx_simple_allowed,
rx_vec_allowed, tx_simple_allowed and tx_vec_allowed) which are used
to impact the selection of Rx/Tx burst function.
The purpose of the design is to provide a way to control the selection
of Rx/Tx burst function
From: Chengwen Feng
Currently, the packet type calculated by
vlan/ovlan/l3id/l4id/ol3id/ol4id fields have the following problems:
1) Identify error when exist vlan strip which will lead to the data
buffer has non vlan header but mbuf's ptype have L2_ETHER_VLAN flag.
2) Some packet identifies erro
From: Chengchang Tang
At the end of the reset, the state of queues need to be restored according
to the states saved in the driver. If the start and stop operations of the
queues are concurrent at this time, it may cause the final status to be
uncertain.
This patch requires queues to acquire the
From: Chengchang Tang
During the task queue pairs reset, the getimeofday is used to obtain the
timestamp to determine whether the command execution times out. But
gettimeofday is not monotonous, it can be modified by system
administrators, so the timing may not be accurate or even cause the loop
This patch adds rollback processing when updating imissed
stats failed in PF init.
Fixes: 3e9f3042d7c8 ("net/hns3: add imissed packet stats")
Signed-off-by: Min Hu (Connor)
---
drivers/net/hns3/hns3_ethdev.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/hns3/hn
This set of patches contains seven bugfixes for hns3 PMD.
Chengchang Tang (3):
net/hns3: fix incorrect timing in resetting queues
net/hns3: fix queue state when concurrent with reset
net/hns3: fix configure FEC when concurrent with reset
Chengwen Feng (2):
net/hns3: remove variables of se
On 3/25/2021 2:21 AM, Li, Xiaoyun wrote:
-Original Message-
From: oulijun
Sent: Wednesday, March 24, 2021 21:40
To: linux...@openeuler.org; Li, Xiaoyun ; dev
Subject: Re: [Linuxarm] Re: [PATCH 1/3] app/testpmd: fix forwarding
configuration when DCB test
在 2021/3/24 10:03, Li, Xiao
The indication fields in the MCDI response are individual
bits, but the current code mistakenly compares the larger
dword with 1. This breaks encap. type discovery. Fix that.
Fixes: 891408c45a63 ("common/sfc_efx/base: indicate MAE support for
encapsulation")
Cc: sta...@dpdk.org
Signed-off-by: Iv
> -Original Message-
> From: Medvedkin, Vladimir
> Sent: Tuesday, April 6, 2021 12:51 PM
> To: dev@dpdk.org
> Cc: Ananyev, Konstantin ; Chilikin, Andrey
> ; Kinsella, Ray ; Wang,
> Yipeng1 ; Gobriel, Sameh
> ; Richardson, Bruce
>
> Subject: [PATCH v2 0/3] Predictable RSS feature
>
> This
> -Original Message-
> From: Medvedkin, Vladimir
> Sent: Tuesday, April 6, 2021 12:51 PM
> To: dev@dpdk.org
> Cc: Ananyev, Konstantin ; Chilikin, Andrey
> ; Kinsella, Ray ; Wang,
> Yipeng1 ; Gobriel, Sameh
> ; Richardson, Bruce
>
> Subject: [PATCH v2 2/3] hash: add predictable RSS impleme
> -Original Message-
> From: Medvedkin, Vladimir
> Sent: Tuesday, April 6, 2021 12:51 PM
> To: dev@dpdk.org
> Cc: Ananyev, Konstantin ; Chilikin, Andrey
> ; Kinsella, Ray ; Wang,
> Yipeng1 ; Gobriel, Sameh
> ; Richardson, Bruce
>
> Subject: [PATCH v2 1/3] hash: add predictable RSS API
>
In case of EAL initialization failure rte_eal_memory_detach() may be
called before mapping memory configuration, which in this case points
to the static structure. Attempt to unmap it yields error:
EAL: Could not unmap shared memory config: Invalid argument
Skip unmapping memory configuration
25/02/2021 20:50, Thomas Monjalon:
> 25/02/2021 16:29, Bruce Richardson:
> > To allow more flexibility in enabling additional build checks to DPDK build
> > process, we can add a developer mode option to the build. With this option
> > enabled, extra checks of interest to DPDK developers rather tha
On Fri, Apr 09, 2021 at 03:30:18PM +0200, Olivier Matz wrote:
> On Thu, Apr 08, 2021 at 09:58:35AM -0300, Flavio Leitner wrote:
> > On Thu, Apr 08, 2021 at 02:05:21PM +0200, Olivier Matz wrote:
> > > On Thu, Apr 08, 2021 at 08:21:58AM -0300, Flavio Leitner wrote:
> > > > On Thu, Apr 08, 2021 at 09:
On 4/9/2021 9:49 PM, Stephen Hemminger wrote:
On Fri, 9 Apr 2021 16:53:56 +0100
Ferruh Yigit wrote:
Fixes: a4862c9e1a98 ("license: introduce SPDX identifiers")
Cc: sta...@dpdk.org
Signed-off-by: Ferruh Yigit
Acked-by: Stephen Hemminger
Acked-by: Hemant Agrawal
On Fri, 9 Apr 2021 16:53:56 +0100
Ferruh Yigit wrote:
> Fixes: a4862c9e1a98 ("license: introduce SPDX identifiers")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Ferruh Yigit
Acked-by: Stephen Hemminger
Fixes: a4862c9e1a98 ("license: introduce SPDX identifiers")
Cc: sta...@dpdk.org
Signed-off-by: Ferruh Yigit
---
Cc: hemant.agra...@nxp.com
---
license/README | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/license/README b/license/README
index 874abaf4cd1e..79dac86440a5 1
On Fri, Apr 09, 2021 at 11:06:20AM -0400, Aaron Conole wrote:
> Bruce Richardson writes:
>
> > On Tue, Apr 06, 2021 at 10:20:37AM -0400, Aaron Conole wrote:
> >> Bruce Richardson writes:
> >>
> >> > On Tue, Apr 06, 2021 at 08:33:07AM -0400, Aaron Conole wrote:
> >> >> Thomas Monjalon writes:
>
On Fri, Apr 09, 2021 at 03:31:31PM +0100, Bruce Richardson wrote:
> On Fri, Apr 09, 2021 at 02:10:08PM +, Juraj Linkeš wrote:
> >
> >
> > > -Original Message-
> > > From: Bruce Richardson
> > > Sent: Friday, April 9, 2021 12:03 PM
> > > To: Juraj Linkeš
> > > Cc: ruifeng.w...@arm.co
09/04/2021 17:21, Ferruh Yigit:
> PMD updates are expected in alphabetical order based on their vendor
> name. Clarify this expectation in the section comment.
>
> Signed-off-by: Ferruh Yigit
> ---
> Cc: Thomas Monjalon
> ---
> --- a/doc/guides/rel_notes/release_21_05.rst
> +++ b/doc/guides/rel_
09/04/2021 17:06, Aaron Conole:
> Bruce Richardson writes:
>
> > On Tue, Apr 06, 2021 at 10:20:37AM -0400, Aaron Conole wrote:
> >> Bruce Richardson writes:
> >>
> >> > On Tue, Apr 06, 2021 at 08:33:07AM -0400, Aaron Conole wrote:
> >> >> Thomas Monjalon writes:
> >> >>
> >> >> > 17/03/2021 1
The purpose of the event dispatcher is primarily to decouple different
parts of an application (e.g., processing pipeline stages), which
share the same underlying event device.
The event dispatcher replaces the conditional logic (often, a switch
statement) that typically follows an event device de
PMD updates are expected in alphabetical order based on their vendor
name. Clarify this expectation in the section comment.
Signed-off-by: Ferruh Yigit
---
Cc: Thomas Monjalon
---
doc/guides/rel_notes/release_21_05.rst | 1 +
1 file changed, 1 insertion(+)
diff --git a/doc/guides/rel_notes/rel
Bruce Richardson writes:
> On Tue, Apr 06, 2021 at 10:20:37AM -0400, Aaron Conole wrote:
>> Bruce Richardson writes:
>>
>> > On Tue, Apr 06, 2021 at 08:33:07AM -0400, Aaron Conole wrote:
>> >> Thomas Monjalon writes:
>> >>
>> >> > 17/03/2021 15:44, Aaron Conole:
>> >> >> The hugepage test rea
On 3/29/2021 3:36 PM, Ferruh Yigit wrote:
KNI runs userspace callback with rtnl lock held, this is not working
fine with some devices that needs to interact with kernel interface in
the callback, like Mellanox devices.
The solution is releasing the rtnl lock before calling the userspace
callback
Ferruh Yigit writes:
> On 4/8/2021 9:51 AM, Wei Huang wrote:
>> In max10_staging_area_init(), variable "size" from fdt_get_reg() may
>> be invalid, it should be checked before assigning to member variable
>> "staging_area_size" of structure "intel_max10_device".
>>
>> Coverity issue: 367480, 3674
On 4/6/2021 10:30 AM, Jiawen Wu wrote:
This patch set provides a skeleton of ngbe PMD,
which adapted to Wangxun WX1860 series NICs.
v4:
- Fix compile error.
v3:
- Use rte_ether functions to define marcos.
v2:
- Correct some clerical errors.
- Use ethdev debug flags instead of driver own.
Jiaw
On 4/6/2021 10:30 AM, Jiawen Wu wrote:
Add basic PCIe ethdev probe and remove.
Signed-off-by: Jiawen Wu
<...>
--- a/drivers/net/ngbe/ngbe_ethdev.c
+++ b/drivers/net/ngbe/ngbe_ethdev.c
@@ -1,10 +1,12 @@
- /* SPDX-License-Identifier: BSD-3-Clause
- * Copyright(c) 2018-2020
- */
+/* SPDX-Lic
On 4/6/2021 10:30 AM, Jiawen Wu wrote:
Add device IDs for Wangxun 1Gb NICs, and register rte_ngbe_pmd.
Signed-off-by: Jiawen Wu
<...>
+static struct rte_pci_driver rte_ngbe_pmd = {
+ .id_table = pci_id_ngbe_map,
+ .drv_flags = RTE_PCI_DRV_NEED_MAPPING |
+RTE_
On 4/6/2021 10:30 AM, Jiawen Wu wrote:
Adding bare minimum PMD library and doc build infrastructure
and claim the maintainership for ngbe PMD.
Signed-off-by: Jiawen Wu
<...>
diff --git a/doc/guides/nics/ngbe.rst b/doc/guides/nics/ngbe.rst
new file mode 100644
index 0..007d8e80e
---
On Wed, Apr 7, 2021 at 10:29 PM Luc Pelletier wrote:
>
> The affinity of a control thread is set after it has been launched. If
> setting the affinity fails, pthread_cancel is called followed by a call
> to pthread_join, which can hang forever if the thread's start routine
> doesn't call a pthread
On Fri, Apr 09, 2021 at 02:10:08PM +, Juraj Linkeš wrote:
>
>
> > -Original Message-
> > From: Bruce Richardson
> > Sent: Friday, April 9, 2021 12:03 PM
> > To: Juraj Linkeš
> > Cc: ruifeng.w...@arm.com; honnappa.nagaraha...@arm.com;
> > phil.y...@arm.com; vcchu...@amazon.com; dharm
> -Original Message-
> From: Bruce Richardson
> Sent: Friday, April 9, 2021 12:03 PM
> To: Juraj Linkeš
> Cc: ruifeng.w...@arm.com; honnappa.nagaraha...@arm.com;
> phil.y...@arm.com; vcchu...@amazon.com; dharmik.thak...@arm.com;
> jerinjac...@gmail.com; hemant.agra...@nxp.com;
> ajit.kh
Advertise crypto adapter forward mode capability and set crypto adapter
enqueue function in driver.
Signed-off-by: Shijith Thotton
---
drivers/crypto/octeontx2/otx2_cryptodev_ops.c | 42 ++
drivers/event/octeontx2/otx2_evdev.c | 5 +-
.../event/octeontx2/otx2_evdev_crypto_adptr
This series proposes a new event device enqueue operation if crypto
adapter forward mode is supported. Second patch in the series is the
implementation of the same in PMD. Test application changes are added in
third patch.
v6:
- Rebased.
v5:
- Set rte_errno if crypto adapter enqueue fails in driv
From: Akhil Goyal
In case an event from a previous stage is required to be forwarded
to a crypto adapter and PMD supports internal event port in crypto
adapter, exposed via capability
RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_OP_FWD, we do not have
a way to check in the API rte_event_enqueue_bur
Use rte_event_crypto_adapter_enqueue() API to enqueue events to crypto
adapter if forward mode is supported in driver.
Signed-off-by: Shijith Thotton
---
app/test/test_event_crypto_adapter.c | 33 ++--
1 file changed, 21 insertions(+), 12 deletions(-)
diff --git a/app/te
On Fri, Apr 09, 2021 at 03:27:17PM +0200, David Marchand wrote:
> On Wed, Jan 27, 2021 at 6:43 PM Bruce Richardson
> wrote:
> >
> > While having the ability to run a test based off the DPDK_TEST environment
> > variable is useful, it's often easier to specify the test name as a
> > commandline par
On Thu, Apr 08, 2021 at 10:51:34PM +0300, Dmitry Kozlyuk wrote:
> 2021-04-08 13:45 (UTC+0200), Olivier Matz:
> [...]
> > > diff --git a/drivers/net/i40e/i40e_fdir.c b/drivers/net/i40e/i40e_fdir.c
> > > index c572d003cb..e7361bf520 100644
> > > --- a/drivers/net/i40e/i40e_fdir.c
> > > +++ b/drivers/
On Thu, Apr 08, 2021 at 09:58:35AM -0300, Flavio Leitner wrote:
> On Thu, Apr 08, 2021 at 02:05:21PM +0200, Olivier Matz wrote:
> > On Thu, Apr 08, 2021 at 08:21:58AM -0300, Flavio Leitner wrote:
> > > On Thu, Apr 08, 2021 at 09:41:59AM +0200, Olivier Matz wrote:
> > > > On Wed, Apr 07, 2021 at 05:
On Thu, Apr 1, 2021 at 1:25 PM Burakov, Anatoly
wrote:
>
> On 08-Mar-21 10:24 PM, Thomas Monjalon wrote:
> > The log messages had various issues:
> > - split on 2 lines, making search (grep) difficult
> > - long lines (can be split after the string)
> > - indented for no good reason (parent messag
On Wed, Jan 27, 2021 at 6:43 PM Bruce Richardson
wrote:
>
> While having the ability to run a test based off the DPDK_TEST environment
> variable is useful, it's often easier to specify the test name as a
> commandline parameter to a test binary. This also allows the test runs to
> be saved as par
From: Pavan Nikhilesh
Fix missing parameters in doxygen of function declaration.
Fixes: 5b1337f0a08a ("eventdev: introduce event vector Rx capability")
Signed-off-by: Pavan Nikhilesh
---
lib/librte_eventdev/rte_event_eth_rx_adapter.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/lib/
From: Rohit Raj
There were some padding left when a packet gets decrypted. This
patch removes those padding.
This patch also removes the padding left after verifying auth of
the packet.
Fixes: e2cdfbd07c8a ("examples/l2fwd-crypto: fix port id type")
Cc: sta...@dpdk.org
Signed-off-by: Rohit Raj
From: Apeksha Gupta
The devices which are masked by cryptodev mask should not be initialized
and skipped while traversing the device list.
Fixes: 6ae3fb9df66e ("examples/l2fwd-crypto: fix session mempool size")
Cc: sta...@dpdk.org
Signed-off-by: Apeksha Gupta
---
examples/l2fwd-crypto/main.c
On Fri, Apr 09, 2021 at 02:43:34PM +0200, David Marchand wrote:
> Let's try to enforce the convention where drivers use a pmd. logtype
> with their class reflected in it and libraries use a lib. logtype.
>
> RTE_LOG_REGISTER now interprets the log type name:
> - if none or a name starting with a .
From: Gagandeep Singh
L2fwd-crypto is passing 24b private data size while packet
pool creation. This patch aligns that private data size
to cache line size for better performance results.
Signed-off-by: Gagandeep Singh
---
examples/l2fwd-crypto/main.c | 3 ++-
1 file changed, 2 insertions(+),
Let's try to enforce the convention where drivers use a pmd. logtype
with their class reflected in it and libraries use a lib. logtype.
RTE_LOG_REGISTER now interprets the log type name:
- if none or a name starting with a . is passed, then the default logtype
is used as a prefix,
- else the nam
Due to hardware limitations the VLAN push/pop and Decap actions following
the sample action are supported in the FDB Tx steering domain only, the
flows with incorrect action order for other domains are rejected by
rdma-core.
To provide the action order requested in RTE flow this patch checks for
t
The mlx5 PMD allocated the resources of the sample actions, and then
moved these ones to the destination actions array. The original indices
were not cleared and the resources were referenced twice in the
flow object - as the fate actions and in the destination actions array.
This causes the failu
On Mon, Apr 5, 2021 at 9:33 PM Thomas Monjalon wrote:
>
> This series is a part of a bigger one split after v4:
> "improve options help"
>
> v5: no change
Previous versions lgtm, hence for the series,
Acked-by: David Marchand
Series applied, thanks.
--
David Marchand
From: Gabriel Ganne
The pcap PMD and the librte_port both declare their dependency to libpcap
with a line "ext_deps += pcap_dep".
Then meson automatically adds this dependency to the pkg-config file
in the "Requires.private" section for static builds.
The additional update of dpdk_extra_ldflags
On Fri, Apr 9, 2021 at 1:05 PM David Marchand wrote:
>
> Here is a little series, following RFC [1].
> No change in behavior expected, but reviews welcome ! :-)
>
> 1: https://patchwork.dpdk.org/project/dpdk/list/?series=10632
>
> Changes since v2:
> - rebased on main,
> - updated log message in p
On Fri, Apr 9, 2021 at 1:59 PM Thomas Monjalon wrote:
>
> 09/04/2021 13:04, David Marchand:
> > Add a log message when registering log types and changing log levels.
> >
> > __rte_log_register previous handled both legacy and dynamic logtypes.
> > To simplify the code, __rte_log_register is rework
This patch adds closing of the PMD after running the benchmark.
Signed-off-by: Adam Dybkowski
---
app/test-crypto-perf/main.c | 15 +--
1 file changed, 13 insertions(+), 2 deletions(-)
diff --git a/app/test-crypto-perf/main.c b/app/test-crypto-perf/main.c
index 49af812d8b..70cb04a21
This patch adds closing of the PMD after running the tests.
Signed-off-by: Adam Dybkowski
---
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 f91debc168..d68855c442 100644
--- a/app/test/test_cryptodev
This patch adds closing of the PMD after running the tests.
--
v3:
* move the call from ut_teardown to testsuite_teardown
v2:
* display error message on close error
Adam Dybkowski (2):
test/crypto: close PMD after tests
app/crypto-perf: close PMD after benchmark run
app/test-crypto-perf/ma
On Wed, Mar 24, 2021 at 8:32 PM Dmitry Kozlyuk wrote:
> @@ -1050,7 +1055,10 @@ rte_eal_memory_detach(void)
> * config - we can't zero it out because it might still be referenced
> * by other processes.
> */
> - rte_mem_unmap(mcfg, RTE_ALIGN(sizeof(*mcfg), page_sz))
09/04/2021 13:04, David Marchand:
> Add a log message when registering log types and changing log levels.
>
> __rte_log_register previous handled both legacy and dynamic logtypes.
> To simplify the code, __rte_log_register is reworked to only handle
> dynamic logtypes and takes a log level.
>
> E
On 4/9/2021 12:03 PM, Dmitry Kozlyuk wrote:
2021-04-09 11:51 (UTC+0100), Ferruh Yigit:
On 3/24/2021 12:50 AM, Dmitry Kozlyuk wrote:
[...]
Depends-on: series-15801 ("eal/windows: do not expose POSIX symbols")
Depends-on: patch-89639 ("[v2,3/3] meson: remove unnecessary explicit link to
libpcap
--log-level option is handled early, no need to reevaluate it later in
EAL init.
Before:
$ echo quit | ./build/app/test/dpdk-test --no-huge -m 512 \
--log-level=lib.eal:debug \
--log-level=lib.ethdev:debug --log-level=lib.ethdev:info \
|& grep -i logtype.level
EAL: lib.eal logtype level cha
Add a log message when registering log types and changing log levels.
__rte_log_register previous handled both legacy and dynamic logtypes.
To simplify the code, __rte_log_register is reworked to only handle
dynamic logtypes and takes a log level.
Example:
$ DPDK_TEST=logs_autotest ./build/app/te
Add checks on log levels:
- default values for rte_log_register and RTE_LOG_REGISTER,
- level changes with rte_log_set_level and consorts
Signed-off-by: David Marchand
---
Changes since v1:
- dropped unrelated change moving logtype1 and logtype2 as global
static variables,
- dropped __FILE__ +
Here is a little series, following RFC [1].
No change in behavior expected, but reviews welcome ! :-)
1: https://patchwork.dpdk.org/project/dpdk/list/?series=10632
Changes since v2:
- rebased on main,
- updated log message in patch 2,
Changes since v1:
- some cleanup in patch 1,
- some precision
2021-04-09 11:51 (UTC+0100), Ferruh Yigit:
> On 3/24/2021 12:50 AM, Dmitry Kozlyuk wrote:
[...]
> > Depends-on: series-15801 ("eal/windows: do not expose POSIX symbols")
> > Depends-on: patch-89639 ("[v2,3/3] meson: remove unnecessary explicit link
> > to libpcap")
> >
>
> Hi Dmitry,
>
> What
Hi Olivier,
> On Thu, Apr 08, 2021 at 01:47:18PM +0530, Tejasree Kondoj wrote:
> > Adding new mbuf packet type for UDP encapsulated
> > ESP packets.
> >
> > Signed-off-by: Tejasree Kondoj
> > ---
> > doc/guides/rel_notes/release_21_05.rst | 5 +
> > lib/librte_mbuf/rte_mbuf_ptype.h | 2
On Thu, Apr 8, 2021 at 6:47 PM Thomas Monjalon wrote:
>
> This series is a part of a bigger one split after v4:
> "improve options help"
>
> The main intent of this series is to provide a nice help
> for the --log-level option.
Series applied, thanks.
--
David Marchand
On 3/24/2021 12:50 AM, Dmitry Kozlyuk wrote:
NOTE: dependency patches are requirements, CI may fail.
Most modern and actively maintained distribution of libpcap for Windows
is Npcap [1]. It currently doesn't provide a pkg-config file [2].
By default it is installed to C:\Program Files\Npcap and
On 4/8/2021 5:07 PM, Ferruh Yigit wrote:
On 3/30/2021 2:23 PM, Lance Richardson wrote:
Initialize TCP data offset field with TCP header length, this
field is used to derive L4 header length and by hardware to
validate a TCP header.
Fixes: 41f72ec94074 ("test: add packet burst generator function
> > > > > Fill up dev->tx_pkt_prepare to i40e_pkt_prepare when on vector and
> > > > > simple data path selection, as the sanity check is needed ideally.
> > > > >
> > > > > Signed-off-by: Leyi Rong
> > > > > ---
> > > > > drivers/net/i40e/i40e_rxtx.c | 2 +-
> > > > > 1 file changed, 1 inser
> -Original Message-
> From: Li, Xiaoyun
> Sent: Friday, April 9, 2021 3:50 PM
> To: Yu, DapengX ; Yigit, Ferruh
> ; Zhang, Qi Z
> Cc: dev@dpdk.org; sta...@dpdk.org
> Subject: RE: [dpdk-stable] [PATCH] app/testpmd: fix queue Rx and Tx offload
> reconfig cmd
>
>
>
> > -Original Me
From: Chengwen Feng
This patch adds debug info for Rx/Tx burst function which was choosing.
Signed-off-by: Chengwen Feng
Signed-off-by: Min Hu (Connor)
---
drivers/net/hns3/hns3_rxtx.c | 17 +
1 file changed, 17 insertions(+)
diff --git a/drivers/net/hns3/hns3_rxtx.c b/driver
From: Chengchang Tang
During the task queue pairs reset, the getimeofday is used to obtain the
timestamp to determine whether the command execution times out. But
gettimeofday is not monotonous, it can be modified by system
administrators, so the timing may not be accurate or even cause the loop
From: Chengchang Tang
At the end of the reset, the state of queues need to be restored according
to the states saved in the driver. If the start and stop operations of the
queues are concurrent at this time, it may cause the final status to be
uncertain.
This patch requires queues to acquire the
From: Chengchang Tang
Currently, after the reset is complete, the PMD restores the FEC according
to the FEC configuration reserved in the driver. If there is a concurrency
between the FEC setup operation and the restore operation after a reset,
the FEC status of the last hardware may be unknown.
From: Chengwen Feng
Currently, the packet type calculated by
vlan/ovlan/l3id/l4id/ol3id/ol4id fields have the following problems:
1) Identify error when exist vlan strip which will lead to the data
buffer has non vlan header but mbuf's ptype have L2_ETHER_VLAN flag.
2) Some packet identifies erro
From: Chengwen Feng
Currently, there are four control variables (rx_simple_allowed,
rx_vec_allowed, tx_simple_allowed and tx_vec_allowed) which are used
to impact the selection of Rx/Tx burst function.
The purpose of the design is to provide a way to control the selection
of Rx/Tx burst function
This patch adds rollback processing when updating imissed
stats failed in PF init.
Fixes: 3e9f3042d7c8 ("net/hns3: add imissed packet stats")
Signed-off-by: Min Hu (Connor)
---
drivers/net/hns3/hns3_ethdev.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/hns3/hn
From: Hongbo Zheng
Currently, if RAS interrupt and FLR occurred at the same time, FLR will
be detected and corresponding schedule state will be set during RAS
interrupt processing. However, the schedule state value will be
overridden in subsequent RAS processing, resulting in FLR processing
failu
This set of patches contains seven bugfixes for hns3 PMD.
Chengchang Tang (3):
net/hns3: fix incorrect timing in resetting queues
net/hns3: fix queue state when concurrent with reset
net/hns3: fix configure FEC when concurrent with reset
Chengwen Feng (2):
net/hns3: remove ariables of sel
Performance improvement: use a write combining store
instead of a regular mmio write to update queue tail
registers.
Signed-off-by: Radu Nicolau
---
drivers/net/ice/ice_rxtx_vec_avx512.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ice/ice_rxtx_vec_avx512.c
On Fri, Apr 09, 2021 at 10:41:17AM +0200, Juraj Linkeš wrote:
> Add support for enabling or disabling drivers for Arm cross build. Do
> not implement any enable/disable lists yet.
>
> Enabling drivers is useful when building for an SoC where we only want
> to build a few drivers. That way the list
On 4/8/2021 10:17 AM, Hemant Agrawal wrote:
This patch adds missing MPLS tokens in for RSS config.
Fixes: d810252857c9 ("ethdev: add MPLS RSS offload type")
Cc: hemant.agra...@nxp.com
Signed-off-by: Hemant Agrawal
Reviewed-by: Ferruh Yigit
Applied to dpdk-next-net/main, thanks.
On Thu, Apr 8, 2021 at 6:47 PM Thomas Monjalon wrote:
>
> The option --log-level was not completely described in the usage text,
> and it was difficult to guess the names of the log types and levels.
>
> A new value "help" is accepted after --log-level to give more details
> about the syntax and l
On 4/8/2021 5:31 PM, Ivan Malov wrote:
On 4/8/2021 7:48 AM, Gregory Etelson wrote:
In GCC compiler, __builtin_constant_p(exp) is a function.
The function returns the integer 1 if the argument is known to be
a compile-time constant.
Therefore, __builtin_constant_p(0xff << 8) returned 1.
As th
09/04/2021 10:31, Gabriel Ganne:
> Hi Thomas,
>
> Thanks for the review.
>
> The use case that made me see this is that I configured the dpdk using meson
> option "-- default_library=static" in conjunction with buildroot which is
> patching
> meson to prefer static libraries in this case.
> This
09/04/2021 11:08, David Marchand:
> On Thu, Apr 8, 2021 at 5:55 PM Thomas Monjalon wrote:
> >
> > 24/03/2021 11:32, David Marchand:
> > > Add a log message when registering log types and changing log levels.
> > [...]
> > > + RTE_LOG(DEBUG, EAL, "%s logtype level changed from %s to
>
On 4/8/2021 10:17 AM, Hemant Agrawal wrote:
RSS distribution field is U64, while the DPAA2 code
was using integer value, which is causing a issue
for MPLS having bit position 33.
This patch fixes the code to use unsigned long long.
Fixes: 1832bc8ee8d4 ("net/dpaa2: support MPLS distribution")
Cc:
On Thu, Apr 8, 2021 at 5:55 PM Thomas Monjalon wrote:
>
> 24/03/2021 11:32, David Marchand:
> > Add a log message when registering log types and changing log levels.
> [...]
> > + RTE_LOG(DEBUG, EAL, "%s logtype level changed from %s to
> > %s\n",
>
> I would prefer "foo log level cha
On 4/9/2021 12:06 AM, John Daley wrote:
Fix a rare case in rte_flow initialization where the action hash table
is not freed if allocating a NIC match table fails.
Fixes: ea7768b5bba8 ("net/enic: add flow implementation based on Flow Manager
API")
Cc: sta...@dpdk.org
Signed-off-by: John Daley
Thanks for spotting this.
I'll move the call to rte_cryptodev_close into the function testsuite_teardown
and send v3.
Adam
> -Original Message-
> From: Akhil Goyal
> Sent: Thursday, 8 April, 2021 14:16
> To: Dybkowski, AdamX ; dev@dpdk.org;
> Doherty, Declan ; Kusztal, ArkadiuszX
>
> S
On Fri, Apr 9, 2021 at 2:11 PM Juraj Linkeš wrote:
>
> The current way of specifying Arm configuration options is insufficient
> since we can't identify the SoC we're building for from the MIDR
> information. For example, we can't distinguish between N1SDP, Graviton2
> or Ampere Altra.
>
> We also
> -Original Message-
> From: Juraj Linkeš
> Sent: Friday, April 9, 2021 4:41 PM
> To: bruce.richard...@intel.com; Ruifeng Wang ;
> Honnappa Nagarahalli ; Phil Yang
> ; vcchu...@amazon.com; Dharmik Thakkar
> ; jerinjac...@gmail.com;
> hemant.agra...@nxp.com; Ajit Khaparde (ajit.khapa...@bro
Add Arm SoC configuration sets to Arm meson.build and add an arch
agnostic meson option, 'platform', to select from these SoC
configurations for meson native builds. This is preferable to
specifying a cross file when doing aarch64 -> aarch64 builds, since the
cross file specifies the toolchain as w
1 - 100 of 126 matches
Mail list logo