PR/55702 CVS commit: [netbsd-9] src

["Martin Husemann" <martin%netbsd.org@localhost>]

The following reply was made to PR kern/55702; it has been noted by GNATS.

From: "Martin Husemann" <martin%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/55702 CVS commit: [netbsd-9] src
Date: Tue, 6 Jul 2021 04:22:35 +0000

 Module Name:	src
 Committed By:	martin
 Date:		Tue Jul  6 04:22:35 UTC 2021
 
 Modified Files:
 	src/external/cddl/osnet/dist/uts/common/fs/zfs [netbsd-9]: zfs_vnops.c
 	src/sys/rump/librump/rumpkern [netbsd-9]: vm.c
 	src/sys/rump/librump/rumpvfs [netbsd-9]: vm_vfs.c
 	src/sys/uvm [netbsd-9]: uvm_anon.c uvm_page.c uvm_pager.c
 	src/tests/rump/rumpkern [netbsd-9]: t_vm.c
 
 Log Message:
 Pull up following revision(s) - all via patch -
 (requested by riastradh in ticket #1317):
 
 	sys/uvm/uvm_page.c: revision 1.248
 	sys/uvm/uvm_anon.c: revision 1.80
 	sys/rump/librump/rumpvfs/vm_vfs.c: revision 1.40
 	sys/rump/librump/rumpvfs/vm_vfs.c: revision 1.41
 	sys/rump/librump/rumpkern/vm.c: revision 1.191
 	sys/uvm/uvm_pager.c: revision 1.130
 	external/cddl/osnet/dist/uts/common/fs/zfs/zfs_vnops.c: revision 1.71
 	tests/rump/rumpkern/t_vm.c: revision 1.5
 	tests/rump/rumpkern/t_vm.c: revision 1.6
 	sys/rump/librump/rumpvfs/vm_vfs.c: revision 1.39
 
 Move the handling of PG_PAGEOUT from uvm_aio_aiodone_pages() to
 uvm_page_unbusy() so that all callers of uvm_page_unbusy() don't need to
 handle this flag separately.  Split out the pages part of uvm_aio_aiodone()
 into uvm_aio_aiodone_pages() in rump just like in the real kernel.
 
 In ZFS functions that can fail to copy data between the ARC and VM pages,
 use uvm_aio_aiodone_pages() rather than uvm_page_unbusy() so that we can
 handle these "I/O" errors.  Fixes PR 55702.
 
 fix an incorrect assertion in the previous commit.
 
 Handle PG_PAGEOUT in uvm_anon_release() too.
 
 Commit the ZFS file that I forgot in this previous commit:
 
 Move the handling of PG_PAGEOUT from uvm_aio_aiodone_pages() to
 uvm_page_unbusy() so that all callers of uvm_page_unbusy() don't need to
 handle this flag separately.  Split out the pages part of uvm_aio_aiodone()
 into uvm_aio_aiodone_pages() in rump just like in the real kernel.
 
 In ZFS functions that can fail to copy data between the ARC and VM pages,
 use uvm_aio_aiodone_pages() rather than uvm_page_unbusy() so that we can
 handle these "I/O" errors.  Fixes PR 55702.
 update the rump copy of uvm_page_unbusy() to match the real version,
 in particular handle PG_PAGEOUT.  fixes a few atf tests.
 the busypage test is buggy, expect it to fail.
 
 make rump's uvm_aio_aiodone_pages() look more like the kernel version.
 fixes some more rumpy assertions.
 
 for the busypage test, replace atf_tc_expect_fail() with atf_tc_skip()
 because atf apparently has no way to expect a test program to crash.
 fixes PR 55945.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.50.2.9 -r1.50.2.10 \
     src/external/cddl/osnet/dist/uts/common/fs/zfs/zfs_vnops.c
 cvs rdiff -u -r1.173 -r1.173.14.1 src/sys/rump/librump/rumpkern/vm.c
 cvs rdiff -u -r1.34 -r1.34.34.1 src/sys/rump/librump/rumpvfs/vm_vfs.c
 cvs rdiff -u -r1.64 -r1.64.8.1 src/sys/uvm/uvm_anon.c
 cvs rdiff -u -r1.199 -r1.199.4.1 src/sys/uvm/uvm_page.c
 cvs rdiff -u -r1.111.8.1 -r1.111.8.2 src/sys/uvm/uvm_pager.c
 cvs rdiff -u -r1.4 -r1.4.16.1 src/tests/rump/rumpkern/t_vm.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.