]> git.baikalelectronics.ru Git - kernel.git/commit
mmc: block: fix read single on recovery logic
authorChristian Löhle <CLoehle@hyperstone.com>
Fri, 4 Feb 2022 15:11:37 +0000 (15:11 +0000)
committerUlf Hansson <ulf.hansson@linaro.org>
Tue, 8 Feb 2022 15:04:49 +0000 (16:04 +0100)
commit9282e96096984c78d25a296720c58460cd7d9889
tree81ddbb9fa104e7401bdffd1f498e38e64239aef6
parent6ba326228f42b846b8462ff17866a8bc965c3366
mmc: block: fix read single on recovery logic

On reads with MMC_READ_MULTIPLE_BLOCK that fail,
the recovery handler will use MMC_READ_SINGLE_BLOCK for
each of the blocks, up to MMC_READ_SINGLE_RETRIES times each.
The logic for this is fixed to never report unsuccessful reads
as success to the block layer.

On command error with retries remaining, blk_update_request was
called with whatever value error was set last to.
In case it was last set to BLK_STS_OK (default), the read will be
reported as success, even though there was no data read from the device.
This could happen on a CRC mismatch for the response,
a card rejecting the command (e.g. again due to a CRC mismatch).
In case it was last set to BLK_STS_IOERR, the error is reported correctly,
but no retries will be attempted.

Fixes: 1095e107a07d3e ("mmc: block: Add blk-mq support")
Cc: stable@vger.kernel.org
Signed-off-by: Christian Loehle <cloehle@hyperstone.com>
Reviewed-by: Adrian Hunter <adrian.hunter@intel.com>
Link: https://lore.kernel.org/r/bc706a6ab08c4fe2834ba0c05a804672@hyperstone.com
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/core/block.c