* Re: [Linux-ia64] modutils 2.4.2 and USB
2001-02-12 17:14 [Linux-ia64] modutils 2.4.2 and USB Michael Madore
@ 2001-02-12 19:41 ` Johannes Erdfelt
2001-02-12 19:49 ` Michael Madore
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Johannes Erdfelt @ 2001-02-12 19:41 UTC (permalink / raw)
To: linux-ia64
On Mon, Feb 12, 2001, Michael Madore <mmadore@turbolinux.com> wrote:
> I have just built kernel 2.4.1 with the IA64 patch, and also the latest
> (2.4.2) modutils. All of the USB options are compiled as modules, with the
> exception of USB-Core, which is compiled statically. After installing
> everything, running depmod -a yields:
>
> depmod: Module /lib/modules/2.4.1-010131-1smp/kernel/drivers/usb/acm.o has
> an unknown version (2) usb_device table
> It is likely that the kernel structure has changed, if so then
> you probably need a new version of modutils to handle this kernel.
> Check linux/Documentation/Changes.
>
> Is this a known problem?
That is definately weird. Keith Owens wanted to add a version information
to help fix a problem we ran into with 2.4.0.
Right before 2.4.0 we had to modify the usb_device_id structure to add some
extra information to make it work correctly. However, this didn't change
the size of the structure (because of alignment reasons) and as a result
it was difficult for modutils to tell the difference between the old style
structure and the new style structure.
Both Linus and I didn't want to add the version information in 2.4 and we
just told him to drop the support for the old style structure since it was
only found in development kernels.
The code stayed in modutils it appears since it wouldn't get in the way
of anything and it would be used in 2.5.
But, the patch never went into the kernel and it shouldn't be finding any
version information. Is there a __module_usb_device_ver symbol in acm.o?
JE
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [Linux-ia64] modutils 2.4.2 and USB
2001-02-12 17:14 [Linux-ia64] modutils 2.4.2 and USB Michael Madore
2001-02-12 19:41 ` Johannes Erdfelt
@ 2001-02-12 19:49 ` Michael Madore
2001-02-12 19:53 ` Johannes Erdfelt
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Michael Madore @ 2001-02-12 19:49 UTC (permalink / raw)
To: linux-ia64
On Mon, Feb 12, 2001 at 11:41:13AM -0800, Johannes Erdfelt wrote:
> On Mon, Feb 12, 2001, Michael Madore <mmadore@turbolinux.com> wrote:
> > I have just built kernel 2.4.1 with the IA64 patch, and also the latest
> > (2.4.2) modutils. All of the USB options are compiled as modules, with the
> > exception of USB-Core, which is compiled statically. After installing
> > everything, running depmod -a yields:
> >
> > depmod: Module /lib/modules/2.4.1-010131-1smp/kernel/drivers/usb/acm.o has
> > an unknown version (2) usb_device table
> > It is likely that the kernel structure has changed, if so then
> > you probably need a new version of modutils to handle this kernel.
> > Check linux/Documentation/Changes.
> >
> > Is this a known problem?
>
> That is definately weird. Keith Owens wanted to add a version information
> to help fix a problem we ran into with 2.4.0.
>
> Right before 2.4.0 we had to modify the usb_device_id structure to add some
> extra information to make it work correctly. However, this didn't change
> the size of the structure (because of alignment reasons) and as a result
> it was difficult for modutils to tell the difference between the old style
> structure and the new style structure.
>
> Both Linus and I didn't want to add the version information in 2.4 and we
> just told him to drop the support for the old style structure since it was
> only found in development kernels.
>
> The code stayed in modutils it appears since it wouldn't get in the way
> of anything and it would be used in 2.5.
>
> But, the patch never went into the kernel and it shouldn't be finding any
> version information. Is there a __module_usb_device_ver symbol in acm.o?
Yes, acm.o has a symbol called __module_usb_device_ver. If I temporarily
remove acm.o, I get the same message for audio.o, so it isn't a problem just
with that module.
--
Mike Madore
Software Engineer
TurboLinux, Inc.
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [Linux-ia64] modutils 2.4.2 and USB
2001-02-12 17:14 [Linux-ia64] modutils 2.4.2 and USB Michael Madore
2001-02-12 19:41 ` Johannes Erdfelt
2001-02-12 19:49 ` Michael Madore
@ 2001-02-12 19:53 ` Johannes Erdfelt
2001-02-12 22:04 ` Keith Owens
2001-02-12 22:16 ` Michael Madore
4 siblings, 0 replies; 6+ messages in thread
From: Johannes Erdfelt @ 2001-02-12 19:53 UTC (permalink / raw)
To: linux-ia64
On Mon, Feb 12, 2001, Michael Madore <mmadore@turbolinux.com> wrote:
> On Mon, Feb 12, 2001 at 11:41:13AM -0800, Johannes Erdfelt wrote:
> > That is definately weird. Keith Owens wanted to add a version information
> > to help fix a problem we ran into with 2.4.0.
> >
> > Right before 2.4.0 we had to modify the usb_device_id structure to add some
> > extra information to make it work correctly. However, this didn't change
> > the size of the structure (because of alignment reasons) and as a result
> > it was difficult for modutils to tell the difference between the old style
> > structure and the new style structure.
> >
> > Both Linus and I didn't want to add the version information in 2.4 and we
> > just told him to drop the support for the old style structure since it was
> > only found in development kernels.
> >
> > The code stayed in modutils it appears since it wouldn't get in the way
> > of anything and it would be used in 2.5.
> >
> > But, the patch never went into the kernel and it shouldn't be finding any
> > version information. Is there a __module_usb_device_ver symbol in acm.o?
>
> Yes, acm.o has a symbol called __module_usb_device_ver. If I temporarily
> remove acm.o, I get the same message for audio.o, so it isn't a problem just
> with that module.
Exactly which patches did you apply? My ia64 development system only has
the symbols which should be there.
0000000000000020 ? __module_author
0000000000000068 ? __module_description
0000000000000000 ? __module_kernel_version
0000000000000000 g __module_usb_device_size
0000000000000008 g __module_usb_device_table
U __this_module
JE
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [Linux-ia64] modutils 2.4.2 and USB
2001-02-12 17:14 [Linux-ia64] modutils 2.4.2 and USB Michael Madore
` (2 preceding siblings ...)
2001-02-12 19:53 ` Johannes Erdfelt
@ 2001-02-12 22:04 ` Keith Owens
2001-02-12 22:16 ` Michael Madore
4 siblings, 0 replies; 6+ messages in thread
From: Keith Owens @ 2001-02-12 22:04 UTC (permalink / raw)
To: linux-ia64
On Mon, 12 Feb 2001 09:14:27 -0800,
Michael Madore <mmadore@turbolinux.com> wrote:
>depmod: Module /lib/modules/2.4.1-010131-1smp/kernel/drivers/usb/acm.o has
>an unknown version (2) usb_device table
> It is likely that the kernel structure has changed, if so then
> you probably need a new version of modutils to handle this kernel.
> Check linux/Documentation/Changes.
Somebody has included patch-2.4.0-hotplug.gz in your source tree. That
patch is not to be used.
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [Linux-ia64] modutils 2.4.2 and USB
2001-02-12 17:14 [Linux-ia64] modutils 2.4.2 and USB Michael Madore
` (3 preceding siblings ...)
2001-02-12 22:04 ` Keith Owens
@ 2001-02-12 22:16 ` Michael Madore
4 siblings, 0 replies; 6+ messages in thread
From: Michael Madore @ 2001-02-12 22:16 UTC (permalink / raw)
To: linux-ia64
Yes, you are correct. I've removed the patch and given myself 20 lashes
with a wet noodle.
On Tue, Feb 13, 2001 at 09:04:10AM +1100, Keith Owens wrote:
> On Mon, 12 Feb 2001 09:14:27 -0800,
> Michael Madore <mmadore@turbolinux.com> wrote:
> >depmod: Module /lib/modules/2.4.1-010131-1smp/kernel/drivers/usb/acm.o has
> >an unknown version (2) usb_device table
> > It is likely that the kernel structure has changed, if so then
> > you probably need a new version of modutils to handle this kernel.
> > Check linux/Documentation/Changes.
>
> Somebody has included patch-2.4.0-hotplug.gz in your source tree. That
> patch is not to be used.
--
Mike Madore
Software Engineer
TurboLinux, Inc.
^ permalink raw reply [flat|nested] 6+ messages in thread