From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3505C3EC683; Tue, 6 Oct 2026 12:45:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791290740; cv=none; b=nPNKsf+OSaXYRtJbSEQMjg8b75PwTNjhnb7KlBQjFwbVsoDgFOgshTBUU5qlAgzha/UKl2O/wYbdVtE8KKU03X+CXaw2rY7f2tShlfKR9Mn9AVjsAhDwCrT9fB0AIqoW+oLsmGdkQX8ZwkMIfFRP2jm5fvjDgxTcwtgaqxtlsEc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791290740; c=relaxed/simple; bh=xbFvBa+TUnnlddB06UOuH8yLVlYrJv/hs+4MAT4utNA=; h=From:To:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=BVLTKgV7YeHJ0L+38yF0X8nL4lP6oeVUA83Q+I/F0tltfSChrHQx8A16q4qfcnY0UBfrWCkifp9/5vu/JTz7Z5Jx8F7Mhszg7moizYKdrOahOYIpTwjdauWUL87n2yQ9CakiCoBN/7EMRbecFFlZestuiiYsZC2cTwEC1ZFUwgg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=C4DxMn4o; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="C4DxMn4o" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 64BCE1F0089C; Tue, 6 Oct 2026 12:45:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1791290718; bh=O/PxZAbzQe04VVNkcUUw/+EKUtlBgPt0Wp0fZer6l8g=; h=From:To:Subject:Date:In-Reply-To:References; b=C4DxMn4ogRJRBdkhtTSBoPri4OLztPMpppJwvmVv+OC8hTndncHNRu363oo2fGXwp c2rQ+7OLa15Ozfx/gu/UGnxyJrI/vecTW6wM5gE3Bzr+zQRLwqeGqVfUhrPRDB0WiE IFgOxVGdIM1MgK3ohkol8lht3mtDsLHNUy2SLVsD8wqorlJ+WFKA2jBSt4hulEAhBi sagFSXv43dnIyRa+hZYz6twRZfz86J5j6rpHO98PbQI67z2+4fxyXQOLVfE5HIznKo O6u7phxRFdW9yxi7hF0zjTgRNRGvjDrW2/SqqMi1MbdTGqI8NVkU1EZBDW7YEmXgec c4b/Wtlojld1A== From: Damien Le Moal To: Jens Axboe , linux-block@vger.kernel.org, Christoph Hellwig , linux-scsi@vger.kernel.org, "Martin K . Petersen" Subject: [PATCH v2 6/7] scsi: sd_zbc: always revalidate zones for disks supporting head depopulation Date: Tue, 6 Oct 2026 21:45:09 +0900 Message-ID: <20261006124510.882017-7-dlemoal@kernel.org> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20261006124510.882017-1-dlemoal@kernel.org> References: <20261006124510.882017-1-dlemoal@kernel.org> Precedence: bulk X-Mailing-List: linux-block@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit sd_zbc_revalidate_zones() skips revalidating the zones of a ZBC device if the zone size and total number of zones of the disk has not changed. This is to avoid a call to the rather slow blk_revalidate_disk_zones(). However, for ZBC devices that support data preserving head depopulation (REMOVE ELEMENT AND MODIFY ZONES command), a disk capacity and number of zones does not change after a head is depopulated but the condition of zones changes as the zones served by the head that was depopulated become either read-only or offline. In this case, not calling blk_revalidate_disk_zones() prevents the block layer from taking appropriate actions on the zone write plugs of the disk for the zones that became read-only or offline. Avoid any issue with the block layer view of the zone conditions by not skipping the call to blk_revalidate_disk_zones() for disks that support the REMOVE ELEMENT AND MODIFY ZONES command. This check is done from sd_zbc_read_zones() using the helper function sd_zbc_check_modify_zones(). The new scsi disk flag modify_zones_supported is defined to remember the result of this check. Signed-off-by: Damien Le Moal --- drivers/scsi/sd.h | 1 + drivers/scsi/sd_zbc.c | 26 +++++++++++++++++++++++++- 2 files changed, 26 insertions(+), 1 deletion(-) diff --git a/drivers/scsi/sd.h b/drivers/scsi/sd.h index 574af8243016..6a72371fee78 100644 --- a/drivers/scsi/sd.h +++ b/drivers/scsi/sd.h @@ -156,6 +156,7 @@ struct scsi_disk { unsigned ignore_medium_access_errors : 1; unsigned rscs : 1; /* reduced stream control support */ unsigned use_atomic_write_boundary : 1; + unsigned modify_zones_supported : 1; }; #define to_scsi_disk(obj) container_of(obj, struct scsi_disk, disk_dev) diff --git a/drivers/scsi/sd_zbc.c b/drivers/scsi/sd_zbc.c index 456beaf2e769..2c77f878ab3f 100644 --- a/drivers/scsi/sd_zbc.c +++ b/drivers/scsi/sd_zbc.c @@ -516,6 +516,21 @@ static int sd_zbc_check_capacity(struct scsi_disk *sdkp, unsigned char *buf, return 0; } +/* + * sd_zbc_check_modify_zones - Check if the device supports depopulation + * @sdkp: Target disk + * @buf: command buffer + * + * Check if the device supports the REMOVE ELEMENT AND MODIFY ZONES command. + */ +static inline bool sd_zbc_check_modify_zones(struct scsi_disk *sdkp, + unsigned char *buf) +{ + return scsi_report_opcode(sdkp->device, buf, SD_BUF_SIZE, + SERVICE_ACTION_IN_16, + SAI_REMOVE_ELEMENT_AND_MODIFY_ZONES) == 1; +} + static void sd_zbc_print_zones(struct scsi_disk *sdkp) { if (sdkp->device->type != TYPE_ZBC || !sdkp->capacity) @@ -554,9 +569,15 @@ int sd_zbc_revalidate_zones(struct scsi_disk *sdkp) if (!blk_queue_is_zoned(q)) return 0; + /* + * If the zone size and number of zones has not changed, and the disk + * does not support depopulating heads, skip the rather slow call to + * blk_revalidate_disk_zones(). + */ if (sdkp->zone_info.zone_blocks == zone_blocks && sdkp->zone_info.nr_zones == nr_zones && - disk->nr_zones == nr_zones) + disk->nr_zones == nr_zones && + !sdkp->modify_zones_supported) return 0; sdkp->zone_info.zone_blocks = zone_blocks; @@ -620,6 +641,9 @@ int sd_zbc_read_zones(struct scsi_disk *sdkp, struct queue_limits *lim, if (ret != 0) goto err; + /* Check if REMOVE ELEMENT AND MODIFY ZONES is supported. */ + sdkp->modify_zones_supported = sd_zbc_check_modify_zones(sdkp, buf); + nr_zones = round_up(sdkp->capacity, zone_blocks) >> ilog2(zone_blocks); if (nr_zones > INT_MAX) { sd_printk(KERN_ERR, sdkp, "Too many zones (%llu)\n", -- 2.55.0