Messages in this thread | | | Date | Tue, 14 Aug 2012 22:35:58 +0400 | From | Cyrill Gorcunov <> | Subject | Re: [patch 3/8] procfs: Add ability to plug in auxiliary fdinfo providers |
| |
On Tue, Aug 14, 2012 at 07:31:42PM +0100, Al Viro wrote: > > Initially we considered to inject some "show" metod to > > file_operations but since there really a number of > > file_operations declared inside kernel (and in real the > > further patches cover onle eventfd/epoll/inotify) the > > waste of memory space will be inacceptable I think. > > NAK. This is too ugly to live. Put it into file_operations, > with NULL meaning default output, or don't do it at all. > > And no, "it's only if you enable CONFIG_SOME_SHIT" gambit won't > fly - we have all seen it played too many times. All it takes > is one politically-inclined induhvidual adding a dependency to > some "vertically integrated" turd (*cough* systemd *spit* udev > *cough*) and we are stuck with the damn thing. CGROUP shite > is already there, DEVTMPFS is well on its way, etc.
OK, I'll put it into file_operations then (actually Pavel was proposing the same but I've been scared by amount of file_operations declared). Thanks!
Cyrill
| |