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 5FDDF3237 for ; Tue, 19 Jul 2016 17:51:39 +0200 (CEST) Received: from alille-653-1-293-182.w90-1.abo.wanadoo.fr ([90.1.53.182] 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 1bPXLZ-0004xm-OB; Tue, 19 Jul 2016 17:54:14 +0200 To: "Chandran, Sugesh" , "dev@dpdk.org" , "Ananyev, Konstantin" , "Zhang, Helin" References: <1467893942-4048-1-git-send-email-olivier.matz@6wind.com> <2EF2F5C0CC56984AA024D0B180335FCB13DEDCB2@IRSMSX102.ger.corp.intel.com> Cc: "Richardson, Bruce" , Maxime Leroy From: Olivier Matz Message-ID: <61364fe7-344f-5cb8-8411-87bd234428ce@6wind.com> Date: Tue, 19 Jul 2016 17:51:31 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.1.0 MIME-Version: 1.0 In-Reply-To: <2EF2F5C0CC56984AA024D0B180335FCB13DEDCB2@IRSMSX102.ger.corp.intel.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH] ixgbe: support checksum flags in sse vector Rx function 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: Tue, 19 Jul 2016 15:51:39 -0000 Hi Sugesh, On 07/14/2016 11:24 AM, Chandran, Sugesh wrote: > Hi Olivier, > Thank you for working on this. > We tried to enable checksum offload in OVS-DPDK and couldn't proceed due to the performance impact. > I assume this patch will fix that issue by enabling checksum offloading with vectorization ON at Rx side. > > Few questions, > 1) Is there any plan to extend this to other NIC drivers, other than ixgbe? What are the implications of it? On my side no plan for other drivers, but this is more a question for pmd maintainers. > 2) Is it possible to enable it on the Tx side as well? Yes, vector tx is enabled or not depending on the feature you request at init (offload, multisegments, ...). See ixgbe_set_tx_function() for details. My patch does not change this behavior. > > I haven't looked into patch very detail and very little context on it. So please forgive me if any of these queries make no sense. > > Regards > _Sugesh Regards, Olivier