Re: svn commit: r246293 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2013-02-03 Thread Andriy Gapon
on 03/02/2013 21:23 Andriy Gapon said the following: > on 03/02/2013 20:42 Andriy Gapon said the following: >> -vm_page_busy(pp); >> -vm_page_undirty(pp); > > ZFS lookup_page previously had two interesting bugs that almost canceled each > other. > > The fi

Re: svn commit: r246293 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2013-02-03 Thread Andriy Gapon
on 03/02/2013 20:42 Andriy Gapon said the following: > - vm_page_busy(pp); > - vm_page_undirty(pp); ZFS lookup_page previously had two interesting bugs that almost canceled each other. The first bug was that vm_page_undirty was called even if we read from

svn commit: r246293 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2013-02-03 Thread Andriy Gapon
Author: avg Date: Sun Feb 3 18:42:20 2013 New Revision: 246293 URL: http://svnweb.freebsd.org/changeset/base/246293 Log: zfs: fix, improve and re-organize page_lookup and page_unlock Now they are split into two pairs: page_hold/page_unhold for mappedread and page_busy/page_unbusy for upd