Acked-by: Vladimir Medvedkin
On 10/07/2023 22:59, Bili Dong wrote:
An XOR32 hash is needed in the Software Switch (SWX) Pipeline for its
use case in P4. We implement it in this patch so it could be easily
registered in the pipeline later.
Signed-off-by: Bili Dong
--
Regards,
Vladimir
Acked-by: Vladimir Medvedkin
On 30/09/2022 13:40, Radu Nicolau wrote:
Fix a typo in computing port mask for Tx checksum offload capability.
Fixes: 4edcee19fc20 ("examples/ipsec-secgw: use Tx checksum offload
conditionally")
Cc: ndabilpu...@marvell.com
Signed-off-by: Radu Nicolau
---
examp
On 21/04/2021 15:48, Conor Walsh wrote:
The commit a8f8b672d575 ("examples/l3fwd: skip Tx queue drain on first
iteration") implemented a change to the em and lpm lookup methods to
prevent the TX queue drain running in the first iteration of their
packet processing loops. This patch introduces
Hello,
This patch breaks the packet validation test.
https://bugs.dpdk.org/show_bug.cgi?id=695
Please take a look.
Thanks!
On 15/04/2021 10:22, Tejasree Kondoj wrote:
Adding lookaside IPsec UDP encapsulation support
for NAT traversal.
Application has to add udp-encap option to sa config file
t
Hello David,
Oh, interesting, thanks for the report, I'll take a look!
On 29/04/2021 12:13, David Marchand wrote:
Hello Vladimir,
On Mon, Apr 19, 2021 at 6:00 PM Vladimir Medvedkin
wrote:
+static int
+test_adjust_tuple(void)
+{
+ struct rte_thash_ctx *ctx;
+ struct rte_thash_subt
Hi Stanislaw,
On 29/04/2021 21H:45, Stanislaw Kardach wrote:
On Thu, Apr 29, 2021 at 12:17:08PM +0300, Medvedkin, Vladimir wrote:
Test Failed
RTE>>
--- stderr ---
EAL: Detected 2 lcore(s)
EAL: Detected 1 NUMA nodes
EAL: Detected shared linkage of DPDK
EAL: Multi-process socket /var/ru
Hi Owen,
Thanks for the fix.
I like your solution with removing the loop. However, while this fixes
the buffer overflow, IMO it is not complete, because get_dir() shouldn't
be called in cases where depth = 128. In this case checking the MSB of
the ip is not quite right thing.
The only place w
Hi Owen,
Just a few nits inlined below
On 16/06/2021 21:18, ohily...@iol.unh.edu wrote:
From: Owen Hilyard
ASAN found a stack buffer overflow in lib/rib/rte_rib6.c:get_dir.
The fix for the stack buffer overflow was to make sure depth
was always < 128, since when depth = 128 it caused the inde
On 18/06/2021 19:22, Medvedkin, Vladimir wrote:
Hi Owen,
Just a few nits inlined below
On 16/06/2021 21:18, ohily...@iol.unh.edu wrote:
From: Owen Hilyard
ASAN found a stack buffer overflow in lib/rib/rte_rib6.c:get_dir.
The fix for the stack buffer overflow was to make sure depth
was
Hi Owen, David,
Apart from David's comments looks good to me.
On 22/06/2021 10:10, David Marchand wrote:
On Mon, Jun 21, 2021 at 3:28 PM wrote:
From: Owen Hilyard
Hi Owen, Vladimir,
Owen, two comments on the patch title.
- We (try to) never prefix with lib/, as it gives no additional
Acked-by: Vladimir Medvedkin
On 21/06/2021 16:28, ohily...@iol.unh.edu wrote:
From: Owen Hilyard
ASAN found a stack buffer overflow in lib/rib/rte_rib6.c:get_dir.
The fix for the stack buffer overflow was to make sure depth
was always < 128, since when depth = 128 it caused the index
into the
Hi Bruce,
Sure, will submit v2.
Thanks!
-Original Message-
From: Richardson, Bruce
Sent: Thursday, July 11, 2024 6:05 PM
To: Medvedkin, Vladimir
Cc: dev@dpdk.org; ting...@intel.com; Stolarchuk, Michael
Subject: Re: [PATCH] net/ice: fix possible memory leak
On Thu, Jul 11, 2024 at
Hi Morten,
On 19/07/2024 10:12, Morten Brørup wrote:
From: Robin Jarry [mailto:rja...@redhat.com]
Vladimir Medvedkin, Jul 18, 2024 at 23:25:
I think alignment should be 1 since in FIB6 users usually don't copy
IPv6
address and just provide a pointer to the memory inside the packet.
How can
-Original Message-
From: Richardson, Bruce
Sent: Monday, July 22, 2024 11:42 AM
To: Medvedkin, Vladimir
Cc: dev@dpdk.org; sta...@dpdk.org; Stolarchuk, Michael
Subject: Re: [PATCH v3 1/3] net/ice: fix possible memory leak
On Mon, Jul 22, 2024 at 08:28:34AM +, Vladimir Medvedkin
-Original Message-
From: Richardson, Bruce
Sent: Monday, July 22, 2024 12:25 PM
To: Medvedkin, Vladimir
Cc: dev@dpdk.org
Subject: Re: [PATCH v4 2/3] net/ice: refactor raw pattern parsing function
On Mon, Jul 22, 2024 at 10:59:49AM +, Vladimir Medvedkin wrote:
> Replace str
Hi all,
This patch breaks ipsec tests with ipsec-secgw:
./examples/ipsec-secgw/test/run_test.sh -4 trs_aesctr_sha1
...
ERROR: ./examples/ipsec-secgw/test/linux_test.sh failed for dst=192.168.31.14,
sz=1
test IPv4 trs_aesctr_sha1 finished with status 1
ERROR test trs_aesctr_sha1 FAILED
On 03
Hi Robin,
Apologies for the delayed response
On 19/03/2024 20:38, Robin Jarry wrote:
Hi Vladimir,
Medvedkin, Vladimir, Mar 19, 2024 at 18:16:
> 2) Is it OK/safe to modify a fib from a control thread (read/write)
> while it is used by data path threads (read only)?
This part is
Acked-by: Vladimir Medvedkin
On 23/07/2024 09:27, Robin Jarry wrote:
In 24.11, all IPv6 public APIs will be modified to use a structure
instead of fixed size arrays.
Signed-off-by: Robin Jarry
Acked-by: Morten Brørup
---
Notes:
v2: updated with the exhaustive list of symbols
doc/gui
Hi Robin,
On 31/07/2024 16:04, Robin Jarry wrote:
Hi Vladimir,
I noticed that the fib/rib APIs (both IPv4 and IPv6) require the next
hops to be represented as integer indexes. Reading the code, I noticed
that they are stored as uint64_t with the MSB used for internal purposes.
I believe yo
: rajesh goel
Sent: Tuesday, August 27, 2024 7:04 AM
To: Ferruh Yigit
Cc: Wang, Yipeng1 ; Gobriel, Sameh
; Richardson, Bruce ;
Medvedkin, Vladimir ; dev@dpdk.org
Subject: Re: Bihash Support in DPDK
Hi All,
Can we get some reply.
Thanks
Rajesh
On Thu, Aug 22, 2024 at 9:32 PM Ferruh Yigit
Hi Rajesh,
rte_hash does not support per bucket locks, instead it uses global rwlock.
But you can try lock free mode (see documentation, in particular
RTE_HASH_EXTRA_FLAGS_RW_CONCURRENCY_LF flag)
From: rajesh goel
Sent: Tuesday, August 27, 2024 4:57 PM
To: Medvedkin, Vladimir
Cc: Ferruh
I am not aware of such plans.
From: rajesh goel
Sent: Wednesday, August 28, 2024 11:22 AM
To: Medvedkin, Vladimir
Cc: Ferruh Yigit ; Wang, Yipeng1
; Gobriel, Sameh ; Richardson,
Bruce ; dev@dpdk.org
Subject: Re: Bihash Support in DPDK
Thanks Vladimir for the confirmation.
Is there any plan
Hi David,
On 03/06/2022 10:41, David Marchand wrote:
On Fri, Jun 3, 2022 at 9:56 AM Bruce Richardson
wrote:
On Fri, Jun 03, 2022 at 09:45:45AM +0200, David Marchand wrote:
Hello Vladimir,
On Thu, Jun 2, 2022 at 8:42 PM Medvedkin, Vladimir
wrote:
if (!dst
On 17/06/2022 13:06, David Marchand wrote:
On Thu, Jun 16, 2022 at 11:33 AM David Marchand
wrote:
GCC 12 raises the following warning:
In function ‘_mm256_loadu_si256’,
inlined from ‘rte_mov32’ at
../lib/eal/x86/include/rte_memcpy.h:319:9,
inlined from ‘rte_mov128’ at
Acked-by: Vladimir Medvedkin
On 22/06/2022 21:41, Stephen Hemminger wrote:
The comments in rte_rib6 were cut-and-pasted from rte_rib
and because of that some references to rte_rib_node were
not updated.
Signed-off-by: Stephen Hemminger
---
lib/rib/rte_rib6.h | 6 +++---
1 file changed, 3 i
Tested-by: Vladimir Medvedkin
On 30/06/2022 12:45, Radu Nicolau wrote:
Fix fallback session create for inline sessions.
Fixes: a8ade12123c3 ("examples/ipsec-secgw: create lookaside sessions at init")
Cc: vfia...@marvell.com
Signed-off-by: Radu Nicolau
---
v2: create the session rather than j
Hi Konstantin,
Thanks for the review,
On 07/04/2021 15:53, Ananyev, Konstantin wrote:
Hi Vladimir,
Few comments below, mostly minor.
One generic one - doc seems missing.
With that in place:
Acked-by: Konstantin Ananyev
This patch implements predictable RSS functionality.
Signed-off-by: Vl
Hi Stephen,
Thanks for the feedback,
On 08/04/2021 18:56, Stephen Hemminger wrote:
On Tue, 6 Apr 2021 20:50:40 +0100
Vladimir Medvedkin wrote:
This patch series introduces predictable RSS feature.
It is based on the idea of searching for partial hash collisions
within Toeplitz hash.
The To
Hi Yipeng,
Thanks for the review,
On 10/04/2021 03:32, Wang, Yipeng1 wrote:
-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
On 10/04/2021 03:05, Wang, Yipeng1 wrote:
-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
On 10/04/2021 03:10, Wang, Yipeng1 wrote:
-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
Hi Konstantin,
On 12/04/2021 12:47, Ananyev, Konstantin wrote:
+#defineRETA_SZ_MIN2U
+#defineRETA_SZ_MAX16U
Should these RETA_SZ defines be in public header?
So user can know what are allowed values?
I don't think this is necessary, because the user chooses it not
arbitrary, but dependi
Hi Yaron,
On 09/01/2022 13:20, Yaron Illouz wrote:
I am using Mellanox Technologies MT27800 Family [ConnectX-5], using dpdk
19 with multi rx queue with rss
port_conf.rx_adv_conf.rss_conf.rss_hf=(ETH_RSS_IP | ETH_RSS_UDP |
ETH_RSS_TCP)
I analyze traffic and need all packet of same session to
Hi Kevin,
On 11/01/2022 17:15, Kevin Traynor wrote:
On 23/12/2021 15:25, Vladimir Medvedkin wrote:
This patch fixes the division by 0, which occurs if the number of
routes is less than 10.
Can be triggered by passing -n argument with value < 10:
9 causing a divide by zero - another example
Hi all,
On 08/02/2022 10:44, Ananyev, Konstantin wrote:
This patchset introduces config file support for l3fwd
and its lookup methods LPM, FIB, and EM, similar to
that of l3fwd-acl. This allows for route rules to be
defined in configuration files and edited there instead
of in each of the look
Hi,
On 09/02/2022 13:54, Bruce Richardson wrote:
On Wed, Feb 09, 2022 at 12:00:40PM +, Ananyev, Konstantin wrote:
Or have a generic library for reading LPM entries. L3fwd is supposed
to be as small as possible (it no longer is), and the real work should
be done by libraries to make it e
Hi David,
On 11/10/2021 13:03, David Marchand wrote:
On Fri, Oct 8, 2021 at 11:28 PM Vladimir Medvedkin
wrote:
This patch fixes buffer overflow reported by ASAN,
please reference https://bugs.dpdk.org/show_bug.cgi?id=818
Some tests for the rte_hash table use the rte_jhash_32b() as
the hash f
Hi Honnappa,
On 14/10/2021 02:19, Honnappa Nagarahalli wrote:
On Wed, 13 Oct 2021 16:38:58 -0500
Honnappa Nagarahalli wrote:
--- a/lib/hash/rte_hash.h
+++ b/lib/hash/rte_hash.h
@@ -430,7 +430,6 @@ rte_hash_get_key_with_position(const struct
rte_hash *h, const int32_t position,
* -
Hi David,
On 14/10/2021 10:34, David Marchand wrote:
On Wed, Oct 13, 2021 at 9:28 PM Vladimir Medvedkin
wrote:
This patch fixes buffer overflow reported by ASAN,
please reference https://bugs.dpdk.org/show_bug.cgi?id=818
Some tests for the rte_hash table use the rte_jhash_32b() as
the hash f
Hi David,
On 14/10/2021 09:04, David Marchand wrote:
Hello Vladimir,
On Wed, Oct 13, 2021 at 9:27 PM Medvedkin, Vladimir
wrote:
With patch applied, ASan reports another issue.
Did you test your fix with ASan?
You're right, for some reason ASAN wasn't enabled.
I applied patch
Hi Konstantin,
Thanks for the review,
On 07/10/2021 20:23, Ananyev, Konstantin wrote:
This patch add a new Toeplitz hash implementation using
Galios Fields New Instructions (GFNI).
Signed-off-by: Vladimir Medvedkin
---
doc/api/doxy-api-index.md | 1 +
lib/hash/meson.build | 1 +
Hi Konstantin,
On 08/10/2021 13:31, Ananyev, Konstantin wrote:
This patch enables new GFNI Toeplitz hash in
predictable RSS library.
Signed-off-by: Vladimir Medvedkin
---
lib/hash/rte_thash.c | 43 +++
lib/hash/rte_thash.h | 19 +++
Hi David,
On 15/10/2021 11:33, David Marchand wrote:
On Thu, Oct 14, 2021 at 7:55 PM Vladimir Medvedkin
wrote:
@@ -1607,6 +1611,17 @@ static struct rte_hash_parameters hash_params_ex = {
};
/*
+ * Wrapper function around rte_jhash_32b.
+ * It is required because rte_jhash_32b() accepts th
On 15/10/2021 12:55, Ananyev, Konstantin wrote:
+/**
+ * Calculate Toeplitz hash.
+ *
+ * @warning
+ * @b EXPERIMENTAL: this API may change without prior notice.
+ *
+ * @param m
+ * Pointer to the matrices generated from the corresponding
+ * RSS hash key using rte_thash_complete_matrix()
Hi Stephen,
Thanks for reviewing
On 15/10/2021 18:58, Stephen Hemminger wrote:
On Fri, 15 Oct 2021 10:30:02 +0100
Vladimir Medvedkin wrote:
+ m[i * 8 + j] = (rss_key[i] << j)|
+ (uint8_t)((uint16_t)(rss_key[i + 1]) >>
+
Hi Stephen,
On 19/10/2021 03:15, Stephen Hemminger wrote:
On Mon, 18 Oct 2021 10:40:00 +
"Ananyev, Konstantin" wrote:
On Fri, 15 Oct 2021 10:30:02 +0100
Vladimir Medvedkin wrote:
+ m[i * 8 + j] = (rss_key[i] << j)|
+ (uint8_t)((uin
Hi David,
On 19/10/2021 09:02, David Marchand wrote:
On Fri, Oct 15, 2021 at 3:02 PM Medvedkin, Vladimir
wrote:
I am confused.
Does it mean that rte_jhash_32b is not compliant with rte_hash_create API?
I think so too, because despite the fact that the ABI is the same, the
API remains
Hi Stanislaw, Michal,
As far as I can see, this implementation almost completely repeats other
lookupx4() implementations, except for the use of vector instructions.
On my board (x86_64) in lpm_perf_autotest your implementation takes about:
LPM LookupX4: 29.5 cycles (fails = 12.5%)
replacing
Hi Stanislaw,
On 27/05/2022 12:16, Stanisław Kardach wrote:
On Tue, May 24, 2022 at 6:28 PM Stanisław Kardach wrote:
That said I wonder why do we have different const requirements for
rte_lpm_lookup() and rte_lpm_lookupx4():
static inline int rte_lpm_lookup(struct rte_lpm *lpm, uint32_t ip
On 30/05/2022 19:24, Stanislaw Kardach wrote:
All other rte_lpm_lookup* functions take lpm argument as a const. As the
basic rte_lpm_lookup() performs the same function, it should also do
that.
As this function is inline, no API/ABI change happens.
Signed-off-by: Stanislaw Kardach
---
lib
Hi Stanislaw,
On 30/05/2022 19:24, Stanislaw Kardach wrote:
From: Michal Mazurek
Add an implementation of the rte_lpm_lookupx4() function for platforms
without support for vector operations.
This will be useful in the upcoming RISC-V port as well as any platform
which may want to start with
On 01/06/2022 12:15, Stanislaw Kardach wrote:
From: Michal Mazurek
Add an implementation of the rte_lpm_lookupx4() function for platforms
without support for vector operations.
This will be useful in the upcoming RISC-V port as well as any platform
which may want to start with a basic level
Hi David,
On 18/05/2022 11:16, David Marchand wrote:
GCC 12 raises the following warning:
In function ‘_mm256_loadu_si256’,
inlined from ‘rte_mov32’ at
../lib/eal/x86/include/rte_memcpy.h:319:9,
inlined from ‘rte_mov128’ at
../lib/eal/x86/include/rte_memcpy.h:344:2,
Hi David,
On 20/10/2021 21:55, David Marchand wrote:
Hello Vladimir,
On Fri, Oct 8, 2021 at 11:29 PM Vladimir Medvedkin
wrote:
This patch fixes buffer overflow reported by ASAN,
please reference https://bugs.dpdk.org/show_bug.cgi?id=819
The rte_lpm6 keeps routing information for control pla
Hi Konstantin,
On 21/10/2021 11:42, Ananyev, Konstantin wrote:
This patch add a new Toeplitz hash implementation using
Galios Fields New Instructions (GFNI).
Signed-off-by: Vladimir Medvedkin
---
doc/api/doxy-api-index.md | 1 +
lib/hash/meson.build | 1 +
lib/hash/rte_t
Hi Thomas,
Thanks for the review, I'll address your comments in v6.
Please find my comment below
On 25/10/2021 19:02, Thomas Monjalon wrote:
21/10/2021 20:54, Vladimir Medvedkin:
This patch adds performance tests for different implementations
of the Toeplitz hash function.
Please name them.
On 25/10/2021 19:04, Thomas Monjalon wrote:
Vladimir, your patches are late and not perfect.
You need reviews. Please ask other maintainers to help with reviews.
21/10/2021 20:54, Vladimir Medvedkin:
This patch adds documentation for the new optimized Toeplitz hash
implementation using GFNI
Hi Thomas,
On 25/10/2021 19:23, Thomas Monjalon wrote:
06/09/2021 17:55, Vladimir Medvedkin:
This patch adds a new parameter to the fib configuration to specify
the size of the extension for internal RIB structure.
It looks to be an announced API change.
What happens if the new field is not i
Hi Stephen,
On 25/10/2021 19:27, Stephen Hemminger wrote:
On Thu, 21 Oct 2021 19:54:29 +0100
Vladimir Medvedkin wrote:
+static uint8_t default_rss_key[] = {
Should this be const?
That way you can make sure API isn't modifying it.
Thanks, I'll fix this in v6
--
Regards,
Vladimir
Hi Thomas,
On 27/10/2021 10:28, Thomas Monjalon wrote:
26/10/2021 22:30, Medvedkin, Vladimir:
Hi Thomas,
On 25/10/2021 19:23, Thomas Monjalon wrote:
06/09/2021 17:55, Vladimir Medvedkin:
This patch adds a new parameter to the fib configuration to specify
the size of the extension for
Hi Thomas,
On 27/10/2021 10:29, Thomas Monjalon wrote:
26/10/2021 22:29, Medvedkin, Vladimir:
Hi Thomas,
Thanks for the review, I'll address your comments in v6.
Please find my comment below
On 25/10/2021 19:02, Thomas Monjalon wrote:
21/10/2021 20:54, Vladimir Medvedkin:
This patch
Hi Usama,
On 04/11/2021 12:11, Walsh, Conor wrote:
From: dev On Behalf Of Usama Nadeem
Sent: Thursday 14 October 2021 19:43
To: tho...@monjalon.net
Cc: dev@dpdk.org; Usama Nadeem
Subject: [dpdk-dev] [PATCH v4] examples/l3fwd: ipv4 and udp/tcp cksum
verification through software
checks if ipv4
@dpdk.org; Wang, Yipeng1
; Gobriel, Sameh ;
Richardson, Bruce ; Medvedkin, Vladimir
Subject: Re: [dpdk-dev] Doubt regarding DPDK hash Library
implementation
+Cc hash lib maintainers
01/11/2021 11:55, Syam Prasad N Pearson:
Dear Sir/Madam,
I am a developer trying to get familiar with the DPDK hash
Hi all,
It looks like the problem caused by the lack of _mm512_set1_epi8()
implementation inside the gcc8, however _mm512_set1_epi[16,32,64] has
been implemented.
I replaced it with _mm512_set1_epi32 and on gcc8 it builds fine. I'll
send a fix soon.
On 09/11/2021 17:22, Lance Richardson wrot
Hi Honnappa,
On 08/11/2021 23:04, Honnappa Nagarahalli wrote:
This patch adds a comment for RTE_HASH_BUCKET_ENTRIES explaining why a
particular value was chosen.
Signed-off-by: Vladimir Medvedkin
---
lib/hash/rte_cuckoo_hash.h | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
On 14/09/2021 09:05, David Marchand wrote:
On Mon, Sep 13, 2021 at 8:47 PM Ben Pfaff wrote:
This is probably due to conversion from ms word to rst format.
Fixes: fc1f2750a3ec ("doc: programmers guide")
Cc: sta...@dpdk.org
Signed-off-by: Ben Pfaff
Reviewed-by: David Marchand
Acke
Hi Jan,
The RIB is always used as a control plane struct intended to maintain
the correct content of the dataplane struct, such as DIR24_8 for
example. So it is always used on _add()/_delete(). For simplicity you
can consider it as an LPM's rule_info. But instead of keeping routes in
a plane
On 05/08/2021 15:32, Jan Viktorin wrote:
On Thu, 5 Aug 2021 15:27:15 +0200
"Medvedkin, Vladimir" wrote:
Hi Jan,
The RIB is always used as a control plane struct intended to maintain
the correct content of the dataplane struct, such as DIR24_8 for
example. So it is always us
On 05/08/2021 16:07, Jan Viktorin wrote:
On Thu, 5 Aug 2021 15:57:14 +0200
"Medvedkin, Vladimir" wrote:
On 05/08/2021 15:32, Jan Viktorin wrote:
On Thu, 5 Aug 2021 15:27:15 +0200
"Medvedkin, Vladimir" wrote:
Hi Jan,
The RIB is always used as a control plan
On 05/08/2021 16:34, Jan Viktorin wrote:
On Thu, 5 Aug 2021 16:29:50 +0200
"Medvedkin, Vladimir" wrote:
On 05/08/2021 16:07, Jan Viktorin wrote:
On Thu, 5 Aug 2021 15:57:14 +0200
"Medvedkin, Vladimir" wrote:
On 05/08/2021 15:32, Jan Viktorin wrote:
On Thu, 5 Aug
On 12/11/2021 01:02, Stephen Hemminger wrote:
Lots of typos in comments found by codespell
Signed-off-by: Stephen Hemminger
Acked-by: Konstantin Ananyev
---
lib/ipsec/esp_inb.c | 4 ++--
lib/ipsec/esp_outb.c | 2 +-
lib/ipsec/ipsec_sad.c | 2 +-
lib/ipsec/sa.c| 2 +-
lib/ip
On 12/11/2021 01:02, Stephen Hemminger wrote:
Yet another spelling error found by codespell.
Signed-off-by: Stephen Hemminger
---
lib/hash/rte_thash.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/hash/rte_thash.c b/lib/hash/rte_thash.c
index 394e1328c531..6847e3
On 12/11/2021 01:02, Stephen Hemminger wrote:
Yet another spelling error found by codespell.
Signed-off-by: Stephen Hemminger
---
lib/rib/rte_rib6.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/rib/rte_rib6.c b/lib/rib/rte_rib6.c
index b00c7fbf5ab0..70405113b419
On 12/11/2021 15:01, Conor Walsh wrote:
This patch fixes unused but set variables within the following test apps:
- test distributor perf
- test thash perf
Bugzilla ID: 881
Fixes: c0de0eb82e40 ("distributor: switch over to new API")
Fixes: 239fffe0402e ("test/thash: add performance tests
Hi David,
On 16/11/2021 14:53, David Marchand wrote:
On Fri, Nov 12, 2021 at 3:17 PM Vladimir Medvedkin
wrote:
1. This patch replaces _mm512_set_epi8 with _mm512_set_epi32
due to the lack of support by some compilers.
Ok, it was the initial report from Lance.
2. This patch checks if AVX51
On 26/01/2021 14:18, Bruce Richardson wrote:
The rte_rib6 header was using RTE_MIN macro from rte_common.h but not
including the header file.
Fixes: f7e861e21c46 ("rib: support IPv6")
Cc: vladimir.medved...@intel.com
Signed-off-by: Bruce Richardson
---
lib/librte_rib/rte_rib6.h | 1 +
1
On 26/01/2021 21:38, Bruce Richardson wrote:
The rte_rib6 header was using RTE_MIN macro from rte_common.h but not
including the header file.
Fixes: f7e861e21c46 ("rib: support IPv6")
Cc: vladimir.medved...@intel.com
Signed-off-by: Bruce Richardson
---
lib/librte_rib/rte_rib6.h | 1 +
1
Hi Tal,
On 10/02/2021 13:33, Burakov, Anatoly wrote:
On 06-Jan-21 8:35 PM, Tal Shnaiderman wrote:
Add support for TLS functionality in EAL.
The following functions are added:
rte_thread_tls_key_create - create a TLS data key.
rte_thread_tls_key_delete - delete a TLS data key.
rte_thread_tls_va
Hi Stepthen,
There is a problem with the build on a machine that supports GFNI:
../lib/hash/rte_thash_gfni.c:15:1: error: redefinition of ‘rte_thash_gfni’
15 | rte_thash_gfni(const uint64_t *mtrx __rte_unused,
| ^~
In file included from ../lib/hash/rte_thash_gfni.h:13,
Hi Bill,
On 15/02/2023 11:06, Bili Dong wrote:
An XOR32 hash is needed in the Software Switch (SWX) Pipeline for its
use case in P4. We implement it in this patch so it could be easily
registered in the pipeline later.
Signed-off-by: Bili Dong
---
+static inline uint32_t
+rte_hash_xor(const vo
Hi Wenjing,
Please find comments inlined
On 05/01/2024 08:16, wenjing.q...@intel.com wrote:
From: Wenjing Qiao
To supporting P4-programmed network controller, reuse devargs
"flow_parser" to specify the path of a p4 context JSON configure
file. The cpfl PMD use the JSON configuration file to t
Hi Wenjing,
Please find comments inlined
On 05/01/2024 08:16, wenjing.q...@intel.com wrote:
From: Wenjing Qiao
Add TDI implementation to a flow engine.
Signed-off-by: Wenjing Qiao
---
--- /dev/null
+++ b/drivers/net/cpfl/cpfl_tdi.c
@@ -0,0 +1,1282 @@
+/* SPDX-License-Identifier: BSD-3-Clau
Hi Abdullah,
Could you please tell more about use cases where this API may be useful?
a new API to get the hidden key count in the hash table if the rcu qsbr is
enabled
Here in commit message and down below in doxygen comments, I think this
statement should be more specific because rcu can
On 30/01/2024 10:00, Kevin Traynor wrote:
When vf issues a reset to pf there is a 50 msec wait plus an additional
max of 1 msec for the pf to indicate the reset is complete before
timeout.
In some cases, it is seen that the reset is timing out, in which case
the reset does not complete and an er
On 15/02/2024 13:31, edwin.brosse...@6wind.com wrote:
From: Edwin Brossette
Different Intel nics with the igxbe pmd do not handle RSS in the same
way when working with virtualization. While some nics like Intel 82599ES
only have a single RSS table in the device and leave all rss features to
b
Hi Beilei,
Just a one nit below, besides that LGTM
Acked-by: Vladimir Medvedkin
On 22/12/2023 16:51, beilei.x...@intel.com wrote:
From: Beilei Xing
Since the response of virtual channel virtchnl2_get_ptype_info is
changed on IMC side, driver needs to be updated when requiring
the virtual ch
Hi Mingjin,
On 11/04/2024 10:45, Mingjin Ye wrote:
This patch enable three Forward Error Correction(FEC) related ops
in ice driver. As no speed information can get from HW, this patch
only show FEC capability.
Signed-off-by: Mingjin Ye
---
doc/guides/nics/features/ice.ini | 1 +
doc/guides
Hi Zhichao,
On 11/04/2024 10:29, Zhichao Zeng wrote:
This patch enabled querying Forward Error Correction(FEC) capabilities,
set FEC mode and get current FEC mode functions.
Signed-off-by: Qiming Yang
Signed-off-by: Zhichao Zeng
---
v5: fix some judgments
v4: fix some logic
v3: optimize code
Acked-by: Vladimir Medvedkin
On 02/07/2024 09:40, Zhichao Zeng wrote:
This patch enabled querying Forward Error Correction(FEC) capabilities,
set FEC mode and get current FEC mode functions.
Signed-off-by: Zhichao Zeng
---
v6: fix some judgments
v5: fix some judgments
v4: fix some logic
v3:
Hi Minjin,
- please update release notes
- see comments inline
On 02/07/2024 09:02, Mingjin Ye wrote:
This patch enable three Forward Error Correction(FEC) related ops
in ice driver. As no speed information can get from HW, this patch
only show FEC capability.
Signed-off-by: Qiming Yang
Signe
Acked-by: Vladimir Medvedkin
On 04/07/2024 07:50, Mingjin Ye wrote:
This patch enable three Forward Error Correction(FEC) related ops
in ice driver. As no speed information can get from HW, this patch
only show FEC capability.
Signed-off-by: Qiming Yang
Signed-off-by: Mingjin Ye
---
v3: opt
Hi Satananda, Anoob, Chengwen, Jerin, all,
After a number of internal discussions we have decided that we're going
to postpone this feature/patchset till next release.
>[Satananda] Have you considered extending rte_dma_port_param and
rte_dma_vchan_conf to represent interdomain memory transfe
Hi Jeffrey,
Have you tried FIB library? This problem has been solved there for a
long time, and it also has a number of other improvements.
Thanks!
On 10/11/2023 14:08, Warrington, Jeffrey wrote:
On 11/10/23, 2:42 AM, "David Marchand" mailto:david.march...@redhat.com>> wrote:
On Fri, Nov 1
Acked-by: Vladimir Medvedkin
On 28/02/2024 18:47, Tyler Retzlaff wrote:
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 +-
Acked-by: Vladimir Medvedkin
On 06/03/2024 20:13, Tyler Retzlaff wrote:
Zero length arrays are GNU extension. Replace with
standard flex array.
Signed-off-by: Tyler Retzlaff
Reviewed-by: Honnappa Nagarahalli
Reviewed-by: Morten Brørup
Acked-by: Chengwen Feng
Acked-by: Stephen Hemminger
--
Acked-by: Vladimir Medvedkin
On 06/03/2024 20:13, Tyler Retzlaff wrote:
Zero length arrays are GNU extension. Replace with
standard flex array.
Signed-off-by: Tyler Retzlaff
Reviewed-by: Morten Brørup
Acked-by: Stephen Hemminger
---
lib/fib/dir24_8.h | 2 +-
lib/fib/trie.h| 2 +-
2
Acked-by: Vladimir Medvedkin
On 06/03/2024 20:13, Tyler Retzlaff wrote:
Zero length arrays are GNU extension. Replace with
standard flex array.
Signed-off-by: Tyler Retzlaff
Reviewed-by: Morten Brørup
---
lib/lpm/rte_lpm6.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Hi Zhicha,
It would be good to reflect FEC feature here:
https://doc.dpdk.org/guides/nics/overview.html
in "/Table 1.1 //Features availability in networking drivers/"
please find the rest comments inline
On 06/03/2024 10:41, Zhichao Zeng wrote:
This patch enabled querying Forward Error Corre
Hi Zhichao,
On 12/03/2024 08:44, Zeng, ZhichaoX wrote:
Hi Vladimir:
Thanks for your comments, some responses and questions are inline.
From: Medvedkin, Vladimir
Sent: Monday, March 11, 2024 11:59 PM
To: Zeng, ZhichaoX ; dev@dpdk.org
Cc: Cui, KaixinX ; Yang, Qiming ;
Zhang, Yuying
Subject
Hi Robin,
On 19/03/2024 08:30, Robin Jarry wrote:
Hi Vladimir,
I have been using rte_fib for a while and stumbled upon a few quirks.
I was wondering if you would answer some questions:
1) Is it OK/safe to share the same fib to perform route lookups from
multiple lcores in parallel? So far
1 - 100 of 265 matches
Mail list logo
