From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.droids-corp.org (zoll.droids-corp.org [94.23.50.67]) by dpdk.org (Postfix) with ESMTP id E66022142 for ; Wed, 5 Oct 2016 16:18:28 +0200 (CEST) Received: from lfbn-1-5996-232.w90-110.abo.wanadoo.fr ([90.110.195.232] helo=[192.168.1.13]) by mail.droids-corp.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.84_2) (envelope-from ) id 1brn4j-0001fq-TE; Wed, 05 Oct 2016 16:21:39 +0200 To: "De Lara Guarch, Pablo" , "dev@dpdk.org" References: <1473407734-11253-1-git-send-email-olivier.matz@6wind.com> <1473407734-11253-5-git-send-email-olivier.matz@6wind.com> From: Olivier Matz Message-ID: Date: Wed, 5 Oct 2016 16:18:22 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.2.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v2 4/8] app/testpmd: add option to enable lro X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 05 Oct 2016 14:18:29 -0000 Hi Pablo, On 10/05/2016 08:26 AM, De Lara Guarch, Pablo wrote: > Hi Olivier, > >> -----Original Message----- >> From: Olivier Matz [mailto:olivier.matz@6wind.com] >> Sent: Friday, September 09, 2016 12:56 AM >> To: dev@dpdk.org; De Lara Guarch, Pablo >> Subject: [PATCH v2 4/8] app/testpmd: add option to enable lro >> >> Introduce a new argument '--enable-lro' to ask testpmd to enable the LRO >> feature on enabled ports, like it's done for '--enable-rx-cksum' for >> instance. >> >> Signed-off-by: Olivier Matz >> --- >> app/test-pmd/parameters.c | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/app/test-pmd/parameters.c b/app/test-pmd/parameters.c >> index 8792c2c..612ad37 100644 >> --- a/app/test-pmd/parameters.c >> +++ b/app/test-pmd/parameters.c >> @@ -150,6 +150,7 @@ usage(char* progname) >> "If the drop-queue doesn't exist, the packet is dropped. " >> "By default drop-queue=127.\n"); >> printf(" --crc-strip: enable CRC stripping by hardware.\n"); >> + printf(" --enable-lro: enable large receive offload.\n"); >> printf(" --enable-rx-cksum: enable rx hardware checksum >> offload.\n"); >> printf(" --disable-hw-vlan: disable hardware vlan.\n"); >> printf(" --disable-hw-vlan-filter: disable hardware vlan filter.\n"); >> @@ -525,6 +526,7 @@ launch_args_parse(int argc, char** argv) >> { "pkt-filter-size", 1, 0, 0 }, >> { "pkt-filter-drop-queue", 1, 0, 0 }, >> { "crc-strip", 0, 0, 0 }, >> + { "enable-lro", 0, 0, 0 }, >> { "enable-rx-cksum", 0, 0, 0 }, >> { "enable-scatter", 0, 0, 0 }, >> { "disable-hw-vlan", 0, 0, 0 }, >> @@ -765,6 +767,8 @@ launch_args_parse(int argc, char** argv) >> } >> if (!strcmp(lgopts[opt_idx].name, "crc-strip")) >> rx_mode.hw_strip_crc = 1; >> + if (!strcmp(lgopts[opt_idx].name, "enable-lro")) >> + rx_mode.enable_lro = 1; >> if (!strcmp(lgopts[opt_idx].name, "enable-scatter")) >> rx_mode.enable_scatter = 1; >> if (!strcmp(lgopts[opt_idx].name, "enable-rx-cksum")) >> -- >> 2.8.1 > > Could you add this new parameter in the testpmd documentation? Yes, I'll do it. Thanks, Olivier