* [PATCH] scsi: sd: Optimal I/O size should be a multiple of physical block size
[not found] <207561f82d5638c8f513f9aaf970a07098bfbb0f.camel@scientia.net>
@ 2018-06-07 1:51 ` Martin K. Petersen
2018-06-20 1:29 ` Martin K. Petersen
0 siblings, 1 reply; 4+ messages in thread
From: Martin K. Petersen @ 2018-06-07 1:51 UTC (permalink / raw)
To: linux-scsi, Christoph Anton Mitterer; +Cc: Martin K. Petersen, stable
It was reported that some devices report an OPTIMAL TRANSFER LENGTH of
0xFFFF blocks. That looks bogus, especially for a device with a
4096-byte physical block size.
Ignore OPTIMAL TRANSFER LENGTH if it is not a multiple of the device's
reported physical block size.
Cc: <stable@vger.kernel.org>
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=199759
Reported-by: Christoph Anton Mitterer <calestyo@scientia.net>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
---
Before:
NAME ALIGNMENT MIN-IO OPT-IO PHY-SEC LOG-SEC ROTA SCHED RQ-SIZE RA WSAME
sda 0 4096 33553920 4096 512 0 mq-deadline 256 128 32M
After:
NAME ALIGNMENT MIN-IO OPT-IO PHY-SEC LOG-SEC ROTA SCHED RQ-SIZE RA WSAME
sda 0 4096 0 4096 512 0 mq-deadline 256 4096 32M
---
drivers/scsi/sd.c | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c
index 9421d9877730..2d19eed3dfbc 100644
--- a/drivers/scsi/sd.c
+++ b/drivers/scsi/sd.c
@@ -3067,7 +3067,7 @@ static int sd_revalidate_disk(struct gendisk *disk)
struct request_queue *q = sdkp->disk->queue;
sector_t old_capacity = sdkp->capacity;
unsigned char *buffer;
- unsigned int dev_max, rw_max;
+ unsigned int dev_max, rw_max, opt_xfer_bytes;
SCSI_LOG_HLQUEUE(3, sd_printk(KERN_INFO, sdkp,
"sd_revalidate_disk\n"));
@@ -3126,13 +3126,16 @@ static int sd_revalidate_disk(struct gendisk *disk)
/*
* Determine the device's preferred I/O size for reads and writes
- * unless the reported value is unreasonably small, large, or
- * garbage.
+ * unless the reported value is unreasonably small, large, not a
+ * multiple of the physical block size, or simply garbage.
*/
+ opt_xfer_bytes = logical_to_bytes(sdp, sdkp->opt_xfer_blocks);
+
if (sdkp->opt_xfer_blocks &&
sdkp->opt_xfer_blocks <= dev_max &&
sdkp->opt_xfer_blocks <= SD_DEF_XFER_BLOCKS &&
- logical_to_bytes(sdp, sdkp->opt_xfer_blocks) >= PAGE_SIZE) {
+ opt_xfer_bytes >= PAGE_SIZE &&
+ (opt_xfer_bytes & (sdkp->physical_block_size - 1)) != 0) {
q->limits.io_opt = logical_to_bytes(sdp, sdkp->opt_xfer_blocks);
rw_max = logical_to_sectors(sdp, sdkp->opt_xfer_blocks);
} else
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] scsi: sd: Optimal I/O size should be a multiple of physical block size
@ 2019-02-08 23:41 Martin K. Petersen
2019-02-12 8:04 ` Christoph Hellwig
0 siblings, 1 reply; 4+ messages in thread
From: Martin K. Petersen @ 2019-02-08 23:41 UTC (permalink / raw)
To: linux-scsi; +Cc: Martin K. Petersen, stable
It was reported that some devices report an OPTIMAL TRANSFER LENGTH of
0xFFFF blocks. That looks bogus, especially for a device with a
4096-byte physical block size.
Ignore OPTIMAL TRANSFER LENGTH if it is not a multiple of the device's
reported physical block size.
Cc: <stable@vger.kernel.org>
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=199759
Reported-by: Christoph Anton Mitterer <calestyo@scientia.net>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
---
Before:
NAME ALIGNMENT MIN-IO OPT-IO PHY-SEC LOG-SEC ROTA SCHED RQ-SIZE RA WSAME
sda 0 4096 33553920 4096 512 0 mq-deadline 256 128 32M
After:
NAME ALIGNMENT MIN-IO OPT-IO PHY-SEC LOG-SEC ROTA SCHED RQ-SIZE RA WSAME
sda 0 4096 0 4096 512 0 mq-deadline 256 4096 32M
---
drivers/scsi/sd.c | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c
index 9aa409b38765..d5ede5d65838 100644
--- a/drivers/scsi/sd.c
+++ b/drivers/scsi/sd.c
@@ -3060,7 +3060,7 @@ static int sd_revalidate_disk(struct gendisk *disk)
struct request_queue *q = sdkp->disk->queue;
sector_t old_capacity = sdkp->capacity;
unsigned char *buffer;
- unsigned int dev_max, rw_max;
+ unsigned int dev_max, rw_max, opt_xfer_bytes;
SCSI_LOG_HLQUEUE(3, sd_printk(KERN_INFO, sdkp,
"sd_revalidate_disk\n"));
@@ -3119,13 +3119,16 @@ static int sd_revalidate_disk(struct gendisk *disk)
/*
* Determine the device's preferred I/O size for reads and writes
- * unless the reported value is unreasonably small, large, or
- * garbage.
+ * unless the reported value is unreasonably small, large, not a
+ * multiple of the physical block size, or simply garbage.
*/
+ opt_xfer_bytes = logical_to_bytes(sdp, sdkp->opt_xfer_blocks);
+
if (sdkp->opt_xfer_blocks &&
sdkp->opt_xfer_blocks <= dev_max &&
sdkp->opt_xfer_blocks <= SD_DEF_XFER_BLOCKS &&
- logical_to_bytes(sdp, sdkp->opt_xfer_blocks) >= PAGE_SIZE) {
+ opt_xfer_bytes >= PAGE_SIZE &&
+ (opt_xfer_bytes & (sdkp->physical_block_size - 1)) != 0) {
q->limits.io_opt = logical_to_bytes(sdp, sdkp->opt_xfer_blocks);
rw_max = logical_to_sectors(sdp, sdkp->opt_xfer_blocks);
} else
--
2.19.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] scsi: sd: Optimal I/O size should be a multiple of physical block size
2019-02-08 23:41 Martin K. Petersen
@ 2019-02-12 8:04 ` Christoph Hellwig
0 siblings, 0 replies; 4+ messages in thread
From: Christoph Hellwig @ 2019-02-12 8:04 UTC (permalink / raw)
To: Martin K. Petersen; +Cc: linux-scsi, stable
On Fri, Feb 08, 2019 at 06:41:47PM -0500, Martin K. Petersen wrote:
> It was reported that some devices report an OPTIMAL TRANSFER LENGTH of
> 0xFFFF blocks. That looks bogus, especially for a device with a
> 4096-byte physical block size.
>
> Ignore OPTIMAL TRANSFER LENGTH if it is not a multiple of the device's
> reported physical block size.
Looks generally fine, but should we log a warning in case of a bogus
OPTIMAL TRANSFER LENGTH?
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-02-12 8:04 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <207561f82d5638c8f513f9aaf970a07098bfbb0f.camel@scientia.net>
2018-06-07 1:51 ` [PATCH] scsi: sd: Optimal I/O size should be a multiple of physical block size Martin K. Petersen
2018-06-20 1:29 ` Martin K. Petersen
2019-02-08 23:41 Martin K. Petersen
2019-02-12 8:04 ` Christoph Hellwig
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).