LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Mark Lord <lkml@rtr.ca>
To: Sergei Shtylyov <sshtylyov@ru.mvista.com>
Cc: Mark Lord <liml@rtr.ca>, Alan Cox <alan@lxorguk.ukuu.org.uk>,
	Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>,
	Suleiman Souhlal <ssouhlal@freebsd.org>,
	linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/3] Make the IDE DMA timeout modifiable
Date: Fri, 13 Jul 2007 15:46:14 -0400	[thread overview]
Message-ID: <4697D686.6090104@rtr.ca> (raw)
In-Reply-To: <4697D518.7010108@ru.mvista.com>

Sergei Shtylyov wrote:
> Mark Lord wrote:
> 
>>>    The original question concerned specifically the DMA command 
>>> timeout which is twice more than the usual one, WAIT_CMD (10 seconds).
..
>> When a drive is in standby, we don't send it anything special to wake up.
>> So even DMA commands have to have a long enough timeout to allow
>> for spinning up.
> 
>    Yes, but why *twice* as long as the others?

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.

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
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).

I'm guessing that this was the original concern of mine,
way back in 1994 or whenever it was.

Cheers

  reply	other threads:[~2007-07-13 19:46 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-21  1:19 Suleiman Souhlal
2007-02-21  1:44 ` Bartlomiej Zolnierkiewicz
2007-02-21  2:13   ` Suleiman Souhlal
2007-02-21  2:42     ` Bartlomiej Zolnierkiewicz
2007-06-12 19:11       ` Sergei Shtylyov
2007-06-15 23:23         ` Bartlomiej Zolnierkiewicz
2007-07-10 19:10           ` Sergei Shtylyov
2007-07-13 15:16             ` Alan Cox
2007-07-13 15:12               ` Mark Lord
2007-07-13 15:21                 ` Sergei Shtylyov
2007-07-13 19:36                   ` Mark Lord
2007-07-13 19:40                     ` Sergei Shtylyov
2007-07-13 19:46                       ` Mark Lord [this message]
2007-07-13 20:03                         ` Sergei Shtylyov
2007-07-13 21:00                           ` Mark Lord
2007-07-16 19:29                             ` Rogier Wolff
2007-07-17 14:18                               ` Sergei Shtylyov
2007-07-13 20:08                     ` Sergei Shtylyov
2007-07-13 20:54                       ` Mark Lord
2007-07-13 21:00                         ` Mark Lord

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4697D686.6090104@rtr.ca \
    --to=lkml@rtr.ca \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=bzolnier@gmail.com \
    --cc=liml@rtr.ca \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sshtylyov@ru.mvista.com \
    --cc=ssouhlal@freebsd.org \
    --subject='Re: [PATCH 1/3] Make the IDE DMA timeout modifiable' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).