Skip to content

Commit 4f9eedf

Browse files
Yihang Limartinkpetersen
authored andcommitted
scsi: sd: Ignore command SYNCHRONIZE CACHE error if format in progress
If formatting a suspended disk (such as formatting with different DIF type), the disk will be resuming first, and then the format command will submit to the disk through SG_IO ioctl. When the disk is processing the format command, the system does not submit other commands to the disk. Therefore, the system attempts to suspend the disk again and sends the SYNCHRONIZE CACHE command. However, the SYNCHRONIZE CACHE command will fail because the disk is in the formatting process. This will cause the runtime_status of the disk to error and it is difficult for user to recover it. Error info like: [ 669.925325] sd 6:0:6:0: [sdg] Synchronizing SCSI cache [ 670.202371] sd 6:0:6:0: [sdg] Synchronize Cache(10) failed: Result: hostbyte=0x00 driverbyte=DRIVER_OK [ 670.216300] sd 6:0:6:0: [sdg] Sense Key : 0x2 [current] [ 670.221860] sd 6:0:6:0: [sdg] ASC=0x4 ASCQ=0x4 To solve the issue, ignore the error and return success/0 when format is in progress. Cc: stable@vger.kernel.org Signed-off-by: Yihang Li <liyihang9@huawei.com> Link: https://lore.kernel.org/r/20240819090934.2130592-1-liyihang9@huawei.com Reviewed-by: Bart Van Assche <bvanassche@acm.org> Reviewed-by: Damien Le Moal <dlemoal@kernel.org> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
1 parent 919ddf8 commit 4f9eedf

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

drivers/scsi/sd.c

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1823,13 +1823,15 @@ static int sd_sync_cache(struct scsi_disk *sdkp)
18231823
(sshdr.asc == 0x74 && sshdr.ascq == 0x71)) /* drive is password locked */
18241824
/* this is no error here */
18251825
return 0;
1826+
18261827
/*
1827-
* This drive doesn't support sync and there's not much
1828-
* we can do because this is called during shutdown
1829-
* or suspend so just return success so those operations
1830-
* can proceed.
1828+
* If a format is in progress or if the drive does not
1829+
* support sync, there is not much we can do because
1830+
* this is called during shutdown or suspend so just
1831+
* return success so those operations can proceed.
18311832
*/
1832-
if (sshdr.sense_key == ILLEGAL_REQUEST)
1833+
if ((sshdr.asc == 0x04 && sshdr.ascq == 0x04) ||
1834+
sshdr.sense_key == ILLEGAL_REQUEST)
18331835
return 0;
18341836
}
18351837

0 commit comments

Comments
 (0)