On Thu, Oct 03, 2019 at 12:47:41PM +0200, Michal Koutný wrote: > On Wed, Oct 02, 2019 at 10:00:07PM +0900, Suleiman Souhlal <suleiman@xxxxxxxxxx> wrote: > > kmem.slabinfo has been absolutely invaluable for debugging, in my experience. > > I am however not aware of any automation based on it. > My experience is the same. However, the point is that this has been > exposed since ages, so the safe assumption is that there may be users. Yes, but kernel memory accounting was an opt-in feature for years, and also it can be disabled on boot time, so displaying an empty memory.slabinfo file doesn't break the interface. > > > Maybe it might be worth adding it to cgroup v2 and have a CONFIG > > option to enable it? > I don't think v2 file is necessary given the cost of obtaining the > information. But I concur the idea of making the per-object tracking > switchable at boot time or at least CONFIGurable. As I said, the cost is the same and should be paid in any case, no matter if cgroup v1 or v2 is used. A user can dynamically switch between v1 and v2, and there is no way to obtain this information afterwards, so we need to collect it from scratch. Another concern I have is that it will require adding a non-trivial amount of new code (as we don't have dynamically creating and destroying kmem_caches anymore). It's perfectly doable, but I'm not sure we need it so much to postpone the merging of the main thing. But I'm happy to hear any arguments why it's not true. Thanks!
