Date: Fri, 10 Mar 2006 09:17:52 +0100 From: Pawel Jakub Dawidek <pjd@FreeBSD.org> To: Tor Egge <tegge@FreeBSD.org> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern vfs_mount.c vfs_subr.c vfs_vnops.c src/sys/sys mount.h vnode.h src/sys/ufs/ffs ffs_extern.h ffs_softdep.c ffs_vfsops.c src/sys/ufs/ufs ufs_inode.c ufs_quota.c ufsmount.h Message-ID: <20060310081752.GC12357@garage.freebsd.pl> In-Reply-To: <200603082343.k28Nhd7T053744@repoman.freebsd.org> References: <200603082343.k28Nhd7T053744@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] On Wed, Mar 08, 2006 at 11:43:39PM +0000, Tor Egge wrote: +> tegge 2006-03-08 23:43:39 UTC +> +> FreeBSD src repository +> +> Modified files: +> sys/kern vfs_mount.c vfs_vnops.c vfs_subr.c +> sys/sys vnode.h mount.h +> sys/ufs/ffs ffs_extern.h ffs_softdep.c ffs_vfsops.c +> sys/ufs/ufs ufs_inode.c ufs_quota.c ufsmount.h +> Log: +> Use vn_start_secondary_write() and vn_finished_secondary_write() as a +> replacement for vn_write_suspend_wait() to better account for secondary write +> processing. +> +> Close race where secondary writes could be started after ffs_sync() returned +> but before the file system was marked as suspended. +> +> Detect if secondary writes or softdep processing occurred during vnode sync +> loop in ffs_sync() and retry the loop if needed. When exactly new 'suspended' argument for ffs_sbupdate() should be set to 1? When I call ffs_sbupdate() after calling vfs_write_suspend(), should I set it to 1? -- Pawel Jakub Dawidek http://www.wheel.pl pjd@FreeBSD.org http://www.FreeBSD.org FreeBSD committer Am I Evil? Yes, I Am! [-- Attachment #2 --] -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.2 (FreeBSD) iD8DBQFEETYwForvXbEpPzQRAqutAJ96Vnizw4HDwgEP2+aV7hbNimhL1wCgnI2y Tt1oCAyBjR1I7x3AzL3cdfg= =En6A -----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060310081752.GC12357>