public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Mel Gorman <mel@csn.ul.ie>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: petkovbb@gmail.com, sshtylyov@ru.mvista.com,
	linux-kernel@vger.kernel.org
Subject: Re: Is the change to IDE probing really necessary?
Date: Tue, 25 Nov 2008 00:36:45 +0000	[thread overview]
Message-ID: <20081125003644.GA16241@csn.ul.ie> (raw)
In-Reply-To: <20081125003002.5dcd9783@lxorguk.ukuu.org.uk>

On Tue, Nov 25, 2008 at 12:30:02AM +0000, Alan Cox wrote:
> > > The mask is set to 0 and then the master/slave are detected based upon
> > > the presence of PCI IDE devices on the master/slave ports.
> > > 
> > 
> > This detection was not happening but then again the proper driver was not
> > compiled. The .config I used was very minimal and lacked most drivers.
> 
> The lspci shows the correct thing happened. You have PCI devices and
> didn't load the right driver. No driver, no support, no suprise.
> 

Yeah, luck seemed to have been working up until this point :/. ide-generic
has been working on the machine since at least 2.6.23 for sure and probably
longer than that which is why it had the hallmarks of a regression even though
ide-generic appears to be doing something more safe and sensible these days.

> > It is certainly the case I was not using the correct driver. Using
> > CONFIG_BLK_DEV_PIIX instead of CONFIG_IDE_GENERIC makes everything work as
> > expected without having to pass special options.
> 
> It would do - and you don't want to use pata_legacy/ide-generic for PCI
> devices. There are several with quirks, some even corruptors that the
> legacy drivers do not know how to handle.
> 

Understood.

> So I think this is a case of 'I put best steam coal in my car and it
> didn't go'

Using piix, I won't have to call in Fred Flintstone to push it with his
legs instead.

Thanks

-- 
Mel Gorman
Part-time Phd Student                          Linux Technology Center
University of Limerick                         IBM Dublin Software Lab

  reply	other threads:[~2008-11-25  0:36 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-24 15:56 Is the change to IDE probing really necessary? Mel Gorman
2008-11-24 18:43 ` Mel Gorman
2008-11-24 18:54   ` Alan Cox
2008-11-24 23:09     ` Mel Gorman
2008-11-25  0:03       ` Alan Cox
2008-11-25  0:16         ` Mel Gorman
2008-11-25  0:30           ` Alan Cox
2008-11-25  0:36             ` Mel Gorman [this message]
2008-11-24 22:51 ` Borislav Petkov
2008-11-25  0:01   ` Mel Gorman
2008-11-25  7:21     ` Borislav Petkov
2008-11-25 10:45       ` Mel Gorman
     [not found] <bBM1f-6TS-19@gated-at.bofh.it>
     [not found] ` <bBOFD-1WX-21@gated-at.bofh.it>
     [not found]   ` <bBOYV-2w9-7@gated-at.bofh.it>
     [not found]     ` <bBSST-7ut-11@gated-at.bofh.it>
     [not found]       ` <bBTFg-8vL-15@gated-at.bofh.it>
     [not found]         ` <bBTOS-dn-3@gated-at.bofh.it>
     [not found]           ` <bBU8c-w0-1@gated-at.bofh.it>
2008-11-26 21:48             ` Bodo Eggert

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=20081125003644.GA16241@csn.ul.ie \
    --to=mel@csn.ul.ie \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=petkovbb@gmail.com \
    --cc=sshtylyov@ru.mvista.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