diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2010-10-24 09:45:22 -0400 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2010-10-25 21:23:00 -0400 |
| commit | 8e3b9a072d071700e83e88b0bf59115c59042885 (patch) | |
| tree | 504d77766446dcb5937f1e50cb0560da42872e5a /fs/ext2 | |
| parent | 309f77ad9bea057d55b04580b5a711e9e3727e83 (diff) | |
| download | lwn-8e3b9a072d071700e83e88b0bf59115c59042885.tar.gz lwn-8e3b9a072d071700e83e88b0bf59115c59042885.zip | |
ext2_remount: don't bother with invalidate_inodes()
It's pointless - we *do* have busy inodes (root directory,
for one), so that call will fail and attempt to change
XIP flag will be ignored.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/ext2')
| -rw-r--r-- | fs/ext2/super.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/ext2/super.c b/fs/ext2/super.c index 85df87d0f7b7..0901320671da 100644 --- a/fs/ext2/super.c +++ b/fs/ext2/super.c @@ -1221,9 +1221,7 @@ static int ext2_remount (struct super_block * sb, int * flags, char * data) } es = sbi->s_es; - if (((sbi->s_mount_opt & EXT2_MOUNT_XIP) != - (old_mount_opt & EXT2_MOUNT_XIP)) && - invalidate_inodes(sb)) { + if ((sbi->s_mount_opt ^ old_mount_opt) & EXT2_MOUNT_XIP) { ext2_msg(sb, KERN_WARNING, "warning: refusing change of " "xip flag with busy inodes while remounting"); sbi->s_mount_opt &= ~EXT2_MOUNT_XIP; |