[PATCH] net/idpf: fix comma operator warning

Bruce Richardson bruce.richardson at intel.com
Thu Mar 12 17:57:46 CET 2026


On Thu, Mar 12, 2026 at 09:43:44AM -0700, Stephen Hemminger wrote:
> Fix comma operator misuse warnings with clang compiler in idpf that
> was missed in previous change. Shows up  when -Wcomma enabled.
> 
> Fixes: 57560a92167a ("net/idpf: add AVX2 Tx path for split queue config")
> 
> Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>
> ---

Acked-by: Bruce Richardson <bruce.richardson at intel.com>

Yep, this was missed because the previous commit [1] was on your tree but
not applied to main, so missing in mine. :-(

[1] https://patches.dpdk.org/project/dpdk/patch/20260219152302.2103018-1-bruce.richardson@intel.com/

>  drivers/net/intel/idpf/idpf_common_rxtx_avx2.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/intel/idpf/idpf_common_rxtx_avx2.c b/drivers/net/intel/idpf/idpf_common_rxtx_avx2.c
> index 4e1062f22d..db7728afad 100644
> --- a/drivers/net/intel/idpf/idpf_common_rxtx_avx2.c
> +++ b/drivers/net/intel/idpf/idpf_common_rxtx_avx2.c
> @@ -842,7 +842,7 @@ idpf_splitq_vtx_avx2(struct idpf_flex_tx_sched_desc *txdp,
>  	/* align if needed */
>  	if (((uintptr_t)txdp & 0x1F) != 0 && nb_pkts != 0) {
>  		idpf_splitq_vtx1_avx2(txdp, *pkt, flags);
> -		txdp++, pkt++, nb_pkts--;
> +		txdp++; pkt++; nb_pkts--;
>  	}
>  
>  	for (; nb_pkts >= IDPF_VPMD_DESCS_PER_LOOP; txdp += IDPF_VPMD_DESCS_PER_LOOP,
> -- 
> 2.51.0
> 


More information about the dev mailing list