Received: by 10.213.65.68 with SMTP id h4csp607280imn; Sat, 17 Mar 2018 17:07:02 -0700 (PDT) X-Google-Smtp-Source: AG47ELsmUIcdApzILkogLC2sYP0g1VHA9JJ62+Wg2PJ7kFWtcYpGu3LXCCE566J8433MGuas0sO3 X-Received: by 2002:a17:902:7082:: with SMTP id z2-v6mr7096221plk.130.1521331622576; Sat, 17 Mar 2018 17:07:02 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1521331622; cv=none; d=google.com; s=arc-20160816; b=Wzwjr+CWuJB2H+D25rFzKLTRJ6eNnGz2au3W48wvYnSdUCMqDWu6Vg47degVNGLQBe nJsNRezRSs+XhdNTpfLXCZ7Q8FUC3gJwM5L6zR10Gh+p5ef20XTxWGOjkaCvN0xcagcH wRhVZqi7fxU1fV/yeWKaq9PUO0pv4AXszuum2tnu/EnjQhUwCngms8ZBl77Qbms06oj/ LF9abkHGCeGXCf2pRiO3UK1aCin19jq8eyzj2YvgM64HPuywy4eVX3E3YiMUPdHnusMS 8jSceZ2OYpvBEQHbBaHxX2nufZil8dfEJo98m+UfUdVGS3q2O09pOVR3e+S45KQ2hA4P /Jcw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:from:subject:cc:to:message-id:date :arc-authentication-results; bh=jgPiLi+ogWgDsMG6nrj1cEH6cGBZxEXLq7q0SFwSwhU=; b=OIBrs7Sqmkd21tmkD8kzRqDXoW+5EKwHwI5iKIPNzHwea2qPRaNyHH7Rbfbea0FGXv 8k2/yBqkAu44wcJaJctlIGv7jabfMpD3XNoRJPDnqEWabyPQw89wXXRnAskVmcYOjApp CIxngOD3GPWG9KwY/YDl2/sunyblFue04eTmeYfB+pj0Fkxmh+geddZUe1TI7pXTqD1G UqcNQf7lriPuuCFk85m6/We2c3PhXLBjJgGM8dI7eJxsKqY8IwPEUxgYscu/Qg/RuZ8G BacEf6lJXG0wgFAxArvX78/0GhFdpgDb1nlJ7+MMWhDrJEVfNUm/lMzRlcsLik99hkoC PlXg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id j66si7994028pfj.181.2018.03.17.17.06.47; Sat, 17 Mar 2018 17:07:02 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753429AbeCRAFT (ORCPT + 99 others); Sat, 17 Mar 2018 20:05:19 -0400 Received: from shards.monkeyblade.net ([184.105.139.130]:50088 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753301AbeCRAFO (ORCPT ); Sat, 17 Mar 2018 20:05:14 -0400 Received: from localhost (pool-173-77-163-229.nycmny.fios.verizon.net [173.77.163.229]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) (Authenticated sender: davem-davemloft) by shards.monkeyblade.net (Postfix) with ESMTPSA id 1232714214D79; Sat, 17 Mar 2018 17:05:14 -0700 (PDT) Date: Sat, 17 Mar 2018 20:05:13 -0400 (EDT) Message-Id: <20180317.200513.1916546646223366804.davem@davemloft.net> To: doshir@vmware.com Cc: netdev@vger.kernel.org, rachel_lunnon@stormagic.com, skhare@vmware.com, pv-drivers@vmware.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 net 2/2] vmxnet3: use correct flag to indicate LRO feature From: David Miller In-Reply-To: <20180316214919.20716-1-doshir@vmware.com> References: <20180316214919.20716-1-doshir@vmware.com> X-Mailer: Mew version 6.7 on Emacs 25.3 / Mule 6.0 (HANACHIRUSATO) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.12 (shards.monkeyblade.net [149.20.54.216]); Sat, 17 Mar 2018 17:05:14 -0700 (PDT) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Ronak Doshi Date: Fri, 16 Mar 2018 14:49:19 -0700 > 'Commit 45dac1d6ea04 ("vmxnet3: Changes for vmxnet3 adapter version 2 > (fwd)")' introduced a flag "lro" in structure vmxnet3_adapter which is > used to indicate whether LRO is enabled or not. However, the patch > did not set the flag and hence it was never exercised. > > So, when LRO is enabled, it resulted in poor TCP performance due to > delayed acks. This issue is seen with packets which are larger than > the mss getting a delayed ack rather than an immediate ack, thus > resulting in high latency. > > This patch removes the lro flag and directly uses device features > against NETIF_F_LRO to check if lro is enabled. > > Fixes: 45dac1d6ea04 ("vmxnet3: Changes for vmxnet3 adapter version 2 (fwd)") > Reported-by: Rachel Lunnon > Signed-off-by: Ronak Doshi > Acked-by: Shrikrishna Khare Applied.