Hi Steve, thanks for the patch! A suggestion/fix below, inline. On 06/06/2024 12:01, Steven Rostedt wrote: > [...] > +static char *mem_name; > +module_param_named(mem_name, mem_name, charp, 0400); > +MODULE_PARM_DESC(mem_name, > + "name of kernel param that holds addr (builtin only)"); s/ (builtin only)// Likely a vestige of a previous version heh Cheers, Guilherme