Re: PR/41977 CVS commit: src/sys/ufs/ffs

[Nicolas Joly <njoly%pasteur.fr@localhost>]

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

From: Nicolas Joly <njoly%pasteur.fr@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: PR/41977 CVS commit: src/sys/ufs/ffs
Date: Wed, 27 Apr 2011 18:20:20 +0200

 On Wed, Apr 27, 2011 at 07:25:07AM +0000, Juergen Hannken-Illjes wrote:
 > The following reply was made to PR kern/41977; it has been noted by GNATS.
 > 
 > From: "Juergen Hannken-Illjes" <hannken%netbsd.org@localhost>
 > To: gnats-bugs%gnats.NetBSD.org@localhost
 > Cc: 
 > Subject: PR/41977 CVS commit: src/sys/ufs/ffs
 > Date: Wed, 27 Apr 2011 07:24:53 +0000
 > 
 >  Module Name:        src
 >  Committed By:       hannken
 >  Date:               Wed Apr 27 07:24:53 UTC 2011
 >  
 >  Modified Files:
 >      src/sys/ufs/ffs: ffs_extern.h ffs_vfsops.c ffs_vnops.c
 >  
 >  Log Message:
 >  Cleanup ffs fsync and make devices on wapbl enabled file systems work here:
 >  
 >  - Replace the ugly sync loop in ffs_full_fsync() and ffs_vfs_fsync() with
 >    vflushbuf().  This loop is a relic of softdeps and not needed anymore.
 >  
 >  - Add ffs_spec_fsync() for device nodes on ffs file systems that calls
 >    spec_fsync() like all other file systems do and then updates the ctime.
 >  
 >  Discussed on tech-kern.
 >  
 >  Should fix PRs:
 [...]
 >  PR #41977 kernel diagnostic assertion "rw_lock_held(&wl->wl_rwlock)" failed
 
 I just checked again, and the problem is gone.
 
 Many thanks.
 
 -- 
 Nicolas Joly
 
 Projects and Developments in Bioinformatics
 Institut Pasteur, Paris.