lkml.org 
[lkml]   [2005]   [Jan]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: 2.6.11-rc2-mm1: SuperIO scx200 breakage
On Mon, 24 Jan 2005 20:03:20 +0100
Adrian Bunk <bunk@stusta.de> wrote:

> On Mon, Jan 24, 2005 at 10:19:29PM +0300, Evgeniy Polyakov wrote:
> > On Mon, 24 Jan 2005 19:29:26 +0100
> > Adrian Bunk <bunk@stusta.de> wrote:
> >
> > > On Mon, Jan 24, 2005 at 09:43:36PM +0300, Evgeniy Polyakov wrote:
> > > > On Mon, 24 Jan 2005 18:54:49 +0100
> > > > Adrian Bunk <bunk@stusta.de> wrote:
> > > >
> > > > > It seems noone who reviewed the SuperIO patches noticed that there are
> > > > > now two modules "scx200" in the kernel...
> > > >
> > > > They are almost mutually exlusive(SuperIO contains more advanced),
> > > > so I do not see any problem here.
> > >
> > > The Kconfig files allow building both modular at the same time.
> > >
> > > > Only one of them can be loaded in a time.
> > >
> > > You are assuming the module support was in able to correctly handle two
> > > modules with the same name...
> > >
> > > > So what does exactly bother you?
> > >
> > > if [ -r System.map ]; then /sbin/depmod -ae -F System.map 2.6.11-rc2-mm1; fi
> > > WARNING: /lib/modules/2.6.11-rc2-mm1/kernel/drivers/i2c/busses/scx200_i2c.ko needs unknown symbol scx200_gpio_base
> > > WARNING: /lib/modules/2.6.11-rc2-mm1/kernel/drivers/i2c/busses/scx200_i2c.ko needs unknown symbol scx200_gpio_configure
> > > WARNING: /lib/modules/2.6.11-rc2-mm1/kernel/drivers/i2c/busses/scx200_i2c.ko needs unknown symbol scx200_gpio_shadow
> > > WARNING: /lib/modules/2.6.11-rc2-mm1/kernel/drivers/char/scx200_gpio.ko needs unknown symbol scx200_gpio_base
> > > WARNING: /lib/modules/2.6.11-rc2-mm1/kernel/drivers/char/scx200_gpio.ko needs unknown symbol scx200_gpio_configure
> > > WARNING: /lib/modules/2.6.11-rc2-mm1/kernel/drivers/char/scx200_gpio.ko needs unknown symbol scx200_gpio_shadow
> >
> > Sorry, I can not buy it.
> > Above symbols are defined in old scx200 driver, and I it is depmod
> > who tries to get them from superio.
>
> More exactly, "make modules_install" does install only one of the two
> drivers.
>
> > I definitely sure that it must be solved on the other layers.
> >...
>
> Two modules with the same name are simply a _very_ bad idea.
>
> Even if they weren't allowed to be compiled at the same time, they
> should be named differently or it will cause much confusion for
> everyone (or don't you want to see from the output of "lsmod" which of
> the two modules is loaded?).

I do not agree with you, Adrian, but I will not contend.
As I say, noone protects against the same program names and there are
mechnisms to differ modules by simply looking in lsmod output.
Noone can damage systrem by loading "wrong" module.

So I still do not see problems here.

As I say I will change superio scx200 name since it is easier than
flood about unmatched points of view.

I will send patch through Greg and Andrew later.

Thank you, Adrian, for your comments.

> cu
> Adrian
>
> --
>
> "Is there not promise of rain?" Ling Tan asked suddenly out
> of the darkness. There had been need of rain for many days.
> "Only a promise," Lao Er said.
> Pearl S. Buck - Dragon Seed


Evgeniy Polyakov

Only failure makes us experts. -- Theo de Raadt
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:09    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site