On Fri, Dec 13, 2024 at 12:30:49PM +0100, Andreas Hindborg wrote: > This patch includes changes required for Rust kernel modules to utilize > module parameters. This code implements read only support for integer > types without `sysfs` support. read-only is VERY limited, and as such, only good for boot options, which as I mentioned before, is not how any "modern" kernel driver should be doing things. And no sysfs interaction? That's going to confuse the heck out of people wondering why the option they added doesn't show up in the same place it normally would if they did it in C, right? Not that I'm saying this should be done at all, just that this is going to be confusing right off the bat which is probably not a good idea. Friends don't let friends add new module parameters to the kernel :) thanks, greg k-h
