login
Header Space

 
 

Re: changeset: Make forced module loading optional

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Rusty Russell <rusty@...>
Cc: <linux-kernel@...>, Jon Masters <jonathan@...>, Sam Ravnborg <sam@...>
Date: Monday, May 5, 2008 - 1:07 pm

On Mon, 5 May 2008, Rusty Russell wrote:

Umm. modversions is in general a whole lot *more* reliable than just 
looking at the kernel version.

The kernel version is pretty good if you use CONFIG_LOCALVERSION_AUTO and 
have a git kernel tree, but if not, then modversions is much more likely 
to stop modules across big infrastructure changes during (say) the merge 
window.

So I agree that modversions is not "reliable", but I think that the 
alternative is often even *less* reliable, so I find the "don't recommend 
modversions" comment to be pretty debatable.

Since I personally try to avoid modules, and if I do use them I'd prefer 
the checking to be as strict as possible, I'd really not mind a "strict"  
mode that tests both MODVERSIONS _and_ the full kernel version string. 
Along with not allowing forced module loads, of course.

I also find it sad that apparently I'm one of the few ones that test with 
modules turned off. It's both more secure and simpler, but it does cause 
lots of noise at least during a Fedora boot, and it occasionally breaks 
the /etc/rc.d scripts because they assume that they have to load modules, 
and that it's an error if that fails. We had that happen with the iptables 
scripts not that long ago (and note how that was unrelated to initrd: this 
is past the point when things have switched to the normal root 
filesystem).

IOW, I wish distros did some testing with non-modular kernels too. Oh 
well. At least I can generally fix the problems, and make error reports, 
but I bet it means that most other kernel users simply turn on modules 
whether they need them or not.

			Linus
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: changeset: Make forced module loading optional, Rusty Russell, (Mon May 5, 12:55 am)
Re: changeset: Make forced module loading optional, Jan Engelhardt, (Mon May 5, 2:35 am)
Re: changeset: Make forced module loading optional, Linus Torvalds, (Mon May 5, 1:05 am)
Re: changeset: Make forced module loading optional, Jan Engelhardt, (Mon May 5, 2:43 am)
Re: changeset: Make forced module loading optional, Dave Jones, (Mon May 5, 11:32 am)
Re: changeset: Make forced module loading optional, Jan Engelhardt, (Mon May 5, 12:01 pm)
Re: changeset: Make forced module loading optional, Linus Torvalds, (Mon May 5, 11:48 am)
Re: changeset: Make forced module loading optional, Linus Torvalds, (Mon May 5, 10:37 am)
Re: changeset: Make forced module loading optional, Jeff Garzik, (Mon May 5, 10:50 am)
Re: changeset: Make forced module loading optional, Linus Torvalds, (Mon May 5, 11:01 am)
Re: changeset: Make forced module loading optional, Linus Torvalds, (Mon May 5, 11:08 am)
Re: changeset: Make forced module loading optional, Rusty Russell, (Mon May 5, 1:35 am)
Re: changeset: Make forced module loading optional, Linus Torvalds, (Mon May 5, 1:07 pm)
Re: changeset: Make forced module loading optional, Rusty Russell, (Mon May 5, 2:42 pm)
Re: changeset: Make forced module loading optional, David Miller, (Mon May 5, 3:47 pm)
speck-geostationary