[git commit master] modprobe-small: added comment about multiple alias matches

Denys Vlasenko vda.linux at googlemail.com
Thu Oct 8 20:54:41 UTC 2009


commit: http://git.busybox.net/busybox/commit/?id=6332151641f2c80c0ca57bd2f7dd11393619bc3a
branch: http://git.busybox.net/busybox/commit/?id=refs/heads/master

Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
---
 modutils/modprobe-small.c |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/modutils/modprobe-small.c b/modutils/modprobe-small.c
index e2359d0..8ef9101 100644
--- a/modutils/modprobe-small.c
+++ b/modutils/modprobe-small.c
@@ -573,6 +573,14 @@ static void process_module(char *name, const char *cmdline_options)
 		info = find_alias(name);
 	}
 
+// Problem here: there can be more than one module
+// for the given alias. For example,
+// "pci:v00008086d00007010sv00000000sd00000000bc01sc01i80" matches
+// ata_piix because it has an alias "pci:v00008086d00007010sv*sd*bc*sc*i*"
+// and ata_generic, it has an alias "alias=pci:v*d*sv*sd*bc01sc01i*"
+// Standard modprobe would load them both.
+// In this code, find_alias() returns only the first matching module.
+
 	/* rmmod? unload it by name */
 	if (is_rmmod) {
 		if (delete_module(name, O_NONBLOCK | O_EXCL) != 0
-- 
1.6.3.3



More information about the busybox-cvs mailing list