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

[Pawel Jakub Dawidek]

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 at FreeBSD.org                           http://www.FreeBSD.org
FreeBSD committer                         Am I Evil? Yes, I Am!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-src/attachments/20060310/887432b9/attachment.bin