Re: [PATCH 05/17] block: use an on-stack bio in blkdev_issue_flush

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

Subject: Re: [PATCH 05/17] block: use an on-stack bio in blkdev_issue_flush
From: Chaitanya Kulkarni <Chaitanya.Kulkarni@xxxxxxx>
Date: Tue, 26 Jan 2021 19:15:02 +0000
Accept-language: en-US
Cc: Mike Snitzer <snitzer@xxxxxxxxxx>, David Sterba <dsterba@xxxxxxxx>, "dm-devel@xxxxxxxxxx" <dm-devel@xxxxxxxxxx>, Naohiro Aota <Naohiro.Aota@xxxxxxx>, "linux-nilfs@xxxxxxxxxxxxxxx" <linux-nilfs@xxxxxxxxxxxxxxx>, Ryusuke@xxxxxxxxxxx, Chao Yu <chao@xxxxxxxxxx>, Josef Bacik <josef@xxxxxxxxxxxxxx>, Coly Li <colyli@xxxxxxx>, "linux-raid@xxxxxxxxxxxxxxx" <linux-raid@xxxxxxxxxxxxxxx>, "linux-bcache@xxxxxxxxxxxxxxx" <linux-bcache@xxxxxxxxxxxxxxx>, "drbd-dev@xxxxxxxxxxxxxxxx" <drbd-dev@xxxxxxxxxxxxxxx>, Jaegeuk Kim <jaegeuk@xxxxxxxxxx>, Konishi <konishi.ryusuke@xxxxxxxxx>, "linux-block@xxxxxxxxxxxxxxx" <linux-block@xxxxxxxxxxxxxxx>, Damien Le Moal <Damien.LeMoal@xxxxxxx>, Philipp@xxxxxxxxxxx, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, "linux-mm@xxxxxxxxx" <linux-mm@xxxxxxxxx>, Reisner <philipp.reisner@xxxxxxxxxx>, "linux-f2fs-devel@xxxxxxxxxxxxxxxxxxxxx" <linux-f2fs-devel@xxxxxxxxxxxxxxxxxxxxx>, "linux-fsdevel@xxxxxxxxxxxxxxx" <linux-fsdevel@xxxxxxxxxxxxxxx>, Lars Ellenberg <lars.ellenberg@xxxxxxxxxx>, "linux-nfs@xxxxxxxxxxxxxxx" <linux-nfs@xxxxxxxxxxxxxxx>, "linux-btrfs@xxxxxxxxxxxxxxx" <linux-btrfs@xxxxxxxxxxxxxxx>
Ironport-sdr: 9F2eAYyHkgpATgkwLy1uhYiWsWZ5LUuQnRgygSejEf4CRXwtTGSIB+/CXjZ+orLDwIb2vr4hz8 JWLrr9ZTteiUk4tXxaXO6ez5X4SQZqIRVo56fAxonHw4cBna5z1++vvKDZ8uaVB86eBC7ww5Mq 5zz2samSRkv59suuw3mqY6j/cwogoIKOScSh+7+WLbAhywqZKASd2+LuCqNCUIBrhh2xyqtigO nezlYVMAxxnHYeMq7i/fxkvk2fmTgcalMTqHvEnWauWTZN/pLxuCv+MzlAMzO5Nvu8fy8Vmg1X yHw=
References: <20210126145247.1964410-1-hch@lst.de> <20210126145247.1964410-6-hch@lst.de>
Wdcipoutbound: EOP-TRUE

On 1/26/21 7:07 AM, Christoph Hellwig wrote:
> There is no point in allocating memory for a synchronous flush.
>
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>

True, looks good.

Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@xxxxxxx>




--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel

References:
- misc bio allocation cleanups
  - From: Christoph Hellwig
- [PATCH 05/17] block: use an on-stack bio in blkdev_issue_flush
  - From: Christoph Hellwig

Prev by Date: Re: [PATCH 01/17] zonefs: use bio_alloc in zonefs_file_dio_append
Next by Date: Re: [PATCH 03/17] blk-crypto: use bio_kmalloc in blk_crypto_clone_bio
Previous by thread: [PATCH 05/17] block: use an on-stack bio in blkdev_issue_flush
Next by thread: [PATCH 06/17] dm-clone: use blkdev_issue_flush in commit_metadata
Index(es):
- Date
- Thread

[Index of Archives] [DM Crypt] [Fedora Desktop] [ATA RAID] [Fedora Marketing] [Fedora Packaging] [Fedora SELinux] [Yosemite Discussion] [KDE Users] [Fedora Docs]