mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Alan Cox <alan@lxorguk.ukuu.org.uk>
To: martin@dalecki.de
Cc: Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl>,
	Pete Zaitcev <zaitcev@redhat.com>,
	Bill Davidsen <davidsen@tmr.com>,
	linux-kernel@vger.kernel.org
Subject: Re: Safety of IRQ during i/o
Date: 25 Jul 2002 11:26:24 +0100	[thread overview]
Message-ID: <1027592784.9489.11.camel@irongate.swansea.linux.org.uk> (raw)
In-Reply-To: <3D3FAEB1.6070704@evision.ag>

On Thu, 2002-07-25 at 08:54, Marcin Dalecki wrote:
> > Yup, for PIO unmask (if possible) is a must.
> 
> It's even for DMA a good thing, since the IRQ handler in question can
> reenter the RQ handler. The invention of the not unmasking
> behaviour in Linux is the result of some not entierly ATA-2 compliant
> devices long long time ago gone. Basically XT disks on PC. They did have 
> the habbit of splewing IRQs too early for command ACK.

There are also some older systems where if the block transfer of the IDE
data didn't keep up with the controller instead of handshaking properly
it kind of dribbled random numbers onto the disk.

Unless anyone knows of PCI era devices with this problem I would be
inclined to agree that we should default to IRQ unmasking in the 2.5 IDE
code if the IDE controller is PCI.

For old ISA/VLB controllers its safer left as is, and nobody running a
machine like that can realistically expect good performance without hand
tuning stuff anyway


  reply	other threads:[~2002-07-25  9:10 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <mailman.1027541521.16533.linux-kernel2news@redhat.com>
2002-07-24 22:37 ` Pete Zaitcev
2002-07-24 22:42   ` Bartlomiej Zolnierkiewicz
2002-07-25  7:54     ` Marcin Dalecki
2002-07-25 10:26       ` Alan Cox [this message]
2002-07-25  9:34         ` Marcin Dalecki
2002-07-25 11:15           ` Roger Larsson
2002-07-25 11:17         ` Bill Davidsen
2002-07-26  0:34           ` Marcin Dalecki
2002-07-24 19:58 Bill Davidsen

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=1027592784.9489.11.camel@irongate.swansea.linux.org.uk \
    --to=alan@lxorguk.ukuu.org.uk \
    --cc=B.Zolnierkiewicz@elka.pw.edu.pl \
    --cc=davidsen@tmr.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=martin@dalecki.de \
    --cc=zaitcev@redhat.com \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®