diff options
author | Dave Hansen <haveblue@us.ibm.com> | 2007-07-16 02:41:14 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-07-16 12:05:48 -0400 |
commit | e3a68e30d28dbc6981dfc3d6ceddbfa2f885fe4e (patch) | |
tree | f15372ae04f4eaea734c1c6935ae65887acf4c3c /fs | |
parent | 6b86e854f71600c809536502a0efa9d4e384fb23 (diff) |
ext3: remove extra IS_RDONLY() check
ext3_change_inode_journal_flag() is only called from one location:
ext3_ioctl(EXT3_IOC_SETFLAGS). That ioctl case already has a IS_RDONLY()
call in it so this one is superfluous.
Signed-off-by: Dave Hansen <haveblue@us.ibm.com>
Cc: <linux-ext4@vger.kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/ext3/inode.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ext3/inode.c b/fs/ext3/inode.c index 2a85ddee4740..de4e3161e479 100644 --- a/fs/ext3/inode.c +++ b/fs/ext3/inode.c | |||
@@ -3195,7 +3195,7 @@ int ext3_change_inode_journal_flag(struct inode *inode, int val) | |||
3195 | */ | 3195 | */ |
3196 | 3196 | ||
3197 | journal = EXT3_JOURNAL(inode); | 3197 | journal = EXT3_JOURNAL(inode); |
3198 | if (is_journal_aborted(journal) || IS_RDONLY(inode)) | 3198 | if (is_journal_aborted(journal)) |
3199 | return -EROFS; | 3199 | return -EROFS; |
3200 | 3200 | ||
3201 | journal_lock_updates(journal); | 3201 | journal_lock_updates(journal); |