Re: [PATCH 08/14] sd: simplify the disable case in sd_config_discard

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Christoph Hellwig <hch@xxxxxx>
Subject: Re: [PATCH 08/14] sd: simplify the disable case in sd_config_discard
From: "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>
Date: Thu, 13 Jun 2024 21:43:06 -0400
Cc: Jens Axboe <axboe@xxxxxxxxx>, "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>, Richard Weinberger <richard@xxxxxx>, Anton Ivanov <anton.ivanov@xxxxxxxxxxxxxxxxxx>, Johannes Berg <johannes@xxxxxxxxxxxxxxxx>, Josef Bacik <josef@xxxxxxxxxxxxxx>, Ilya Dryomov <idryomov@xxxxxxxxx>, Dongsheng Yang <dongsheng.yang@xxxxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, linux-um@xxxxxxxxxxxxxxxxxxx, linux-block@xxxxxxxxxxxxxxx, nbd@xxxxxxxxxxxxxxxx, ceph-devel@xxxxxxxxxxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxxx, linux-scsi@xxxxxxxxxxxxxxx, Bart Van Assche <bvanassche@xxxxxxx>, Damien Le Moal <dlemoal@xxxxxxxxxx>
In-reply-to: <20240531074837.1648501-9-hch@lst.de> (Christoph Hellwig's message of "Fri, 31 May 2024 09:48:03 +0200")
Organization: Oracle Corporation

Christoph,

> Fall through to the main call to blk_queue_max_discard_sectors given
> that max_blocks has been initialized to zero above instead of
> duplicating the call.

Reviewed-by: Martin K. Petersen <martin.petersen@xxxxxxxxxx>

-- 
Martin K. Petersen	Oracle Linux Engineering

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

References:
- convert the SCSI ULDs to the atomic queue limits API v2
  - From: Christoph Hellwig
- [PATCH 08/14] sd: simplify the disable case in sd_config_discard
  - From: Christoph Hellwig

Prev by Date: Re: [PATCH 07/14] sd: add a sd_disable_write_same helper
Next by Date: Re: [PATCH 09/14] sd: factor out a sd_discard_mode helper
Previous by thread: [PATCH 08/14] sd: simplify the disable case in sd_config_discard
Next by thread: [PATCH 09/14] sd: factor out a sd_discard_mode helper
Index(es):
- Date
- Thread

[Index of Archives] [SCSI Target Devel] [Linux SCSI Target Infrastructure] [Kernel Newbies] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Linux IIO] [Samba] [Device Mapper]