Skip to content

btrfs: return all mirror whether need_raid_map set or not

Zhao Lei linux-btrfs 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 patches 2 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4btrfs: Disable raid56 readaheadfs/btrfs/reada.c+5 -00 replies
2/4btrfs: return all mirror whether need_raid_map set or notfs/btrfs/volumes.c+2 -30 replies
3/4btrfs: Small cleanup for get index_srcdev loopfs/btrfs/volumes.c+20 -220 replies
4/4btrfs: Use direct way to determine raid56 write/recover modefs/btrfs/volumes.c+2 -10 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/4] btrfs: return all mirror whether need_raid_map set or not

__btrfs_map_block() should return all mirror on WRITE, REQ_GET_READ_MIRRORS, and RECOVERY case, whether need_raid_map set or not.

need_raid_map only used to control is to set bbio->raid_map.

Current code works right becuase there is only one caller can trigger above bug, which is readahead, and this function happened to bypass on less mirror.

But after we fixed __btrfs_map_block(), readahead will be really works, and exit with warning at another bug.
This patchset fixed __btrfs_map_block(), and disable raid56 readahead temp temporary, (actually, it already disable by this bug), and I'll fix raid56 readahead next.

Zhao Lei (4):
btrfs: Disable raid56 readahead
btrfs: return all mirror whether need_raid_map set or not
btrfs: Small cleanup for get index_srcdev loop
btrfs: Use direct way to determine raid56 write/recover mode

 fs/btrfs/reada.c   |  5 +++++
 fs/btrfs/volumes.c | 50 ++++++++++++++++++++++++--------------------------
 2 files changed, 29 insertions(+), 26 deletions(-)
Signature · 1 line
1.8.5.1