Re: [PATCH v12 01/41] block: add bio_add_zone_append_page

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

 



On 15/01/2021 07:55, Naohiro Aota wrote:
> From: Johannes Thumshirn <johannes.thumshirn@xxxxxxx>
> 
> Add bio_add_zone_append_page(), a wrapper around bio_add_hw_page() which
> is intended to be used by file systems that directly add pages to a bio
> instead of using bio_iov_iter_get_pages().
> 
> Cc: Jens Axboe <axboe@xxxxxxxxx>

Jens, can I have an Ack fro you on this one? Christoph and Josef are fine with
it.

> Reviewed-by: Christoph Hellwig <hch@xxxxxx>
> Signed-off-by: Johannes Thumshirn <johannes.thumshirn@xxxxxxx>
> ---
>  block/bio.c         | 33 +++++++++++++++++++++++++++++++++
>  include/linux/bio.h |  2 ++
>  2 files changed, 35 insertions(+)
> 
> diff --git a/block/bio.c b/block/bio.c
> index fa01bef35bb1..a5c534bfe999 100644
> --- a/block/bio.c
> +++ b/block/bio.c
> @@ -851,6 +851,39 @@ int bio_add_pc_page(struct request_queue *q, struct bio *bio,
>  }
>  EXPORT_SYMBOL(bio_add_pc_page);
>  
> +/**
> + * bio_add_zone_append_page - attempt to add page to zone-append bio
> + * @bio: destination bio
> + * @page: page to add
> + * @len: vec entry length
> + * @offset: vec entry offset
> + *
> + * Attempt to add a page to the bio_vec maplist of a bio that will be submitted
> + * for a zone-append request. This can fail for a number of reasons, such as the
> + * bio being full or the target block device is not a zoned block device or
> + * other limitations of the target block device. The target block device must
> + * allow bio's up to PAGE_SIZE, so it is always possible to add a single page
> + * to an empty bio.
> + *
> + * Returns: number of bytes added to the bio, or 0 in case of a failure.
> + */
> +int bio_add_zone_append_page(struct bio *bio, struct page *page,
> +			     unsigned int len, unsigned int offset)
> +{
> +	struct request_queue *q = bio->bi_disk->queue;
> +	bool same_page = false;
> +
> +	if (WARN_ON_ONCE(bio_op(bio) != REQ_OP_ZONE_APPEND))
> +		return 0;
> +
> +	if (WARN_ON_ONCE(!blk_queue_is_zoned(q)))
> +		return 0;
> +
> +	return bio_add_hw_page(q, bio, page, len, offset,
> +			       queue_max_zone_append_sectors(q), &same_page);
> +}
> +EXPORT_SYMBOL_GPL(bio_add_zone_append_page);
> +
>  /**
>   * __bio_try_merge_page - try appending data to an existing bvec.
>   * @bio: destination bio
> diff --git a/include/linux/bio.h b/include/linux/bio.h
> index c6d765382926..7ef300cb4e9a 100644
> --- a/include/linux/bio.h
> +++ b/include/linux/bio.h
> @@ -442,6 +442,8 @@ void bio_chain(struct bio *, struct bio *);
>  extern int bio_add_page(struct bio *, struct page *, unsigned int,unsigned int);
>  extern int bio_add_pc_page(struct request_queue *, struct bio *, struct page *,
>  			   unsigned int, unsigned int);
> +int bio_add_zone_append_page(struct bio *bio, struct page *page,
> +			     unsigned int len, unsigned int offset);
>  bool __bio_try_merge_page(struct bio *bio, struct page *page,
>  		unsigned int len, unsigned int off, bool *same_page);
>  void __bio_add_page(struct bio *bio, struct page *page,
> 





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

  Powered by Linux