Linux-Fsdevel Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Hannes Reinecke <hare@suse.de>
To: Johannes Thumshirn <johannes.thumshirn@wdc.com>,
	Jens Axboe <axboe@kernel.dk>
Cc: Christoph Hellwig <hch@infradead.org>,
	linux-block <linux-block@vger.kernel.org>,
	Damien Le Moal <Damien.LeMoal@wdc.com>,
	Keith Busch <kbusch@kernel.org>,
	"linux-scsi @ vger . kernel . org" <linux-scsi@vger.kernel.org>,
	"Martin K . Petersen" <martin.petersen@oracle.com>,
	"linux-fsdevel @ vger . kernel . org"
	<linux-fsdevel@vger.kernel.org>, Christoph Hellwig <hch@lst.de>,
	Daniel Wagner <dwagner@suse.de>
Subject: Re: [PATCH v10 1/9] block: rename __bio_add_pc_page to bio_add_hw_page
Date: Wed, 6 May 2020 20:16:15 +0200	[thread overview]
Message-ID: <df3cc93b-0e71-868a-30a9-dac7048e15b5@suse.de> (raw)
In-Reply-To: <20200506161145.9841-2-johannes.thumshirn@wdc.com>

On 5/6/20 6:11 PM, Johannes Thumshirn wrote:
> From: Christoph Hellwig <hch@lst.de>
> 
> Rename __bio_add_pc_page() to bio_add_hw_page() and explicitly pass in a
> max_sectors argument.
> 
> This max_sectors argument can be used to specify constraints from the
> hardware.
> 
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> [ jth: rebased and made public for blk-map.c ]
> Signed-off-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
> Reviewed-by: Daniel Wagner <dwagner@suse.de>
> Reviewed-by: Martin K. Petersen <martin.petersen@oracle.com>
> ---
>   block/bio.c     | 65 ++++++++++++++++++++++++++++++-------------------
>   block/blk-map.c |  5 ++--
>   block/blk.h     |  4 +--
>   3 files changed, 45 insertions(+), 29 deletions(-)
> 
> diff --git a/block/bio.c b/block/bio.c
> index 21cbaa6a1c20..aad0a6dad4f9 100644
> --- a/block/bio.c
> +++ b/block/bio.c
> @@ -748,9 +748,14 @@ static inline bool page_is_mergeable(const struct bio_vec *bv,
>   	return true;
>   }
>   
> -static bool bio_try_merge_pc_page(struct request_queue *q, struct bio *bio,
> -		struct page *page, unsigned len, unsigned offset,
> -		bool *same_page)
> +/*
> + * Try to merge a page into a segment, while obeying the hardware segment
> + * size limit.  This is not for normal read/write bios, but for passthrough
> + * or Zone Append operations that we can't split.
> + */
> +static bool bio_try_merge_hw_seg(struct request_queue *q, struct bio *bio,
> +				 struct page *page, unsigned len,
> +				 unsigned offset, bool *same_page)
>   {
>   	struct bio_vec *bv = &bio->bi_io_vec[bio->bi_vcnt - 1];
>   	unsigned long mask = queue_segment_boundary(q);
> @@ -765,38 +770,32 @@ static bool bio_try_merge_pc_page(struct request_queue *q, struct bio *bio,
>   }
>   
>   /**
> - *	__bio_add_pc_page	- attempt to add page to passthrough bio
> - *	@q: the target queue
> - *	@bio: destination bio
> - *	@page: page to add
> - *	@len: vec entry length
> - *	@offset: vec entry offset
> - *	@same_page: return if the merge happen inside the same page
> - *
> - *	Attempt to add a page to the bio_vec maplist. This can fail for a
> - *	number of reasons, such as the bio being full or target block device
> - *	limitations. 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.
> + * bio_add_hw_page - attempt to add a page to a bio with hw constraints
> + * @q: the target queue
> + * @bio: destination bio
> + * @page: page to add
> + * @len: vec entry length
> + * @offset: vec entry offset
> + * @max_sectors: maximum number of sectors that can be added
> + * @same_page: return if the segment has been merged inside the same page
>    *
> - *	This should only be used by passthrough bios.
> + * Add a page to a bio while respecting the hardware max_sectors, max_segment
> + * and gap limitations.
>    */
> -int __bio_add_pc_page(struct request_queue *q, struct bio *bio,
> +int bio_add_hw_page(struct request_queue *q, struct bio *bio,
>   		struct page *page, unsigned int len, unsigned int offset,
> -		bool *same_page)
> +		unsigned int max_sectors, bool *same_page)
>   {
>   	struct bio_vec *bvec;
>   
> -	/*
> -	 * cloned bio must not modify vec list
> -	 */
> -	if (unlikely(bio_flagged(bio, BIO_CLONED)))
> +	if (WARN_ON_ONCE(bio_flagged(bio, BIO_CLONED)))
>   		return 0;
>   
> -	if (((bio->bi_iter.bi_size + len) >> 9) > queue_max_hw_sectors(q))
> +	if (((bio->bi_iter.bi_size + len) >> 9) > max_sectors)
>   		return 0;
>   
>   	if (bio->bi_vcnt > 0) {
> -		if (bio_try_merge_pc_page(q, bio, page, len, offset, same_page))
> +		if (bio_try_merge_hw_seg(q, bio, page, len, offset, same_page))
>   			return len;
>   
>   		/*
> @@ -823,11 +822,27 @@ int __bio_add_pc_page(struct request_queue *q, struct bio *bio,
>   	return len;
>   }
>   
> +/**
> + * bio_add_pc_page	- attempt to add page to passthrough bio
> + * @q: the target queue
> + * @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. This can fail for a
> + * number of reasons, such as the bio being full or target block device
> + * limitations. 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.
> + *
> + * This should only be used by passthrough bios.
> + */
>   int bio_add_pc_page(struct request_queue *q, struct bio *bio,
>   		struct page *page, unsigned int len, unsigned int offset)
>   {
>   	bool same_page = false;
> -	return __bio_add_pc_page(q, bio, page, len, offset, &same_page);
> +	return bio_add_hw_page(q, bio, page, len, offset,
> +			queue_max_hw_sectors(q), &same_page);
>   }
>   EXPORT_SYMBOL(bio_add_pc_page);
>   
> diff --git a/block/blk-map.c b/block/blk-map.c
> index b6fa343fea9f..e3e4ac48db45 100644
> --- a/block/blk-map.c
> +++ b/block/blk-map.c
> @@ -257,6 +257,7 @@ static struct bio *bio_copy_user_iov(struct request_queue *q,
>   static struct bio *bio_map_user_iov(struct request_queue *q,
>   		struct iov_iter *iter, gfp_t gfp_mask)
>   {
> +	unsigned int max_sectors = queue_max_hw_sectors(q);
>   	int j;
>   	struct bio *bio;
>   	int ret;
> @@ -294,8 +295,8 @@ static struct bio *bio_map_user_iov(struct request_queue *q,
>   				if (n > bytes)
>   					n = bytes;
>   
> -				if (!__bio_add_pc_page(q, bio, page, n, offs,
> -						&same_page)) {
> +				if (!bio_add_hw_page(q, bio, page, n, offs,
> +						     max_sectors, &same_page)) {
>   					if (same_page)
>   						put_page(page);
>   					break;
> diff --git a/block/blk.h b/block/blk.h
> index 73bd3b1c6938..1ae3279df712 100644
> --- a/block/blk.h
> +++ b/block/blk.h
> @@ -453,8 +453,8 @@ static inline void part_nr_sects_write(struct hd_struct *part, sector_t size)
>   
>   struct request_queue *__blk_alloc_queue(int node_id);
>   
> -int __bio_add_pc_page(struct request_queue *q, struct bio *bio,
> +int bio_add_hw_page(struct request_queue *q, struct bio *bio,
>   		struct page *page, unsigned int len, unsigned int offset,
> -		bool *same_page);
> +		unsigned int max_sectors, bool *same_page);
>   
>   #endif /* BLK_INTERNAL_H */
> 
Reviewed-by: Hannes Reinecke <hare@suse.de>

Cheers,

Hannes
-- 
Dr. Hannes Reinecke            Teamlead Storage & Networking
hare@suse.de                               +49 911 74053 688
SUSE Software Solutions GmbH, Maxfeldstr. 5, 90409 Nürnberg
HRB 36809 (AG Nürnberg), Geschäftsführer: Felix Imendörffer

  reply	other threads:[~2020-05-06 18:16 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-06 16:11 [PATCH v10 0/9] Introduce Zone Append for writing to zoned block devices Johannes Thumshirn
2020-05-06 16:11 ` [PATCH v10 1/9] block: rename __bio_add_pc_page to bio_add_hw_page Johannes Thumshirn
2020-05-06 18:16   ` Hannes Reinecke [this message]
2020-05-06 16:11 ` [PATCH v10 2/9] block: Introduce REQ_OP_ZONE_APPEND Johannes Thumshirn
2020-05-06 16:11 ` [PATCH v10 3/9] block: introduce blk_req_zone_write_trylock Johannes Thumshirn
2020-05-06 16:11 ` [PATCH v10 4/9] block: Modify revalidate zones Johannes Thumshirn
2020-05-06 16:11 ` [PATCH v10 5/9] scsi: sd_zbc: factor out sanity checks for zoned commands Johannes Thumshirn
2020-05-06 16:11 ` [PATCH v10 6/9] scsi: sd_zbc: emulate ZONE_APPEND commands Johannes Thumshirn
2020-05-06 18:18   ` Hannes Reinecke
2020-05-06 16:11 ` [PATCH v10 7/9] null_blk: Support REQ_OP_ZONE_APPEND Johannes Thumshirn
2020-05-06 16:11 ` [PATCH v10 8/9] block: export bio_release_pages and bio_iov_iter_get_pages Johannes Thumshirn
2020-05-06 16:11 ` [PATCH v10 9/9] zonefs: use REQ_OP_ZONE_APPEND for sync DIO Johannes Thumshirn

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=df3cc93b-0e71-868a-30a9-dac7048e15b5@suse.de \
    --to=hare@suse.de \
    --cc=Damien.LeMoal@wdc.com \
    --cc=axboe@kernel.dk \
    --cc=dwagner@suse.de \
    --cc=hch@infradead.org \
    --cc=hch@lst.de \
    --cc=johannes.thumshirn@wdc.com \
    --cc=kbusch@kernel.org \
    --cc=linux-block@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=martin.petersen@oracle.com \
    --subject='Re: [PATCH v10 1/9] block: rename __bio_add_pc_page to bio_add_hw_page' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).