Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5;
From:   Benjamin Coddington <bcodding@redhat.com>
To:     Anna Schumaker <anna@kernel.org>
Cc:     trond.myklebust@hammerspace.com, linux-nfs@vger.kernel.org
Subject: Re: [PATCH 2/2] NFS: drop unused nfs_direct_req bytes_left
Date:   Thu, 16 Nov 2023 17:03:17 -0500
Message-ID: <CCFAA747-7BC9-40AB-8E37-A18B33A01511@redhat.com>
In-Reply-To: <CAFX2Jfkc7p+22aK0KvN4yUerS1HuKDC+Njo_AJV1=5pWW0sUYQ@mail.gmail.com>
References: <21a1f2a6155398965f79ed64f0bd23bf38a50367.1700083991.git.bcodding@redhat.com>
 <952eea7e97246870f83e7a4592e9338007dfe625.1700083991.git.bcodding@redhat.com>
 <CAFX2Jfkc7p+22aK0KvN4yUerS1HuKDC+Njo_AJV1=5pWW0sUYQ@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Precedence: bulk

On 16 Nov 2023, at 16:44, Anna Schumaker wrote:

> Hi Ben,
>
> On Wed, Nov 15, 2023 at 4:34 PM Benjamin Coddington <bcodding@redhat.com> wrote:
>>
>> Now that we're calculating how large a remaining IO should be based
>> on the current request's offset, we no longer need to track bytes_left on
>> each struct nfs_direct_req.  Drop the field, and clean up the direct
>> request tracepoints.
>
> I've been having problems with xfstests generic/465 on all NFS
> versions after applying this patch. Looking at wireshark, the client
> appears to be resending the same reads over and over again. Have you
> seen anything like this in your testing?

I have generic/465 failing before and after these two patches on pNFS SCSI..
but at least it completes.  If I run it without pNFS I can see the same
thing.. it just sends the same reads over and over.  I'll figure out why.

Ben