Re: [PATCH] rtc: Make rtc-ds1742 driver hotplug-aware

From: Kay Sievers
Date: Fri Aug 17 2007 - 13:15:30 EST


On Fri, 2007-08-17 at 09:55 -0700, David Brownell wrote:
> On Friday 17 August 2007, Kay Sievers wrote:
> > Again,
>
> "Again"?

We exchanges several mails a few weeks ago after the Debian bug caused
by a modprobe loop.

> > the only sane solution is to provide MODALIAS="platform:<name>"
> > from the platform bus, and adding the aliases to drivers who support
> > autoloading. Modalias strings are not free-text strings, they are
> > required to be prefixed by the subsystem.
>
> If so, then whoever tried to change the usage of module aliases
> in that way goofed in several ways. First, by not changing all
> the in-kernel uses. Second, by not changing all the out-of-tree
> uses in various distros, toolchains, etc. Third, by not even
> documenting that change...

Yes, please fix the misuse of MODALIAS.

> Module aliases have been around since very early versions of
> stuff like "modprobe" ... without that new rule. Aliases have
> always been

The environment key MODALIAS= and the sysfs file 'modalias" are part of
the driver core, and have a well defined meaning. And it's a much
younger construct, and _not_ free-text.

> > I still fail to see why platform must be different from everything
> > else in the kernel.
>
> It isn't. See above.

Sure, it is. Please prefix it with "platform:".

> > Please change that stuff, and the bugs which we
> > are fighting magically go away, because module-init-tools alias
> > resolving works like it does for every other subsystem in the kernel.
>
> Those bugs would be ... what? "We" haven't observed anything
> attributable to this usage, which has been common for years now.

Loops of modprobe by modules requesting itself, you worked around that
in the recent kernel, but that broke other setups. And module-init-tools
"blacklist" does not work, because platform uses no alias, but a plain
driver name. That should be fixed, please, please, please...

Every distro has problems with platform, and believe me, I as the udev
maintainer, get all the questions and the trouble. Some blacklist the
whole platform event, some only a few devices. All just because platform
thinks it must not follow the rules everybody else plays. Please,
please, please do what everybody else is doing with the driver core.

Thanks again,
Kay

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