Date: Sun, 3 Apr 2005 10:29:55 +0000 (UTC) From: Jeff Roberson <jeff@FreeBSD.org> To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/ufs/ffs ffs_extern.h ffs_softdep.c ffs_vfsops.c Message-ID: <200504031029.j33ATtAX021544@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200504031029.j33ATtAX021544>