Re: 2.6.6-mm2

[Chris Wright <chrisw@osdl.org>]

* Kevin O'Connor (kevin@koconnor.net) wrote:
> On Thu, May 13, 2004 at 12:38:09PM -0700, Andrew Morton wrote:
> > Chris Wright <chrisw@osdl.org> wrote:
> > >
> > > 
> > >  +static int capability_mask;
> > >  +module_param_named(mask, capability_mask, int, 0);
> > >  +MODULE_PARM_DESC(mask, "Mask of capability checks to ignore");
> > 
> > Is there a way to make this tunable at runtime, btw?
> 
> I thought that was what the fourth argument to module_param_named was for..
> 
> 
> /* This is the fundamental function for registering boot/module
>    parameters.  perm sets the visibility in driverfs: 000 means it's
>    not there, read bits mean it's readable, write bits mean it's
>    writable. */
> #define __module_param_call(prefix, name, set, get, arg, perm)          \
> 
> Did I miss something?

No, that's right, but I didn't think it was safe enough, because it
doesn't let you register your own function to manage how the variable is
set.  For example, only clearing set bits.

thanks,
-chris
-- 
Linux Security Modules     http://lsm.immunix.org     http://lsm.bkbits.net