Messages in this thread | | | Date | Fri, 24 Jan 2020 20:42:31 -0500 | From | "Theodore Y. Ts'o" <> | Subject | Re: [PATCH v3] dynamic_debug: allow to work if debugfs is disabled |
| |
On Fri, Jan 24, 2020 at 08:29:40AM +0100, Greg Kroah-Hartman wrote: > > It's likely that people who normally use > > distribution kernels where debugfs is disabled will have scripts which > > are hard-coded to look in /proc, and then when they build a kernel > > with debugfs enabled, the /proc entry will go **poof**, and their > > script will break. > > **poof** they didn't test it :) > > Seriously, I am doing this change to make it _easier_ for those people > who want debugfs disabled, yet want this type of debugging. This is > much better than having no debugging at all. > > Don't put extra complexity in the kernel for when it can be trivially > handled in userspace, you know this :)
It's also trivial to handle this in the kernel by potentially having the control file appear in two places. Consider what it would be like to explain this in the Linux documentation --- "the control file could be here, or it could be there", depending on how the kernel is configured. The complexity of documenting this, and the complexity in userspace; and we have to have the code in the source code for the file to be in the appear in both places *anyway*.
We've done a lot more to maintain userspace backwards compatibility that Linus demands; and while I recognize this is not exactly the same thing, being consistent about where to find the control file would be even *easier* for userspace programmers. And is it really that hard to provide this consistency in the kernel?
Cheers,
- Ted
| |