2005-04-17 02:20:36 +04:00
/* Helpers for initial module or kernel cmdline parsing
Copyright ( C ) 2001 Rusty Russell .
This program is free software ; you can redistribute it and / or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation ; either version 2 of the License , or
( at your option ) any later version .
This program is distributed in the hope that it will be useful ,
but WITHOUT ANY WARRANTY ; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE . See the
GNU General Public License for more details .
You should have received a copy of the GNU General Public License
along with this program ; if not , write to the Free Software
Foundation , Inc . , 59 Temple Place , Suite 330 , Boston , MA 02111 - 1307 USA
*/
# include <linux/moduleparam.h>
# include <linux/kernel.h>
# include <linux/string.h>
# include <linux/errno.h>
# include <linux/module.h>
# include <linux/device.h>
# include <linux/err.h>
2005-10-31 02:03:48 +03:00
# include <linux/slab.h>
2005-04-17 02:20:36 +04:00
2009-03-31 23:05:29 +04:00
/* We abuse the high bits of "perm" to record whether we kmalloc'ed. */
# define KPARAM_KMALLOCED 0x80000000
2005-04-17 02:20:36 +04:00
#if 0
# define DEBUGP printk
# else
# define DEBUGP(fmt, a...)
# endif
2006-03-28 13:56:24 +04:00
static inline char dash2underscore ( char c )
2005-04-17 02:20:36 +04:00
{
if ( c = = ' - ' )
return ' _ ' ;
return c ;
}
static inline int parameq ( const char * input , const char * paramname )
{
unsigned int i ;
for ( i = 0 ; dash2underscore ( input [ i ] ) = = paramname [ i ] ; i + + )
if ( input [ i ] = = ' \0 ' )
return 1 ;
return 0 ;
}
static int parse_one ( char * param ,
char * val ,
struct kernel_param * params ,
unsigned num_params ,
int ( * handle_unknown ) ( char * param , char * val ) )
{
unsigned int i ;
/* Find parameter */
for ( i = 0 ; i < num_params ; i + + ) {
if ( parameq ( param , params [ i ] . name ) ) {
DEBUGP ( " They are equal! Calling %p \n " ,
params [ i ] . set ) ;
return params [ i ] . set ( val , & params [ i ] ) ;
}
}
if ( handle_unknown ) {
DEBUGP ( " Unknown argument: calling %p \n " , handle_unknown ) ;
return handle_unknown ( param , val ) ;
}
DEBUGP ( " Unknown argument `%s' \n " , param ) ;
return - ENOENT ;
}
/* You can use " around spaces, but can't escape ". */
/* Hyphens and underscores equivalent in parameter names. */
static char * next_arg ( char * args , char * * param , char * * val )
{
unsigned int i , equals = 0 ;
int in_quote = 0 , quoted = 0 ;
char * next ;
if ( * args = = ' " ' ) {
args + + ;
in_quote = 1 ;
quoted = 1 ;
}
for ( i = 0 ; args [ i ] ; i + + ) {
if ( args [ i ] = = ' ' & & ! in_quote )
break ;
if ( equals = = 0 ) {
if ( args [ i ] = = ' = ' )
equals = i ;
}
if ( args [ i ] = = ' " ' )
in_quote = ! in_quote ;
}
* param = args ;
if ( ! equals )
* val = NULL ;
else {
args [ equals ] = ' \0 ' ;
* val = args + equals + 1 ;
/* Don't include quotes in value. */
if ( * * val = = ' " ' ) {
( * val ) + + ;
if ( args [ i - 1 ] = = ' " ' )
args [ i - 1 ] = ' \0 ' ;
}
if ( quoted & & args [ i - 1 ] = = ' " ' )
args [ i - 1 ] = ' \0 ' ;
}
if ( args [ i ] ) {
args [ i ] = ' \0 ' ;
next = args + i + 1 ;
} else
next = args + i ;
2005-09-28 08:45:34 +04:00
/* Chew up trailing spaces. */
while ( * next = = ' ' )
next + + ;
2005-04-17 02:20:36 +04:00
return next ;
}
/* Args looks like "foo=bar,bar2 baz=fuz wiz". */
int parse_args ( const char * name ,
char * args ,
struct kernel_param * params ,
unsigned num ,
int ( * unknown ) ( char * param , char * val ) )
{
char * param , * val ;
DEBUGP ( " Parsing ARGS: %s \n " , args ) ;
2005-09-28 08:45:34 +04:00
/* Chew leading spaces */
while ( * args = = ' ' )
args + + ;
2005-04-17 02:20:36 +04:00
while ( * args ) {
int ret ;
2007-01-06 03:36:20 +03:00
int irq_was_disabled ;
2005-04-17 02:20:36 +04:00
args = next_arg ( args , & param , & val ) ;
2007-01-06 03:36:20 +03:00
irq_was_disabled = irqs_disabled ( ) ;
2005-04-17 02:20:36 +04:00
ret = parse_one ( param , val , params , num , unknown ) ;
2007-01-06 03:36:20 +03:00
if ( irq_was_disabled & & ! irqs_disabled ( ) ) {
printk ( KERN_WARNING " parse_args(): option '%s' enabled "
" irq's! \n " , param ) ;
}
2005-04-17 02:20:36 +04:00
switch ( ret ) {
case - ENOENT :
printk ( KERN_ERR " %s: Unknown parameter `%s' \n " ,
name , param ) ;
return ret ;
case - ENOSPC :
printk ( KERN_ERR
" %s: `%s' too large for parameter `%s' \n " ,
name , val ? : " " , param ) ;
return ret ;
case 0 :
break ;
default :
printk ( KERN_ERR
" %s: `%s' invalid for parameter `%s' \n " ,
name , val ? : " " , param ) ;
return ret ;
}
}
/* All parsed OK. */
return 0 ;
}
/* Lazy bastard, eh? */
# define STANDARD_PARAM_DEF(name, type, format, tmptype, strtolfn) \
int param_set_ # # name ( const char * val , struct kernel_param * kp ) \
{ \
tmptype l ; \
Add new string functions strict_strto* and convert kernel params to use them
Currently, for every sysfs node, the callers will be responsible for
implementing store operation, so many many callers are doing duplicate
things to validate input, they have the same mistakes because they are
calling simple_strtol/ul/ll/uul, especially for module params, they are
just numeric, but you can echo such values as 0x1234xxx, 07777888 and
1234aaa, for these cases, module params store operation just ignores
succesive invalid char and converts prefix part to a numeric although input
is acctually invalid.
This patch tries to fix the aforementioned issues and implements
strict_strtox serial functions, kernel/params.c uses them to strictly
validate input, so module params will reject such values as 0x1234xxxx and
returns an error:
write error: Invalid argument
Any modules which export numeric sysfs node can use strict_strtox instead of
simple_strtox to reject any invalid input.
Here are some test results:
Before applying this patch:
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000g > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000gggggggg > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 010000 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0100008 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 010000aaaaa > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]#
After applying this patch:
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000g > /sys/module/e1000/parameters/copybreak
-bash: echo: write error: Invalid argument
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000gggggggg > /sys/module/e1000/parameters/copybreak
-bash: echo: write error: Invalid argument
[root@yangyi-dev /]# echo 010000 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# echo 0100008 > /sys/module/e1000/parameters/copybreak
-bash: echo: write error: Invalid argument
[root@yangyi-dev /]# echo 010000aaaaa > /sys/module/e1000/parameters/copybreak
-bash: echo: write error: Invalid argument
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo -n 4096 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]#
[akpm@linux-foundation.org: fix compiler warnings]
[akpm@linux-foundation.org: fix off-by-one found by tiwai@suse.de]
Signed-off-by: Yi Yang <yi.y.yang@intel.com>
Cc: Greg KH <greg@kroah.com>
Cc: "Randy.Dunlap" <rdunlap@xenotime.net>
Cc: Takashi Iwai <tiwai@suse.de>
Cc: Hugh Dickins <hugh@veritas.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2008-02-08 15:21:57 +03:00
int ret ; \
2005-04-17 02:20:36 +04:00
\
if ( ! val ) return - EINVAL ; \
Add new string functions strict_strto* and convert kernel params to use them
Currently, for every sysfs node, the callers will be responsible for
implementing store operation, so many many callers are doing duplicate
things to validate input, they have the same mistakes because they are
calling simple_strtol/ul/ll/uul, especially for module params, they are
just numeric, but you can echo such values as 0x1234xxx, 07777888 and
1234aaa, for these cases, module params store operation just ignores
succesive invalid char and converts prefix part to a numeric although input
is acctually invalid.
This patch tries to fix the aforementioned issues and implements
strict_strtox serial functions, kernel/params.c uses them to strictly
validate input, so module params will reject such values as 0x1234xxxx and
returns an error:
write error: Invalid argument
Any modules which export numeric sysfs node can use strict_strtox instead of
simple_strtox to reject any invalid input.
Here are some test results:
Before applying this patch:
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000g > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000gggggggg > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 010000 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0100008 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 010000aaaaa > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]#
After applying this patch:
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000g > /sys/module/e1000/parameters/copybreak
-bash: echo: write error: Invalid argument
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000gggggggg > /sys/module/e1000/parameters/copybreak
-bash: echo: write error: Invalid argument
[root@yangyi-dev /]# echo 010000 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# echo 0100008 > /sys/module/e1000/parameters/copybreak
-bash: echo: write error: Invalid argument
[root@yangyi-dev /]# echo 010000aaaaa > /sys/module/e1000/parameters/copybreak
-bash: echo: write error: Invalid argument
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo -n 4096 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]#
[akpm@linux-foundation.org: fix compiler warnings]
[akpm@linux-foundation.org: fix off-by-one found by tiwai@suse.de]
Signed-off-by: Yi Yang <yi.y.yang@intel.com>
Cc: Greg KH <greg@kroah.com>
Cc: "Randy.Dunlap" <rdunlap@xenotime.net>
Cc: Takashi Iwai <tiwai@suse.de>
Cc: Hugh Dickins <hugh@veritas.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2008-02-08 15:21:57 +03:00
ret = strtolfn ( val , 0 , & l ) ; \
if ( ret = = - EINVAL | | ( ( type ) l ! = l ) ) \
2005-04-17 02:20:36 +04:00
return - EINVAL ; \
* ( ( type * ) kp - > arg ) = l ; \
return 0 ; \
} \
int param_get_ # # name ( char * buffer , struct kernel_param * kp ) \
{ \
return sprintf ( buffer , format , * ( ( type * ) kp - > arg ) ) ; \
}
Add new string functions strict_strto* and convert kernel params to use them
Currently, for every sysfs node, the callers will be responsible for
implementing store operation, so many many callers are doing duplicate
things to validate input, they have the same mistakes because they are
calling simple_strtol/ul/ll/uul, especially for module params, they are
just numeric, but you can echo such values as 0x1234xxx, 07777888 and
1234aaa, for these cases, module params store operation just ignores
succesive invalid char and converts prefix part to a numeric although input
is acctually invalid.
This patch tries to fix the aforementioned issues and implements
strict_strtox serial functions, kernel/params.c uses them to strictly
validate input, so module params will reject such values as 0x1234xxxx and
returns an error:
write error: Invalid argument
Any modules which export numeric sysfs node can use strict_strtox instead of
simple_strtox to reject any invalid input.
Here are some test results:
Before applying this patch:
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000g > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000gggggggg > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 010000 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0100008 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 010000aaaaa > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]#
After applying this patch:
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000g > /sys/module/e1000/parameters/copybreak
-bash: echo: write error: Invalid argument
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo 0x1000gggggggg > /sys/module/e1000/parameters/copybreak
-bash: echo: write error: Invalid argument
[root@yangyi-dev /]# echo 010000 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# echo 0100008 > /sys/module/e1000/parameters/copybreak
-bash: echo: write error: Invalid argument
[root@yangyi-dev /]# echo 010000aaaaa > /sys/module/e1000/parameters/copybreak
-bash: echo: write error: Invalid argument
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]# echo -n 4096 > /sys/module/e1000/parameters/copybreak
[root@yangyi-dev /]# cat /sys/module/e1000/parameters/copybreak
4096
[root@yangyi-dev /]#
[akpm@linux-foundation.org: fix compiler warnings]
[akpm@linux-foundation.org: fix off-by-one found by tiwai@suse.de]
Signed-off-by: Yi Yang <yi.y.yang@intel.com>
Cc: Greg KH <greg@kroah.com>
Cc: "Randy.Dunlap" <rdunlap@xenotime.net>
Cc: Takashi Iwai <tiwai@suse.de>
Cc: Hugh Dickins <hugh@veritas.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2008-02-08 15:21:57 +03:00
STANDARD_PARAM_DEF ( byte , unsigned char , " %c " , unsigned long , strict_strtoul ) ;
STANDARD_PARAM_DEF ( short , short , " %hi " , long , strict_strtol ) ;
STANDARD_PARAM_DEF ( ushort , unsigned short , " %hu " , unsigned long , strict_strtoul ) ;
STANDARD_PARAM_DEF ( int , int , " %i " , long , strict_strtol ) ;
STANDARD_PARAM_DEF ( uint , unsigned int , " %u " , unsigned long , strict_strtoul ) ;
STANDARD_PARAM_DEF ( long , long , " %li " , long , strict_strtol ) ;
STANDARD_PARAM_DEF ( ulong , unsigned long , " %lu " , unsigned long , strict_strtoul ) ;
2005-04-17 02:20:36 +04:00
int param_set_charp ( const char * val , struct kernel_param * kp )
{
if ( ! val ) {
printk ( KERN_ERR " %s: string parameter expected \n " ,
kp - > name ) ;
return - EINVAL ;
}
if ( strlen ( val ) > 1024 ) {
printk ( KERN_ERR " %s: string parameter too long \n " ,
kp - > name ) ;
return - ENOSPC ;
}
2009-03-31 23:05:29 +04:00
if ( kp - > perm & KPARAM_KMALLOCED )
kfree ( * ( char * * ) kp - > arg ) ;
/* This is a hack. We can't need to strdup in early boot, and we
* don ' t need to ; this mangled commandline is preserved . */
if ( slab_is_available ( ) ) {
kp - > perm | = KPARAM_KMALLOCED ;
* ( char * * ) kp - > arg = kstrdup ( val , GFP_KERNEL ) ;
if ( ! kp - > arg )
return - ENOMEM ;
} else
* ( const char * * ) kp - > arg = val ;
2005-04-17 02:20:36 +04:00
return 0 ;
}
int param_get_charp ( char * buffer , struct kernel_param * kp )
{
return sprintf ( buffer , " %s " , * ( ( char * * ) kp - > arg ) ) ;
}
int param_set_bool ( const char * val , struct kernel_param * kp )
{
/* No equals means "set"... */
if ( ! val ) val = " 1 " ;
/* One of =[yYnN01] */
switch ( val [ 0 ] ) {
case ' y ' : case ' Y ' : case ' 1 ' :
* ( int * ) kp - > arg = 1 ;
return 0 ;
case ' n ' : case ' N ' : case ' 0 ' :
* ( int * ) kp - > arg = 0 ;
return 0 ;
}
return - EINVAL ;
}
int param_get_bool ( char * buffer , struct kernel_param * kp )
{
/* Y and N chosen as being relatively non-coder friendly */
return sprintf ( buffer , " %c " , ( * ( int * ) kp - > arg ) ? ' Y ' : ' N ' ) ;
}
int param_set_invbool ( const char * val , struct kernel_param * kp )
{
int boolval , ret ;
2007-10-17 10:29:34 +04:00
struct kernel_param dummy ;
2005-04-17 02:20:36 +04:00
2007-10-17 10:29:34 +04:00
dummy . arg = & boolval ;
2005-04-17 02:20:36 +04:00
ret = param_set_bool ( val , & dummy ) ;
if ( ret = = 0 )
* ( int * ) kp - > arg = ! boolval ;
return ret ;
}
int param_get_invbool ( char * buffer , struct kernel_param * kp )
{
2007-10-17 10:29:34 +04:00
return sprintf ( buffer , " %c " , ( * ( int * ) kp - > arg ) ? ' N ' : ' Y ' ) ;
2005-04-17 02:20:36 +04:00
}
2007-05-08 11:28:50 +04:00
/* We break the rule and mangle the string. */
2006-03-25 14:07:06 +03:00
static int param_array ( const char * name ,
const char * val ,
unsigned int min , unsigned int max ,
void * elem , int elemsize ,
int ( * set ) ( const char * , struct kernel_param * kp ) ,
2008-02-06 12:37:50 +03:00
unsigned int * num )
2005-04-17 02:20:36 +04:00
{
int ret ;
struct kernel_param kp ;
char save ;
/* Get the name right for errors. */
kp . name = name ;
kp . arg = elem ;
/* No equals sign? */
if ( ! val ) {
printk ( KERN_ERR " %s: expects arguments \n " , name ) ;
return - EINVAL ;
}
* num = 0 ;
/* We expect a comma-separated list of values. */
do {
int len ;
if ( * num = = max ) {
printk ( KERN_ERR " %s: can only take %i arguments \n " ,
name , max ) ;
return - EINVAL ;
}
len = strcspn ( val , " , " ) ;
/* nul-terminate and parse */
save = val [ len ] ;
( ( char * ) val ) [ len ] = ' \0 ' ;
ret = set ( val , & kp ) ;
if ( ret ! = 0 )
return ret ;
kp . arg + = elemsize ;
val + = len + 1 ;
( * num ) + + ;
} while ( save = = ' , ' ) ;
if ( * num < min ) {
printk ( KERN_ERR " %s: needs at least %i arguments \n " ,
name , min ) ;
return - EINVAL ;
}
return 0 ;
}
int param_array_set ( const char * val , struct kernel_param * kp )
{
2007-10-17 10:29:34 +04:00
const struct kparam_array * arr = kp - > arr ;
[PATCH] kernel/param.c: don't use .max when .num is NULL in param_array_set()
there seems to be a bug, at least for me, in kernel/param.c for arrays with
.num == NULL. If .num == NULL, the function param_array_set() uses &.max
for the call to param_array(), wich alters the .max value to the number of
arguments. The result is, you can't set more array arguments as the last
time you set the parameter.
example:
# a module 'example' with
# static int array[10] = { 0, };
# module_param_array(array, int, NULL, 0644);
$ insmod example.ko array=1,2,3
$ cat /sys/module/example/parameters/array
1,2,3
$ echo "4,3,2,1" > /sys/module/example/parameters/array
$ dmesg | tail -n 1
kernel: array: can take only 3 arguments
Signed-off-by: Bert Wesarg <wesarg@informatik.uni-halle.de>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
2005-04-17 02:25:42 +04:00
unsigned int temp_num ;
2005-04-17 02:20:36 +04:00
return param_array ( kp - > name , val , 1 , arr - > max , arr - > elem ,
[PATCH] kernel/param.c: don't use .max when .num is NULL in param_array_set()
there seems to be a bug, at least for me, in kernel/param.c for arrays with
.num == NULL. If .num == NULL, the function param_array_set() uses &.max
for the call to param_array(), wich alters the .max value to the number of
arguments. The result is, you can't set more array arguments as the last
time you set the parameter.
example:
# a module 'example' with
# static int array[10] = { 0, };
# module_param_array(array, int, NULL, 0644);
$ insmod example.ko array=1,2,3
$ cat /sys/module/example/parameters/array
1,2,3
$ echo "4,3,2,1" > /sys/module/example/parameters/array
$ dmesg | tail -n 1
kernel: array: can take only 3 arguments
Signed-off-by: Bert Wesarg <wesarg@informatik.uni-halle.de>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
2005-04-17 02:25:42 +04:00
arr - > elemsize , arr - > set , arr - > num ? : & temp_num ) ;
2005-04-17 02:20:36 +04:00
}
int param_array_get ( char * buffer , struct kernel_param * kp )
{
int i , off , ret ;
2007-10-17 10:29:34 +04:00
const struct kparam_array * arr = kp - > arr ;
2005-04-17 02:20:36 +04:00
struct kernel_param p ;
p = * kp ;
for ( i = off = 0 ; i < ( arr - > num ? * arr - > num : arr - > max ) ; i + + ) {
if ( i )
buffer [ off + + ] = ' , ' ;
p . arg = arr - > elem + arr - > elemsize * i ;
ret = arr - > get ( buffer + off , & p ) ;
if ( ret < 0 )
return ret ;
off + = ret ;
}
buffer [ off ] = ' \0 ' ;
return off ;
}
int param_set_copystring ( const char * val , struct kernel_param * kp )
{
2007-10-17 10:29:34 +04:00
const struct kparam_string * kps = kp - > str ;
2005-04-17 02:20:36 +04:00
2007-04-12 10:28:41 +04:00
if ( ! val ) {
printk ( KERN_ERR " %s: missing param set value \n " , kp - > name ) ;
return - EINVAL ;
}
2005-04-17 02:20:36 +04:00
if ( strlen ( val ) + 1 > kps - > maxlen ) {
printk ( KERN_ERR " %s: string doesn't fit in %u chars. \n " ,
kp - > name , kps - > maxlen - 1 ) ;
return - ENOSPC ;
}
strcpy ( kps - > string , val ) ;
return 0 ;
}
int param_get_string ( char * buffer , struct kernel_param * kp )
{
2007-10-17 10:29:34 +04:00
const struct kparam_string * kps = kp - > str ;
2005-04-17 02:20:36 +04:00
return strlcpy ( buffer , kps - > string , kps - > maxlen ) ;
}
/* sysfs output in /sys/modules/XYZ/parameters/ */
2008-10-22 19:00:22 +04:00
# define to_module_attr(n) container_of(n, struct module_attribute, attr);
# define to_module_kobject(n) container_of(n, struct module_kobject, kobj);
2005-04-17 02:20:36 +04:00
extern struct kernel_param __start___param [ ] , __stop___param [ ] ;
struct param_attribute
{
struct module_attribute mattr ;
struct kernel_param * param ;
} ;
struct module_param_attrs
{
2008-10-22 19:00:22 +04:00
unsigned int num ;
2005-04-17 02:20:36 +04:00
struct attribute_group grp ;
struct param_attribute attrs [ 0 ] ;
} ;
2007-02-14 02:19:06 +03:00
# ifdef CONFIG_SYSFS
2005-04-17 02:20:36 +04:00
# define to_param_attr(n) container_of(n, struct param_attribute, mattr);
static ssize_t param_attr_show ( struct module_attribute * mattr ,
struct module * mod , char * buf )
{
int count ;
struct param_attribute * attribute = to_param_attr ( mattr ) ;
if ( ! attribute - > param - > get )
return - EPERM ;
count = attribute - > param - > get ( buf , attribute - > param ) ;
if ( count > 0 ) {
strcat ( buf , " \n " ) ;
+ + count ;
}
return count ;
}
/* sysfs always hands a nul-terminated string in buf. We rely on that. */
static ssize_t param_attr_store ( struct module_attribute * mattr ,
struct module * owner ,
const char * buf , size_t len )
{
int err ;
struct param_attribute * attribute = to_param_attr ( mattr ) ;
if ( ! attribute - > param - > set )
return - EPERM ;
err = attribute - > param - > set ( buf , attribute - > param ) ;
if ( ! err )
return len ;
return err ;
}
2007-02-14 02:19:06 +03:00
# endif
2005-04-17 02:20:36 +04:00
# ifdef CONFIG_MODULES
# define __modinit
# else
# define __modinit __init
# endif
2007-02-14 02:19:06 +03:00
# ifdef CONFIG_SYSFS
2005-04-17 02:20:36 +04:00
/*
2008-10-22 19:00:22 +04:00
* add_sysfs_param - add a parameter to sysfs
* @ mk : struct module_kobject
* @ kparam : the actual parameter definition to add to sysfs
* @ name : name of parameter
2005-04-17 02:20:36 +04:00
*
2008-10-22 19:00:22 +04:00
* Create a kobject if for a ( per - module ) parameter if mp NULL , and
* create file in sysfs . Returns an error on out of memory . Always cleans up
* if there ' s an error .
2005-04-17 02:20:36 +04:00
*/
2008-10-22 19:00:22 +04:00
static __modinit int add_sysfs_param ( struct module_kobject * mk ,
struct kernel_param * kp ,
const char * name )
2005-04-17 02:20:36 +04:00
{
2008-10-22 19:00:22 +04:00
struct module_param_attrs * new ;
struct attribute * * attrs ;
int err , num ;
/* We don't bother calling this with invisible parameters. */
BUG_ON ( ! kp - > perm ) ;
if ( ! mk - > mp ) {
num = 0 ;
attrs = NULL ;
} else {
num = mk - > mp - > num ;
attrs = mk - > mp - > grp . attrs ;
2005-04-17 02:20:36 +04:00
}
2008-10-22 19:00:22 +04:00
/* Enlarge. */
new = krealloc ( mk - > mp ,
sizeof ( * mk - > mp ) + sizeof ( mk - > mp - > attrs [ 0 ] ) * ( num + 1 ) ,
GFP_KERNEL ) ;
if ( ! new ) {
kfree ( mk - > mp ) ;
err = - ENOMEM ;
goto fail ;
}
attrs = krealloc ( attrs , sizeof ( new - > grp . attrs [ 0 ] ) * ( num + 2 ) , GFP_KERNEL ) ;
if ( ! attrs ) {
err = - ENOMEM ;
goto fail_free_new ;
}
2005-04-17 02:20:36 +04:00
2008-10-22 19:00:22 +04:00
/* Sysfs wants everything zeroed. */
memset ( new , 0 , sizeof ( * new ) ) ;
memset ( & new - > attrs [ num ] , 0 , sizeof ( new - > attrs [ num ] ) ) ;
memset ( & attrs [ num ] , 0 , sizeof ( attrs [ num ] ) ) ;
new - > grp . name = " parameters " ;
new - > grp . attrs = attrs ;
/* Tack new one on the end. */
new - > attrs [ num ] . param = kp ;
new - > attrs [ num ] . mattr . show = param_attr_show ;
new - > attrs [ num ] . mattr . store = param_attr_store ;
new - > attrs [ num ] . mattr . attr . name = ( char * ) name ;
new - > attrs [ num ] . mattr . attr . mode = kp - > perm ;
new - > num = num + 1 ;
/* Fix up all the pointers, since krealloc can move us */
for ( num = 0 ; num < new - > num ; num + + )
new - > grp . attrs [ num ] = & new - > attrs [ num ] . mattr . attr ;
new - > grp . attrs [ num ] = NULL ;
mk - > mp = new ;
return 0 ;
2005-04-17 02:20:36 +04:00
2008-10-22 19:00:22 +04:00
fail_free_new :
kfree ( new ) ;
fail :
mk - > mp = NULL ;
return err ;
}
2005-04-17 02:20:36 +04:00
2008-10-23 23:07:17 +04:00
# ifdef CONFIG_MODULES
2008-10-22 19:00:22 +04:00
static void free_module_param_attrs ( struct module_kobject * mk )
{
kfree ( mk - > mp - > grp . attrs ) ;
kfree ( mk - > mp ) ;
mk - > mp = NULL ;
2005-04-17 02:20:36 +04:00
}
/*
* module_param_sysfs_setup - setup sysfs support for one module
* @ mod : module
* @ kparam : module parameters ( array )
* @ num_params : number of module parameters
*
2008-10-22 19:00:22 +04:00
* Adds sysfs entries for module parameters under
* / sys / module / [ mod - > name ] / parameters /
2005-04-17 02:20:36 +04:00
*/
int module_param_sysfs_setup ( struct module * mod ,
struct kernel_param * kparam ,
unsigned int num_params )
{
2008-10-22 19:00:22 +04:00
int i , err ;
bool params = false ;
for ( i = 0 ; i < num_params ; i + + ) {
if ( kparam [ i ] . perm = = 0 )
continue ;
err = add_sysfs_param ( & mod - > mkobj , & kparam [ i ] , kparam [ i ] . name ) ;
if ( err )
return err ;
params = true ;
}
2005-04-17 02:20:36 +04:00
2008-10-22 19:00:22 +04:00
if ( ! params )
return 0 ;
2005-04-17 02:20:36 +04:00
2008-10-22 19:00:22 +04:00
/* Create the param group. */
err = sysfs_create_group ( & mod - > mkobj . kobj , & mod - > mkobj . mp - > grp ) ;
if ( err )
free_module_param_attrs ( & mod - > mkobj ) ;
return err ;
2005-04-17 02:20:36 +04:00
}
/*
* module_param_sysfs_remove - remove sysfs support for one module
* @ mod : module
*
* Remove sysfs entries for module parameters and the corresponding
* kobject .
*/
void module_param_sysfs_remove ( struct module * mod )
{
2008-10-22 19:00:22 +04:00
if ( mod - > mkobj . mp ) {
sysfs_remove_group ( & mod - > mkobj . kobj , & mod - > mkobj . mp - > grp ) ;
2005-04-17 02:20:36 +04:00
/* We are positive that no one is using any param
* attrs at this point . Deallocate immediately . */
2008-10-22 19:00:22 +04:00
free_module_param_attrs ( & mod - > mkobj ) ;
2005-04-17 02:20:36 +04:00
}
}
# endif
2009-03-31 23:05:29 +04:00
void destroy_params ( const struct kernel_param * params , unsigned num )
{
unsigned int i ;
for ( i = 0 ; i < num ; i + + )
if ( params [ i ] . perm & KPARAM_KMALLOCED )
kfree ( * ( char * * ) params [ i ] . arg ) ;
}
2008-10-22 19:00:22 +04:00
static void __init kernel_add_sysfs_param ( const char * name ,
struct kernel_param * kparam ,
unsigned int name_skip )
2005-04-17 02:20:36 +04:00
{
struct module_kobject * mk ;
2008-10-22 19:00:22 +04:00
struct kobject * kobj ;
int err ;
2005-04-17 02:20:36 +04:00
2008-10-22 19:00:22 +04:00
kobj = kset_find_obj ( module_kset , name ) ;
if ( kobj ) {
/* We already have one. Remove params so we can add more. */
mk = to_module_kobject ( kobj ) ;
/* We need to remove it before adding parameters. */
sysfs_remove_group ( & mk - > kobj , & mk - > mp - > grp ) ;
} else {
mk = kzalloc ( sizeof ( struct module_kobject ) , GFP_KERNEL ) ;
BUG_ON ( ! mk ) ;
mk - > mod = THIS_MODULE ;
mk - > kobj . kset = module_kset ;
err = kobject_init_and_add ( & mk - > kobj , & module_ktype , NULL ,
" %s " , name ) ;
if ( err ) {
kobject_put ( & mk - > kobj ) ;
printk ( KERN_ERR " Module '%s' failed add to sysfs, "
" error number %d \n " , name , err ) ;
printk ( KERN_ERR " The system will be unstable now. \n " ) ;
return ;
}
/* So that exit path is even. */
kobject_get ( & mk - > kobj ) ;
2007-07-30 22:26:38 +04:00
}
2008-10-22 19:00:22 +04:00
/* These should not fail at boot. */
err = add_sysfs_param ( mk , kparam , kparam - > name + name_skip ) ;
BUG_ON ( err ) ;
err = sysfs_create_group ( & mk - > kobj , & mk - > mp - > grp ) ;
BUG_ON ( err ) ;
2007-01-15 22:22:02 +03:00
kobject_uevent ( & mk - > kobj , KOBJ_ADD ) ;
2008-10-22 19:00:22 +04:00
kobject_put ( & mk - > kobj ) ;
2005-04-17 02:20:36 +04:00
}
/*
* param_sysfs_builtin - add contents in / sys / parameters for built - in modules
*
* Add module_parameters to sysfs for " modules " built into the kernel .
*
* The " module " name ( KBUILD_MODNAME ) is stored before a dot , the
* " parameter " name is stored behind a dot in kernel_param - > name . So ,
* extract the " module " name for all built - in kernel_param - eters ,
2008-10-22 19:00:22 +04:00
* and for all who have the same , call kernel_add_sysfs_param .
2005-04-17 02:20:36 +04:00
*/
static void __init param_sysfs_builtin ( void )
{
2008-10-22 19:00:22 +04:00
struct kernel_param * kp ;
unsigned int name_len ;
char modname [ MODULE_NAME_LEN ] ;
2005-04-17 02:20:36 +04:00
2008-10-22 19:00:22 +04:00
for ( kp = __start___param ; kp < __stop___param ; kp + + ) {
2005-04-17 02:20:36 +04:00
char * dot ;
2008-10-22 19:00:22 +04:00
if ( kp - > perm = = 0 )
continue ;
2005-04-17 02:20:36 +04:00
2008-10-22 19:00:22 +04:00
dot = strchr ( kp - > name , ' . ' ) ;
2005-04-17 02:20:36 +04:00
if ( ! dot ) {
2008-10-22 19:00:23 +04:00
/* This happens for core_param() */
strcpy ( modname , " kernel " ) ;
name_len = 0 ;
} else {
name_len = dot - kp - > name + 1 ;
strlcpy ( modname , kp - > name , name_len ) ;
2005-04-17 02:20:36 +04:00
}
2008-10-22 19:00:23 +04:00
kernel_add_sysfs_param ( modname , kp , name_len ) ;
2005-04-17 02:20:36 +04:00
}
}
/* module-related sysfs stuff */
static ssize_t module_attr_show ( struct kobject * kobj ,
struct attribute * attr ,
char * buf )
{
struct module_attribute * attribute ;
struct module_kobject * mk ;
int ret ;
attribute = to_module_attr ( attr ) ;
mk = to_module_kobject ( kobj ) ;
if ( ! attribute - > show )
2005-04-29 10:27:34 +04:00
return - EIO ;
2005-04-17 02:20:36 +04:00
ret = attribute - > show ( attribute , mk - > mod , buf ) ;
return ret ;
}
static ssize_t module_attr_store ( struct kobject * kobj ,
struct attribute * attr ,
const char * buf , size_t len )
{
struct module_attribute * attribute ;
struct module_kobject * mk ;
int ret ;
attribute = to_module_attr ( attr ) ;
mk = to_module_kobject ( kobj ) ;
if ( ! attribute - > store )
2005-04-29 10:27:34 +04:00
return - EIO ;
2005-04-17 02:20:36 +04:00
ret = attribute - > store ( attribute , mk - > mod , buf , len ) ;
return ret ;
}
static struct sysfs_ops module_sysfs_ops = {
. show = module_attr_show ,
. store = module_attr_store ,
} ;
2007-01-18 15:26:15 +03:00
static int uevent_filter ( struct kset * kset , struct kobject * kobj )
{
struct kobj_type * ktype = get_ktype ( kobj ) ;
if ( ktype = = & module_ktype )
return 1 ;
return 0 ;
}
static struct kset_uevent_ops module_uevent_ops = {
. filter = uevent_filter ,
} ;
2007-11-01 20:39:50 +03:00
struct kset * module_kset ;
2007-04-14 00:15:19 +04:00
int module_sysfs_initialized ;
2005-04-17 02:20:36 +04:00
2007-11-01 20:39:50 +03:00
struct kobj_type module_ktype = {
2005-04-17 02:20:36 +04:00
. sysfs_ops = & module_sysfs_ops ,
} ;
/*
* param_sysfs_init - wrapper for built - in params support
*/
static int __init param_sysfs_init ( void )
{
2007-11-01 20:39:50 +03:00
module_kset = kset_create_and_add ( " module " , & module_uevent_ops , NULL ) ;
if ( ! module_kset ) {
printk ( KERN_WARNING " %s (%d): error creating kset \n " ,
__FILE__ , __LINE__ ) ;
return - ENOMEM ;
2006-09-29 12:58:55 +04:00
}
2007-04-14 00:15:19 +04:00
module_sysfs_initialized = 1 ;
2005-04-17 02:20:36 +04:00
param_sysfs_builtin ( ) ;
return 0 ;
}
2006-09-29 12:59:34 +04:00
subsys_initcall ( param_sysfs_init ) ;
2005-04-17 02:20:36 +04:00
2007-11-01 20:39:50 +03:00
# endif /* CONFIG_SYSFS */
2007-02-14 02:19:06 +03:00
2005-04-17 02:20:36 +04:00
EXPORT_SYMBOL ( param_set_byte ) ;
EXPORT_SYMBOL ( param_get_byte ) ;
EXPORT_SYMBOL ( param_set_short ) ;
EXPORT_SYMBOL ( param_get_short ) ;
EXPORT_SYMBOL ( param_set_ushort ) ;
EXPORT_SYMBOL ( param_get_ushort ) ;
EXPORT_SYMBOL ( param_set_int ) ;
EXPORT_SYMBOL ( param_get_int ) ;
EXPORT_SYMBOL ( param_set_uint ) ;
EXPORT_SYMBOL ( param_get_uint ) ;
EXPORT_SYMBOL ( param_set_long ) ;
EXPORT_SYMBOL ( param_get_long ) ;
EXPORT_SYMBOL ( param_set_ulong ) ;
EXPORT_SYMBOL ( param_get_ulong ) ;
EXPORT_SYMBOL ( param_set_charp ) ;
EXPORT_SYMBOL ( param_get_charp ) ;
EXPORT_SYMBOL ( param_set_bool ) ;
EXPORT_SYMBOL ( param_get_bool ) ;
EXPORT_SYMBOL ( param_set_invbool ) ;
EXPORT_SYMBOL ( param_get_invbool ) ;
EXPORT_SYMBOL ( param_array_set ) ;
EXPORT_SYMBOL ( param_array_get ) ;
EXPORT_SYMBOL ( param_set_copystring ) ;
EXPORT_SYMBOL ( param_get_string ) ;