Re: [PATCH] mmc_spi: use EILSEQ for possible transmission errors

From: Pierre Ossman
Date: Wed May 20 2009 - 06:09:16 EST


On Wed, 20 May 2009 02:20:46 -0700
David Brownell <david-b@xxxxxxxxxxx> wrote:

> On Wednesday 20 May 2009, Wolfgang MÃes wrote:
> > R1_SPI_COM_CRC is a transmission error - no doubt. Mapping to EILSEQ is OK.
> >
> > R1_SPI_ERASE_SEQ and R1_SPI_ERASE_RESET are responses to a sector erase
> > command. block.c is not sending such commands, so block.c can safely assume
> > that each such response is a transmission error.
>
> Hmm, would hope that's not a long-term plan. Remember that the
> firmware in the card can leverage "that's erased" knowledge for
> things like wear leveling. SSDs and other storage devices would
> likewise benefit from such knowledge. I'm quite certain there's
> been discussion about adding support for that in the block layer.
>

It's already in AFAIK. I even had code for hooking it up to mmc_block.
But it didn't produce any measurable results so I never merged it.

Rgds
--
-- Pierre Ossman

WARNING: This correspondence is being monitored by the
Swedish government. Make sure your server uses encryption
for SMTP traffic and consider using PGP for end-to-end
encryption.

Attachment: signature.asc
Description: PGP signature