On Fri, Feb 13, 2026 at 12:37:49PM +0000, Bruce Richardson wrote:
> 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.
>
Changed my mind :-). We might as well take this patch. Even if other
warnings creep in, it still makes any subsequent work easier.
Applied to dpdk-next-net-intel.
Thanks,
/Bruce
