public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Padraig Brady <padraig.brady@corvil.com>
To: "Heater, Daniel (IndSys, GEFanuc, VMIC)" <Daniel.Heater@gefanuc.com>
Cc: "'Linux Kernel'" <linux-kernel@vger.kernel.org>,
	"'andre@linux-ide.org'" <andre@linux-ide.org>,
	"Warner, Bill (IndSys, GEFanuc, VMIC)" <Bill.Warner@gefanuc.com>
Subject: Re: IDE-flash device and hard disk on same controller
Date: Tue, 20 Aug 2002 09:44:10 +0100	[thread overview]
Message-ID: <3D62015A.4000707@corvil.com> (raw)
In-Reply-To: A9713061F01AD411B0F700D0B746CA6802FC1463@vacho6misge.cho.ge.com

Heater, Daniel (IndSys, GEFanuc, VMIC) wrote:
> The IDE driver, file ide-probe.c currently contains this test do prevent
> hard drives and IDE-flash devices (ex CompactFlash) from co-existing on the
> same IDE controller. 
> 
>         /*
>          * Prevent long system lockup probing later for non-existant
>          * slave drive if the hwif is actually a flash memory card of some
> variety:
>          */
>         if (drive_is_flashcard(drive)) {
>                 ide_drive_t *mate =
> &HWIF(drive)->drives[1^drive->select.b.unit];
>                 if (!mate->ata_flash) {
>                         mate->present = 0;
>                         mate->noprobe = 1;
>                 }
>         }
> 
> This test's assumption that a spinning hard drive cannot coexist on the same
> controller as an IDE-flash device is incorrect.  I have a working setup with
> such a configuration.  I don't think that the IDE subsystem should punish
> everyone because _some_ hardware cannot tolerate this configuration.
> 
> One solution may be to remove this test from the IDE subsystem and force
> users with buggy hardware to explicitly  disable probing for a second
> device.  I think the parameters hdx=none or hdx=noprobe should work for
> them.
> 
> Comments??

Mentioned several times (and there is a workaround), see:
http://marc.theaimsgroup.com/?l=linux-kernel&m=100446144028502&w=2

I really think some of the default CF logic is bogus.

Pádraig.


  parent reply	other threads:[~2002-08-20  8:40 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-08-19 18:31 IDE-flash device and hard disk on same controller Heater, Daniel (IndSys, GEFanuc, VMIC)
2002-08-19 19:21 ` Alan Cox
2002-08-20  8:44 ` Padraig Brady [this message]
  -- strict thread matches above, loose matches on Subject: below --
2002-08-20 19:52 Heater, Daniel (IndSys, GEFanuc, VMIC)
2002-08-20 21:57 ` Andre Hedrick
2002-08-20 22:00   ` Jeff Garzik
2002-08-20 22:28     ` Jeff Garzik
2002-08-21  6:36       ` Geert Uytterhoeven
2002-08-21  6:55         ` Andre Hedrick
2002-08-21  7:16           ` Geert Uytterhoeven
2002-08-22  5:34       ` Eric W. Biederman
2002-08-22 23:12         ` Jeff Garzik
2002-08-23  0:50           ` Eric W. Biederman
2002-08-23  1:26             ` Jeff Garzik
2002-08-23 11:03               ` Russell King
2002-08-23 11:41               ` Benjamin Herrenschmidt
2002-08-23 10:12                 ` Alan Cox
2002-08-23 13:09               ` Eric W. Biederman
2002-08-23  3:14             ` Andre Hedrick
2002-08-23 11:44               ` Benjamin Herrenschmidt
2002-08-24  2:02                 ` Jeff Garzik
2002-08-24  0:19                   ` Benjamin Herrenschmidt
2002-08-24  8:41                   ` Russell King
2002-08-24 20:28                     ` Andre Hedrick
2002-08-23  3:19             ` Andre Hedrick
2002-08-23  7:14               ` Helge Hafting
2002-08-23  7:49                 ` Andre Hedrick
2002-08-23 10:50               ` Geert Uytterhoeven
2002-08-23 11:08                 ` Russell King
2002-08-23 11:07               ` Russell King
2002-08-23 11:36           ` Benjamin Herrenschmidt
2002-08-22 13:41 ` bill davidsen
2002-08-21 13:22 Warner, Bill (IndSys, GEFanuc, VMIC)
2002-08-23  6:54 Adam J. Richter
2002-08-23  7:45 ` Andre Hedrick
2002-08-23 11:10   ` Russell King
2002-08-23 17:08     ` Andre Hedrick
2002-08-23 17:42       ` Andre Hedrick
2002-08-23  8:31 Adam J. Richter
2002-08-23  8:59 ` Andre Hedrick
2002-08-23  9:54 ` Andries Brouwer
2002-08-23 10:14 Adam J. Richter
2002-08-23 10:48 Adam J. Richter

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=3D62015A.4000707@corvil.com \
    --to=padraig.brady@corvil.com \
    --cc=Bill.Warner@gefanuc.com \
    --cc=Daniel.Heater@gefanuc.com \
    --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