linux-toradex.git/include/linux/moduleparam.h, branch tegra-10.9.1

param: fix lots of bugs with writing charp params from sysfs, by leaking mem.

2009-10-28T22:26:17+00:00

e180a6b7759a "param: fix charp parameters set via sysfs" fixed the case
where charp parameters written via sysfs were freed, leaving drivers
accessing random memory.

Unfortunately, storing a flag in the kparam struct was a bad idea: it's
rodata so setting it causes an oops on some archs.  But that's not all:

1) module_param_array() on charp doesn't work reliably, since we use an
   uninitialized temporary struct kernel_param.
2) there's a fundamental race if a module uses this parameter and then
   it's changed: they will still access the old, freed, memory.

The simplest fix (ie. for 2.6.32) is to never free the memory.  This
prevents all these problems, at cost of a memory leak.  In practice, there
are only 18 places where a charp is writable via sysfs, and all are
root-only writable.

Reported-by: Takashi Iwai 
Cc: Sitsofe Wheeler 
Cc: Frederic Weisbecker 
Cc: Christof Schmitt 
Signed-off-by: Rusty Russell 
Cc: stable@kernel.org

module_param: allow 'bool' module_params to be bool, not just int.

2009-06-12T12:16:58+00:00

Impact: API cleanup

For historical reasons, 'bool' parameters must be an int, not a bool.
But there are around 600 users, so a conversion seems like useless churn.

So we use __same_type() to distinguish, and handle both cases.

Signed-off-by: Rusty Russell

module_param: split perm field into flags and perm

2009-06-12T12:16:56+00:00

Impact: cleanup

Rather than hack KPARAM_KMALLOCED into the perm field, separate it out.
Since the perm field was 32 bits and only needs 16, we don't add bloat.

Signed-off-by: Rusty Russell

module_param: invbool should take a 'bool', not an 'int'

2009-06-12T12:16:56+00:00

It takes an 'int' for historical reasons, and there are only two
users: simply switch it over to bool.

The other user (uvesafb.c) will get a (harmless-on-x86) warning until
the next patch is applied.

Cc: Brad Douglas 
Cc: Michal Januszewski 
Signed-off-by: Rusty Russell

param: fix charp parameters set via sysfs

2009-03-31T02:35:30+00:00

Impact: fix crash on reading from /sys/module/.../ieee80211_default_rc_algo

The module_param type "charp" simply sets a char * pointer in the
module to the parameter in the commandline string: this is why we keep
the (mangled) module command line around.  But when set via sysfs (as
about 11 charp parameters can be) this memory is freed on the way
out of the write().  Future reads hit random mem.

So we kstrdup instead: we have to check we're not in early commandline
parsing, and we have to note when we've used it so we can reliably
kfree the parameter when it's next overwritten, and also on module
unload.

(Thanks to Randy Dunlap for CONFIG_SYSFS=n fixes)

Reported-by: Sitsofe Wheeler 
Diagnosed-by: Frederic Weisbecker 
Tested-by: Frederic Weisbecker 
Tested-by: Christof Schmitt 
Signed-off-by: Rusty Russell

core_param() for genuinely core kernel parameters

2008-10-21T23:00:23+00:00

There are a lot of one-liner uses of __setup() in the kernel: they're
cumbersome and not queryable (definitely not settable) via /sys.  Yet
it's ugly to simplify them to module_param(), because by default that
inserts a prefix of the module name (usually filename).

So, introduce a "core_param".  The parameter gets no prefix, but
appears in /sys/module/kernel/parameters/ (if non-zero perms arg).  I
thought about using the name "core", but that's more common than
"kernel".  And if you create a module called "kernel", you will die
a horrible death.

Signed-off-by: Rusty Russell

module: check kernel param length at compile time, not runtime

2008-10-21T23:00:22+00:00

The kparam code tries to handle over-length parameter prefixes at
runtime.  Not only would I bet this has never been tested, it's not
clear that truncating names is a good idea either.

So let's check at compile time.  We need to move the #define to
moduleparam.h to do this, though.

Signed-off-by: Rusty Russell

moduleparam: fix alpha, ia64 and ppc64 compile failures

2008-02-14T00:21:19+00:00

On alpha, ia64 and ppc64 only relocations to local data can go into
read-only sections. The vast majority of module parameters use the global
generic param_set_*/param_get_* functions, so the 'const' attribute for
struct kernel_param is not only useless, but it also causes compile
failures due to 'section type conflict' in those rare cases where
param_set/get are local functions.

This fixes http://bugzilla.kernel.org/show_bug.cgi?id=8964

Signed-off-by: Ivan Kokshaysky 
Cc: Richard Henderson 
Cc: "Luck, Tony" 
Cc: Anton Blanchard 
Cc: Paul Mackerras 
Cc: Adrian Bunk 
Cc: Kamalesh Babulal 
Cc: Rusty Russell 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

remove __attribute_used__

2008-01-28T22:21:18+00:00

Remove the deprecated __attribute_used__.

[Introduce __section in a few places to silence checkpatch /sam]

Signed-off-by: Adrian Bunk 
Signed-off-by: Sam Ravnborg

constify string/array kparam tracking structures

2007-10-17T15:42:56+00:00

.. in an effort to make read-only whatever can be made, so that
CONFIG_DEBUG_RODATA can catch as many issues as possible.

Signed-off-by: Jan Beulich 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds