cvs commit: src/sys/ufs/ffs ffs_extern.h ffs_softdep.c ffs_vfsops.c

[Jeff Roberson]

jeff        2005-04-03 10:29:55 UTC

  FreeBSD src repository

  Modified files:
    sys/ufs/ffs          ffs_extern.h ffs_softdep.c ffs_vfsops.c 
  Log:
   - Move the contents of softdep_disk_prewrite into ffs_geom_strategy to fix
     two bugs.
   - ffs_disk_prewrite was pulling the vp from the buf and checking for
     COPYONWRITE, when really it wanted the vp from the bufobj that we're
     writing to, which is the devvp.  This lead to us skipping the copy on
     write to all file data, which significantly broke snapshots for the
     last few months.
   - When the SOFTUPDATES option was not included in the kernel config we
     would also skip the copy on write check, which would effectively disable
     snapshots.
   - Remove an invalid mp_fixme().
  
  Debugging tips from:    mckusick
  Reported by:            iedowse, others
  Discussed with:         phk
  
  Revision  Changes    Path
  1.69      +0 -1      src/sys/ufs/ffs/ffs_extern.h
  1.180     +0 -29     src/sys/ufs/ffs/ffs_softdep.c
  1.288     +21 -2     src/sys/ufs/ffs/ffs_vfsops.c