2002-04-15 09:23:47

by Morten Helgesen

[permalink] [raw]
Subject: [COMMENTS IDE 2.5] - "idebus=66" in 2.5.8 results in "ide_setup: idebus=66 -- BAD OPTION"

Hey Martin (and the rest of you)

Been testing 2.5.8 for an hour or so, and noticed the following :

[11:14][root@sexything:/home/admin]$ dmesg | grep idebus
Kernel command line: BOOT_IMAGE=2.5.8-without-TCQ ro root=303 video=matrox:vesa:0x118 idebus=66 profile=2
ide_setup: idebus=66 -- BAD OPTION

With 2.5.8-pre3 :

[11:15][root@sexything:/home/admin]$ cat ./IDEBUS
Kernel command line: BOOT_IMAGE=2.5.8-pre3 ro root=303 video=matrox:vesa:0x118 idebus=66 profile=2
ide_setup: idebus=66

Now, I do not know the reasons for changing the code that handles "idebus=" stuff in ide.c (except from the fact
that it now looks cleaner :) - just thought I should let you know. I do not have the time right now to hunt down
the bug(?) and cook up a patch, but if you want me to, I`ll do it later today.

One more thing, Martin - I compiled a 2.5.8 with TCQ enabled (yep, I`m aware of the fact that this one is _really_ alpha :),
and tested it for, oh ... 10 minutes ... the system gave me all sorts of funny responses - segfaults,
"inconsistency in ld.so" and so on ... would you like me to collect 'funnies' and send them to you ? If so, just
give me a hint.

== Morten

--

"Livet er ikke for nybegynnere" - sitat fra en klok person.

mvh
Morten Helgesen
UNIX System Administrator & C Developer
Nextframe AS
[email protected] / 93445641
http://www.nextframe.net


2002-04-15 09:27:34

by Martin Dalecki

[permalink] [raw]
Subject: Re: [COMMENTS IDE 2.5] - "idebus=66" in 2.5.8 results in "ide_setup: idebus=66 -- BAD OPTION"

Morten Helgesen wrote:
> Hey Martin (and the rest of you)

> Now, I do not know the reasons for changing the code that handles "idebus=" stuff in ide.c (except from the fact

Should be an off by one error there.

> that it now looks cleaner :) - just thought I should let you know. I do not have the time right now to hunt down
> the bug(?) and cook up a patch, but if you want me to, I`ll do it later today.

I would love if you could have a look at it...

> One more thing, Martin - I compiled a 2.5.8 with TCQ enabled (yep, I`m aware of the fact that this one is _really_ alpha :),
> and tested it for, oh ... 10 minutes ... the system gave me all sorts of funny responses - segfaults,
> "inconsistency in ld.so" and so on ... would you like me to collect 'funnies' and send them to you ? If so, just
> give me a hint.

Thats mostly Jens stuff...

2002-04-15 09:34:37

by Morten Helgesen

[permalink] [raw]
Subject: Re: [COMMENTS IDE 2.5] - "idebus=66" in 2.5.8 results in "ide_setup: idebus=66 -- BAD OPTION"

On Mon, Apr 15, 2002 at 10:25:20AM +0200, Martin Dalecki wrote:
> Morten Helgesen wrote:
> >Hey Martin (and the rest of you)
>
> >Now, I do not know the reasons for changing the code that handles "idebus="
> >stuff in ide.c (except from the fact
>
> Should be an off by one error there.

yep - that`s what struck me.

>
> >that it now looks cleaner :) - just thought I should let you know. I do not
> >have the time right now to hunt down
> >the bug(?) and cook up a patch, but if you want me to, I`ll do it later
> >today.
>
> I would love if you could have a look at it...

sure - I`ll have a look at it in an hour or two.

>
> >One more thing, Martin - I compiled a 2.5.8 with TCQ enabled (yep, I`m
> >aware of the fact that this one is _really_ alpha :), and tested it for, oh
> >... 10 minutes ... the system gave me all sorts of funny responses -
> >segfaults, "inconsistency in ld.so" and so on ... would you like me to
> >collect 'funnies' and send them to you ? If so, just give me a hint.
>
> Thats mostly Jens stuff...

ok - Jens ?

I forgot to mention that 2.5.8 _without_ TCQ works like a charm - that`s why I made TCQ my main suspect :)

>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/

== Morten

--

"Livet er ikke for nybegynnere" - sitat fra en klok person.

mvh
Morten Helgesen
UNIX System Administrator & C Developer
Nextframe AS
[email protected] / 93445641
http://www.nextframe.net

2002-04-15 09:40:42

by Jens Axboe

[permalink] [raw]
Subject: Re: [COMMENTS IDE 2.5] - "idebus=66" in 2.5.8 results in "ide_setup: idebus=66 -- BAD OPTION"

On Mon, Apr 15 2002, Morten Helgesen wrote:
> > >One more thing, Martin - I compiled a 2.5.8 with TCQ enabled (yep, I`m
> > >aware of the fact that this one is _really_ alpha :), and tested it for, oh
> > >... 10 minutes ... the system gave me all sorts of funny responses -
> > >segfaults, "inconsistency in ld.so" and so on ... would you like me to
> > >collect 'funnies' and send them to you ? If so, just give me a hint.
> >
> > Thats mostly Jens stuff...
>
> ok - Jens ?

Would love to see it! Also send me the ide bootup messages. Note that
2.5.8 has at least on irq race with tcq dma that I've fixed, but I'd
like to see the setup anyways. Did you get any dmesg errors?

--
Jens Axboe

2002-04-15 12:15:54

by Morten Helgesen

[permalink] [raw]
Subject: Re: [COMMENTS IDE 2.5] - "idebus=66" in 2.5.8 results in "ide_setup: idebus=66 -- BAD OPTION"

Hey again, Martin :)

What do you think about the following :

--- clean-linux-2.5.8/drivers/ide/ide.c Sun Apr 14 21:18:52 2002
+++ patched-linux-2.5.8/drivers/ide/ide.c Mon Apr 15 14:09:18 2002
@@ -3120,7 +3120,7 @@
/*
* Look for bus speed option: "idebus="
*/
- if (strncmp(s, "idebus", 6)) {
+ if (strncmp(s, "idebus", 6) == 0) {
if (match_parm(&s[6], NULL, vals, 1) != 1)
goto bad_option;
if (vals[0] >= 20 && vals[0] <= 66) {

gives :

Kernel command line: BOOT_IMAGE=2.5.8-without-TCQ ro root=303 video=matrox:vesa:0x118 idebus=66 profile=2
ide_setup: idebus=66
ide: system bus speed 66MHz

works like a charm :)

== Morten

On Mon, Apr 15, 2002 at 10:25:20AM +0200, Martin Dalecki wrote:
> Morten Helgesen wrote:
> >Hey Martin (and the rest of you)
>
> >Now, I do not know the reasons for changing the code that handles "idebus="
> >stuff in ide.c (except from the fact
>
> Should be an off by one error there.
>
> >that it now looks cleaner :) - just thought I should let you know. I do not
> >have the time right now to hunt down
> >the bug(?) and cook up a patch, but if you want me to, I`ll do it later
> >today.
>
> I would love if you could have a look at it...
>
> >One more thing, Martin - I compiled a 2.5.8 with TCQ enabled (yep, I`m
> >aware of the fact that this one is _really_ alpha :), and tested it for, oh
> >... 10 minutes ... the system gave me all sorts of funny responses -
> >segfaults, "inconsistency in ld.so" and so on ... would you like me to
> >collect 'funnies' and send them to you ? If so, just give me a hint.
>
> Thats mostly Jens stuff...
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/

== Morten

--

"Livet er ikke for nybegynnere" - sitat fra en klok person.

mvh
Morten Helgesen
UNIX System Administrator & C Developer
Nextframe AS
[email protected] / 93445641
http://www.nextframe.net

2002-04-15 12:26:40

by Martin Dalecki

[permalink] [raw]
Subject: Re: [COMMENTS IDE 2.5] - "idebus=66" in 2.5.8 results in "ide_setup: idebus=66 -- BAD OPTION"

Morten Helgesen wrote:
> Hey again, Martin :)
>
> What do you think about the following :

What I think? I think it's abviously right... Thank you very much
and sorry for the inconvenience!

And judging from the other typos fixed in patch number 34 I think
that I should stick to my "principles" and don't write code past 02:00 o'clock
... :-).

>
> --- clean-linux-2.5.8/drivers/ide/ide.c Sun Apr 14 21:18:52 2002
> +++ patched-linux-2.5.8/drivers/ide/ide.c Mon Apr 15 14:09:18 2002
> @@ -3120,7 +3120,7 @@
> /*
> * Look for bus speed option: "idebus="
> */
> - if (strncmp(s, "idebus", 6)) {
> + if (strncmp(s, "idebus", 6) == 0) {
> if (match_parm(&s[6], NULL, vals, 1) != 1)
> goto bad_option;
> if (vals[0] >= 20 && vals[0] <= 66) {
>
> gives :
>
> Kernel command line: BOOT_IMAGE=2.5.8-without-TCQ ro root=303 video=matrox:vesa:0x118 idebus=66 profile=2
> ide_setup: idebus=66
> ide: system bus speed 66MHz
>
> works like a charm :)

2002-04-15 17:20:21

by Meelis Roos

[permalink] [raw]
Subject: Re: [COMMENTS IDE 2.5] - "idebus=66" in 2.5.8 results in "ide_setup: idebus=66 -- BAD OPTION"

MH> Kernel command line: BOOT_IMAGE=2.5.8-without-TCQ ro root=303 video=matrox:vesa:0x118 idebus=66 profile=2
MH> ide_setup: idebus=66
MH> ide: system bus speed 66MHz

MH> works like a charm :)

Do you really have an IDE controller that does 66 MHz PCI? What kind on IDE
controller is this?

--
Meelis Roos ([email protected])

2002-04-15 22:05:06

by Maxwell Spangler

[permalink] [raw]
Subject: Re: [COMMENTS IDE 2.5] - "idebus=66" in 2.5.8 results in "ide_setup: idebus=66 -- BAD OPTION"

On Mon, 15 Apr 2002, Meelis Roos wrote:

> MH> Kernel command line: BOOT_IMAGE=2.5.8-without-TCQ ro root=303 video=matrox:vesa:0x118 idebus=66 profile=2
> MH> ide_setup: idebus=66
> MH> ide: system bus speed 66MHz
>
> MH> works like a charm :)
>
> Do you really have an IDE controller that does 66 MHz PCI? What kind on IDE
> controller is this?

I have a Promise Ultra133TX2 card on my shelf that is 32-bit PCI with 66Mhz
operation.

I don't know to what extent it is supported yet.. Andre?
-- ----------------------------------------------------------------------------
Maxwell Spangler Save Futurama!
Program Writer Sign the petition!
Greenbelt, Maryland, U.S.A. http://www.gotfuturama.com/
Washington D.C. Metropolitan Area

2002-04-15 22:39:41

by Vojtech Pavlik

[permalink] [raw]
Subject: Re: [COMMENTS IDE 2.5] - "idebus=66" in 2.5.8 results in "ide_setup: idebus=66 -- BAD OPTION"

On Mon, Apr 15, 2002 at 06:04:54PM -0400, Maxwell Spangler wrote:
> On Mon, 15 Apr 2002, Meelis Roos wrote:
>
> > MH> Kernel command line: BOOT_IMAGE=2.5.8-without-TCQ ro root=303 video=matrox:vesa:0x118 idebus=66 profile=2
> > MH> ide_setup: idebus=66
> > MH> ide: system bus speed 66MHz
> >
> > MH> works like a charm :)
> >
> > Do you really have an IDE controller that does 66 MHz PCI? What kind on IDE
> > controller is this?
>
> I have a Promise Ultra133TX2 card on my shelf that is 32-bit PCI with 66Mhz
> operation.
>
> I don't know to what extent it is supported yet.. Andre?

The promise driver ignores the idebus= value anyway. :) And for others
66 is usually an invalid setting.

--
Vojtech Pavlik
SuSE Labs

2002-04-15 22:55:44

by Andre Hedrick

[permalink] [raw]
Subject: Re: [COMMENTS IDE 2.5] - "idebus=66" in 2.5.8 results in "ide_setup: idebus=66 -- BAD OPTION"


Do not know what is left of the kernel base in 2.5 as I am out of the
development tree. Somewhat offically kicked out of Linux. After 2.4 pass
on so will I.

Cheers,

Andre Hedrick
LAD Storage Consulting Group

On Mon, 15 Apr 2002, Maxwell Spangler wrote:

> On Mon, 15 Apr 2002, Meelis Roos wrote:
>
> > MH> Kernel command line: BOOT_IMAGE=2.5.8-without-TCQ ro root=303 video=matrox:vesa:0x118 idebus=66 profile=2
> > MH> ide_setup: idebus=66
> > MH> ide: system bus speed 66MHz
> >
> > MH> works like a charm :)
> >
> > Do you really have an IDE controller that does 66 MHz PCI? What kind on IDE
> > controller is this?
>
> I have a Promise Ultra133TX2 card on my shelf that is 32-bit PCI with 66Mhz
> operation.
>
> I don't know to what extent it is supported yet.. Andre?
> -- ----------------------------------------------------------------------------
> Maxwell Spangler Save Futurama!
> Program Writer Sign the petition!
> Greenbelt, Maryland, U.S.A. http://www.gotfuturama.com/
> Washington D.C. Metropolitan Area
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>

2002-04-15 23:44:33

by Maxwell Spangler

[permalink] [raw]
Subject: Re: [COMMENTS IDE 2.5] - "idebus=66" in 2.5.8 results in "ide_setup: idebus=66 -- BAD OPTION"

On Mon, 15 Apr 2002, Andre Hedrick wrote:

>
> Do not know what is left of the kernel base in 2.5 as I am out of the
> development tree. Somewhat offically kicked out of Linux. After 2.4 pass
> on so will I.

I'm interested in production systems, and that means staying with 2.4 for a
LONG time from now. So your code for 2.4.x is very relevant to me and
that's why I was asking..

Is the Promise Ultra133TX2 supported with 2.4.18, or do I need to patch in
your major ATA upgrade? (Which I assume is still a patch and not included in
the mainstream kernel?)

Thanks!

-- ----------------------------------------------------------------------------
Maxwell Spangler Save Futurama!
Program Writer Sign the petition!
Greenbelt, Maryland, U.S.A. http://www.gotfuturama.com/
Washington D.C. Metropolitan Area