CVS commit: src/sys/ufs/ffs

[Andrew Doran <ad%netbsd.org@localhost>]

Module Name:    src
Committed By:   ad
Date:           Sun Mar 29 10:29:00 UTC 2009

Modified Files:
        src/sys/ufs/ffs: ffs_vfsops.c ffs_vnops.c

Log Message:
fsync:

- atime updates were not being synced.

ffs_sync:

- In some cases the sync vnode was acting like now dead /usr/sbin/update.
  It was examining vnodes that it should have ignored.

- It would find dirty inodes and try to flush them. Often ffs_fsync()
  cheerfully ignored the flush request due to the fsync bug. Such inodes
  remained dirty and were repeatedly re-examined by the syncer until
  vnode reclaim or system shutdown.

- We were marking our place in the per-mount vnode list even though in
  most cases there was not flush to perform. While not a bug, this wasted
  CPU cycles because a TAILQ_NEXT would have sufficed.


To generate a diff of this commit:
cvs rdiff -u -r1.244 -r1.245 src/sys/ufs/ffs/ffs_vfsops.c
cvs rdiff -u -r1.111 -r1.112 src/sys/ufs/ffs/ffs_vnops.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.