sys/vfs/msdosfs: Add mprintf()
authorTomohiro Kusumi <kusumi.tomohiro@gmail.com>
Sun, 1 Sep 2019 19:44:01 +0000 (04:44 +0900)
committerTomohiro Kusumi <kusumi.tomohiro@gmail.com>
Sun, 1 Sep 2019 20:11:34 +0000 (05:11 +0900)
sys/vfs/msdosfs/msdosfs_conv.c
sys/vfs/msdosfs/msdosfs_denode.c
sys/vfs/msdosfs/msdosfs_fat.c
sys/vfs/msdosfs/msdosfs_lookup.c
sys/vfs/msdosfs/msdosfs_vfsops.c
sys/vfs/msdosfs/msdosfs_vnops.c
sys/vfs/msdosfs/msdosfsmount.h

index 9dc08d1..bd31c9c 100644 (file)
@@ -1005,11 +1005,8 @@ winChkName(struct mbnambuf *nbp, const u_char *un, size_t unlen, int chksum,
         */
        if (!mbnambuf_flush(nbp, d_name, &d_namlen) || d_namlen == 0)
                return -1;
-#ifdef MSDOSFS_DEBUG
-       kprintf("winChkName(): un=%s:%zd,d_name=%s:%d\n", un, unlen,
-                                                       d_name,
-                                                       d_namlen);
-#endif
+       mprintf("winChkName(): un=%s:%zd,d_name=%s:%d\n",
+               un, unlen, d_name, d_namlen);
        /*
         * Compare the name parts
         */
index bba927c..537c910 100644 (file)
@@ -260,10 +260,8 @@ deget(struct msdosfsmount *pmp,    /* so we know the maj/min number */
        struct buf *bp;
        struct timeval tv;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("deget(pmp %p, dirclust %lu, diroffset %lx, depp %p)\n",
+       mprintf("deget(pmp %p, dirclust %lu, diroffset %lx, depp %p)\n",
            pmp, dirclust, diroffset, depp);
-#endif
 
        /*
         * On FAT32 filesystems, root is a (more or less) normal
@@ -493,10 +491,8 @@ detrunc(struct denode *dep, u_long length, int flags)
        struct buf *bp;
        struct msdosfsmount *pmp = dep->de_pmp;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("detrunc(): file %s, length %lu, flags %x\n", dep->de_Name,
+       mprintf("detrunc(): file %s, length %lu, flags %x\n", dep->de_Name,
                length, flags);
-#endif
 
        /*
         * Disallow attempts to truncate the root directory since it is of
@@ -535,9 +531,7 @@ detrunc(struct denode *dep, u_long length, int flags)
                error = pcbmap(dep, de_clcount(pmp, length) - 1,
                               NULL, &eofentry, NULL);
                if (error) {
-#ifdef MSDOSFS_DEBUG
-                       kprintf("detrunc(): pcbmap fails %d\n", error);
-#endif
+                       mprintf("detrunc(): pcbmap fails %d\n", error);
                        return (error);
                }
        }
@@ -561,9 +555,7 @@ detrunc(struct denode *dep, u_long length, int flags)
                }
                if (error) {
                        brelse(bp);
-#ifdef MSDOSFS_DEBUG
-                       kprintf("detrunc(): bread fails %d\n", error);
-#endif
+                       mprintf("detrunc(): bread fails %d\n", error);
                        return (error);
                }
                /*
@@ -591,10 +583,8 @@ detrunc(struct denode *dep, u_long length, int flags)
        error = deupdat(dep, 1);
        if (error && (allerror == 0))
                allerror = error;
-#ifdef MSDOSFS_DEBUG
-       kprintf("detrunc(): allerror %d, eofentry %lu\n",
+       mprintf("detrunc(): allerror %d, eofentry %lu\n",
                allerror, eofentry);
-#endif
 
        /*
         * If we need to break the cluster chain for the file then do it
@@ -604,9 +594,7 @@ detrunc(struct denode *dep, u_long length, int flags)
                error = fatentry(FAT_GET_AND_SET, pmp, eofentry,
                                 &chaintofree, CLUST_EOFE);
                if (error) {
-#ifdef MSDOSFS_DEBUG
-                       kprintf("detrunc(): fatentry errors %d\n", error);
-#endif
+                       mprintf("detrunc(): fatentry errors %d\n", error);
                        return (error);
                }
                fc_setcache(dep, FC_LASTFC, de_cluster(pmp, length - 1),
@@ -676,10 +664,8 @@ msdosfs_reclaim(struct vop_reclaim_args *ap)
        struct vnode *vp = ap->a_vp;
        struct denode *dep = VTODE(vp);
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_reclaim(): dep %p, file %s, refcnt %ld\n",
+       mprintf("msdosfs_reclaim(): dep %p, file %s, refcnt %ld\n",
            dep, dep ? (char *)dep->de_Name : "?", dep ? dep->de_refcnt : -1);
-#endif
 
        if (prtactive && VREFCNT(vp) > 1)
                vprint("msdosfs_reclaim(): pushing active", vp);
@@ -708,10 +694,8 @@ msdosfs_inactive(struct vop_inactive_args *ap)
        struct denode *dep = VTODE(vp);
        int error = 0;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_inactive(): dep %p, de_Name[0] %x\n",
+       mprintf("msdosfs_inactive(): dep %p, de_Name[0] %x\n",
                dep, (dep ? dep->de_Name[0] : 0));
-#endif
 
        if (prtactive && VREFCNT(vp) > 1)
                vprint("msdosfs_inactive(): pushing active", vp);
@@ -727,10 +711,8 @@ msdosfs_inactive(struct vop_inactive_args *ap)
         * filesystem, then truncate the file, and mark the directory slot
         * as empty.  (This may not be necessary for the dos filesystem.)
         */
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_inactive(): dep %p, refcnt %ld, mntflag %x, MNT_RDONLY %x\n",
+       mprintf("msdosfs_inactive(): dep %p, refcnt %ld, mntflag %x, MNT_RDONLY %x\n",
                dep, dep->de_refcnt, vp->v_mount->mnt_flag, MNT_RDONLY);
-#endif
        if (dep->de_refcnt <= 0 && (vp->v_mount->mnt_flag & MNT_RDONLY) == 0) {
                error = detrunc(dep, (u_long) 0, 0);
                dep->de_flag |= DE_UPDATE;
@@ -743,10 +725,8 @@ out:
         * If we are done with the denode, reclaim it
         * so that it can be reused immediately.
         */
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_inactive(): v_refcnt 0x%08x, de_Name[0] %x\n",
+       mprintf("msdosfs_inactive(): v_refcnt 0x%08x, de_Name[0] %x\n",
                vp->v_refcnt, (dep ? dep->de_Name[0] : 0));
-#endif
        if (dep == NULL || dep->de_Name[0] == SLOT_DELETED)
                vrecycle(vp);
        return (error);
index fbe87b4..9de3f51 100644 (file)
@@ -315,9 +315,7 @@ updatefats(struct msdosfsmount *pmp, struct buf *bp, u_long fatbn)
        int i;
        struct buf *bpn;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("updatefats(pmp %p, bp %p, fatbn %lu)\n", pmp, bp, fatbn);
-#endif
+       mprintf("updatefats(pmp %p, bp %p, fatbn %lu)\n", pmp, bp, fatbn);
 
        /*
         * If we have an FSInfo block, update it.
@@ -477,10 +475,8 @@ fatentry(int function, struct msdosfsmount *pmp, u_long cn, u_long *oldcontents,
        u_long bn, bo, bsize, byteoffset;
        struct buf *bp;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("fatentry(func %d, pmp %p, clust %lu, oldcon %p, newcon %lx)\n",
+       mprintf("fatentry(func %d, pmp %p, clust %lu, oldcon %p, newcon %lx)\n",
                function, pmp, cn, oldcontents, newcontents);
-#endif
 
 #ifdef DIAGNOSTIC
        /*
@@ -579,10 +575,8 @@ fatchain(struct msdosfsmount *pmp, u_long start, u_long count, u_long fillwith)
        u_long bn, bo, bsize, byteoffset, readcn, newc;
        struct buf *bp;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("fatchain(pmp %p, start %lu, count %lu, fillwith %lx)\n",
+       mprintf("fatchain(pmp %p, start %lu, count %lu, fillwith %lx)\n",
            pmp, start, count, fillwith);
-#endif
        /*
         * Be sure the clusters are in the filesystem.
         */
@@ -699,10 +693,8 @@ chainalloc(struct msdosfsmount *pmp, u_long start, u_long count,
        error = fatchain(pmp, start, count, fillwith);
        if (error != 0)
                return (error);
-#ifdef MSDOSFS_DEBUG
-       kprintf("clusteralloc(): allocated cluster chain at %lu (%lu clusters)\n",
+       mprintf("clusteralloc(): allocated cluster chain at %lu (%lu clusters)\n",
            start, count);
-#endif
        if (retcluster)
                *retcluster = start;
        if (got)
@@ -730,9 +722,7 @@ clusteralloc(struct msdosfsmount *pmp, u_long start, u_long count,
        u_long foundcn = 0; /* XXX: foundcn could be used unititialized */
        u_int map;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("clusteralloc(): find %lu clusters\n",count);
-#endif
+       mprintf("clusteralloc(): find %lu clusters\n",count);
        if (start) {
                if ((len = chainlength(pmp, start, count)) >= count)
                        return (chainalloc(pmp, start, count, fillwith,
index 95c1855..d2c5ff2 100644 (file)
@@ -118,18 +118,14 @@ msdosfs_lookup(struct vop_old_lookup_args *ap)
        int olddos = 1;
        cnp->cn_flags &= ~CNP_PDIRUNLOCK;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_lookup(): looking for %s\n", cnp->cn_nameptr);
-#endif
+       mprintf("msdosfs_lookup(): looking for %s\n", cnp->cn_nameptr);
        dp = VTODE(vdp);
        pmp = dp->de_pmp;
        *vpp = NULL;
        lockparent = flags & CNP_LOCKPARENT;
        wantparent = flags & (CNP_LOCKPARENT | CNP_WANTPARENT);
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_lookup(): vdp %p, dp %p, Attr %02x\n",
+       mprintf("msdosfs_lookup(): vdp %p, dp %p, Attr %02x\n",
            vdp, dp, dp->de_Attributes);
-#endif
 
        /*
         * If they are going after the . or .. entry in the root directory,
@@ -141,9 +137,7 @@ msdosfs_lookup(struct vop_old_lookup_args *ap)
                (cnp->cn_namelen == 2 && cnp->cn_nameptr[1] == '.'))) {
                isadir = ATTR_DIRECTORY;
                scn = MSDOSFSROOT;
-#ifdef MSDOSFS_DEBUG
-               kprintf("msdosfs_lookup(): looking for . or .. in root directory\n");
-#endif
+               mprintf("msdosfs_lookup(): looking for . or .. in root directory\n");
                cluster = MSDOSFSROOT;
                blkoff = MSDOSFSROOT_OFS;
                goto foundroot;
@@ -180,10 +174,8 @@ msdosfs_lookup(struct vop_old_lookup_args *ap)
        if (nameiop == NAMEI_CREATE || nameiop == NAMEI_RENAME)
                slotcount = 0;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_lookup(): dos version of filename %s, length %ld\n",
+       mprintf("msdosfs_lookup(): dos version of filename %s, length %ld\n",
            dosfilename, cnp->cn_namelen);
-#endif
        /*
         * Search the directory pointed at by vdp for the name pointed at
         * by cnp->cn_nameptr.
@@ -285,10 +277,8 @@ msdosfs_lookup(struct vop_old_lookup_args *ap)
                                        chksum = -1;
                                        continue;
                                }
-#ifdef MSDOSFS_DEBUG
-                               kprintf("msdosfs_lookup(): match blkoff %d, diroff %d\n",
+                               mprintf("msdosfs_lookup(): match blkoff %d, diroff %d\n",
                                    blkoff, diroff);
-#endif
                                /*
                                 * Remember where this directory
                                 * entry came from for whoever did
@@ -329,12 +319,10 @@ notfound:
         * that's ok if we are creating or renaming and are at the end of
         * the pathname and the directory hasn't been removed.
         */
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_lookup(): op %d, refcnt %ld\n",
+       mprintf("msdosfs_lookup(): op %d, refcnt %ld\n",
            nameiop, dp->de_refcnt);
-       kprintf("               slotcount %d, slotoffset %d\n",
+       mprintf("               slotcount %d, slotoffset %d\n",
               slotcount, slotoffset);
-#endif
        if ((nameiop == NAMEI_CREATE || nameiop == NAMEI_RENAME) &&
            dp->de_refcnt > 0) {
                /*
@@ -560,10 +548,8 @@ createde(struct denode *dep, struct denode *ddep, struct denode **depp,
        daddr_t bn;
        int blsize;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("createde(dep %p, ddep %p, depp %p, cnp %p)\n",
+       mprintf("createde(dep %p, ddep %p, depp %p, cnp %p)\n",
            dep, ddep, depp, cnp);
-#endif
 
        /*
         * If no space left in the directory then allocate another cluster
@@ -727,10 +713,8 @@ dosdirempty(struct denode *dep)
                                if (bcmp(dentp->deName, ".          ", 11) &&
                                    bcmp(dentp->deName, "..         ", 11)) {
                                        brelse(bp);
-#ifdef MSDOSFS_DEBUG
-                                       kprintf("dosdirempty(): entry found %02x, %02x\n",
+                                       mprintf("dosdirempty(): entry found %02x, %02x\n",
                                            dentp->deName[0], dentp->deName[1]);
-#endif
                                        return (0);     /* not empty */
                                }
                        }
@@ -896,10 +880,8 @@ removede(struct denode *pdep,      /* directory where the entry is removed */
        struct msdosfsmount *pmp = pdep->de_pmp;
        u_long offset = pdep->de_fndoffset;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("removede(): filename %s, dep %p, offset %08lx\n",
+       mprintf("removede(): filename %s, dep %p, offset %08lx\n",
            dep->de_Name, dep, offset);
-#endif
 
        KKASSERT(dep->de_refcnt > 0);
        dep->de_refcnt--;
index 76a037f..c70b258 100644 (file)
@@ -696,9 +696,7 @@ msdosfs_root(struct mount *mp, struct vnode **vpp)
        struct denode *ndep;
        int error;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_root(); mp %p, pmp %p\n", mp, pmp);
-#endif
+       mprintf("msdosfs_root(); mp %p, pmp %p\n", mp, pmp);
        error = deget(pmp, MSDOSFSROOT, MSDOSFSROOT_OFS, &ndep);
        if (error)
                return (error);
index 96ad3b8..c45e6d6 100644 (file)
@@ -116,9 +116,7 @@ msdosfs_create(struct vop_old_create_args *ap)
        struct timespec ts;
        int error;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_create(cnp %p, vap %p\n", cnp, ap->a_vap);
-#endif
+       mprintf("msdosfs_create(cnp %p, vap %p\n", cnp, ap->a_vap);
 
        /*
         * If this is the root directory and there is no space left we
@@ -307,10 +305,8 @@ msdosfs_setattr(struct vop_setattr_args *ap)
        struct ucred *cred = ap->a_cred;
        int error = 0;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_setattr(): vp %p, vap %p, cred %p\n",
+       mprintf("msdosfs_setattr(): vp %p, vap %p, cred %p\n",
            ap->a_vp, vap, cred);
-#endif
 
        /*
         * Check for unsettable attributes.
@@ -319,16 +315,14 @@ msdosfs_setattr(struct vop_setattr_args *ap)
            (vap->va_fsid != VNOVAL) || (vap->va_fileid != VNOVAL) ||
            (vap->va_blocksize != VNOVAL) || (vap->va_rmajor != VNOVAL) ||
            (vap->va_bytes != VNOVAL) || (vap->va_gen != VNOVAL)) {
-#ifdef MSDOSFS_DEBUG
-               kprintf("msdosfs_setattr(): returning EINVAL\n");
-               kprintf("    va_type %u, va_nlink %"PRIx64", va_fsid %x, va_fileid %"PRIx64"\n",
+               mprintf("msdosfs_setattr(): returning EINVAL\n");
+               mprintf("    va_type %u, va_nlink %"PRIx64", va_fsid %x, va_fileid %"PRIx64"\n",
                    vap->va_type, vap->va_nlink, vap->va_fsid, vap->va_fileid);
-               kprintf("    va_blocksize %lx, va_rmajor %x, va_bytes %"PRIx64", va_gen %"PRIx64"\n",
+               mprintf("    va_blocksize %lx, va_rmajor %x, va_bytes %"PRIx64", va_gen %"PRIx64"\n",
                    vap->va_blocksize, vap->va_rmajor, vap->va_bytes,
                    vap->va_gen);
-               kprintf("    va_uid %x, va_gid %x\n",
+               mprintf("    va_uid %x, va_gid %x\n",
                    vap->va_uid, vap->va_gid);
-#endif
                return (EINVAL);
        }
        if (vap->va_flags != VNOVAL) {
@@ -584,12 +578,10 @@ msdosfs_write(struct vop_write_args *ap)
        struct proc *p = (td ? td->td_proc : NULL);
        struct lwp *lp = (td ? td->td_lwp : NULL);
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_write(vp %p, uio %p, ioflag %x, cred %p\n",
+       mprintf("msdosfs_write(vp %p, uio %p, ioflag %x, cred %p\n",
            vp, uio, ioflag, ap->a_cred);
-       kprintf("msdosfs_write(): diroff %lu, dirclust %lu, startcluster %lu\n",
+       mprintf("msdosfs_write(): diroff %lu, dirclust %lu, startcluster %lu\n",
            dep->de_diroffset, dep->de_dirclust, dep->de_StartCluster);
-#endif
 
        switch (vp->v_type) {
        case VREG:
@@ -818,10 +810,8 @@ msdosfs_remove(struct vop_old_remove_args *ap)
                error = EPERM;
        else
                error = removede(ddep, dep);
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_remove(), dep %p, v_refcnt 0x%08x\n",
+       mprintf("msdosfs_remove(), dep %p, v_refcnt 0x%08x\n",
                dep, ap->a_vp->v_refcnt);
-#endif
        return (error);
 }
 
@@ -1501,10 +1491,8 @@ msdosfs_readdir(struct vop_readdir_args *ap)
        dep = VTODE(ap->a_vp);
        pmp = dep->de_pmp;
 
-#ifdef MSDOSFS_DEBUG
-       kprintf("msdosfs_readdir(): vp %p, uio %p, cred %p, eofflagp %p\n",
+       mprintf("msdosfs_readdir(): vp %p, uio %p, cred %p, eofflagp %p\n",
            ap->a_vp, uio, ap->a_cred, ap->a_eofflag);
-#endif
 
        /*
         * msdosfs_readdir() won't operate properly on regular files since
index a8958d4..2c4020b 100644 (file)
@@ -268,4 +268,10 @@ struct msdosfs_args {
 
 #define MSDOSFS_ARGSMAGIC      0xe4eff300
 
+#ifdef MSDOSFS_DEBUG
+#define mprintf(fmt, ...)      kprintf(fmt, ## __VA_ARGS__)
+#else
+#define mprintf(fmt, ...)
+#endif
+
 #endif /* !_MSDOSFS_MSDOSFSMOUNT_H_ */