diff options
author | David Sterba <dsterba@suse.com> | 2015-10-10 12:16:38 -0400 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2015-10-10 12:16:38 -0400 |
commit | 730d9ec36bf27e87805aee6ebf462071735e31f9 (patch) | |
tree | 8a89528930f7429db0f6e7014df910ecfad7ba87 /fs/btrfs/dev-replace.c | |
parent | 9ffecb10283508260936b96022d4ee43a7798b4c (diff) |
btrfs: remove waitqueue_active check from btrfs_rm_dev_replace_unblocked
Normally the waitqueue_active would need a barrier, but this is not
necessary here because it's not a performance sensitive context and we
can call wake_up directly.
Suggested-by: Chris Mason <clm@fb.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/dev-replace.c')
-rw-r--r-- | fs/btrfs/dev-replace.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/btrfs/dev-replace.c b/fs/btrfs/dev-replace.c index e54dd5905cee..733ff75b620e 100644 --- a/fs/btrfs/dev-replace.c +++ b/fs/btrfs/dev-replace.c | |||
@@ -454,8 +454,7 @@ static void btrfs_rm_dev_replace_blocked(struct btrfs_fs_info *fs_info) | |||
454 | static void btrfs_rm_dev_replace_unblocked(struct btrfs_fs_info *fs_info) | 454 | static void btrfs_rm_dev_replace_unblocked(struct btrfs_fs_info *fs_info) |
455 | { | 455 | { |
456 | clear_bit(BTRFS_FS_STATE_DEV_REPLACING, &fs_info->fs_state); | 456 | clear_bit(BTRFS_FS_STATE_DEV_REPLACING, &fs_info->fs_state); |
457 | if (waitqueue_active(&fs_info->replace_wait)) | 457 | wake_up(&fs_info->replace_wait); |
458 | wake_up(&fs_info->replace_wait); | ||
459 | } | 458 | } |
460 | 459 | ||
461 | static int btrfs_dev_replace_finishing(struct btrfs_fs_info *fs_info, | 460 | static int btrfs_dev_replace_finishing(struct btrfs_fs_info *fs_info, |