Date: Fri, 20 Jul 2007 16:21:47 +0000 (UTC) From: Bruce Evans <bde@FreeBSD.org> To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/fs/msdosfs msdosfs_fat.c msdosfs_vnops.c Message-ID: <200707201621.l6KGLmHI088298@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
bde 2007-07-20 16:21:47 UTC
FreeBSD src repository
Modified files:
sys/fs/msdosfs msdosfs_fat.c msdosfs_vnops.c
Log:
Clean up before implementing vfs clustering for msdosfs:
In msdosfs_read(), mainly reorder the main loop to the same order as in
ffs_read().
In msdosfs_write() and extendfile(), use vfs_bio_clrbuf() instead of
clrbuf(). I think this just just a bogus optimization, but ffs always
does it and msdosfs already did it in one place, and it is what I've
tested.
In msdosfs_write(), merge good bits from a comment in ffs_write(), and
fix 1 style bug.
In the main comment for msdosfs_pcbmap(), improve wording and catch
up with 13 years of changes in the function. This comment belongs in
VOP_BMAP.9 but that doesn't exist.
In msdosfs_bmap(), return EFBIG if the requested cluster number is out
of bounds instead of blindly truncating it, and fix many style bugs.
Approved by: re (hrs)
Revision Changes Path
1.42 +1 -1 src/sys/fs/msdosfs/msdosfs_fat.c
1.171 +42 -36 src/sys/fs/msdosfs/msdosfs_vnops.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200707201621.l6KGLmHI088298>