ata: libata-core: do not retry reading the log on timeout
authorNiklas Cassel <niklas.cassel@wdc.com>
Fri, 7 Oct 2022 13:23:40 +0000 (15:23 +0200)
committerDamien Le Moal <damien.lemoal@opensource.wdc.com>
Wed, 19 Oct 2022 04:46:08 +0000 (13:46 +0900)
commit5122e53ee784b9a2a2f40d34222751b6b4ab076c
tree7a636ece9bb4c0c65920727a68adfc8dd521e324
parent7834589e81bcf5fe4e7ab60430670c9245183de0
ata: libata-core: do not retry reading the log on timeout

ata_read_log_page() first tries to read the log using READ LOG DMA EXT.
If that fails it will instead try to read the log using READ LOG EXT.

ata_exec_internal_sg() is synchronous, so it will wait for the command to
finish. If we actually got an error back from the device, it is correct
to retry. However, if the command timed out, ata_exec_internal_sg() will
freeze the port.

There is no point in retrying if the port is frozen, as
ata_exec_internal_sg() will return AC_ERR_SYSTEM on a frozen port,
without ever sending the command down to the drive.

Therefore, avoid retrying if the first command froze the port, as that
will result in a misleading AC_ERR_SYSTEM error print, instead of printing
the error that actually caused the port to be frozen (AC_ERR_TIMEOUT).

Signed-off-by: Niklas Cassel <niklas.cassel@wdc.com>
Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
drivers/ata/libata-core.c