[EXTERNAL] [PATCH v2] examples/l3fwd: adjust Tx burst size based on Rx burst

[Tummala, Sivaprasad]

[AMD Official Use Only - AMD Internal Distribution Only]

Hi Pavan,

Please find my response inline.

________________________________
From: Pavan Nikhilesh Bhagavatula <pbhagavatula at marvell.com>
Sent: Tuesday, June 10, 2025 7:59 PM
To: Tummala, Sivaprasad <Sivaprasad.Tummala at amd.com>; david.hunt at intel.com <david.hunt at intel.com>; anatoly.burakov at intel.com <anatoly.burakov at intel.com>; Jerin Jacob <jerinj at marvell.com>; radu.nicolau at intel.com <radu.nicolau at intel.com>; Akhil Goyal <gakhil at marvell.com>; cristian.dumitrescu at intel.com <cristian.dumitrescu at intel.com>; Yigit, Ferruh <Ferruh.Yigit at amd.com>; konstantin.ananyev at huawei.com <konstantin.ananyev at huawei.com>; mb at smartsharesystems.com <mb at smartsharesystems.com>
Cc: dev at dpdk.org <dev at dpdk.org>; haijie1 at huawei.com <haijie1 at huawei.com>; stable at dpdk.org <stable at dpdk.org>; Ande, Venkat Kumar <VenkatKumar.Ande at amd.com>; Dengdui Huang <huangdengdui at huawei.com>
Subject: Re: [EXTERNAL] [PATCH v2] examples/l3fwd: adjust Tx burst size based on Rx burst

Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.


>>Previously, the TX burst size was fixed at 256, leading to performance
>>degradation in certain scenarios.
>>
>>This patch introduces logic to set the TX burst size to match the
>>configured RX burst size (--burst option, default 32, max 512)
>>for better efficiency.
>>
>>Fixes: d5c4897ecfb2 ("examples/l3fwd: add option to set Rx burst size")
>>Cc: haijie1 at huawei.com
>>Cc: stable at dpdk.org
>>
>>Signed-off-by: Sivaprasad Tummala <sivaprasad.tummala at amd.com>
>>Tested-by: Venkat Kumar Ande <VenkatKumar.Ande at amd.com>
>>Tested-by: Dengdui Huang <huangdengdui at huawei.com>

>It would be good if the selected burst sizes of Rx and Tx are logged.
ACK

>On CN10K platform we see upto 5% improvement, and upto 30% improvement on CN9K.

>Tested-by: Pavan Nikhilesh <pbhagavatula at marvell.com>

>---
> examples/l3fwd/l3fwd.h        |  8 ++------
> examples/l3fwd/l3fwd_common.h | 11 +++++++----
> examples/l3fwd/main.c         |  2 ++
> 3 files changed, 11 insertions(+), 10 deletions(-)

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mails.dpdk.org/archives/stable/attachments/20251009/24f21b8c/attachment.htm>