[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 14/26] block: move the nonrot flag to queue_limits
 
- To: Christoph Hellwig <hch@xxxxxx>
 
- From: Keith Busch <kbusch@xxxxxxxxxx>
 
- Date: Mon, 24 Jun 2024 11:08:16 -0600
 
- Cc: Jens Axboe <axboe@xxxxxxxxx>, Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>,	Richard Weinberger <richard@xxxxxx>,	Philipp Reisner <philipp.reisner@xxxxxxxxxx>,	Lars Ellenberg <lars.ellenberg@xxxxxxxxxx>,	Christoph Böhmwalder <christoph.boehmwalder@xxxxxxxxxx>,	Josef Bacik <josef@xxxxxxxxxxxxxx>, Ming Lei <ming.lei@xxxxxxxxxx>,	"Michael S. Tsirkin" <mst@xxxxxxxxxx>,	Jason Wang <jasowang@xxxxxxxxxx>,	Roger Pau Monné <roger.pau@xxxxxxxxxx>,	Alasdair Kergon <agk@xxxxxxxxxx>, Mike Snitzer <snitzer@xxxxxxxxxx>,	Mikulas Patocka <mpatocka@xxxxxxxxxx>, Song Liu <song@xxxxxxxxxx>,	Yu Kuai <yukuai3@xxxxxxxxxx>,	Vineeth Vijayan <vneethv@xxxxxxxxxxxxx>,	"Martin K. Petersen" <martin.petersen@xxxxxxxxxx>,	linux-m68k@xxxxxxxxxxxxxxxxxxxx, linux-um@xxxxxxxxxxxxxxxxxxx,	drbd-dev@xxxxxxxxxxxxxxxx, nbd@xxxxxxxxxxxxxxxx,	linuxppc-dev@xxxxxxxxxxxxxxxx, ceph-devel@xxxxxxxxxxxxxxx,	virtualization@xxxxxxxxxxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxxx,	linux-bcache@xxxxxxxxxxxxxxx, dm-devel@xxxxxxxxxxxxxxx,	linux-raid@xxxxxxxxxxxxxxx, linux-mmc@xxxxxxxxxxxxxxx,	linux-mtd@xxxxxxxxxxxxxxxxxxx, nvdimm@xxxxxxxxxxxxxxx,	linux-nvme@xxxxxxxxxxxxxxxxxxx, linux-s390@xxxxxxxxxxxxxxx,	linux-scsi@xxxxxxxxxxxxxxx, linux-block@xxxxxxxxxxxxxxx,	Damien Le Moal <dlemoal@xxxxxxxxxx>
 
- Delivery-date: Mon, 24 Jun 2024 17:08:32 +0000
 
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
 
 
 
On Mon, Jun 17, 2024 at 08:04:41AM +0200, Christoph Hellwig wrote:
> -#define blk_queue_nonrot(q)  test_bit(QUEUE_FLAG_NONROT, &(q)->queue_flags)
> +#define blk_queue_nonrot(q)  ((q)->limits.features & BLK_FEAT_ROTATIONAL)
This is inverted. Should be:
 #define blk_queue_nonrot(q)    (!((q)->limits.features & BLK_FEAT_ROTATIONAL))
 
 
    
     |