patch 'hash: fix maybe-uninitialized warnings on build' has been queued to stable release 24.11.5
luca.boccassi at gmail.com
luca.boccassi at gmail.com
Fri Feb 20 15:56:36 CET 2026
Hi,
FYI, your patch has been queued to stable release 24.11.5
Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 02/22/26. So please
shout if anyone has objections.
Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.
Queued patches are on a temporary branch at:
https://github.com/bluca/dpdk-stable
This queued commit can be viewed at:
https://github.com/bluca/dpdk-stable/commit/0737c2ae50d370c3ab158ef7d40baf0e3a53a890
Thanks.
Luca Boccassi
---
>From 0737c2ae50d370c3ab158ef7d40baf0e3a53a890 Mon Sep 17 00:00:00 2001
From: Bruce Richardson <bruce.richardson at intel.com>
Date: Thu, 12 Feb 2026 16:27:05 +0000
Subject: [PATCH] hash: fix maybe-uninitialized warnings on build
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
[ upstream commit 32cdf0611b570b49fcee799d080656e3bfbed8e5 ]
Following recent changes to the test-meson-build.sh script[1] to use more
build-types warnings are seen in the hash library GFNI code about values
possibly being uninitialized.
In function ‘_mm512_mask_permutexvar_epi8’,
inlined from ‘__rte_thash_gfni’ at ../lib/hash/rte_thash_x86_gfni.h:150:6:
/usr/lib/gcc/x86_64-linux-gnu/15/include/avx512vbmiintrin.h:97:20:
error: ‘tuple_bytes_2’ may be used uninitialized [-Werror=maybe-uninitialized]
These all appear to be a false positive, but fix these by explicitly
zero-initializing the offending values.
[1] 177acaf5cf75 ("devtools: test different build types")
Fixes: 4fd8c4cb0de1 ("hash: add new Toeplitz hash implementation")
Signed-off-by: Bruce Richardson <bruce.richardson at intel.com>
Acked-by: Vladimir Medvedkin <vladimir.medvedkin at intel.com>
Acked-by: David Marchand <david.marchand at redhat.com>
---
lib/hash/rte_thash_x86_gfni.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/hash/rte_thash_x86_gfni.h b/lib/hash/rte_thash_x86_gfni.h
index 99226b7592..69973503fd 100644
--- a/lib/hash/rte_thash_x86_gfni.h
+++ b/lib/hash/rte_thash_x86_gfni.h
@@ -89,9 +89,9 @@ __rte_thash_gfni(const uint64_t *mtrx, const uint8_t *tuple,
const __m512i shift_8 = _mm512_set1_epi8(8);
__m512i xor_acc = _mm512_setzero_si512();
__m512i perm_bytes = _mm512_setzero_si512();
- __m512i vals, matrixes, tuple_bytes_2;
+ __m512i vals, matrixes, tuple_bytes_2 = _mm512_setzero_si512();
__m512i tuple_bytes = _mm512_setzero_si512();
- __mmask64 load_mask, permute_mask_2;
+ __mmask64 load_mask, permute_mask_2 = 0;
__mmask64 permute_mask = 0;
int chunk_len = 0, i = 0;
uint8_t mtrx_msk;
--
2.47.3
---
Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- - 2026-02-20 14:55:46.834982971 +0000
+++ 0094-hash-fix-maybe-uninitialized-warnings-on-build.patch 2026-02-20 14:55:43.340193380 +0000
@@ -1 +1 @@
-From 32cdf0611b570b49fcee799d080656e3bfbed8e5 Mon Sep 17 00:00:00 2001
+From 0737c2ae50d370c3ab158ef7d40baf0e3a53a890 Mon Sep 17 00:00:00 2001
@@ -8,0 +9,2 @@
+[ upstream commit 32cdf0611b570b49fcee799d080656e3bfbed8e5 ]
+
@@ -24 +25,0 @@
-Cc: stable at dpdk.org
More information about the stable
mailing list