Hi Gowrishankar,
On 2024/2/28 2:35, Amit Prakash Shukla wrote:
> From: Gowrishankar Muthukrishnan
>
> Validate copied memory to ensure DMA copy did not fail.
>
> Signed-off-by: Gowrishankar Muthukrishnan
> Acked-by: Anoob Joseph
> Acked-by: Chengwen Feng
> ---
> app/test-dma-perf/benchmark.
Hi,
> -Original Message-
> From: Morten Brørup
> Sent: Tuesday, February 27, 2024 1:28 PM
> To: Jerin Jacob ; Raslan Darawsheh
>
> Cc: julien_d...@jaube.fr; dev@dpdk.org; step...@networkplumber.org;
> rm...@marvell.com; shsha...@marvell.com; pa...@marvell.com
> Subject: RE: [PATCH v9] net
+To: Thomas, previously joined the discussion
> From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com]
> Sent: Tuesday, 27 February 2024 18.17
>
> On Tue, Feb 27, 2024 at 10:10:03AM +0100, Morten Brørup wrote:
> > > From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com]
> > > Sent: Tuesda
On Tue, Feb 27, 2024 at 11:06:04AM -0800, Stephen Hemminger wrote:
> On Tue, 27 Feb 2024 12:27:31 +0100
> Morten Brørup wrote:
>
> > > > 1. The extra 2 byte copy is effectively harmless due to padding, as
> > > mentioned in the commit message.
> > > > 2. The decorated rte_memcpy (if work on t
On Tue, Feb 27, 2024 at 11:21:58AM -0800, Tyler Retzlaff wrote:
> Enable build of eal & ring library when building with MSVC.
>
> This series depends on 2 other series that seem to be near being
> accepted for merge.
>
> https://patches.dpdk.org/project/dpdk/list/?series=31229
> https://patch
Hi Fengchengwen,
> > + for (i = 0; i < (nr_buf / nb_workers) * nb_workers; i++) {
> > + if (memcmp(rte_pktmbuf_mtod(srcs[i], void *),
> > + rte_pktmbuf_mtod(dsts[i], void *),
> > + cfg->buf_size.cur) != 0) {
> > + printf("Copy
On Tue, Feb 27, 2024 at 10:22 PM Jeremy Spewock wrote:
>
>
> > +
> > +lowercase_suite_to_find = (
> > +f"{self._test_suite_class_prefix}{module_name.replace('_',
> > '')}".lower()
> > +)
> > +for class_name, class_obj in inspect.getmembers(test_suite_module,
Missing GEN2, GEN3 and GEN4 devices updated for the QAT compression.
Signed-off-by: Sivaramakrishnan Venkat
---
doc/guides/compressdevs/qat_comp.rst | 4
1 file changed, 4 insertions(+)
diff --git a/doc/guides/compressdevs/qat_comp.rst
b/doc/guides/compressdevs/qat_comp.rst
index 475c4a9f
> From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com]
> Sent: Wednesday, 28 February 2024 00.08
>
> Use existing __rte_pure macro from rte_common.h instead of directly
Typo: __rte_pure -> __rte_may_alias
The code is correct, only the description has the typo.
> using __attribute__((__may
Hi Gowrishankar,
On 2024/2/28 2:56, Amit Prakash Shukla wrote:
> From: Gowrishankar Muthukrishnan
>
> Add SG copy support.
>
> Signed-off-by: Gowrishankar Muthukrishnan
> Acked-by: Anoob Joseph
> Acked-by: Chengwen Feng
> ---
> v10:
> - SG config variables renamed.
>
> app/test-dma-perf/be
Hi Chengwen,
Please see my reply in-line.
Thanks
Amit Shukla
> -Original Message-
> From: fengchengwen
> Sent: Wednesday, February 28, 2024 12:34 PM
> To: Amit Prakash Shukla ; Cheng Jiang
> ; Gowrishankar Muthukrishnan
>
> Cc: dev@dpdk.org; Jerin Jacob ; Anoob Joseph
> ; Kevin Laatz ;
Add support for matching metadata in HWS ingress rules.
It using REG_B matching which is supported for each device supports HWS.
Signed-off-by: Michael Baum
Reviewed-by: Erez Shitrit
---
drivers/net/mlx5/hws/mlx5dr.h | 1 +
drivers/net/mlx5/hws/mlx5dr_definer.c | 32 ++-
The integrity item uses the DW "oks1" in header layout. It includes the
all supported bits for both inner and outer. When item is integrity
type, the relevant bits are turned on and all DW is submitted.
When user provides more then single integrity item in same pattern, the
last one overrides the
On Mon, Feb 26, 2024 at 6:30 PM Pavan Nikhilesh Bhagavatula
wrote:
>
> > Uninitialized mbufs are enqueued to eventdev which causes segfault
> > on freeing the mbuf in tx adapter.
> > Fixed by initializing mbufs before enqueuing to eventdev.
> >
> > Fixes: 46cf97e4bbfa ("eventdev: add test for eth
Hi,
> -Original Message-
> From: Dariusz Sosnowski
> Sent: Tuesday, February 27, 2024 3:37 PM
> To: Slava Ovsiienko ; Ori Kam ;
> Suanming Mou ; Matan Azrad
>
> Cc: dev@dpdk.org
> Subject: [PATCH v4 1/2] net/mlx5: move meter init functions
>
> Move mlx5_flow_meter_init() and mlx5_flow_m
Hi,
> -Original Message-
> From: Dariusz Sosnowski
> Sent: Tuesday, February 27, 2024 3:52 PM
> To: Slava Ovsiienko ; Ori Kam ;
> Suanming Mou ; Matan Azrad
>
> Cc: dev@dpdk.org
> Subject: [PATCH v3 0/4] net/mlx5: connection tracking changes
>
> Patches 1 and 2 contain fixes for existin
This RFC presents a new API for static per-lcore id
data allocation.
Please refer to the API documentation for both a
rationale for this new API, and a comparison to the alternatives
available.
The adoption of this API would affect many different DPDK modules, but
the author updated only a few,
Introduce DPDK per-lcore id variables, or lcore variables for short.
An lcore variable has one value for every current and future lcore
id-equipped thread.
The primary use case is for statically allocating
small chunks of often-used data, which is related logically, but where
there are performan
Add test suite to exercise the API.
RFC v5:
* Adapt tests to reflect the removal of the GET() and SET() macros.
RFC v4:
* Check all lcore id's values for all variables in the many variables
test case.
* Introduce test case for max-sized lcore variables.
RFC v2:
* Improve alignment-relate
Replace keeping PRNG state in a RTE_MAX_LCORE-sized static array of
cache-aligned and RTE_CACHE_GUARDed struct instances with keeping the
same state in a more cache-friendly lcore variable.
RFC v3:
* Remove cache alignment on unregistered threads' rte_rand_state.
(Morten Brørup)
Signed-off-by
Replace static array of cache-aligned structs with an lcore variable,
to slightly benefit code simplicity and performance.
RFC v3:
* Replace for loop with FOREACH macro.
Signed-off-by: Mattias Rönnblom
Acked-by: Morten Brørup
---
lib/power/rte_power_pmd_mgmt.c | 36 ---
Keep per-lcore power intrinsics state in a lcore variable to reduce
cache working set size and avoid any CPU next-line-prefetching causing
false sharing.
Signed-off-by: Mattias Rönnblom
Acked-by: Morten Brørup
---
lib/eal/x86/rte_power_intrinsics.c | 17 +++--
1 file changed, 11 ins
Replace static array of cache-aligned structs with an lcore variable,
to slightly benefit code simplicity and performance.
RFC v5:
* Fix lcore value pointer bug introduced by RFC v4.
RFC v4:
* Remove strange-looking lcore value lookup potentially containing
invalid lcore id. (Morten Brørup)
Support template table resize API.
Gregory Etelson (3):
net/mlx5: fix parameters verification in HWS table create
net/mlx5: move multi-pattern actions management to table level
net/mlx5: add support for flow table resizing
Maayan Kashani (1):
net/mlx5: add resize function to ipool
drive
From: Maayan Kashani
Before this patch, ipool size could be fixed by
setting max_idx in mlx5_indexed_pool_config upon
ipool creation. Or it can be auto resized to the
maximum limit by setting max_idx to zero upon
ipool creation and the saved value is the maximum
index possible.
This patch adds ip
Modified the conditionals in `flow_hw_table_create()` to use bitwise
AND instead of equality checks when assessing
`table_cfg->attr->specialize` bitmask.
This will allow for greater flexibility as the bitmask may encapsulate
multiple flags.
The patch maintains the previous behavior with single flag
The multi-pattern actions related structures and management code
have been moved to the table level.
That code refactor is required for the upcoming table resize feature.
Signed-off-by: Gregory Etelson
Acked-by: Dariusz Sosnowski
---
drivers/net/mlx5/mlx5_flow.h| 73 ++-
drivers/ne
Support template table API in PMD.
The patch allows to increase existing table capacity.
Signed-off-by: Gregory Etelson
Acked-by: Dariusz Sosnowski
---
drivers/net/mlx5/mlx5.h | 5 +
drivers/net/mlx5/mlx5_flow.c| 51
drivers/net/mlx5/mlx5_flow.h| 84 --
drivers/net/
When tail pointer of Circ buffer rollsover as the Circ buffer becomes full,
crypto adapter is enqueueing ops beyond the size of the Circ buffer
leading to segfault due to invalid ops access.
Fixed by enqueueing ops from head pointer to (size-head) number of ops
when Circ buffer becomes full and th
On Tue, Feb 27, 2024 at 6:23 PM Tyler Retzlaff
wrote:
>
> On Tue, Feb 27, 2024 at 04:18:10PM +0100, David Marchand wrote:
> > Hello Dodji,
> >
> > On Tue, Feb 27, 2024 at 6:44 AM Tyler Retzlaff
> > wrote:
> > >
> > > RTE_MARKER typedefs are a GCC extension unsupported by MSVC. Remove
> > > RTE_MA
>
> - Implemented SVE code for comparing signatures in bulk lookup.
> - Added Defines in code for SVE code support.
> - Optimise NEON code
> - New SVE code is ~5% slower than optimized NEON for N2 processor.
>
> Signed-off-by: Yoan Picchi
> Signed-off-by: Harjot Singh
> Reviewed-by: Nathan Br
Currently IPsec MB provides both the JOB API and direct API.
AESNI_MB PMD is using the JOB API codepath while ZUC, KASUMI, SNOW3G
and CHACHA20_POLY1305 are using the direct API.
Instead of using the direct API for these PMDs, they should now make
use of the JOB API codepath. This would remove all u
While I was implementing the new API, I realized one issue, and it would be
good to discuss it here. First of all rte_rcu_qsbr_dq_reclaim function
checks the state of the qsbr values. It means that all threads should
report the quiescent states. It conflicts with my aim.
Let's think about below sc
On Tue, Feb 27, 2024 at 7:15 PM Tyler Retzlaff
wrote:
>
> On Mon, Feb 26, 2024 at 12:54:36PM -0800, Stephen Hemminger wrote:
> > On Mon, 26 Feb 2024 12:19:30 -0800
> > Tyler Retzlaff wrote:
> >
> > > RTE_LOG_LINE cannot be augmented with a prefix format and arguments
> > > without the user of RTE
> -Original Message-
> From: Brian Dooley
> Sent: Wednesday, February 28, 2024 11:33 AM
> To: Ji, Kai ; De Lara Guarch, Pablo
>
> Cc: dev@dpdk.org; gak...@marvell.com; Dooley, Brian
>
> Subject: [PATCH v4] crypto/ipsec_mb: unified IPsec MB interface
>
> Currently IPsec MB provides bo
On 2/20/2024 3:33 PM, Sivaprasad Tummala wrote:
> + ops = rte_power_get_ops(env);
> + if (ops->status == 0) {
> + POWER_LOG(ERR, WER,
> + "Power Management Environment(%d) not"
> + " registered\n", env);
'WER' seems typo, causing build er
On 2/28/2024 2:27 AM, huangdengdui wrote:
>
>
> On 2024/2/27 0:43, Ferruh Yigit wrote:
>> On 2/26/2024 3:16 AM, Jie Hai wrote:
>>> On 2024/2/23 21:53, Ferruh Yigit wrote:
On 2/20/2024 3:58 AM, Jie Hai wrote:
> Hi, Ferruh,
>
> Thanks for your review.
>
> On 2024/2/7 22:15,
On Mon, Feb 26, 2024 at 9:20 PM Tyler Retzlaff
wrote:
>
> Use RTE_LOG_LINE_PREFIX instead of RTE_LOG_LINE in macro expansions
> which allow a prefix and arguments to be inserted into the log line
> without the need to use the ## args variadic argument pack extension.
>
> Signed-off-by: Tyler Retzl
Support template table resize API.
Gregory Etelson (3):
net/mlx5: fix parameters verification in HWS table create
net/mlx5: move multi-pattern actions management to table level
net/mlx5: add support for flow table resizing
Maayan Kashani (1):
net/mlx5: add resize function to ipool
drive
From: Maayan Kashani
Before this patch, ipool size could be fixed by
setting max_idx in mlx5_indexed_pool_config upon
ipool creation. Or it can be auto resized to the
maximum limit by setting max_idx to zero upon
ipool creation and the saved value is the maximum
index possible.
This patch adds ip
Modified the conditionals in `flow_hw_table_create()` to use bitwise
AND instead of equality checks when assessing
`table_cfg->attr->specialize` bitmask.
This will allow for greater flexibility as the bitmask may encapsulate
multiple flags.
The patch maintains the previous behavior with single flag
The multi-pattern actions related structures and management code
have been moved to the table level.
That code refactor is required for the upcoming table resize feature.
Signed-off-by: Gregory Etelson
Acked-by: Dariusz Sosnowski
---
drivers/net/mlx5/mlx5_flow.h| 73 ++-
drivers/ne
Support template table API in PMD.
The patch allows to increase existing table capacity.
Signed-off-by: Gregory Etelson
Acked-by: Dariusz Sosnowski
---
drivers/net/mlx5/mlx5.h | 5 +
drivers/net/mlx5/mlx5_flow.c| 51 +++
drivers/net/mlx5/mlx5_flow.h| 84 +++--
drivers/net/ml
>
> On Mon, Feb 26, 2024 at 9:20 PM Tyler Retzlaff
> wrote:
> >
> > Use RTE_LOG_LINE_PREFIX instead of RTE_LOG_LINE in macro expansions
> > which allow a prefix and arguments to be inserted into the log line
> > without the need to use the ## args variadic argument pack extension.
> >
> > Signed
Hi
> -Original Message-
> From: Bing Zhao
> Sent: Tuesday, February 27, 2024 5:26 PM
> To: dev@dpdk.org; Suanming Mou ; Raslan
> Darawsheh
> Cc: Ori Kam ; Slava Ovsiienko ;
> Matan Azrad ; Dariusz Sosnowski
> ; Gregory Etelson ;
> sta...@dpdk.org
> Subject: [PATCH] net/mlx5: fix the HWS
Hi,
> -Original Message-
> From: Michael Baum
> Sent: Wednesday, February 28, 2024 11:51 AM
> To: dev@dpdk.org
> Cc: Matan Azrad ; Dariusz Sosnowski
> ; Raslan Darawsheh ; Slava
> Ovsiienko ; Ori Kam ;
> Suanming Mou ; Alex Vesker ;
> sta...@dpdk.org; Erez Shitrit
> Subject: [PATCH] net/
Hi,
> -Original Message-
> From: Dariusz Sosnowski
> Sent: Tuesday, February 27, 2024 3:58 PM
> To: Slava Ovsiienko ; Ori Kam ;
> Suanming Mou ; Matan Azrad
>
> Cc: dev@dpdk.org; sta...@dpdk.org
> Subject: [PATCH v2] net/mlx5: fix VLAN handling in meter split
>
> On the attempt to creat
This patchset adds a new QAT LCE device.
The device currently only supports symmetric crypto,
and only the AES-GCM algorithm.
v6:
- Added documentation and release note changes.
- Removed unused device PCI ID.
v5:
- Fixed compilation issue by replacing __u8 with uint8_t.
v4:
- Fixed cover
Adding GEN LCE specific macros which is required for updating
the support for GEN LCE features.
Also this patch adds other macros which is being used by GEN LCE
Specific APIs.
Signed-off-by: Nishikant Nayak
Acked-by: Ciara Power
---
v6:
- Removed unused PCI device IDs from the device list.
Adding GEN5 files for handling GEN LCE specific operations.
These files are inherited from the existing files/APIs
which has some changes specific GEN5 requirements
Also updated the mailmap file.
Signed-off-by: Nishikant Nayak
Acked-by: Ciara Power
---
v5:
- Replaced usage of __u8 with uint8
This patch handles the changes required for updating the common
header fields specific to GEN LCE, Also added/updated of the response
processing APIs based on GEN LCE requirement.
Signed-off-by: Nishikant Nayak
Acked-by: Ciara Power
---
v2:
- Renamed device from GEN 5 to GEN LCE.
- Remov
Adding one new unit test code for validating the features
added as part of GCM with 64 byte AAD.
The new test case adds one new test for GCM algo for both
encrypt and decrypt operations.
Signed-off-by: Nishikant Nayak
Acked-by: Ciara Power
---
v2:
- Removed unused code.
- Added one new u
Hello,
David Marchand writes:
> Hello Dodji,
o/
[...]
> This change is reported as a potential ABI change.
>
> For the context, this patch
> https://patchwork.dpdk.org/project/dpdk/patch/1709012499-12813-21-git-send-email-roret...@linux.microsoft.com/
> removes null-sized markers (those fiel
On Tue, Feb 27, 2024 at 6:44 AM Tyler Retzlaff
wrote:
>
> RTE_MARKER typedefs are a GCC extension unsupported by MSVC. Remove
> RTE_MARKER fields from rte_mbuf struct.
>
> Maintain alignment of fields after removed cacheline1 marker by placing
> C11 alignas(RTE_CACHE_LINE_MIN_SIZE).
>
> Update imp
On Wed, Feb 28, 2024 at 3:04 PM Dodji Seketeli wrote:
> > Btw, I see no way to suppress this (except a global [suppress_type]
> > name = rte_mbuf)...
>
> Right.
>
> To avoid having subsequent changes to that type from being "overly"
> suppressed, maybe do something like:
>
> [suppress_type]
>
On 2/28/24 10:56, Konstantin Ananyev wrote:
- Implemented SVE code for comparing signatures in bulk lookup.
- Added Defines in code for SVE code support.
- Optimise NEON code
- New SVE code is ~5% slower than optimized NEON for N2 processor.
Signed-off-by: Yoan Picchi
Signed-off-by: Harjot S
Acked-by: Patrick Robb
> On Feb 28, 2024, at 5:44 AM, Abdullah Ömer Yamaç wrote:
>
> While I was implementing the new API, I realized one issue, and it would be
> good to discuss it here. First of all rte_rcu_qsbr_dq_reclaim function checks
> the state of the qsbr values. It means that all threads should report the
> From: David Marchand [mailto:david.march...@redhat.com]
> Sent: Wednesday, 28 February 2024 15.19
>
> On Tue, Feb 27, 2024 at 6:44 AM Tyler Retzlaff
> wrote:
> >
> > RTE_MARKER typedefs are a GCC extension unsupported by MSVC. Remove
> > RTE_MARKER fields from rte_mbuf struct.
> >
> > Maintain
This patch set contains the mlx5 PMD implementation for NAT64.
Series-acked-by: Ori Kam
Update in v4:
1. rebase to solve the conflicts.
2. fix the old NIC startup issue in a separate patch:
https://patches.dpdk.org/project/dpdk/patch/20240227152627.25749-1-bi...@nvidia.com/
Update in
From: Erez Shitrit
Add support of new action mlx5dr_action_create_nat64.
The new action allows to modify IP packets from version to version, IPV6
to IPV4 and vice versa.
Signed-off-by: Erez Shitrit
Signed-off-by: Bing Zhao
---
drivers/net/mlx5/hws/mlx5dr.h| 29 ++
drivers/net/mlx5/hw
REG_C_6 is used as the 1st one and since it is reserved internally
by default, there is no impact.
The remaining 2 registers will be fetched from the available TAGs
array from right to left. They will not be masked in the array due
to the fact that not all the rules will use NAT64 action.
Signed-
The NAT64 DR actions can be shared among the tables. All these
actions can be created during configuring the flow queues and saved
for the future usage.
Even the actions can be shared now, inside per each flow rule, the
actual hardware resources are unique.
Signed-off-by: Bing Zhao
---
doc/guid
The action will handle the IPv4 and IPv6 headers translation. It will
add / remove IPv6 address prefix by default.
To use the user specific address, another rule to modify the
addresses of the IP header is needed.
Signed-off-by: Bing Zhao
---
drivers/net/mlx5/mlx5_flow_hw.c | 20 +++
NAT64 is treated as a modify header action. The action order and
limitation should be the same as that of modify header in each
domain.
Since the last 2 TAG registers will be used implicitly in the
address backup mode, the values in these registers are no longer
valid after the NAT64 action. The a
On Wed, Feb 28, 2024 at 4:01 PM Morten Brørup
wrote:
>
> > From: David Marchand [mailto:david.march...@redhat.com]
> > Sent: Wednesday, 28 February 2024 15.19
> >
> > On Tue, Feb 27, 2024 at 6:44 AM Tyler Retzlaff
> > wrote:
> > >
> > > RTE_MARKER typedefs are a GCC extension unsupported by MSVC
Hi,
> -Original Message-
> From: Gregory Etelson
> Sent: Wednesday, February 28, 2024 3:33 PM
> To: dev@dpdk.org
> Cc: Gregory Etelson ; Maayan Kashani
> ; Raslan Darawsheh ; Dariusz
> Sosnowski
> Subject: [PATCH v3 0/4] net/mlx5: add support for flow table resizing
>
> Support template
https://bugs.dpdk.org/show_bug.cgi?id=1388
Bug ID: 1388
Summary: Add dependency documentation for TG Node
Product: DPDK
Version: unspecified
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: normal
On Wed, Feb 28, 2024 at 09:28:22AM +0100, Morten Brørup wrote:
> +To: Thomas, previously joined the discussion
>
> > From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com]
> > Sent: Tuesday, 27 February 2024 18.17
> >
> > On Tue, Feb 27, 2024 at 10:10:03AM +0100, Morten Brørup wrote:
> > > >
https://bugs.dpdk.org/show_bug.cgi?id=1389
Bug ID: 1389
Summary: Investigate automatically building config schema
documentation
Product: DPDK
Version: unspecified
Hardware: All
OS: All
Status: UN
On 2/27/2024 7:38 PM, Harman Kalra wrote:
> Fixing uninitialized value access.
>
> Coverity issue: 414934
> Fixes: 9a9eb104edf6 ("ethdev: parse multiple representor devargs")
>
> Signed-off-by: Harman Kalra
>
Acked-by: Ferruh Yigit
Applied to dpdk-next-net/main, thanks.
On 2/27/2024 11:15 AM, Chaoyong He wrote:
> From: Peng Zhang
>
> Add the elf module, which can get mip information from the
> firmware ELF file.
>
What is MIP?
Is it in the .note section of the ELF binary?
How it is used?
Source code always refers it as MIP, can you please clarify it in the
com
On 2/27/2024 11:15 AM, Chaoyong He wrote:
> From: Peng Zhang
>
> Add an option to force reload the firmware.
>
> This option can be ignored in some case, for example:
> When using a 2 port NFP card and both with this reload firmware option,
> only the first one will cause the firmware reload and
Goal of this patchset is to improve the throughput of flow insertion
and deletion in mlx5 PMD when HW Steering flow engine is used.
- Patch 1 - Use preallocated per-queue, per-actions template buffer
for storing translated flow actions, instead of allocating and
filling it on demand, on each f
Goal of this is to remove the unnecessary copying of precalculated
mlx5dr_rule_action structures used to create HWS flow rules.
Before this patch, during template table creation an array of these
structures was calculated for each actions template used.
Each of these structures contained either fu
From: Erez Shitrit
The function returns true if that matcher can contain complicated rule,
which means rule that needs more than one writing to the HW in order to
have it.
Signed-off-by: Erez Shitrit
---
drivers/net/mlx5/hws/mlx5dr.h | 8
drivers/net/mlx5/hws/mlx5dr_action.c
From: Erez Shitrit
The user want to know before trying to update a rule if that matcher that
keeps the original rule supports updating.
Signed-off-by: Erez Shitrit
---
drivers/net/mlx5/hws/mlx5dr.h | 8
drivers/net/mlx5/hws/mlx5dr_matcher.c | 12
2 files changed,
mlx5_hw_q_job struct held a reference to temporary flow rule pattern
and contained temporary REPRESENTED_PORT and TAG items structs.
They are used whenever it is required to prepend a flow rule pattern,
provided by the application with one of such items.
If prepending is required, then flow rule pa
From: Bing Zhao
The resource index was introduced to decouple the flow rule and its
resources used by hardware steering. This is needed only when a rule
update is supported.
In some cases, the update is not supported on a table(matcher). E.g.:
* Table is resizable
* FW gets involved
* Root
mlx5_hw_q_job struct held a reference to a temporary flow rule struct,
used during flow rule update operation. It serves as a container for
flow actions data calculated during actions construction.
After flow rule update operation succeeds, data from temporary flow rule
is copied over to original f
mlx5_hw_q_job struct held references to buffers which contained:
- modify header commands array,
- encap/decap data buffer,
- IPv6 routing data buffer.
These buffers were passed as parameters to HWS layer during rule
creation. They were needed only during the call to HWS layer
when flow operation
From: Bing Zhao
The flow rule can be inserted even before the device started. The
only exception is for a queue or RSS action.
For the other interfaces of template API, the start status is not
checked. The checking would cause some cache miss or eviction since
the flag locates on some other cach
While processing async flow operations in mlx5 PMD,
mlx5_hw_q_job struct is used to hold the following data
related to the ongoing operation.
- operation type,
- user data,
- flow reference.
Job itself is then passed to mlx5dr layer as its "user data".
Other types of data required during flow ope
Some of the flow fields are either not always required
or are used very rarely, e.g.:
- AGE action reference,
- direct METER/METER_MARK action reference,
- matcher selector for resizable tables.
This patch moves these fields to rte_flow_hw_aux struct in order to
reduce the overall size of the flo
Each time a flow is allocated in mlx5 PMD the whole buffer,
both rte_flow_hw and mlx5dr_rule parts, are zeroed.
This introduces some wasted work because:
- mlx5dr layer does not assume that mlx5dr_rule must be initialized,
- flow action translation in mlx5 PMD does not need most of the fields
of
> -Original Message-
> From: Kundapura, Ganapati
> Sent: Wednesday, February 28, 2024 4:09 PM
> To: dev@dpdk.org; jer...@marvell.com; Jayatheerthan, Jay
>
> Cc: Naga Harish K, S V ; Gujjar, Abhinandan S
>
> Subject: [PATCH v1] eventdev/crypto: fix enqueueing invalid ops
>
> When tail
On Wed, Feb 28, 2024 at 03:18:40PM +0100, David Marchand wrote:
> On Tue, Feb 27, 2024 at 6:44 AM Tyler Retzlaff
> wrote:
> >
> > RTE_MARKER typedefs are a GCC extension unsupported by MSVC. Remove
> > RTE_MARKER fields from rte_mbuf struct.
> >
> > Maintain alignment of fields after removed cache
On Wed, Feb 28, 2024 at 01:34:59PM +, Konstantin Ananyev wrote:
>
> >
> > On Mon, Feb 26, 2024 at 9:20 PM Tyler Retzlaff
> > wrote:
> > >
> > > Use RTE_LOG_LINE_PREFIX instead of RTE_LOG_LINE in macro expansions
> > > which allow a prefix and arguments to be inserted into the log line
> > >
On Wed, Feb 28, 2024 at 12:45:04PM +0100, David Marchand wrote:
> On Tue, Feb 27, 2024 at 7:15 PM Tyler Retzlaff
> wrote:
> >
> > On Mon, Feb 26, 2024 at 12:54:36PM -0800, Stephen Hemminger wrote:
> > > On Mon, 26 Feb 2024 12:19:30 -0800
> > > Tyler Retzlaff wrote:
> > >
> > > > RTE_LOG_LINE cann
On 2/10/2024 2:09 AM, Stephen Hemminger wrote:
> On Sat, 10 Feb 2024 00:54:15 +
> Ferruh Yigit wrote:
>
>> On 2/8/2024 7:05 PM, Stephen Hemminger wrote:
>>> Rewrite the BPF program used to do queue based RSS.
>>> Important changes:
>>> - uses newer BPF map format BTF
>>> - accepts key
> On Wed, Feb 28, 2024 at 01:34:59PM +, Konstantin Ananyev wrote:
> >
> > >
> > > On Mon, Feb 26, 2024 at 9:20 PM Tyler Retzlaff
> > > wrote:
> > > >
> > > > Use RTE_LOG_LINE_PREFIX instead of RTE_LOG_LINE in macro expansions
> > > > which allow a prefix and arguments to be inserted into the
On Wed, Feb 28, 2024 at 05:27:54PM +, Konstantin Ananyev wrote:
>
>
> > On Wed, Feb 28, 2024 at 01:34:59PM +, Konstantin Ananyev wrote:
> > >
> > > >
> > > > On Mon, Feb 26, 2024 at 9:20 PM Tyler Retzlaff
> > > > wrote:
> > > > >
> > > > > Use RTE_LOG_LINE_PREFIX instead of RTE_LOG_LINE
On Mon, Feb 26, 2024 at 9:20 PM Tyler Retzlaff
wrote:
>
> RTE_LOG_LINE cannot be augmented with a prefix format and arguments
> without the user of RTE_LOG_LINE using the args... and ## args compiler
> extension to conditionally remove trailing comma when the macro receives
> only a single argumen
On Wed, Feb 28, 2024 at 10:29:11AM +0100, Morten Brørup wrote:
> > From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com]
> > Sent: Wednesday, 28 February 2024 00.08
> >
> > Use existing __rte_pure macro from rte_common.h instead of directly
>
> Typo: __rte_pure -> __rte_may_alias
>
> The co
On 2/22/2024 1:46 PM, Loftus, Ciara wrote:
>> Subject: [v9 2/3] net/af_xdp: fix multi interface support for K8s
>>
>> The original 'use_cni' implementation, was added
>> to enable support for the AF_XDP PMD in a K8s env
>> without any escalated privileges.
>> However 'use_cni' used a hardcoded sock
On 2/23/2024 4:19 AM, Oleksandr Kolomeiets wrote:
> "flow update" updates a flow rule specified by a rule ID with a
> new action list by making a call to "rte_flow_actions_update()":
>
> flow update {port_id} {rule_id}
> actions {action} [/ {action} [...]] / end [user_id]
>
> Creating
> -Original Message-
> From: Haifei Luo
> Sent: Monday, February 26, 2024 2:04 PM
> To: Ori Kam ; Slava Ovsiienko ;
> Matan Azrad ; Shahaf Shuler ;
> Dariusz Sosnowski ; Suanming Mou
>
> Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon (EXTERNAL)
> ; Wisam Jaddo ; Raslan
> Darawsheh ; Roni B
Updates the tag of Arm IPsec-MB library to SECLIB-IPSEC-2023.10.13
in snow3g and zuc documentation. Tag SECLIB-IPSEC-2023.10.13 updates
IPSec-MB library version to 1.4.
Signed-off-by: Wathsala Vithanage
Reviewed-by: Ruifeng Wang
Reviewed-by: Honnappa Nagarahalli
Reviewed-by: Jack Bond-Preston
On Wed, Feb 28, 2024 at 06:29:14PM +0100, David Marchand wrote:
> On Mon, Feb 26, 2024 at 9:20 PM Tyler Retzlaff
> wrote:
> >
> > RTE_LOG_LINE cannot be augmented with a prefix format and arguments
> > without the user of RTE_LOG_LINE using the args... and ## args compiler
> > extension to conditi
Use newly added __rte_pure macro from rte_common.h instead of directly
using __attribute__((pure)).
Signed-off-by: Tyler Retzlaff
Reviewed-by: Morten Brørup
---
lib/lpm/rte_lpm.c | 4 ++--
lib/lpm/rte_lpm6.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/lib/lpm/rte_lpm
1 - 100 of 110 matches
Mail list logo
