On Thu, Sep 04, 2014 at 08:10:48PM +0800, Zheng Liu wrote: > > When reading through this again, I realized that we probably don't want > > this file in /proc/fs/ext4 but rather in /sys/kernel/debug/ext4 because > > it's really a debugging interface and we don't want any tools to start > > using it. > > Fair enough. I will fix it in next version. I was actually OK with leaving it in /proc, since we have a number of other debugging files in /proc/fs/ext4/<dev> already, and I'm not sure spreading ext4's foot print so that we have some files in /proc, some in /sys, and some in debugfs is worth decreasing the possibility that some tool would start depending on the contents of that file --- I can't really imagine why non-fs developers would ever care about such statistics. I probably *would* be in favor of creating a new config option which disables /proc/fs/ext4/<dev>/mballoc_groups and /proc/fs/ext4/<dev>/es_shrinker_info, to help with the people who care shrinking the total footprint of the Linux kernel for embedded application (kernel tinification), though. - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
