diff options
author | mpi <mpi@openbsd.org> | 2018-05-07 14:43:01 +0000 |
---|---|---|
committer | mpi <mpi@openbsd.org> | 2018-05-07 14:43:01 +0000 |
commit | 90aa86e4e33ee7a01b3cb9193956fe8779092447 (patch) | |
tree | 4c579f2fbc67fe5aed85249463de20e06d8426f2 /sys/msdosfs/msdosfs_vnops.c | |
parent | Use speed from device tree for serial console on armv7 too. (diff) | |
download | wireguard-openbsd-90aa86e4e33ee7a01b3cb9193956fe8779092447.tar.xz wireguard-openbsd-90aa86e4e33ee7a01b3cb9193956fe8779092447.zip |
Implement VFS read clustering for MSDOSFS, take 3.
With sf@, inputs from krw@, tested by many, ok visa@
Diffstat (limited to 'sys/msdosfs/msdosfs_vnops.c')
-rw-r--r-- | sys/msdosfs/msdosfs_vnops.c | 124 |
1 files changed, 74 insertions, 50 deletions
diff --git a/sys/msdosfs/msdosfs_vnops.c b/sys/msdosfs/msdosfs_vnops.c index a592cd75010..a1e9e50365e 100644 --- a/sys/msdosfs/msdosfs_vnops.c +++ b/sys/msdosfs/msdosfs_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: msdosfs_vnops.c,v 1.119 2018/05/02 02:24:56 visa Exp $ */ +/* $OpenBSD: msdosfs_vnops.c,v 1.120 2018/05/07 14:43:01 mpi Exp $ */ /* $NetBSD: msdosfs_vnops.c,v 1.63 1997/10/17 11:24:19 ws Exp $ */ /*- @@ -78,13 +78,13 @@ static uint32_t fileidhash(uint64_t); +int msdosfs_bmaparray(struct vnode *, uint32_t, daddr_t *, int *); int msdosfs_kqfilter(void *); int filt_msdosfsread(struct knote *, long); int filt_msdosfswrite(struct knote *, long); int filt_msdosfsvnode(struct knote *, long); void filt_msdosfsdetach(struct knote *); - /* * Some general notes: * @@ -510,18 +510,15 @@ int msdosfs_read(void *v) { struct vop_read_args *ap = v; - int error = 0; - uint32_t diff; - int blsize; - int isadir; - uint32_t n; - long on; - daddr_t lbn, rablock, rablkno; - struct buf *bp; struct vnode *vp = ap->a_vp; struct denode *dep = VTODE(vp); struct msdosfsmount *pmp = dep->de_pmp; struct uio *uio = ap->a_uio; + int isadir, error = 0; + uint32_t n, diff, size, on; + struct buf *bp; + uint32_t cn; + daddr_t bn; /* * If they didn't ask for any data, then we are done. @@ -536,7 +533,8 @@ msdosfs_read(void *v) if (uio->uio_offset >= dep->de_FileSize) return (0); - lbn = de_cluster(pmp, uio->uio_offset); + cn = de_cluster(pmp, uio->uio_offset); + size = pmp->pm_bpcluster; on = uio->uio_offset & pmp->pm_crbomask; n = ulmin(pmp->pm_bpcluster - on, uio->uio_resid); @@ -549,31 +547,22 @@ msdosfs_read(void *v) if (diff < n) n = diff; - /* convert cluster # to block # if a directory */ - if (isadir) { - error = pcbmap(dep, lbn, &lbn, 0, &blsize); - if (error) - return (error); - } /* * If we are operating on a directory file then be sure to * do i/o with the vnode for the filesystem instead of the * vnode for the directory. */ if (isadir) { - error = bread(pmp->pm_devvp, lbn, blsize, &bp); + /* convert cluster # to block # */ + error = pcbmap(dep, cn, &bn, 0, &size); + if (error) + return (error); + error = bread(pmp->pm_devvp, bn, size, &bp); } else { - rablock = lbn + 1; - rablkno = de_cn2bn(pmp, rablock); - if (dep->de_lastr + 1 == lbn && - de_cn2off(pmp, rablock) < dep->de_FileSize) - error = breadn(vp, de_cn2bn(pmp, lbn), - pmp->pm_bpcluster, &rablkno, - &pmp->pm_bpcluster, 1, &bp); + if (de_cn2off(pmp, cn + 1) >= dep->de_FileSize) + error = bread(vp, cn, size, &bp); else - error = bread(vp, de_cn2bn(pmp, lbn), - pmp->pm_bpcluster, &bp); - dep->de_lastr = lbn; + error = bread_cluster(vp, cn, size, &bp); } n = min(n, pmp->pm_bpcluster - bp->b_resid); if (error) { @@ -601,8 +590,7 @@ msdosfs_write(void *v) int extended = 0; uint32_t osize; int error = 0; - uint32_t count, lastcn; - daddr_t bn; + uint32_t count, lastcn, cn; struct buf *bp; int ioflag = ap->a_ioflag; struct uio *uio = ap->a_uio; @@ -679,30 +667,30 @@ msdosfs_write(void *v) lastcn = de_clcount(pmp, osize) - 1; do { - if (de_cluster(pmp, uio->uio_offset) > lastcn) { + croffset = uio->uio_offset & pmp->pm_crbomask; + cn = de_cluster(pmp, uio->uio_offset); + + if (cn > lastcn) { error = ENOSPC; break; } - bn = de_blk(pmp, uio->uio_offset); - if ((uio->uio_offset & pmp->pm_crbomask) == 0 - && (de_blk(pmp, uio->uio_offset + uio->uio_resid) > de_blk(pmp, uio->uio_offset) - || uio->uio_offset + uio->uio_resid >= dep->de_FileSize)) { + if (croffset == 0 && + (de_cluster(pmp, uio->uio_offset + uio->uio_resid) > cn || + (uio->uio_offset + uio->uio_resid) >= dep->de_FileSize)) { /* * If either the whole cluster gets written, * or we write the cluster from its start beyond EOF, * then no need to read data from disk. */ - bp = getblk(thisvp, bn, pmp->pm_bpcluster, 0, 0); + bp = getblk(thisvp, cn, pmp->pm_bpcluster, 0, 0); clrbuf(bp); /* * Do the bmap now, since pcbmap needs buffers * for the fat table. (see msdosfs_strategy) */ if (bp->b_blkno == bp->b_lblkno) { - error = pcbmap(dep, - de_bn2cn(pmp, bp->b_lblkno), - &bp->b_blkno, 0, 0); + error = pcbmap(dep, bp->b_lblkno, &bp->b_blkno, 0, 0); if (error) bp->b_blkno = -1; } @@ -714,16 +702,16 @@ msdosfs_write(void *v) } } else { /* - * The block we need to write into exists, so read it in. + * The block we need to write into exists, so + * read it in. */ - error = bread(thisvp, bn, pmp->pm_bpcluster, &bp); + error = bread(thisvp, cn, pmp->pm_bpcluster, &bp); if (error) { brelse(bp); break; } } - croffset = uio->uio_offset & pmp->pm_crbomask; n = ulmin(uio->uio_resid, pmp->pm_bpcluster - croffset); if (uio->uio_offset + n > dep->de_FileSize) { dep->de_FileSize = uio->uio_offset + n; @@ -1755,7 +1743,7 @@ msdosfs_islocked(void *v) /* * vp - address of vnode file the file - * bn - which cluster we are interested in mapping to a filesystem block number. + * bn - which cluster we are interested in mapping to a filesystem block number * vpp - returns the vnode for the block special file holding the filesystem * containing the file of interest * bnp - address of where to return the filesystem relative block number @@ -1765,19 +1753,56 @@ msdosfs_bmap(void *v) { struct vop_bmap_args *ap = v; struct denode *dep = VTODE(ap->a_vp); - struct msdosfsmount *pmp = dep->de_pmp; + uint32_t cn; if (ap->a_vpp != NULL) *ap->a_vpp = dep->de_devvp; if (ap->a_bnp == NULL) return (0); - if (ap->a_runp) { + + cn = ap->a_bn; + if (cn != ap->a_bn) + return (EFBIG); + + return (msdosfs_bmaparray(ap->a_vp, cn, ap->a_bnp, ap->a_runp)); +} + +int +msdosfs_bmaparray(struct vnode *vp, uint32_t cn, daddr_t *bnp, int *runp) +{ + struct denode *dep = VTODE(vp); + struct msdosfsmount *pmp = dep->de_pmp; + struct mount *mp; + int error, maxrun = 0, run; + daddr_t runbn; + + mp = vp->v_mount; + + if (runp) { /* - * Sequential clusters should be counted here. + * XXX + * If MAXBSIZE is the largest transfer the disks can handle, + * we probably want maxrun to be 1 block less so that we + * don't create a block larger than the device can handle. */ - *ap->a_runp = 0; + *runp = 0; + maxrun = min(MAXBSIZE / mp->mnt_stat.f_iosize - 1, + pmp->pm_maxcluster - cn); } - return (pcbmap(dep, de_bn2cn(pmp, ap->a_bn), ap->a_bnp, 0, 0)); + + if ((error = pcbmap(dep, cn, bnp, 0, 0)) != 0) + return (error); + + for (run = 1; run <= maxrun; run++) { + error = pcbmap(dep, cn + run, &runbn, 0, 0); + if (error != 0 || (runbn != *bnp + de_cn2bn(pmp, run))) + break; + } + + if (runp) + *runp = run - 1; + + return (0); } int @@ -1799,8 +1824,7 @@ msdosfs_strategy(void *v) * don't allow files with holes, so we shouldn't ever see this. */ if (bp->b_blkno == bp->b_lblkno) { - error = pcbmap(dep, de_bn2cn(dep->de_pmp, bp->b_lblkno), - &bp->b_blkno, 0, 0); + error = pcbmap(dep, bp->b_lblkno, &bp->b_blkno, 0, 0); if (error) bp->b_blkno = -1; if (bp->b_blkno == -1) |