On Tue, Sep 10, 2024 at 12:45:11PM -0400, Mike Snitzer wrote: > On Fri, Sep 06, 2024 at 04:34:18PM -0400, Mike Snitzer wrote: > > On Fri, Sep 06, 2024 at 03:31:41PM -0400, Anna Schumaker wrote: > > > Hi Mike, > > > > > > I've been running tests on localio this afternoon after finishing up going through v15 of the patches (I was most of the way through when you posted v16, so I haven't updated yet!). Cthon tests passed on all NFS versions, and xfstests passed on NFS v4.x. However, I saw this crash from xfstests with NFS v3: > > > > > > [ 1502.440896] run fstests generic/633 at 2024-09-06 14:04:17 > > > [ 1502.694356] process 'vfstest' launched '/dev/fd/4/file1' with NULL argv: empty string added > > > [ 1502.699514] Oops: general protection fault, probably for non-canonical address 0x6c616e69665f6140: 0000 [#1] PREEMPT SMP NOPTI > > > [ 1502.700970] CPU: 3 UID: 0 PID: 513 Comm: nfsd Not tainted 6.11.0-rc6-g0c79a48cd64d-dirty+ #42323 70d41673e6cbf8e3437eb227e0a9c3c46ed3b289 > > > [ 1502.702506] Hardware name: QEMU Standard PC (Q35 + ICH9, 2009), BIOS unknown 2/2/2022 > > > [ 1502.703593] RIP: 0010:nfsd_cache_lookup+0x2b3/0x840 [nfsd] <snip> > > > > > > Please let me know if there are any other details you need about my setup to help debug this! > > > > Hmm, I haven't seen this issue, my runs of xfstests with LOCALIO > > enabled look solid: > > https://evilpiepirate.org/~testdashboard/ci?user=snitzer&branch=snitm-nfs-next&test=^fs.nfs.fstests.generic.633$ > > > > And I know Chuck has been testing xfstests and more with the patches > > applied but LOCALIO disabled in his kernel config. > > > > The stack seems to indicate nfsd is just handling a request (so it > > isn't using LOCALIO, at least not for this op). > > > > Probably best if you do try v16. v15 has issues v16 addressed. If > > you can reproduce with v16 please share your kernel .config and > > xfstests config. > > > > Note that I've only really tested my changes against v6.11-rc4. But I > > can rebase on v6.11-rc6 if you find v16 still fails for you. > > Hi Anna, > > Just checking back, how is LOCALIO for you at this point? Anything > you're continuing to see as an issue or need from me? In case it helps, I did just rebase LOCALIO (v16 + 1 fix) ontop of cel/nfsd-next (v6.11-rc6 based), and I've pushed the result here: https://git.kernel.org/pub/scm/linux/kernel/git/snitzer/linux.git/log/?h=nfs-localio-for-next ktest is running xfstests against it (LOCALIO enabled and in use): https://evilpiepirate.org/~testdashboard/ci?user=snitzer&branch=snitm-nfs-next And Chuck's kdevops testing should test it tomorrow morning.
