sys/vfs/msdosfs: Cleanups
authorTomohiro Kusumi <kusumi.tomohiro@gmail.com>
Sun, 1 Sep 2019 16:51:01 +0000 (01:51 +0900)
committerTomohiro Kusumi <kusumi.tomohiro@gmail.com>
Sun, 1 Sep 2019 17:38:27 +0000 (02:38 +0900)
sys/vfs/msdosfs/bootsect.h
sys/vfs/msdosfs/bpb.h
sys/vfs/msdosfs/msdosfs_denode.c
sys/vfs/msdosfs/msdosfs_fat.c
sys/vfs/msdosfs/msdosfs_lookup.c

index 91ba9cd..1292386 100644 (file)
@@ -76,7 +76,7 @@ struct bootsector710 {
 #define        BOOTSIG3        0
 };
 
-#ifdef atari
+#ifdef atari
 /*
  * The boot sector on a gemdos fs is a little bit different from the msdos fs
  * format. Currently there is no need to declare a separate structure, the
index 5cfd630..413fe7b 100644 (file)
@@ -84,7 +84,7 @@ struct bpb710 {
        u_int8_t        bpbReserved[12]; /* reserved for future expansion */
 };
 
-#ifdef atari
+#ifdef atari
 /*
  * BPB for gemdos filesystems. Atari leaves the obsolete stuff undefined.
  * Currently there is no need for a separate BPB structure.
index 9c4dcda..bba927c 100644 (file)
@@ -86,9 +86,10 @@ static MALLOC_DEFINE(M_MSDOSFSNODE, "MSDOSFS node", "MSDOSFS vnode private part"
  */
 static struct denode **dehashtbl;
 static u_long dehash;                  /* size of hash table - 1 */
+static struct lwkt_token dehash_token;
+
 #define        DEHASH(dev, dcl, doff)  (dehashtbl[(minor(dev) + (dcl) + (doff) / \
                                sizeof(struct direntry)) & dehash])
-static struct lwkt_token dehash_token;
 
 union _qcvt {
        quad_t qcvt;
@@ -123,7 +124,6 @@ msdosfs_init(struct vfsconf *vfsp)
 int
 msdosfs_uninit(struct vfsconf *vfsp)
 {
-
        if (dehashtbl)
                kfree(dehashtbl, M_MSDOSFSMNT);
        return (0);
@@ -152,7 +152,8 @@ loop:
                 * We must check to see if the inode has been ripped
                 * out from under us after blocking.
                 */
-               for (dep = DEHASH(dev, dirclust, diroff); dep; dep = dep->de_next) {
+               for (dep = DEHASH(dev, dirclust, diroff); dep;
+                    dep = dep->de_next) {
                        if (dirclust == dep->de_dirclust
                            && diroff == dep->de_diroffset
                            && dev == dep->de_dev
@@ -506,12 +507,12 @@ detrunc(struct denode *dep, u_long length, int flags)
         * directory's life.
         */
        if ((DETOV(dep)->v_flag & VROOT) && !FAT32(pmp)) {
-               kprintf("detrunc(): can't truncate root directory, clust %ld, offset %ld\n",
-                   dep->de_dirclust, dep->de_diroffset);
+               kprintf("detrunc(): can't truncate root directory, clust %ld, "
+                       "offset %ld\n",
+                       dep->de_dirclust, dep->de_diroffset);
                return (EINVAL);
        }
 
-
        if (dep->de_FileSize < length) {
                vnode_pager_setsize(DETOV(dep), length);
                return deextend(dep, length);
index 119b51f..fbe87b4 100644 (file)
@@ -91,8 +91,7 @@ fatblock(struct msdosfsmount *pmp, u_long ofs, u_long *bnp, u_long *sizep,
        u_long bn, size;
 
        bn = ofs / pmp->pm_fatblocksize * pmp->pm_fatblocksec;
-       size = min(pmp->pm_fatblocksec, pmp->pm_FATsecs - bn)
-           * DEV_BSIZE;
+       size = min(pmp->pm_fatblocksec, pmp->pm_FATsecs - bn) * DEV_BSIZE;
        bn += pmp->pm_fatblk + pmp->pm_curfat * pmp->pm_FATsecs;
 
        if (bnp)
@@ -250,7 +249,7 @@ pcbmap(struct denode *dep,
                return (0);
        }
 
-hiteof:;
+hiteof:
        if (cnp)
                *cnp = i;
        if (bp)
index 62ff5a4..95c1855 100644 (file)
@@ -601,7 +601,8 @@ createde(struct denode *dep, struct denode *ddep, struct denode **depp,
        diroffset = ddep->de_fndoffset;
        if (dirclust != MSDOSFSROOT)
                diroffset &= pmp->pm_crbomask;
-       if ((error = bread(pmp->pm_devvp, de_bntodoff(pmp, bn), blsize, &bp)) != 0) {
+       error = bread(pmp->pm_devvp, de_bntodoff(pmp, bn), blsize, &bp);
+       if (error != 0) {
                brelse(bp);
                return error;
        }
@@ -775,7 +776,7 @@ doscheckpath(struct denode *source, struct denode *target)
        if (dep->de_StartCluster == MSDOSFSROOT)
                goto out;
        pmp = dep->de_pmp;
-#ifdef DIAGNOSTIC
+#ifdef DIAGNOSTIC
        if (pmp != source->de_pmp)
                panic("doscheckpath: source and target on different filesystems");
 #endif
@@ -824,7 +825,7 @@ doscheckpath(struct denode *source, struct denode *target)
                if ((error = deget(pmp, scn, 0, &dep)) != 0)
                        break;
        }
-out:;
+out:
        if (bp)
                brelse(bp);
        if (error == ENOTDIR)
@@ -852,7 +853,8 @@ readep(struct msdosfsmount *pmp, u_long dirclust, u_long diroffset,
            && de_blk(pmp, diroffset + blsize) > pmp->pm_rootdirsize)
                blsize = de_bn2off(pmp, pmp->pm_rootdirsize) & pmp->pm_crbomask;
        bn = detobn(pmp, dirclust, diroffset);
-       if ((error = bread(pmp->pm_devvp, de_bntodoff(pmp, bn), blsize, bpp)) != 0) {
+       error = bread(pmp->pm_devvp, de_bntodoff(pmp, bn), blsize, bpp);
+       if (error != 0) {
                brelse(*bpp);
                *bpp = NULL;
                return (error);