Re: [PATCH 05/10] block: don't update BLK_FEAT_POLL in __blk_mq_update_nr_hw_queues
- To: Nilay Shroff <nilay@linux.ibm.com>
- Cc: Christoph Hellwig <hch@lst.de>, Jens Axboe <axboe@kernel.dk>, Damien Le Moal <dlemoal@kernel.org>, Ming Lei <ming.lei@redhat.com>, linux-block@vger.kernel.org, linux-nvme@lists.infradead.org, nbd@other.debian.org, virtualization@lists.linux.dev, linux-scsi@vger.kernel.org, usb-storage@lists.one-eyed-alien.net
- Subject: Re: [PATCH 05/10] block: don't update BLK_FEAT_POLL in __blk_mq_update_nr_hw_queues
- From: Christoph Hellwig <hch@lst.de>
- Date: Mon, 6 Jan 2025 12:05:32 +0100
- Message-id: <[🔎] 20250106110532.GA22062@lst.de>
- In-reply-to: <[🔎] 4addcb5e-fc88-4a86-a464-cc25d8674267@linux.ibm.com>
- References: <[🔎] 20250106100645.850445-1-hch@lst.de> <[🔎] 20250106100645.850445-6-hch@lst.de> <[🔎] 4addcb5e-fc88-4a86-a464-cc25d8674267@linux.ibm.com>
On Mon, Jan 06, 2025 at 04:31:23PM +0530, Nilay Shroff wrote:
> > +static bool bdev_can_poll(struct block_device *bdev)
> > +{
> > + struct request_queue *q = bdev_get_queue(bdev);
> > +
> > + if (queue_is_mq(q))
> > + return blk_mq_can_poll(q->tag_set);
> > + return q->limits.features & BLK_FEAT_POLL;
> > +}
> > +
>
> Should we make bdev_can_poll() inline ?
I don't really see the point. It's file local and doesn't have any
magic that could confuse the code generator, so we might as well leave
it to the compiler. Although that might be about to change per the
discussion with Damien, which could require it in blk-sysfs, in
which case it should become an inline in a header.
Reply to: