On Tue, Jul 19, 2011 at 09:32:38AM -0700, Tim Chen wrote: > @@ -1193,6 +1193,7 @@ static void __exit cleanup_mtdchar(void) > { > unregister_mtd_user(&mtdchar_notifier); > mntput(mtd_inode_mnt); > + kern_unmount(mtd_inode_mnt); Surely you want to merge that mntput() in there... > +void kern_unmount(struct vfsmount *mnt) > +{ > + /* release long term mount so mount point can be released */ > + if (!IS_ERR_OR_NULL(mnt)) { > + mnt_make_shortterm(mnt); > + mntput(); > + } > +} ... and if you pass it the argument, it'll be much happier. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html