Re: [PATCH v3] examples/l3fwd: add Tx burst size configuration option

[David Marchand <david.marchand@redhat.com>]

On Thu, 6 Nov 2025 at 15:17, Sivaprasad Tummala
<sivaprasad.tummala@amd.com> wrote:
>
> Previously, the TX burst size in l3fwd was fixed at 256, which could
> lead to suboptimal performance in certain scenarios.
>
> This patch introduces separate --rx-burst and --tx-burst options to
> explicitly configure RX and TX burst sizes. By default, the TX burst
> size now matches the RX burst size for better efficiency and pipeline
> balance.
>
> Fixes: d5c4897ecfb2 ("examples/l3fwd: add option to set Rx burst size")
>
> Signed-off-by: Sivaprasad Tummala <sivaprasad.tummala@amd.com>
> Tested-by: Venkat Kumar Ande <venkatkumar.ande@amd.com>
> Tested-by: Dengdui Huang <huangdengdui@huawei.com>
> Tested-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
>
> v3:
>  - Added --tx-burst option to allow independent TX burst configuration
>  - Renamed --burst to --rx-burst for clarity
>  - Renamed variables for clarity:
>      - nb_pkt_per_burst → rx_burst_size
>      - max_tx_burst     → tx_burst_size
>  - Modified parsing packet burst logic for RX and TX
>  - Default TX burst size now equals RX burst size
>  - Added logging of selected RX and TX burst sizes
>  - Updated documentation for --rx-burst and --tx-burst options

Please don't put changelog in the commitlog itself.
Fixed when applying...

I also fixed [RT]X occurences as [RT]x.

Applied, thanks.


-- 
David Marchand