diff options
| author | Tahsin Erdogan <[email protected]> | 2016-12-13 00:43:20 +0000 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2016-12-13 02:55:08 +0000 |
| commit | bace9248188f64d7490ebe59fc0733db8b6f0e57 (patch) | |
| tree | 593403191c322c4528f4ec8f568a330a149d2947 /fs/fs-writeback.c | |
| parent | mm/filemap.c: add comment for confusing logic in page_cache_tree_insert() (diff) | |
| download | kernel-bace9248188f64d7490ebe59fc0733db8b6f0e57.tar.gz kernel-bace9248188f64d7490ebe59fc0733db8b6f0e57.zip | |
fs/fs-writeback.c: remove redundant if check
b_more_io non-empty check is already preceded by an opposite check.
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Tahsin Erdogan <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'fs/fs-writeback.c')
| -rw-r--r-- | fs/fs-writeback.c | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/fs/fs-writeback.c b/fs/fs-writeback.c index 05713a5da083..ef600591d96f 100644 --- a/fs/fs-writeback.c +++ b/fs/fs-writeback.c @@ -1769,15 +1769,13 @@ static long wb_writeback(struct bdi_writeback *wb, * become available for writeback. Otherwise * we'll just busyloop. */ - if (!list_empty(&wb->b_more_io)) { - trace_writeback_wait(wb, work); - inode = wb_inode(wb->b_more_io.prev); - spin_lock(&inode->i_lock); - spin_unlock(&wb->list_lock); - /* This function drops i_lock... */ - inode_sleep_on_writeback(inode); - spin_lock(&wb->list_lock); - } + trace_writeback_wait(wb, work); + inode = wb_inode(wb->b_more_io.prev); + spin_lock(&inode->i_lock); + spin_unlock(&wb->list_lock); + /* This function drops i_lock... */ + inode_sleep_on_writeback(inode); + spin_lock(&wb->list_lock); } spin_unlock(&wb->list_lock); blk_finish_plug(&plug); |