[patch-2.3.99-pre7-3] correction to may_umount()

From: Tigran Aivazian (tigran@veritas.com)
Date: Wed May 03 2000 - 08:25:58 EST


Hi Alexander,

Since I assume it was you that changed may_umount() I am sending this to
you instead of Linus. The 'sb' variable in may_umount() is superfluous
(unless you plan on adding some more conditions in may_umount() that
involve 'sb' directly...)

Regards,
Tigran

--- linux/fs/super.c Wed May 3 14:17:28 2000
+++ work/super.c Wed May 3 14:20:16 2000
@@ -874,8 +874,7 @@
  */
 int may_umount(struct vfsmount *mnt)
 {
- struct super_block * sb = mnt->mnt_sb;
- struct dentry * root;
+ struct dentry * root = mnt->mnt_sb->s_root;
         int count;
 
         if (atomic_read(&mnt->mnt_count) > 2)
@@ -891,8 +890,6 @@
          * active reference from ->mnt_mountpoint (if mnt->mnt_parent == mnt).
          * Anything above that means that tree is busy.
          */
-
- root = sb->s_root;
 
         count = d_active_refs(root);
         if (mnt->mnt_parent == mnt)

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun May 07 2000 - 21:00:11 EST