Messages in this thread | | | Subject | Re: [PATCH 17/18] dyndbg: rename dynamic_debug to dyndbg | From | Rasmus Villemoes <> | Date | Fri, 6 Dec 2019 08:49:19 +0100 |
| |
On 05/12/2019 23.24, Andy Shevchenko wrote: > On Thu, Dec 5, 2019 at 11:54 PM Jim Cromie <jim.cromie@gmail.com> wrote: >> >> This rename fixes a subtle usage wrinkle; the __setup() names didn't >> match the fake "dyndbg" module parameter used to enable dynamic-printk >> callsites in modules. See the last change in Docs for the effect. >> >> It also shortens the "__FILE__:__func__" prefix in dyndbg.verbose >> messages, effectively s/dynamic_debug/dyndbg/ >> >> This is a 99.9% rename; trim_prefix and debugfs_create_dir arg excepted. >> Nonetheless, it also changes both /sys appearances: >> >> bash-5.0# ls -R /sys/kernel/debug/dyndbg/ /sys/module/dyndbg/parameters/ >> /sys/kernel/debug/dyndbg/: >> control > >> /sys/module/dyndbg/parameters/: > > Isn't this path a part of ABI?
Yeah, I think this is a somewhat dangerous change, and I don't really see the point.
Unrelated: Jim, if you want these patches picked up eventually, you have to put akpm on the recipient list (he is on this one, but AFAICT not on any of the others).
Rasmus
| |