linux-kernel - CONFIG_KMOD breakage in next kernel

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 20 Aug 2008 18:16:13 +0200
From:	Takashi Iwai <tiwai@...e.de>
To:	linux-next@...r.kernel.org
Cc:	Andrew Morton <akpm@...ux-foundation.org>,
	Stephen Rothwell <sfr@...b.auug.org.au>,
	Johannes Berg <johannes@...solutions.net>,
	Rusty Russell <rusty@...tcorp.com.au>,
	LKML <linux-kernel@...r.kernel.org>, netdev@...r.kernel.org,
	video4linux-list@...hat.com
Subject: CONFIG_KMOD breakage in next kernel

Hi,

I found that CONFIG_KMOD was already removed in linux-next tree.
This seems to result in many breakages.  For example, I cannot mount a
FAT device automatically because of failure of automatic loading of
nls modules.

My particular problem would be fixed by the patch below, but I found
CONFIG_KMOD is still referred in many places.

% find * -name '*.[ch]' | xargs grep -l CONFIG_KMOD
drivers/media/video/w9968cf.c
drivers/media/video/v4l2-common.c
drivers/media/video/v4l1-compat.c
drivers/media/video/usbvision/usbvision-video.c
drivers/media/video/usbvision/usbvision-core.c
drivers/media/video/vino.c
drivers/media/video/cpia.c
drivers/video/fbmem.c
drivers/md/md.c
drivers/net/irda/sir_dongle.c
drivers/net/pppox.c
drivers/net/ppp_generic.c
drivers/mtd/mtdpart.c
fs/char_dev.c
fs/exec.c
kernel/kmod.c
net/dccp/ccid.c
net/sunrpc/auth.c
net/ipv4/devinet.c
net/ipv4/inet_diag.c
net/ipv4/tcp_cong.c
net/bluetooth/af_bluetooth.c
net/sched/cls_api.c
net/sched/act_api.c
net/sched/ematch.c
net/sched/sch_api.c
net/can/af_can.c
net/core/rtnetlink.c
net/core/dev.c
net/decnet/dn_dev.c
net/netlink/af_netlink.c
net/socket.c
net/netfilter/nfnetlink.c
net/bridge/netfilter/ebtables.c


Anyone working on this?  It must be trivial.

Or, revert the following for the time being until all these are fixed:
commit ba90ad7a589abdf59553a622c35cc149a348034f
Author: Johannes Berg <johannes@...solutions.net>
Date:   Tue Jul 8 19:00:26 2008 +0200

    remove CONFIG_KMOD
    
    Now that nothing depends on it any more, remove CONFIG_KMOD.
    
    Signed-off-by: Johannes Berg <johannes@...solutions.net>
    Signed-off-by: Rusty Russell <rusty@...tcorp.com.au>


thanks,

Takashi

---
diff --git a/fs/nls/nls_base.c b/fs/nls/nls_base.c
index 64965e1..9f1a54c 100644
--- a/fs/nls/nls_base.c
+++ b/fs/nls/nls_base.c
@@ -13,9 +13,6 @@
 #include <linux/nls.h>
 #include <linux/kernel.h>
 #include <linux/errno.h>
-#ifdef CONFIG_KMOD
-#include <linux/kmod.h>
-#endif
 #include <linux/spinlock.h>
 
 static struct nls_table default_table;
@@ -216,7 +213,7 @@ static struct nls_table *find_nls(char *charset)
 struct nls_table *load_nls(char *charset)
 {
 	struct nls_table *nls;
-#ifdef CONFIG_KMOD
+#ifdef CONFIG_MODULES
 	int ret;
 #endif
 
@@ -224,7 +221,7 @@ struct nls_table *load_nls(char *charset)
 	if (nls)
 		return nls;
 
-#ifdef CONFIG_KMOD
+#ifdef CONFIG_MODULES
 	ret = request_module("nls_%s", charset);
 	if (ret != 0) {
 		printk("Unable to load NLS charset %s\n", charset);
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/