Re: [RFC v2 5/5] Atmel MCI: Driver for Atmel on-chip MMC controllers

[Dan Williams]

On Thu, Feb 14, 2008 at 1:36 AM, Haavard Skinnemoen
<hskinnemoen@xxxxxxxxx> wrote:
[..]
>
>  > 2/ Fall back to PIO for a few transfers
>
>  Which means killing performance for large transfers. Not really an
>  option.
>
>  It's ok to use PIO for small and/or odd transfers like "read 2 bytes
>  from this SDIO register", something that the mmc-block driver would
>  never ask us to do. Using PIO for huge block data transfers will really
>  hurt.
>

Except your testing has already shown that running out of descriptors
rarely, if ever happens.  It just becomes an exercise in tuning the
pool size, and letting this simple fall back mechanism be a safety net
for the corner cases.

>
>  > Descriptor availability is improved if the code interleaves allocation
>  > and submission.  Currently it looks like we wait until all descriptors
>  > for the scatterlist are allocated before we start submitting.
>
>  No, none of the descriptors will appear before the command has been
>  sent, the card has responded, and a full block of data has been
>  transferred. I suppose we could send the command earlier, but I don't
>  think it will help a lot and it will complicate error handling.
>
>  There may be room for improvement though. The current scheme of
>  splitting DMA preparation and submission was initially used because
>  older versions of the controller would instantly fail with an overrun
>  or underrun if the data wasn't available immediately when the command
>  had been sent. Sending the command earlier and doing interleaved
>  allocation and submission might improve performance a bit.
>
>  I think I'll try triggering the mmc tasklet from the DMA callback for
>  now. Scheduling a tasklet from another tasklet should work just fine,
>  right?
>

Yeah that should work because you are no longer under the channel lock.

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