On Tue, May 13, 2025 at 05:37:11PM -0700, Josh Poimboeuf wrote: > 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(). Sure, but this has no bearing on that - unless I've missed them, there aren't separate methods for just setting and checking the value, which is all we're doing here.
