From: Andreas Robinson <andr345@gmail.com>
To: sam@ravnborg.org, rusty@rustcorp.com.au
Cc: linux-kernel@vger.kernel.org
Subject: [RFC PATCH 3/6] module: always prefix module parameters with the module name
Date: Sun, 15 Feb 2009 19:20:25 +0100 [thread overview]
Message-ID: <1234722028-8110-4-git-send-email-andr345@gmail.com> (raw)
In-Reply-To: <1234722028-8110-1-git-send-email-andr345@gmail.com>
The parameters in statically linked modules now end up in the
proper sysfs directories, ie /sys/module/<modname>/parameter/*
---
include/linux/moduleparam.h | 2 +-
kernel/params.c | 17 ++++++++++++++++-
2 files changed, 17 insertions(+), 2 deletions(-)
diff --git a/include/linux/moduleparam.h b/include/linux/moduleparam.h
index e4af339..0cb2867 100644
--- a/include/linux/moduleparam.h
+++ b/include/linux/moduleparam.h
@@ -7,7 +7,7 @@
/* You can override this manually, but generally this should match the
module name. */
-#ifdef MODULE
+#if defined(MODULE) && !defined(CONFIG_MODULE_STATIC)
#define MODULE_PARAM_PREFIX /* empty */
#else
#define MODULE_PARAM_PREFIX KBUILD_MODNAME "."
diff --git a/kernel/params.c b/kernel/params.c
index a1e3025..22af6ee 100644
--- a/kernel/params.c
+++ b/kernel/params.c
@@ -518,6 +518,19 @@ static void free_module_param_attrs(struct module_kobject *mk)
mk->mp = NULL;
}
+#ifdef CONFIG_MODULE_STATIC
+static const char *skip_param_modname(const char *name)
+{
+ const char *dot = strchr(name, '.');
+ return (!dot) ? name : dot + 1;
+}
+#else
+static const char *skip_param_modname(const char *name)
+{
+ return name;
+}
+#endif
+
/*
* module_param_sysfs_setup - setup sysfs support for one module
* @mod: module
@@ -535,9 +548,11 @@ int module_param_sysfs_setup(struct module *mod,
bool params = false;
for (i = 0; i < num_params; i++) {
+ const char *name;
if (kparam[i].perm == 0)
continue;
- err = add_sysfs_param(&mod->mkobj, &kparam[i], kparam[i].name);
+ name = skip_param_modname(kparam[i].name);
+ err = add_sysfs_param(&mod->mkobj, &kparam[i], name);
if (err)
return err;
params = true;
--
1.5.6.3
next prev parent reply other threads:[~2009-02-15 18:21 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-15 18:20 [RFC PATCH 0/6] module, kbuild: Faster boot with custom kernel Andreas Robinson
2009-02-15 18:20 ` [RFC PATCH 1/6] New option: Static linking of external modules Andreas Robinson
2009-02-15 18:20 ` [RFC PATCH 2/6] module: add module ELF section with module_init() pointer Andreas Robinson
2009-02-15 18:20 ` Andreas Robinson [this message]
2009-02-15 18:20 ` [RFC PATCH 4/6] kbuild: allow linking of an external object into vmlinux Andreas Robinson
2009-02-15 18:20 ` [RFC PATCH 5/6] scripts: new module preprocessor for static linking Andreas Robinson
2009-02-15 18:20 ` [RFC PATCH 6/6] kbuild: enable relinking of vmlinux without full kernel tree Andreas Robinson
2009-02-16 22:51 ` [RFC PATCH 0/6] module, kbuild: Faster boot with custom kernel Rusty Russell
2009-02-17 10:42 ` Andreas Robinson
2009-02-17 11:53 ` Kay Sievers
2009-02-18 4:58 ` Rusty Russell
2009-02-18 9:15 ` Kay Sievers
2009-02-18 10:25 ` Andreas Robinson
2009-02-20 0:37 ` Andreas Robinson
2009-02-20 1:55 ` Kay Sievers
2009-02-21 11:43 ` Andreas Robinson
2009-03-02 14:32 ` Andreas Robinson
2009-03-02 15:59 ` Kay Sievers
2009-03-02 16:20 ` Arjan van de Ven
2009-03-02 16:29 ` Kay Sievers
2009-03-02 18:27 ` Arjan van de Ven
2009-03-02 21:41 ` Andreas Robinson
2009-03-04 18:47 ` Andreas Robinson
2009-03-06 0:18 ` Arjan van de Ven
2009-03-06 15:15 ` Andreas Robinson
2009-03-06 15:45 ` Arjan van de Ven
2009-03-08 10:47 ` Andreas Robinson
2009-03-08 16:01 ` Arjan van de Ven
2009-03-08 20:13 ` [PATCH] sata_nv: add a module parameter to enable async scanning Andreas Robinson
2009-03-09 17:12 ` [RFC PATCH 0/6] module, kbuild: Faster boot with custom kernel Will Newton
2009-03-06 7:05 ` fastboot kernel parameter Sitsofe Wheeler
2009-03-06 11:23 ` Arjan van de Ven
2009-02-24 1:27 ` [RFC PATCH 0/6] module, kbuild: Faster boot with custom kernel Rusty Russell
2009-02-18 11:57 ` Rusty Russell
2009-02-18 13:57 ` Kay Sievers
2009-02-19 11:15 ` Rusty Russell
2009-02-19 11:41 ` Kay Sievers
2009-02-19 20:48 ` Kay Sievers
2009-02-19 21:59 ` Kay Sievers
2009-02-20 0:58 ` Rusty Russell
2009-02-20 1:33 ` Kay Sievers
2009-02-24 1:39 ` Rusty Russell
2009-02-20 11:32 ` Rusty Russell
2009-02-23 16:42 ` Kay Sievers
2009-02-25 7:03 ` Rusty Russell
2009-02-25 18:12 ` Kay Sievers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1234722028-8110-4-git-send-email-andr345@gmail.com \
--to=andr345@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rusty@rustcorp.com.au \
--cc=sam@ravnborg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.