Re: [PATCHv2 1/3] block/bio: remove duplicate append pages code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Keith Busch <kbusch@xxxxxx>, "linux-fsdevel@xxxxxxxxxxxxxxx" <linux-fsdevel@xxxxxxxxxxxxxxx>, "linux-block@xxxxxxxxxxxxxxx" <linux-block@xxxxxxxxxxxxxxx>
Subject: Re: [PATCHv2 1/3] block/bio: remove duplicate append pages code
From: Chaitanya Kulkarni <chaitanyak@xxxxxxxxxx>
Date: Wed, 18 May 2022 20:21:40 +0000
Accept-language: en-US
Cc: "axboe@xxxxxxxxx" <axboe@xxxxxxxxx>, Kernel Team <Kernel-team@xxxxxx>, "hch@xxxxxx" <hch@xxxxxx>, "bvanassche@xxxxxxx" <bvanassche@xxxxxxx>, "damien.lemoal@xxxxxxxxxxxxxxxxxx" <damien.lemoal@xxxxxxxxxxxxxxxxxx>, Keith Busch <kbusch@xxxxxxxxxx>
In-reply-to: <20220518171131.3525293-2-kbusch@fb.com>
Thread-index: AQHYatr5/qCg/QuXGkWhAPWwreYOAa0lFA0A
Thread-topic: [PATCHv2 1/3] block/bio: remove duplicate append pages code
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.8.0

On 5/18/22 10:11, Keith Busch wrote:
> From: Keith Busch <kbusch@xxxxxxxxxx>
> 
> The setup for getting pages are identical for zone append and normal IO.
> Use common code for each.
> 
> Signed-off-by: Keith Busch <kbusch@xxxxxxxxxx>
> ---
>

Looks good.

Reviewed-by: Chaitanya Kulkarni <kch@xxxxxxxxxx>

-ck

References:
- [PATCHv2 0/3] direct io alignment relax
  - From: Keith Busch
- [PATCHv2 1/3] block/bio: remove duplicate append pages code
  - From: Keith Busch

Prev by Date: Re: [PATCH -next] exec: Remove redundant check in do_open_execat/uselib
Next by Date: Re: [PATCHv2 2/3] block: export dma_alignment attribute
Previous by thread: [PATCHv2 1/3] block/bio: remove duplicate append pages code
Next by thread: Re: [PATCHv2 1/3] block/bio: remove duplicate append pages code
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [NTFS 3] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [NTFS 3] [Samba] [Device Mapper] [CEPH Development]