Thread (17 messages) 17 messages, 4 authors, 2021-05-03

Re: [PATCH v2] block: Improve limiting the bio size

From: Bart Van Assche <bvanassche@acm.org>
Date: 2021-04-26 16:18:00

On 4/26/21 1:34 AM, Changheun Lee wrote:
Should we check queue point in bio_max_size()?
__device_add_disk() can be called with "register_queue=false" like as
device_add_disk_no_queue_reg(). How about below?

unsigned int bio_max_size(struct bio *bio)
{
	struct request_queue *q;

	q = (bio->bi_bdev) ? bio->bi_bdev->bd_disk->queue : NULL;
	return q ? q->limits.bio_max_bytes : UINT_MAX;
}
How could bio_max_size() get called from inside __device_add_disk() if
no request queue is registered? Did I perhaps miss something?

Thanks,

Bart.

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help