[dpdk-dev] [PATCH 2/5] examples/kni: add optional parameter to enable LRO

Patil, Harish Harish.Patil at cavium.com
Tue Dec 5 20:43:35 CET 2017



-----Original Message-----
From: Shahaf Shuler <shahafs at mellanox.com>
Date: Monday, December 4, 2017 at 10:05 PM
To: Ferruh Yigit <ferruh.yigit at intel.com>, "Mody, Rasesh"
<Rasesh.Mody at cavium.com>, "dev at dpdk.org" <dev at dpdk.org>
Cc: Harish Patil <Harish.Patil at cavium.com>, Dept-Eng DPDK Dev
<Dept-EngDPDKDev at cavium.com>
Subject: RE: [dpdk-dev] [PATCH 2/5] examples/kni: add optional parameter
to enable LRO

>Tuesday, December 5, 2017 1:25 AM, Ferruh Yigit:
>> On 11/24/2017 12:35 PM, Rasesh Mody wrote:
>> > From: Harish Patil <harish.patil at cavium.com>
>> >
>> > Add an optional cmdline parameter to enable LRO. This is useful to
>> > test LRO feature by being able to run linux utils like iperf over KNI
>> > interface which generates consistent packet aggregations.
>> >
>> > Signed-off-by: Harish Patil <harish.patil at cavium.com>
>> 
>> Acked-by: Ferruh Yigit <ferruh.yigit at intel.com>
>> 
>> I think this patch has no dependency for rest of the patchset and can be
>> separately applied.
>
>Snipped from the patch [1]
>
>Please use the new ethdev offloads API. there is already a series to
>convert the examples including kni[2], I think it is better to work on
>top of it. 
>
>[1]
>@@ -611,6 +618,10 @@ struct kni_interface_stats {
> 	/* Initialise device and RX/TX queues */
> 	RTE_LOG(INFO, APP, "Initialising port %u ...\n", (unsigned)port);
> 	fflush(stdout);
>+
>+	if (enable_lro)
>+		port_conf.rxmode.enable_lro = 1;
>+
>
>[2] http://dpdk.org/dev/patchwork/patch/31558/
>
Okay, I will take a look at it.

Thanks,
Harish


>



More information about the dev mailing list