mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Neil Conway" <nconway_kernel@yahoo.co.uk>
To: Andre Hedrick <andre@linux-ide.org>
Cc: linux-kernel@vger.kernel.org
Subject: Re: PATCH, IDE corruption, 2.4.18
Date: Sun, 5 May 2002 03:10:11 +0100 (BST)	[thread overview]
Message-ID: <20020505021011.52472.qmail@web21507.mail.yahoo.com> (raw)
In-Reply-To: <Pine.LNX.4.10.10205041556460.19145-100000@master.linux-ide.org>

Ah good stuff, an honest-to-god expert with brains for the picking :)

 --- Andre Hedrick <andre@linux-ide.org> wrote: > 
> It is a viable check but it the need for it shows another problem in
> the
> code.  In the IDEDMA recovery kludges it must be losing the the
> hwgroup
> busy state.

Have I misunderstood then?  My understanding was this:
ide_config_drive_speed is doing a SELECT_DRIVE, which is quite liable
to happen during a DMA transfer to/from the other unit on the hwif, and
thus screw up that transfer.

The recovery or not (from the screw-up) is a secondary factor (and it
would be nice if such recoveries worked) but the primary fix is surely
to make sure we don't gratuitously screw up transfers in the first
place.

Or have I just demonstrated my utter ignorance of IDE?

Neil
PS: a secondary issue is that after ide_config_drive_speed buggers up a
transfer, the selected drive is NOT the one that ide_intr expects to be
selected.  I haven't yet got any clue if this is also relevant though.


__________________________________________________
Do You Yahoo!?
Everything you'll ever need on one web page
from News and Sport to Email and Music Charts
http://uk.my.yahoo.com

  reply	other threads:[~2002-05-05  2:10 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-05-04 12:15 Neil Conway
2002-05-04 22:58 ` Andre Hedrick
2002-05-05  2:10   ` Neil Conway [this message]
2002-05-05  0:22 ` Mike Fedyk
2002-05-05  0:47   ` Andre Hedrick
2002-05-05  1:54   ` Neil Conway
2002-05-05  7:36     ` Mike Fedyk
2002-05-05  9:44       ` Neil Conway
  -- strict thread matches above, loose matches on Subject: below --
2002-05-05 15:49 Bartlomiej Zolnierkiewicz
2002-05-05 15:04 ` Martin Dalecki
2002-05-05 20:44 ` Neil Conway
2002-05-06  0:23   ` Mike Fedyk
2002-05-03 20:39 Neil Conway
2002-05-03 20:36 Neil Conway

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=20020505021011.52472.qmail@web21507.mail.yahoo.com \
    --to=nconway_kernel@yahoo.co.uk \
    --cc=andre@linux-ide.org \
    --cc=linux-kernel@vger.kernel.org \
    /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®