diff options
| author | Miao Xie | 2014-11-25 09:39:28 +0100 |
|---|---|---|
| committer | Miao Xie | 2014-12-03 03:18:47 +0100 |
| commit | 4245215d6a8dba1a51c50533b6667919687c0b89 (patch) | |
| tree | d3d988c61ff1dcf8fe6886e3c2ba3783814d614e /fs/btrfs/ctree.h | |
| parent | Btrfs, replace: write raid56 parity into the replace target device (diff) | |
| download | kernel-qcow2-linux-4245215d6a8dba1a51c50533b6667919687c0b89.tar.gz kernel-qcow2-linux-4245215d6a8dba1a51c50533b6667919687c0b89.tar.xz kernel-qcow2-linux-4245215d6a8dba1a51c50533b6667919687c0b89.zip | |
Btrfs, raid56: fix use-after-free problem in the final device replace procedure on raid56
The commit c404e0dc (Btrfs: fix use-after-free in the finishing
procedure of the device replace) fixed a use-after-free problem
which happened when removing the source device at the end of device
replace, but at that time, btrfs didn't support device replace
on raid56, so we didn't fix the problem on the raid56 profile.
Currently, we implemented device replace for raid56, so we need
kick that problem out before we enable that function for raid56.
The fix method is very simple, we just increase the bio per-cpu
counter before we submit a raid56 io, and decrease the counter
when the raid56 io ends.
Signed-off-by: Miao Xie <miaox@cn.fujitsu.com>
Diffstat (limited to 'fs/btrfs/ctree.h')
| -rw-r--r-- | fs/btrfs/ctree.h | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h index fe69edda11fb..470e3177a7e8 100644 --- a/fs/btrfs/ctree.h +++ b/fs/btrfs/ctree.h @@ -4097,7 +4097,12 @@ int btrfs_scrub_progress(struct btrfs_root *root, u64 devid, /* dev-replace.c */ void btrfs_bio_counter_inc_blocked(struct btrfs_fs_info *fs_info); void btrfs_bio_counter_inc_noblocked(struct btrfs_fs_info *fs_info); -void btrfs_bio_counter_dec(struct btrfs_fs_info *fs_info); +void btrfs_bio_counter_sub(struct btrfs_fs_info *fs_info, s64 amount); + +static inline void btrfs_bio_counter_dec(struct btrfs_fs_info *fs_info) +{ + btrfs_bio_counter_sub(fs_info, 1); +} /* reada.c */ struct reada_control { |