Re: [PATCH] [MMC] Multi sector write transfers

[Pierre Ossman <drzeus-list@drzeus.cx>]

Andrew Morton wrote:
> Russell is moving away from mmc maintainership so we'll need to do
> something different here.
> 
> The patches look good to my untrained eye, except...
> 
> On Sun, 01 Oct 2006 14:42:57 +0200
> Pierre Ossman <drzeus@drzeus.cx> wrote:
> 
>> SD cards extend the protocol by allowing the host to query a card how many
>> blocks were successfully stored on the medium. This allows us to safely write
>> chunks of blocks at once.
> 
> I recall Russell nacked multisector mmc-writing when it came up six or
> twelve months ago.  I don't recall the exact details - lack of trust in
> manufacturers supporting it correctly?
> 
> Is that concern relevant to this patch?

The concern then was that we enabled multi-sector writes across the
board. Russell's concern was that some host controllers weren't properly
reporting back how far they had gotten when a failure occurred. We now
have two solutions to this:

 * Let the drivers set a flag, indicating that the controller properly
reports the number of sent blocks, even when a fault occurs. Russell put
together a patch for this and it is already in Linus' tree.

 * Ask the card, not the controller, how many blocks got there ok. This
is only supported by SD though, so MMC still use only one block per
write (except for the case above).

The only problem I see is if the card has decided to completely shut
down so it's impossible to query the number of blocks written. In this
case, reporting correctly to the upper layers might not be a big issue
(as they can't take any action on the card anyway), so I do not see it
as a problem in practice. I have yet to see any comments on the patch
though (other than end users that like the speed boost).

Rgds
Pierre
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/