Commit 545b9fd3 authored by Nick Piggin's avatar Nick Piggin Committed by Al Viro

fs: remove incorrect I_NEW warnings

Some filesystems can call in to sync an inode that is still in the
I_NEW state (eg. ext family, when mounted with -osync). This is OK
because the filesystem has sole access to the new inode, so it can
modify i_state without races (because no other thread should be
modifying it, by definition of I_NEW). Ie. a false positive, so
remove the warnings.

The races are described here 7ef0d737,
which is also where the warnings were introduced.
Reported-by: default avatarStephen Hemminger <shemminger@vyatta.com>
Signed-off-by: default avatarNick Piggin <npiggin@suse.de>
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 8688b863
...@@ -289,7 +289,6 @@ __sync_single_inode(struct inode *inode, struct writeback_control *wbc) ...@@ -289,7 +289,6 @@ __sync_single_inode(struct inode *inode, struct writeback_control *wbc)
int ret; int ret;
BUG_ON(inode->i_state & I_SYNC); BUG_ON(inode->i_state & I_SYNC);
WARN_ON(inode->i_state & I_NEW);
/* Set I_SYNC, reset I_DIRTY */ /* Set I_SYNC, reset I_DIRTY */
dirty = inode->i_state & I_DIRTY; dirty = inode->i_state & I_DIRTY;
...@@ -314,7 +313,6 @@ __sync_single_inode(struct inode *inode, struct writeback_control *wbc) ...@@ -314,7 +313,6 @@ __sync_single_inode(struct inode *inode, struct writeback_control *wbc)
} }
spin_lock(&inode_lock); spin_lock(&inode_lock);
WARN_ON(inode->i_state & I_NEW);
inode->i_state &= ~I_SYNC; inode->i_state &= ~I_SYNC;
if (!(inode->i_state & I_FREEING)) { if (!(inode->i_state & I_FREEING)) {
if (!(inode->i_state & I_DIRTY) && if (!(inode->i_state & I_DIRTY) &&
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment