[lkml]   [2001]   [May]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [patch]: ide dma timeout retry in pio
    On Mon, May 28 2001, Mark Hahn wrote:
    > > request, when we hit a dma timout. In this case, what we really want to
    > > do is retry the request in pio mode and revert to normal dma operations
    > > later again.
    > really? do we know the nature of the DMA engine problem well enough?
    > is there a reason to believe that it'll work better "later"?
    > I guess I was surprised at resorting to PIO - couldn't we just
    > break the request up into smaller chunks, still using DMA?

    That is indeed possible, it will require some surgery to the dma request
    path though. IDE has no concept of doing part of a request for dma
    currently, it's an all-or-nothing approach. That's why it falls back to
    pio right now.

    > I seem to recall Andre saying that the problem arises when the
    > ide DMA engine looses PCI arbitration during a burst. shorter
    > bursts would seem like the best workaround if this is the problem...

    It's worth a shot. My patch was not meant as the end-all solution,
    however we need something _now_. Loosing sectors is not funny.
    Dynamically limiting general request size for to make dma work is a
    piece of cake, that'll be about a one-liner addition to the current
    patch. So the logic could be something of the order of:

    - 1st dma timeout
    - scale max size down from 128kB (127.5kB really) to half that
    - things aren't working, 2nd dma timeout. Scale down to 32kB.

    and so forth, revert to pio and reset full size if it's really no good.
    If limiting transfer sizes solves the problem, this would be the way to
    go. I'll do another version that does this.

    Testers? Who has frequent ide dma timeout problems??

    > resorting to PIO would be such a shame, not only because it eats
    > CPU so badly, but also because it has no checksum like UDMA...

    Look at the patch -- we resort to pio for _one_ hunk. That's 8 sectors
    tops, then back to dma. Hardly a big issue.

    Jens Axboe

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:17    [W:0.023 / U:0.364 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site