<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div>
<div style="font-family: Calibri; text-align: left; color: rgb(0, 0, 255); margin-left: 5pt; font-size: 10pt;">
[AMD Official Use Only - AMD Internal Distribution Only]</div>
<br>
</div>
<div style="font-family: Verdana, Geneva, sans-serif; font-size: 9pt; color: rgb(0, 0, 0);" class="elementToProof">
Hi Pavan, </div>
<div style="font-family: Verdana, Geneva, sans-serif; font-size: 9pt; color: rgb(0, 0, 0);" class="elementToProof">
<br>
</div>
<div style="font-family: Verdana, Geneva, sans-serif; font-size: 9pt; color: rgb(0, 0, 0);" class="elementToProof">
Please find my response inline. </div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
<br>
</div>
<hr style="display: inline-block; width: 98%;">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<b>From:</b> Pavan Nikhilesh Bhagavatula <pbhagavatula@marvell.com><br>
<b>Sent:</b> Tuesday, June 10, 2025 7:59 PM<br>
<b>To:</b> Tummala, Sivaprasad <Sivaprasad.Tummala@amd.com>; david.hunt@intel.com <david.hunt@intel.com>; anatoly.burakov@intel.com <anatoly.burakov@intel.com>; Jerin Jacob <jerinj@marvell.com>; radu.nicolau@intel.com <radu.nicolau@intel.com>; Akhil Goyal <gakhil@marvell.com>;
cristian.dumitrescu@intel.com <cristian.dumitrescu@intel.com>; Yigit, Ferruh <Ferruh.Yigit@amd.com>; konstantin.ananyev@huawei.com <konstantin.ananyev@huawei.com>; mb@smartsharesystems.com <mb@smartsharesystems.com><br>
<b>Cc:</b> dev@dpdk.org <dev@dpdk.org>; haijie1@huawei.com <haijie1@huawei.com>; stable@dpdk.org <stable@dpdk.org>; Ande, Venkat Kumar <VenkatKumar.Ande@amd.com>; Dengdui Huang <huangdengdui@huawei.com><br>
<b>Subject:</b> Re: [EXTERNAL] [PATCH v2] examples/l3fwd: adjust Tx burst size based on Rx burst
</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-size: 11pt;" class="elementToProof">Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.<br>
<br>
<br>
>>Previously, the TX burst size was fixed at 256, leading to performance<br>
>>degradation in certain scenarios.<br>
>><br>
>>This patch introduces logic to set the TX burst size to match the<br>
>>configured RX burst size (--burst option, default 32, max 512)<br>
>>for better efficiency.<br>
>><br>
>>Fixes: d5c4897ecfb2 ("examples/l3fwd: add option to set Rx burst size")<br>
>>Cc: haijie1@huawei.com<br>
>>Cc: stable@dpdk.org<br>
>><br>
>>Signed-off-by: Sivaprasad Tummala <sivaprasad.tummala@amd.com><br>
>>Tested-by: Venkat Kumar Ande <VenkatKumar.Ande@amd.com><br>
>>Tested-by: Dengdui Huang <huangdengdui@huawei.com><br>
<br>
>It would be good if the selected burst sizes of Rx and Tx are logged.<br>
<span style="color: rgb(0, 0, 0);">ACK</span></div>
<div style="font-size: 11pt;" class="elementToProof"><span style="color: rgb(0, 0, 0);"></span><br>
</div>
<div style="font-size: 11pt;" class="elementToProof">>On CN10K platform we see upto 5% improvement, and upto 30% improvement on CN9K.<br>
<br>
>Tested-by: Pavan Nikhilesh <pbhagavatula@marvell.com><br>
<br>
>---<br>
> examples/l3fwd/l3fwd.h | 8 ++------<br>
> examples/l3fwd/l3fwd_common.h | 11 +++++++----<br>
> examples/l3fwd/main.c | 2 ++<br>
> 3 files changed, 11 insertions(+), 10 deletions(-)<br>
<br>
</div>
</body>
</html>