diff options
| author | Abhi Das <adas@redhat.com> | 2018-01-30 18:00:09 +0100 |
|---|---|---|
| committer | Bob Peterson <rpeterso@redhat.com> | 2018-01-30 18:00:27 +0100 |
| commit | 957a7acd46e64c52d2a1d59cd7273ed49455afb6 (patch) | |
| tree | 799a8c20a32b18690203eb0f69f71a83cc1d2ff2 /fs/gfs2 | |
| parent | GFS2: Don't try to end a non-existent transaction in unlink (diff) | |
| download | linux-957a7acd46e64c52d2a1d59cd7273ed49455afb6.tar.xz linux-957a7acd46e64c52d2a1d59cd7273ed49455afb6.zip | |
gfs2: Remove inode from ordered write list in gfs2_write_inode()
The vfs clears the I_DIRTY inode flag before calling gfs2_write_inode()
having queued any data that needed to be written to disk.
This is a good time to remove such inodes from our ordered write list
so they don't hang around for long periods of time.
Signed-off-by: Abhi Das <adas@redhat.com>
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Diffstat (limited to 'fs/gfs2')
| -rw-r--r-- | fs/gfs2/super.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/fs/gfs2/super.c b/fs/gfs2/super.c index 50a297b920fc..620be0521866 100644 --- a/fs/gfs2/super.c +++ b/fs/gfs2/super.c @@ -768,6 +768,12 @@ static int gfs2_write_inode(struct inode *inode, struct writeback_control *wbc) ret = filemap_fdatawait(metamapping); if (ret) mark_inode_dirty_sync(inode); + else { + spin_lock(&inode->i_lock); + if (!(inode->i_flags & I_DIRTY)) + gfs2_ordered_del_inode(ip); + spin_unlock(&inode->i_lock); + } return ret; } |