Hello,

While using the mmc_spi driver occasionally errors like this popped up:

mmcblk0: error -84 transferring data
end_request: I/O error, dev mmcblk0, sector 581756

I looked on the Internet for occurences of the same problem and came across a 
helpful post [1]. It includes source code to reproduce the bug. There is also 
an analysis about the cause. During transmission data in the supplied buffer is 
beeing modified. Thus the previously calculated check sum is not correct 
anymore.

After some digging I found out that device drivers are supposed to report they 
need stable writes.  To fix this I set the appropriate flag at queue 
initialization if CRC checksumming is enabled for that SPI host.

[1]
https://groups.google.com/forum/#!msg/sim1/gLlzWeXGFr8/KevXinUXfc8J

Signed-off-by: Andreas Koop <andreas.koop <at> zf.com>
---
diff --git a/drivers/mmc/card/queue.c b/drivers/mmc/card/queue.c
index fa9632e..3fdef37 100644
--- a/drivers/mmc/card/queue.c
+++ b/drivers/mmc/card/queue.c
@@ -207,6 +207,9 @@ int mmc_init_queue(struct mmc_queue *mq, struct mmc_card 
*card,
        mq->mqrq_prev = mqrq_prev;
        mq->queue->queuedata = mq;
 
+       if (mmc_host_is_spi(host) && host->use_spi_crc)
+               mq->queue->backing_dev_info.capabilities |= 
BDI_CAP_STABLE_WRITES;
+
        blk_queue_prep_rq(mq->queue, mmc_prep_request);
        queue_flag_set_unlocked(QUEUE_FLAG_NONROT, mq->queue);
        if (mmc_can_erase(card))
--
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to