On Tue, Jan 20, 2026 at 06:41:56AM +0000, Venkatesh Vemula wrote:
> fixed comma operator misuse warnings with clang compiler
> for intel PMDs when -Wcomma enabled.
>
> Bugzilla ID: 1809
> Fixes: b3fa893d84d6 ("i40e: fix using memory after free")
> Fixes: aed68d5b0e81 ("net/i40e: add AVX2 Tx function")
> Fixes: e6a6a138919f ("net/i40e: add AVX512 vector path")
> Fixes: af0c246a3800 ("net/iavf: enable AVX2 for iavf")
> Fixes: 9ab9514c150e ("net/iavf: enable AVX512 for Tx")
> Fixes: f9cf4f864150 ("net/ice: support device initialization")
> Fixes: 2d5f6953d56d ("net/ice: support vector AVX2 in Tx")
> Fixes: 7f85d5ebcfe1 ("net/ice: add AVX512 vector path")
> Fixes: afac6261461f ("common/idpf: enable AVX2 for single queue Tx")
> Fixes: 5bf87b45b2c8 ("net/idpf: add AVX512 data path for single queue model")
> Fixes: e528d7c74819 ("common/idpf: add AVX512 data path for split queue
> model")
> Cc: [email protected]
>
> Signed-off-by: Venkatesh Vemula <[email protected]>
> ---
> drivers/net/intel/i40e/i40e_ethdev.c | 10 ++++------
> drivers/net/intel/i40e/i40e_rxtx_vec_avx2.c | 4 ++--
> drivers/net/intel/i40e/i40e_rxtx_vec_avx512.c | 2 +-
> drivers/net/intel/iavf/iavf_rxtx_vec_avx2.c | 4 ++--
> drivers/net/intel/iavf/iavf_rxtx_vec_avx512.c | 6 +++---
> drivers/net/intel/ice/ice_ethdev.c | 10 ++++------
> drivers/net/intel/ice/ice_rxtx_vec_avx2.c | 4 ++--
> drivers/net/intel/ice/ice_rxtx_vec_avx512.c | 2 +-
> drivers/net/intel/idpf/idpf_common_rxtx_avx2.c | 4 ++--
> drivers/net/intel/idpf/idpf_common_rxtx_avx512.c | 8 ++++----
> 10 files changed, 25 insertions(+), 29 deletions(-)
>
For changes, so far as they go:
Acked-by: Bruce Richardson <[email protected]>
However, there is little point in this fix right now because the warnings
are disabled at a global driver level. I think if we do this, we need to
re-enable the warnings for these drivers to prevent new issues from
creeping in.
