[dm-devel] [PATCH next] Btrfs: fix comparison in __btrfs_map_block()
Mike Christie
mchristi at redhat.com
Sun Jul 17 20:51:03 UTC 2016
On 07/15/2016 10:03 AM, Vincent Stehlé wrote:
> Add missing comparison to op in expression, which was forgotten when doing
> the REQ_OP transition.
>
> Fixes: b3d3fa519905 ("btrfs: update __btrfs_map_block for REQ_OP transition")
> Signed-off-by: Vincent Stehlé <vincent.stehle at intel.com>
> Cc: Mike Christie <mchristi at redhat.com>
> Cc: Jens Axboe <axboe at fb.com>
> ---
>
>
> Hi,
>
> I saw that issue in linux next.
>
> Not sure if it is too late to squash the fix with commit b3d3fa519905 or
> not...
>
> Best regards,
>
> Vincent.
>
>
> fs/btrfs/volumes.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c
> index a69203a..6ee1e36 100644
> --- a/fs/btrfs/volumes.c
> +++ b/fs/btrfs/volumes.c
> @@ -5533,7 +5533,7 @@ static int __btrfs_map_block(struct btrfs_fs_info *fs_info, int op,
> }
>
> } else if (map->type & BTRFS_BLOCK_GROUP_DUP) {
> - if (op == REQ_OP_WRITE || REQ_OP_DISCARD ||
> + if (op == REQ_OP_WRITE || op == REQ_OP_DISCARD ||
> op == REQ_GET_READ_MIRRORS) {
> num_stripes = map->num_stripes;
> } else if (mirror_num) {
>
Shoot. Dumb mistake by me. It is of course correct.
Reviewed-by: Mike Christie <mchristi at redhat.com>
More information about the dm-devel
mailing list