From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr1-f45.google.com (mail-wr1-f45.google.com [209.85.221.45]) by mail19.linbit.com (LINBIT Mail Daemon) with ESMTP id 885A54205C7 for ; Thu, 4 Jul 2024 13:11:20 +0200 (CEST) Received: by mail-wr1-f45.google.com with SMTP id ffacd0b85a97d-3626c29d3f0so329821f8f.1 for ; Thu, 04 Jul 2024 04:11:20 -0700 (PDT) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 13.4 \(3608.120.23.2.7\)) Subject: Re: [PATCH 14/26] block: move the nonrot flag to queue_limits From: Simon Fernandez In-Reply-To: Date: Thu, 4 Jul 2024 12:11:16 +0100 Content-Transfer-Encoding: quoted-printable Message-Id: <78BDDF6A-1FC7-4DD7-AABF-E0B055772CBF@gmail.com> References: <20240617060532.127975-1-hch@lst.de> <20240617060532.127975-15-hch@lst.de> To: Keith Busch Cc: nvdimm@lists.linux.dev, "Michael S. Tsirkin" , Jason Wang , linux-nvme@lists.infradead.org, Song Liu , linux-mtd@lists.infradead.org, Vineeth Vijayan , linux-bcache@vger.kernel.org, Christoph Hellwig , Alasdair Kergon , drbd-dev@lists.linbit.com, linux-s390@vger.kernel.org, linux-scsi@vger.kernel.org, Richard Weinberger , Geert Uytterhoeven , Yu Kuai , dm-devel@lists.linux.dev, linux-um@lists.infradead.org, Mike Snitzer , Josef Bacik , Ming Lei , linux-raid@vger.kernel.org, linux-m68k@lists.linux-m68k.org, Damien Le Moal , Mikulas Patocka , xen-devel@lists.xenproject.org, ceph-devel@vger.kernel.org, nbd@other.debian.org, Jens Axboe , linux-block@vger.kernel.org, "Martin K. Petersen" , linux-mmc@vger.kernel.org, Philipp Reisner , virtualization@lists.linux.dev, Lars Ellenberg , linuxppc-dev@lists.ozlabs.org, =?utf-8?Q?Roger_Pau_Monn=C3=A9?= List-Id: "*Coordination* of development, patches, contributions -- *Questions* \(even to developers\) go to drbd-user, please." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hi folks, how can I unsubscribe from this group.? Thanks in advance. S > On 24 Jun 2024, at 18:08, Keith Busch wrote: >=20 > 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) >=20 > This is inverted. Should be: >=20 > #define blk_queue_nonrot(q) (!((q)->limits.features & = BLK_FEAT_ROTATIONAL)) >=20