[RFC PATCH v2 05/33] net: remove shadowed variable
Bruce Richardson
bruce.richardson at intel.com
Fri Nov 7 16:50:01 CET 2025
The "mask" local variable shadowed a global definition. The local var
was only used on two lines as a temporary value to pass the return value
of one function as a parameter to another. Therefore, we can remove the
shadowing issue by removing the variable.
Fixes: 17a937baed3e ("net: add CRC AVX512 implementation")
Cc: stable at dpdk.org
Signed-off-by: Bruce Richardson <bruce.richardson at intel.com>
---
lib/net/net_crc_avx512.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/lib/net/net_crc_avx512.c b/lib/net/net_crc_avx512.c
index d18eb96971..7cd681b1cd 100644
--- a/lib/net/net_crc_avx512.c
+++ b/lib/net/net_crc_avx512.c
@@ -180,7 +180,6 @@ crc32_eth_calc_vpclmulqdq(const uint8_t *data, uint32_t data_len, uint32_t crc,
__m512i temp, k;
__m512i qw0 = _mm512_set1_epi64(0), qw1, qw2, qw3;
__m512i fold0, fold1, fold2, fold3;
- __mmask16 mask;
uint32_t n = 0;
int reduction = 0;
@@ -260,8 +259,7 @@ crc32_eth_calc_vpclmulqdq(const uint8_t *data, uint32_t data_len, uint32_t crc,
res = _mm_xor_si128(res, d);
} else {
res = _mm_cvtsi32_si128(crc);
- mask = byte_len_to_mask_table[data_len];
- d = _mm_maskz_loadu_epi8(mask, data);
+ d = _mm_maskz_loadu_epi8(byte_len_to_mask_table[data_len], data);
res = _mm_xor_si128(res, d);
if (data_len > 3) {
--
2.48.1
More information about the dev
mailing list