kernel: mfs -- Fix unmount panic for MFS filesystems.
authorVenkatesh Srinivas <me@endeavour.zapto.org>
Thu, 16 Feb 2012 15:53:22 +0000 (07:53 -0800)
committerVenkatesh Srinivas <me@endeavour.zapto.org>
Thu, 16 Feb 2012 16:17:31 +0000 (08:17 -0800)
mfs_start() is active for the life of an MFS mount, handling copyin/out
requests. It is directed to exit when an MFS filesystem is unmounted.
The VFS accounting code and VFS MPLOCK code assumed that the mountpoint
structure was stable on return of mfs_start, which was not the case.

To work around that, don't call acinit if the vfs_start vfsop returns
EMOUNTEXIT, a kernel-specific pseudo-errno. Also allow marking vfs_start
MPSAFE and mark mfs vfs_start MPSAFE, to avoid accessing a per-mount
token field. A future commit will rework MFS's locking so that critical
sections are not used to protect the MFS bioq.

This is a hack for MFS; future work should either rework the filesystem
or replace it outright with tmpfs.

Closes: Bug 2276

sys/kern/vfs_vfsops.c
sys/sys/errno.h
sys/sys/mount.h
sys/vfs/mfs/mfs_vfsops.c

index c21f333..f5b0937 100644 (file)
@@ -110,13 +110,15 @@ vfs_start(struct mount *mp, int flags)
        VFS_MPLOCK_DECLARE;
        int error;
 
-       VFS_MPLOCK1(mp);
+       VFS_MPLOCK_FLAG(mp, MNTK_ST_MPSAFE);
        error = (mp->mnt_op->vfs_start)(mp, flags);
        if (error == 0)
                /* do not call vfs_acinit on mount updates */
                if ((mp->mnt_flag & MNT_UPDATE) == 0)
                        error = (mp->mnt_op->vfs_acinit)(mp);
        VFS_MPUNLOCK(mp);
+       if (error == EMOUNTEXIT)
+               error = 0;
        return (error);
 }
 
index 15ea636..b6e7b70 100644 (file)
@@ -195,6 +195,7 @@ static __inline int *__error(void)
 #define        ERESTART        (-1)            /* restart syscall */
 #define        EJUSTRETURN     (-2)            /* don't modify regs, just return */
 #define        ENOIOCTL        (-3)            /* ioctl not handled by this layer */
+#define        EMOUNTEXIT      (-4)            /* mfs: mountpoint released via vfs_start */
 #endif
 
 #endif
index f4bd4c1..30c0760 100644 (file)
@@ -343,9 +343,11 @@ struct mount {
 #define MNTK_NOSTKMNT  0x10000000      /* no stacked mount point allowed */
 #define MNTK_NOMSYNC   0x20000000      /* used by tmpfs */
 #define MNTK_THR_SYNC  0x40000000      /* fs sync thread requested */
+#define MNTK_ST_MPSAFE 0x80000000      /* (mfs) vfs_start is MPSAFE */
 
 #define MNTK_ALL_MPSAFE        (MNTK_MPSAFE | MNTK_RD_MPSAFE | MNTK_WR_MPSAFE | \
-                        MNTK_GA_MPSAFE | MNTK_IN_MPSAFE | MNTK_SG_MPSAFE)
+                        MNTK_GA_MPSAFE | MNTK_IN_MPSAFE | MNTK_SG_MPSAFE | \
+                        MNTK_ST_MPSAFE)
 
 /*
  * mountlist_*() defines
index 4c95554..dbd36b7 100644 (file)
@@ -396,6 +396,12 @@ mfs_mount(struct mount *mp, char *path, caddr_t data, struct ucred *cred)
         */
        VFS_STATFS(mp, &mp->mnt_stat, cred);
 
+       /*
+        * Mark VFS_START MPSAFE; this is to avoid accessing
+        * per-mount token after VFS_START exits
+        */
+       mp->mnt_kern_flag |= MNTK_ST_MPSAFE;
+
        goto success;
 
 error_2:       /* error with devvp held*/
@@ -491,7 +497,7 @@ mfs_start(struct mount *mp, int flags)
                 mfsp->mfs_dev = NULL;
         }
        kfree(mfsp, M_MFSNODE);
-       return (0);
+       return (EMOUNTEXIT);
 }
 
 /*