-----Original Message----- From: Shahaf Shuler <[email protected]> Date: Monday, December 4, 2017 at 10:05 PM To: Ferruh Yigit <[email protected]>, "Mody, Rasesh" <[email protected]>, "[email protected]" <[email protected]> Cc: Harish Patil <[email protected]>, Dept-Eng DPDK Dev <[email protected]> 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 <[email protected]> >> > >> > 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 <[email protected]> >> >> Acked-by: Ferruh Yigit <[email protected]> >> >> 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 >
