Re: Problem with IDE driver not claiming the disk

[Scott James Remnant <scott@ubuntu.com>]

[-- Attachment #1: Type: text/plain, Size: 1600 bytes --]

On Tue, 2005-12-06 at 09:23 +0100, Bartlomiej Zolnierkiewicz wrote:

> On 12/6/05, Scott James Remnant <scott@ubuntu.com> wrote:
> > I was first hoping it was a glaringly obvious thing we were doing wrong,
> > and wouldn't come to comparing kernel code;  the kernel is the current
> > Ubuntu one, it's based off 2.6.15 -- our patches are all in git:
> >
> > http://www.kernel.org/git/?p=linux/kernel/git/bcollins/ubuntu-2.6.git
> >
> > I'm not best placed to answer the patches that are in the exact version
> > I used ("our" 2.6.15-6.8), Ben Collins is (Cc'd).
> 
> You have IDE patches in your tree, please try vanilla 2.6.15.
> 
Yeah, one was compiling over night (still waiting for my AMD64 beast to
arrive) and I tried it out this morning ... I see your point, it works
exactly as advertised in vanilla 2.6.15-rc5 -- loading the real IDE
driver finds the devices itself, and there's no need to load
ide-generic.

> Changes to setup-pci.c and ide-generic.c in the following patch are wrong:
> http://www.kernel.org/git/?p=linux/kernel/git/bcollins/ubuntu-2.6.git;a=commitdiff;h=149744c1475cdaf2c6babfd92f163cca8f10c540;hp=5a373d68631531fff09047364be4565d67335f08
> 
> It seems that with this patch 'pre_init' in setup-pci.c. will
> become '0' (it equals '1' initially) only after ide-generic is loaded...
> and ide_pci_register_driver() checks 'pre_init' value.
> 
> Sigh, this patch was NACK-ed upstream for some (important) reasons...
> 
I'll go bludgeon our kernel guy for a bit then!

Thanks for your time,

Scott
-- 
Scott James Remnant
scott@ubuntu.com

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]