Re: [PATCH 1/3] Make the IDE DMA timeout modifiable

[Mark Lord]

Sergei Shtylyov wrote:
> Mark Lord wrote:
> ..
> > I would guess simply because DMA has to transfer up to 256 sectors of 
> > data,
> > possibly with sector reallocations, in addition to waiting for the drive
> > to spin up.  Other commands don't.
>
>    What?! PIO commands don't have to do this as well? :-)

"Other commands" don't have to transfer up to 256 sectors before interrupting.

> > At the time that was coded (?), I suspect that PIO READ/WRITE commands
> > were fed data as it became available to/from the drive.  This may or may
>
>    I don't see *any* difference with the DMA commands here -- the drive 
> has always been free to assert and deassert DMARQ at its well.

Yeah, but that's not what our software timeout is for.
The timeout is for the next interrupt, which will only 
be given (for DMA) once the drive completes *all* transfers.

> > not still be the case, but it does imply that they don't need to hang
> > around as long on the timeouts as do DMA commands (which have to wait
> > for *everything* to be transferred).
>
>    Ah, that makes sense -- during PIO interrupts happen a lot more often.
> 20 secs still seem to be too much.

I don't think so, even for modern drives.
Figure 8-10 seconds max for spin-up,
plus 6-9 seconds to do a sector re-assignment
or retries on a bad block (a measured *real-life* value).

That adds up to 14-19 seconds, so 20 seconds is probably good.

Still, this does need to be adjustable for faster (CF) devices,
and slower (optical/tape) devices, rather than just a single
set of fixed timeout values.

Cheers

> PS: Your mail server still keeps bouncing me off. :-(

PS: your mail server still looks like a spammer, because it is
(1) missing a reverse DNS entry for 63.81.120.155,
and (2) is not RFC compliant on the SMTP "HELO" message
(bad hostname given there).
-
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/