[RFC PATCH 1/7] block: share more code for bio addition helpers
Bart Van Assche
bvanassche at acm.org
Thu Oct 31 13:55:11 PDT 2024
On 10/27/24 7:21 AM, Leon Romanovsky wrote:
> +static int bio_add_zone_append_page_int(struct bio *bio, struct page *page,
> + unsigned int len, unsigned int offset, bool *same_page)
> +{
> + struct block_device *bdev = bio->bi_bdev;
> +
> + if (WARN_ON_ONCE(bio_op(bio) != REQ_OP_ZONE_APPEND))
> + return 0;
> + if (WARN_ON_ONCE(!bdev_is_zoned(bdev)))
> + return 0;
> + return bio_add_hw_page(bdev_get_queue(bdev), bio, page, len, offset,
> + bdev_max_zone_append_sectors(bdev), same_page);
> +}
Does "_int" stand for "_internal"? If so, please consider changing it
into "_impl". I think that will prevent that anyone confuses this suffix
with the "int" data type.
Thanks,
Bart.
More information about the Linux-nvme
mailing list