Bug 1967496
Summary: | [virtio-fs] nfs/xfstest generic/089 generic/478 failed | ||
---|---|---|---|
Product: | Red Hat Enterprise Linux 8 | Reporter: | xiagao |
Component: | qemu-kvm | Assignee: | Hanna Czenczek <hreitz> |
qemu-kvm sub component: | virtio-fs | QA Contact: | xiagao |
Status: | CLOSED ERRATA | Docs Contact: | |
Severity: | low | ||
Priority: | low | CC: | kkiwi, qzhang, virt-maint, xinma, yidliu, yihyu, yimsong, zhenyzha |
Version: | 8.5 | Keywords: | Triaged |
Target Milestone: | beta | ||
Target Release: | --- | ||
Hardware: | x86_64 | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | qemu-kvm-4.2.0-58.module+el8.5.0+12272+74ace547 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2021-11-09 18:01:39 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
xiagao
2021-06-03 08:59:23 UTC
Hi xiagao a) I can't reproduce generic/089 generic/478 error (I tested on aarch64). ``` [root@localhost xfstests]# cat local.config.nfs export TEST_DEV=nfsmyfs0 export TEST_DIR=/mnt/test export SCRATCH_DEV=nfsmyfs1 export SCRATCH_MNT=/mnt/scratch export FSX_AVOID="-E" [root@localhost xfstests]# ./check -virtiofs generic/089 generic/478 generic/632 FSTYP -- virtiofs PLATFORM -- Linux/aarch64 localhost 4.18.0-310.el8.aarch64 #1 SMP Thu May 27 14:52:00 EDT 2021 MKFS_OPTIONS -- nfsmyfs1 MOUNT_OPTIONS -- -o context=system_u:object_r:root_t:s0 nfsmyfs1 /mnt/scratch generic/089 96s generic/478 4s generic/632 [failed, exit status 1]- output mismatch (see /root/xfstests/results//generic/632.out.bad) --- tests/generic/632.out 2021-06-04 09:50:17.829040740 +0800 +++ /root/xfstests/results//generic/632.out.bad 2021-06-04 09:56:10.199040740 +0800 @@ -1,2 +1,2 @@ QA output created by 632 -silence is golden +No space left on device - Buggy mount countingsilence is golden ... (Run 'diff -u /root/xfstests/tests/generic/632.out /root/xfstests/results//generic/632.out.bad' to see the entire diff) Ran: generic/089 generic/478 generic/632 Failures: generic/632 Failed 1 of 3 tests ``` My virtiofsd info ``` [root@fujitsu-fx700-01-n01 ~]# ps aux | grep virtiofsd root 70386 0.0 0.0 87808 4736 ? Sl 21:41 0:00 /usr/libexec/virtiofsd --fd=37 -o source=/mnt/share0,no_flock,no_posix_lock root 70390 0.0 0.0 87808 4736 ? Sl 21:41 0:00 /usr/libexec/virtiofsd --fd=37 -o source=/mnt/share1,no_flock,no_posix_lock root 70394 0.0 0.0 87808 4736 ? Sl 21:41 0:00 /usr/libexec/virtiofsd --fd=37 -o source=/mnt/nfs/share0,no_flock,no_posix_lock root 70398 0.0 0.0 87808 4736 ? Sl 21:41 0:00 /usr/libexec/virtiofsd --fd=37 -o source=/mnt/nfs/share1,no_flock,no_posix_lock root 70458 0.0 0.0 29464448 2880 ? Sl 21:41 0:00 /usr/libexec/virtiofsd --fd=37 -o source=/mnt/share0,no_flock,no_posix_lock root 70459 0.0 0.0 29464448 2880 ? Sl 21:41 0:00 /usr/libexec/virtiofsd --fd=37 -o source=/mnt/share1,no_flock,no_posix_lock root 70460 0.0 0.0 29529984 5184 ? Sl 21:41 0:00 /usr/libexec/virtiofsd --fd=37 -o source=/mnt/nfs/share1,no_flock,no_posix_lock root 70463 3.5 0.0 29595520 19840 ? Sl 21:41 0:46 /usr/libexec/virtiofsd --fd=37 -o source=/mnt/nfs/share0,no_flock,no_posix_lock ``` Env: Host kernel: 4.18.0-310.el8.aarch64 libvirt: libvirt-7.4.0-1.module+el8.5.0+11218+83343022.src.rpm qemu-kvm: qemu-kvm-6.0.0-18.module+el8.5.0+11243+5269aaa1.src.rpm Guest kernel: 4.18.0-310.el8.aarch64 b) generic/632 only passed on the kernel that include below commit ee2e3f50629f ("mount: fix mounting of detached mounts onto targets that reside on shared mounts" Max, can you take this one? Thanks. My findings so far: 089 and 478 fail because of -o posix_lock. This is the default in RHEL 8, but has been changed upstream with qemu commit 88fc107956a5812649e5918e0c092d3f78bb28ad (“virtiofsd: Disable remote posix locks by default”). This is why in comment 1, both passed, because -o no_posix_lock was passed. I can’t think of a good reason not to backport this commit. 632 fails only with the 4.18.0-310.el8 in the guest, not with a current upstream kernel. The qemu/virtiofsd version doesn’t matter. I’ll have to investigate/bisect this one further. So turns out 632 is fixed by the kernel commit ee2e3f50629f17b0752b55b2566c15ce8dafb557, which in hindsight I guess is kind of obvious, because according to the commit that added 632 to the xfstests, it was added as a regression test for said commit ee2e3f50629f17b0752b55b2566c15ce8dafb557. I don’t think its failure has anything to do with virtio-fs. (In reply to Max Reitz from comment #4) > So turns out 632 is fixed by the kernel commit > ee2e3f50629f17b0752b55b2566c15ce8dafb557, which in hindsight I guess is kind > of obvious, because according to the commit that added 632 to the xfstests, > it was added as a regression test for said commit > ee2e3f50629f17b0752b55b2566c15ce8dafb557. > > I don’t think its failure has anything to do with virtio-fs. Maybe I should be a bit more verbose: ee2e3f50629f17b0752b55b2566c15ce8dafb557 is “mount: fix mounting of detached mounts onto targets that reside on shared mounts”, and it’s a general vfs commit that has nothing to do with virtio-fs in particular. It’s about mounts in general. So if we want 632 fixed, we would need a separate BZ, and that wouldn’t have anything to do with virtio-fs. If we don’t particularly care, then 632 just shouldn’t be run with a RHEL guest kernel. Can we get a qa_ack+ please? I also set ITM=26 since that's +2 on DTM and the last ITM before exceptions required. Series has 1 downstream ack and we'll push to get the last 2, so bz should be on_qa soon. thanks! Hit the same issue on rhel850 when run virtiofs test loop, case: virtio_fs_share_data.run_stress.with_xfstest.with_local_source.with_cache.auto.default.default virtio_fs_share_data.run_stress.with_xfstest.with_nfs_source.with_cache.auto.default.default pkg: qemu-kvm-4.2.0-56.module+el8.5.0+12039+0434c559.x86_64 kernel-4.18.0-325.el8.x86_64 seabios-1.13.0-2.module+el8.3.0+7353+9de0a3cc.x86_64 QE bot(pre verify): Set 'Verified:Tested,SanityOnly' as gating/tier1 test pass. Changing the title to reflect that the 632 failure has nothing to do with virtiofs. (In reply to Hanna Reitz from comment #14) > Changing the title to reflect that the 632 failure has nothing to do with > virtiofs. Yes, I only can reproduce generic/089 generic/478 failed on qemu-kvm-4.2.0-57.module+el8.5.0+12118+4998563d.x86_64. Thanks. generic/089 generic/478 passed on qemu-kvm-4.2.0-58.module+el8.5.0+12272+74ace547.x86_64. based on comment 15, verify this bug. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory (Moderate: virt:rhel and virt-devel:rhel security, bug fix, and enhancement update), and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2021:4191 |