* [PATCH] libata: Increase the timeout of command ATA_CMD_READ_LOG_DMA_EXT
@ 2020-10-15 1:57 chenxiang
2020-11-03 8:23 ` chenxiang (M)
2021-03-30 1:54 ` chenxiang (M)
0 siblings, 2 replies; 3+ messages in thread
From: chenxiang @ 2020-10-15 1:57 UTC (permalink / raw)
To: axboe, tj; +Cc: john.garry, linux-ide, linuxarm, Xiang Chen
From: Xiang Chen <chenxiang66@hisilicon.com>
ATA_CMD_READ_LOG_DMA_EXT command normally uses the common internal command
default timeout (5s), and it is adequate in most situations. But it is not
enough for some disks such as ST4000NM0035 which causes qc timeout
sometimes as follows:
...
[ 157.284284] sas: Enter sas_scsi_recover_host busy: 0 failed: 0
[ 157.290124] sas: ata8: end_device-4:0:2: dev error handler
[ 157.290128] sas: ata9: end_device-4:0:8: dev error handler
[ 157.290131] sas: ata10: end_device-4:0:11: dev error handler
[ 164.588391] ata9.00: qc timeout (cmd 0x47)
[ 164.588393] ata8.00: qc timeout (cmd 0x47)
[ 164.588395] ata10.00: qc timeout (cmd 0x47)
[ 166.582572] ata10.00: READ LOG DMA EXT failed, trying PIO
[ 166.587960] ata10.00: NCQ Send/Recv Log not supported
[ 166.592997] ata10.00: ATA-10: ST4000NM0035-1V4107, TN03, max UDMA/133
[ 166.599414] ata10.00: 7814037168 sectors, multi 0: LBA48 NCQ (depth 32)
[ 166.606011] ata10.00: failed to set xfermode (err_mask=0x40)
[ 166.659253] ata8.00: READ LOG DMA EXT failed, trying PIO
[ 166.664559] ata8.00: NCQ Send/Recv Log not supported
[ 166.669506] ata8.00: ATA-10: ST4000NM0035-1V4107, TN03, max UDMA/133
[ 166.675837] ata8.00: 7814037168 sectors, multi 0: LBA48 NCQ (depth 32)
[ 166.682343] ata8.00: failed to set xfermode (err_mask=0x40)
[ 166.743458] ata9.00: READ LOG DMA EXT failed, trying PIO
[ 166.748765] ata9.00: NCQ Send/Recv Log not supported
[ 166.753713] ata9.00: ATA-10: ST4000NM0035-1V4107, TN03, max UDMA/133
[ 166.760045] ata9.00: 7814037168 sectors, multi 0: LBA48 NCQ (depth 32)
[ 166.766553] ata9.00: failed to set xfermode (err_mask=0x40)
[ 168.886378] ata10.00: configured for UDMA/133
[ 168.999943] ata8.00: configured for UDMA/133
[ 169.050801] ata9.00: configured for UDMA/133
[ 169.055071] sas: --- Exit sas_scsi_recover_host: busy: 0 failed: 0 tries: 1
...
Actually it may takes about 9s to complete the command for disk ST4000NM0035
sometimes. So to avoid the issue, increase the timeout of command
ATA_CMD_READ_LOG_DMA_EXT from 5s to 15s.
Signed-off-by: Xiang Chen <chenxiang66@hisilicon.com>
Reviewed-by: John Garry <john.garry@huawei.com>
---
drivers/ata/libata-eh.c | 9 +++++++++
include/linux/libata.h | 2 +-
2 files changed, 10 insertions(+), 1 deletion(-)
diff --git a/drivers/ata/libata-eh.c b/drivers/ata/libata-eh.c
index 474c6c3..6db9474 100644
--- a/drivers/ata/libata-eh.c
+++ b/drivers/ata/libata-eh.c
@@ -100,6 +100,13 @@ static const unsigned long ata_eh_flush_timeouts[] = {
ULONG_MAX,
};
+static const unsigned long ata_eh_read_log_ext_timeouts[] = {
+ 15000,
+ 15000,
+ 30000,
+ ULONG_MAX,
+};
+
static const unsigned long ata_eh_other_timeouts[] = {
5000, /* same rationale as identify timeout */
10000, /* ditto */
@@ -139,6 +146,8 @@ ata_eh_cmd_timeout_table[ATA_EH_CMD_TIMEOUT_TABLE_SIZE] = {
.timeouts = ata_eh_other_timeouts, },
{ .commands = CMDS(ATA_CMD_FLUSH, ATA_CMD_FLUSH_EXT),
.timeouts = ata_eh_flush_timeouts },
+ { .commands = CMDS(ATA_CMD_READ_LOG_DMA_EXT, ATA_CMD_READ_LOG_EXT),
+ .timeouts = ata_eh_read_log_ext_timeouts },
};
#undef CMDS
diff --git a/include/linux/libata.h b/include/linux/libata.h
index b0bd30e..bcc68b3 100644
--- a/include/linux/libata.h
+++ b/include/linux/libata.h
@@ -390,7 +390,7 @@ enum {
/* This should match the actual table size of
* ata_eh_cmd_timeout_table in libata-eh.c.
*/
- ATA_EH_CMD_TIMEOUT_TABLE_SIZE = 6,
+ ATA_EH_CMD_TIMEOUT_TABLE_SIZE = 7,
/* Horkage types. May be set by libata or controller on drives
(some horkage may be drive/controller pair dependent */
--
2.8.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] libata: Increase the timeout of command ATA_CMD_READ_LOG_DMA_EXT
2020-10-15 1:57 [PATCH] libata: Increase the timeout of command ATA_CMD_READ_LOG_DMA_EXT chenxiang
@ 2020-11-03 8:23 ` chenxiang (M)
2021-03-30 1:54 ` chenxiang (M)
1 sibling, 0 replies; 3+ messages in thread
From: chenxiang (M) @ 2020-11-03 8:23 UTC (permalink / raw)
To: axboe, tj; +Cc: john.garry, linux-ide, linuxarm
在 2020/10/15 9:57, chenxiang 写道:
> From: Xiang Chen <chenxiang66@hisilicon.com>
>
> ATA_CMD_READ_LOG_DMA_EXT command normally uses the common internal command
> default timeout (5s), and it is adequate in most situations. But it is not
> enough for some disks such as ST4000NM0035 which causes qc timeout
> sometimes as follows:
> ...
> [ 157.284284] sas: Enter sas_scsi_recover_host busy: 0 failed: 0
> [ 157.290124] sas: ata8: end_device-4:0:2: dev error handler
> [ 157.290128] sas: ata9: end_device-4:0:8: dev error handler
> [ 157.290131] sas: ata10: end_device-4:0:11: dev error handler
> [ 164.588391] ata9.00: qc timeout (cmd 0x47)
> [ 164.588393] ata8.00: qc timeout (cmd 0x47)
> [ 164.588395] ata10.00: qc timeout (cmd 0x47)
> [ 166.582572] ata10.00: READ LOG DMA EXT failed, trying PIO
> [ 166.587960] ata10.00: NCQ Send/Recv Log not supported
> [ 166.592997] ata10.00: ATA-10: ST4000NM0035-1V4107, TN03, max UDMA/133
> [ 166.599414] ata10.00: 7814037168 sectors, multi 0: LBA48 NCQ (depth 32)
> [ 166.606011] ata10.00: failed to set xfermode (err_mask=0x40)
> [ 166.659253] ata8.00: READ LOG DMA EXT failed, trying PIO
> [ 166.664559] ata8.00: NCQ Send/Recv Log not supported
> [ 166.669506] ata8.00: ATA-10: ST4000NM0035-1V4107, TN03, max UDMA/133
> [ 166.675837] ata8.00: 7814037168 sectors, multi 0: LBA48 NCQ (depth 32)
> [ 166.682343] ata8.00: failed to set xfermode (err_mask=0x40)
> [ 166.743458] ata9.00: READ LOG DMA EXT failed, trying PIO
> [ 166.748765] ata9.00: NCQ Send/Recv Log not supported
> [ 166.753713] ata9.00: ATA-10: ST4000NM0035-1V4107, TN03, max UDMA/133
> [ 166.760045] ata9.00: 7814037168 sectors, multi 0: LBA48 NCQ (depth 32)
> [ 166.766553] ata9.00: failed to set xfermode (err_mask=0x40)
> [ 168.886378] ata10.00: configured for UDMA/133
> [ 168.999943] ata8.00: configured for UDMA/133
> [ 169.050801] ata9.00: configured for UDMA/133
> [ 169.055071] sas: --- Exit sas_scsi_recover_host: busy: 0 failed: 0 tries: 1
> ...
> Actually it may takes about 9s to complete the command for disk ST4000NM0035
> sometimes. So to avoid the issue, increase the timeout of command
> ATA_CMD_READ_LOG_DMA_EXT from 5s to 15s.
>
> Signed-off-by: Xiang Chen <chenxiang66@hisilicon.com>
> Reviewed-by: John Garry <john.garry@huawei.com>
Hi Jens, do you have any idea about this patch?
> ---
> drivers/ata/libata-eh.c | 9 +++++++++
> include/linux/libata.h | 2 +-
> 2 files changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/ata/libata-eh.c b/drivers/ata/libata-eh.c
> index 474c6c3..6db9474 100644
> --- a/drivers/ata/libata-eh.c
> +++ b/drivers/ata/libata-eh.c
> @@ -100,6 +100,13 @@ static const unsigned long ata_eh_flush_timeouts[] = {
> ULONG_MAX,
> };
>
> +static const unsigned long ata_eh_read_log_ext_timeouts[] = {
> + 15000,
> + 15000,
> + 30000,
> + ULONG_MAX,
> +};
> +
> static const unsigned long ata_eh_other_timeouts[] = {
> 5000, /* same rationale as identify timeout */
> 10000, /* ditto */
> @@ -139,6 +146,8 @@ ata_eh_cmd_timeout_table[ATA_EH_CMD_TIMEOUT_TABLE_SIZE] = {
> .timeouts = ata_eh_other_timeouts, },
> { .commands = CMDS(ATA_CMD_FLUSH, ATA_CMD_FLUSH_EXT),
> .timeouts = ata_eh_flush_timeouts },
> + { .commands = CMDS(ATA_CMD_READ_LOG_DMA_EXT, ATA_CMD_READ_LOG_EXT),
> + .timeouts = ata_eh_read_log_ext_timeouts },
> };
> #undef CMDS
>
> diff --git a/include/linux/libata.h b/include/linux/libata.h
> index b0bd30e..bcc68b3 100644
> --- a/include/linux/libata.h
> +++ b/include/linux/libata.h
> @@ -390,7 +390,7 @@ enum {
> /* This should match the actual table size of
> * ata_eh_cmd_timeout_table in libata-eh.c.
> */
> - ATA_EH_CMD_TIMEOUT_TABLE_SIZE = 6,
> + ATA_EH_CMD_TIMEOUT_TABLE_SIZE = 7,
>
> /* Horkage types. May be set by libata or controller on drives
> (some horkage may be drive/controller pair dependent */
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] libata: Increase the timeout of command ATA_CMD_READ_LOG_DMA_EXT
2020-10-15 1:57 [PATCH] libata: Increase the timeout of command ATA_CMD_READ_LOG_DMA_EXT chenxiang
2020-11-03 8:23 ` chenxiang (M)
@ 2021-03-30 1:54 ` chenxiang (M)
1 sibling, 0 replies; 3+ messages in thread
From: chenxiang (M) @ 2021-03-30 1:54 UTC (permalink / raw)
To: axboe, tj; +Cc: john.garry, linux-ide, linuxarm
Ping....
在 2020/10/15 9:57, chenxiang 写道:
> From: Xiang Chen <chenxiang66@hisilicon.com>
>
> ATA_CMD_READ_LOG_DMA_EXT command normally uses the common internal command
> default timeout (5s), and it is adequate in most situations. But it is not
> enough for some disks such as ST4000NM0035 which causes qc timeout
> sometimes as follows:
> ...
> [ 157.284284] sas: Enter sas_scsi_recover_host busy: 0 failed: 0
> [ 157.290124] sas: ata8: end_device-4:0:2: dev error handler
> [ 157.290128] sas: ata9: end_device-4:0:8: dev error handler
> [ 157.290131] sas: ata10: end_device-4:0:11: dev error handler
> [ 164.588391] ata9.00: qc timeout (cmd 0x47)
> [ 164.588393] ata8.00: qc timeout (cmd 0x47)
> [ 164.588395] ata10.00: qc timeout (cmd 0x47)
> [ 166.582572] ata10.00: READ LOG DMA EXT failed, trying PIO
> [ 166.587960] ata10.00: NCQ Send/Recv Log not supported
> [ 166.592997] ata10.00: ATA-10: ST4000NM0035-1V4107, TN03, max UDMA/133
> [ 166.599414] ata10.00: 7814037168 sectors, multi 0: LBA48 NCQ (depth 32)
> [ 166.606011] ata10.00: failed to set xfermode (err_mask=0x40)
> [ 166.659253] ata8.00: READ LOG DMA EXT failed, trying PIO
> [ 166.664559] ata8.00: NCQ Send/Recv Log not supported
> [ 166.669506] ata8.00: ATA-10: ST4000NM0035-1V4107, TN03, max UDMA/133
> [ 166.675837] ata8.00: 7814037168 sectors, multi 0: LBA48 NCQ (depth 32)
> [ 166.682343] ata8.00: failed to set xfermode (err_mask=0x40)
> [ 166.743458] ata9.00: READ LOG DMA EXT failed, trying PIO
> [ 166.748765] ata9.00: NCQ Send/Recv Log not supported
> [ 166.753713] ata9.00: ATA-10: ST4000NM0035-1V4107, TN03, max UDMA/133
> [ 166.760045] ata9.00: 7814037168 sectors, multi 0: LBA48 NCQ (depth 32)
> [ 166.766553] ata9.00: failed to set xfermode (err_mask=0x40)
> [ 168.886378] ata10.00: configured for UDMA/133
> [ 168.999943] ata8.00: configured for UDMA/133
> [ 169.050801] ata9.00: configured for UDMA/133
> [ 169.055071] sas: --- Exit sas_scsi_recover_host: busy: 0 failed: 0 tries: 1
> ...
> Actually it may takes about 9s to complete the command for disk ST4000NM0035
> sometimes. So to avoid the issue, increase the timeout of command
> ATA_CMD_READ_LOG_DMA_EXT from 5s to 15s.
>
> Signed-off-by: Xiang Chen <chenxiang66@hisilicon.com>
> Reviewed-by: John Garry <john.garry@huawei.com>
> ---
> drivers/ata/libata-eh.c | 9 +++++++++
> include/linux/libata.h | 2 +-
> 2 files changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/ata/libata-eh.c b/drivers/ata/libata-eh.c
> index 474c6c3..6db9474 100644
> --- a/drivers/ata/libata-eh.c
> +++ b/drivers/ata/libata-eh.c
> @@ -100,6 +100,13 @@ static const unsigned long ata_eh_flush_timeouts[] = {
> ULONG_MAX,
> };
>
> +static const unsigned long ata_eh_read_log_ext_timeouts[] = {
> + 15000,
> + 15000,
> + 30000,
> + ULONG_MAX,
> +};
> +
> static const unsigned long ata_eh_other_timeouts[] = {
> 5000, /* same rationale as identify timeout */
> 10000, /* ditto */
> @@ -139,6 +146,8 @@ ata_eh_cmd_timeout_table[ATA_EH_CMD_TIMEOUT_TABLE_SIZE] = {
> .timeouts = ata_eh_other_timeouts, },
> { .commands = CMDS(ATA_CMD_FLUSH, ATA_CMD_FLUSH_EXT),
> .timeouts = ata_eh_flush_timeouts },
> + { .commands = CMDS(ATA_CMD_READ_LOG_DMA_EXT, ATA_CMD_READ_LOG_EXT),
> + .timeouts = ata_eh_read_log_ext_timeouts },
> };
> #undef CMDS
>
> diff --git a/include/linux/libata.h b/include/linux/libata.h
> index b0bd30e..bcc68b3 100644
> --- a/include/linux/libata.h
> +++ b/include/linux/libata.h
> @@ -390,7 +390,7 @@ enum {
> /* This should match the actual table size of
> * ata_eh_cmd_timeout_table in libata-eh.c.
> */
> - ATA_EH_CMD_TIMEOUT_TABLE_SIZE = 6,
> + ATA_EH_CMD_TIMEOUT_TABLE_SIZE = 7,
>
> /* Horkage types. May be set by libata or controller on drives
> (some horkage may be drive/controller pair dependent */
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-03-30 1:55 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-15 1:57 [PATCH] libata: Increase the timeout of command ATA_CMD_READ_LOG_DMA_EXT chenxiang
2020-11-03 8:23 ` chenxiang (M)
2021-03-30 1:54 ` chenxiang (M)
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.