Re: [PATCH] Module alias and device table support.

From: Roman Zippel (zippel@linux-m68k.org)
Date: Fri Jan 31 2003 - 20:22:38 EST


Hi,

On Fri, 31 Jan 2003, Kai Germaschewski wrote:

> > > missing
> > > EXPORT_SYMBOL()s tend to go unnoticed quite often otherwise.
> >
> > The problem here is that we use System.map, it's not that difficult to
> > extract the exported symbols:
> > objcopy -j .kstrtab -O binary vmlinux .export.tmp
> > tr \\0 \\n < .export.tmp > Export.map
>
> What you say is right (except that it misses symbols exported from
> modules), but I don't see what you mean the problem is?

See above, maybe I quoted to much. The other exported symbols are
already extracted by depmod, so it had exactly the information it needs
and would give more correct warnings.

> > It makes sense to keep depmod close to the linker, as both need the same
> > knowledge about resolving symbols, but I still don't know why that would
> > be a reason to put it into the kernel.
>
> Well, I hope you mean into the kernel tree, it sure doesn't make sense to
> put it into the kernel itself.
>
> Anyway, I think rusty's approach is to deal with the kernel-internal data
> structures from inside the kernel tree (during the build, that is) and
> generate data in a fixed format (.modalias) for depmod to read. Since
> depmod is external, it needs a fixed interface. Makes sense to me.

You have to define a fixed format somewhere anyway, either you have to do
it for depmod or for modprobe. This only moves the problem around and if
we already break interfaces, we should look at all the possibilities.
What I'm really missing is an analysis of the problem(s) and a description
of how the solution solves it. After reading most of the patches I think I
understand what Rusty is trying to do, but I still think there are better
solutions, unfortunately Rusty doesn't talk with me anymore :(, if anyone
else knows what I'm doing wrong, I'd be really happy to know about it.

bye, Roman

-
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/



This archive was generated by hypermail 2b29 : Fri Jan 31 2003 - 22:00:27 EST