On Tue, May 13, 2025 at 07:34:59PM -0400, Kent Overstreet wrote: > On Tue, May 13, 2025 at 02:24:46PM -0700, Josh Poimboeuf wrote: > > > +++ b/include/linux/moduleparam.h > > > @@ -122,6 +122,7 @@ struct kparam_array > > > * charp: a character pointer > > > * bool: a bool, values 0/1, y/n, Y/N. > > > * invbool: the above, only sense-reversed (N = true). > > > + * static_key_t: same as bool, but updates a 'struct static_key' > > > > The static_key_*() interfaces are deprecated because they're really easy > > to use incorrectly. I don't think we want to propagate that misuse to > > modules. > > > > It would be better to have type(s) based on static_key_false and/or > > static_key_true, depending on whatever the default is. > > Except those are just wrappers around struct static_key, so why does > that matter here? Those struct wrappers are needed to work with static_branch_likely() and static_branch_unlikely(). We really need to get rid of old static_key_true() and static_key_false(), they don't have many users left at this point. -- Josh
