Michal Marek | 4 Nov 2009 21:23
Picon
Gravatar

softdep in .modinfo

Hi Andreas,

good example how additional dependencies in the .modinfo section could
help: https://bugzilla.novell.com/552443
libcrc32c.ko loads crc32c.ko via request_module(), but modprobe
--show-depends won't tell you that in advance.

Michal
Jon Masters | 5 Nov 2009 00:05
Picon
Favicon

Re: softdep in .modinfo

On Wed, 2009-11-04 at 21:23 +0100, Michal Marek wrote:

> good example how additional dependencies in the .modinfo section could
> help: https://bugzilla.novell.com/552443
> libcrc32c.ko loads crc32c.ko via request_module(), but modprobe
> --show-depends won't tell you that in advance.

Yeah. Softdeps are certainly going to be useful for that and many other
things - a common example is pulling in ioctl-control style modules that
are optional (no symbol deps) but needed by management software.

Jon.

Andreas Robinson | 5 Nov 2009 16:54
Picon

Re: softdep in .modinfo

Hi guys,

I have updated depmod to generate modules.softdep.
http://github.com/andr345/module-init-tools/commits/modprobe-softdep/

Should modprobe always read modules.softdep or let the user choose
through some command line option?

For now, modprobe does nothing with it. Instead depmod places a
comment in the beginning, asking the user to link or copy the file to
/etc/modprobe.d if needed.

A pair of kernel patches (against 2.6.32-rc6) are also attached. When
you feel they're ready and you can sign off on them, I'll pass them on
to Rusty Russell.

Cheers,
Andreas
Peter Simons | 6 Nov 2009 19:44
Picon
Gravatar

Re: [PATCH] ax_enable_builddir: improve quality of generated top-level Makefile

Hi Alan,

 > The top-level Makefile is generated by mangling rules, "a: b".  We
 > already avoid mangling "a := b".  But we should also exclude "a = :".
 >
 > Apparently this was causing unbalanced parenthesis errors on some
 > systems.
 >
 > Reported-by: Mike Frysinger <vapier@...>
 > Signed-off-by: Alan Jenkins <alan-jenkins@...>
 > [...]

I have applied your patch to the latest version of that macro in the
Autoconf Archive. Thank you very much.

Take care,
Peter

Alan Jenkins | 9 Nov 2009 11:21

Re: [PATCH] depmod: initial support for split module directories

On 10/1/09, Jon Masters <jcm@...> wrote:
> Traditionally, kernel modules lived only in /lib/modules/≤kernelversion>. In
> the past, we added support for prioritized "update" directories within this
> hierarchy so that distributions and their users could insert updates to
> existing drivers and have those take precendence. But there are times when
> it is also necessary to search a completely different

*
> (and kernel specific)
*

> directory, perhaps on a different filesystem, for module updates.
> This patch introduces a new depmod.conf configuration option called
> "path", which can be used in combination with the existing "search"
> option to specify which directories will be searched before building
> the kernel specific modules.dep and modules.dep.bin files. Paths are
> searched in the order given, with preference within the path determined
> by the value of the "search" option, allowing flexible configurations.
>
> Signed-off-by: Jon Masters <jcm@...>

I can't see the "kernel specific" part here, outside of the default
"current-version" path.  I don't see how you specify a custom path
which is dependent on the kernel version.  Could you please
demonstrate it with an example?

Incentive: if you outline how this will be used, I could write a test
case for it :-).

Thanks
(Continue reading)

Jon Masters | 9 Nov 2009 21:55
Picon
Favicon

Re: [PATCH] depmod: initial support for split module directories

On Mon, 2009-11-09 at 10:21 +0000, Alan Jenkins wrote:

> I can't see the "kernel specific" part here, outside of the default
> "current-version" path.  I don't see how you specify a custom path
> which is dependent on the kernel version.  Could you please
> demonstrate it with an example?

The initial reason for this is for split module trees as used e.g. by an
installer - so you might need to provide (kernel specific) drivers that
aren't in the usual location, but instead in a tmpfs after the installer
read them in from individual driver disks. This is the case for Fedora
and RHEL, and probably other distributions.

> Incentive: if you outline how this will be used, I could write a test
> case for it :-).

I will send an example. Also, I'm really glad to see your work on symbol
sorting going in upstream. Thanks for your hard work!

Jon.

Jon Masters | 25 Nov 2009 08:36
Picon
Favicon

Re: softdep in .modinfo

On Thu, 2009-11-05 at 16:54 +0100, Andreas Robinson wrote:
> Hi guys,
> 
> I have updated depmod to generate modules.softdep.
> http://github.com/andr345/module-init-tools/commits/modprobe-softdep/
> 
> Should modprobe always read modules.softdep or let the user choose
> through some command line option?
> 
> For now, modprobe does nothing with it. Instead depmod places a
> comment in the beginning, asking the user to link or copy the file to
> /etc/modprobe.d if needed.
> 
> A pair of kernel patches (against 2.6.32-rc6) are also attached. When
> you feel they're ready and you can sign off on them, I'll pass them on
> to Rusty Russell.

I will look at these again over the US Thanksgiving holiday since I need
to test some examples - last week was nuts with RHEL stuff. I just
pushed a couple of other pending patches though in the interim.

Jon.


Gmane