Re: [PATCH 0/2] mark some slabs as visible not mergeable

2018-02-26 Thread David Miller
From: Stephen Hemminger Date: Mon, 26 Feb 2018 13:46:13 -0800 > This is ancient original iproute2 code that dumpster dives into > slabinfo to get summary statistics on active objects. > > 1) open sockets (sock_inode_cache) The sockets inuse counter from /proc/net/sockstat is really suffic

Re: [PATCH 0/2] mark some slabs as visible not mergeable

2018-02-26 Thread Stephen Hemminger
On Mon, 26 Feb 2018 15:15:02 -0500 (EST) David Miller wrote: > From: Stephen Hemminger > Date: Sat, 24 Feb 2018 11:04:52 -0800 > > > This fixes an old bug in iproute2's ss command because it was > > reading slabinfo to get statistics. There isn't a better API > > to do this, and one can argue t

Re: [PATCH 0/2] mark some slabs as visible not mergeable

2018-02-26 Thread David Miller
From: Stephen Hemminger Date: Sat, 24 Feb 2018 11:04:52 -0800 > This fixes an old bug in iproute2's ss command because it was > reading slabinfo to get statistics. There isn't a better API > to do this, and one can argue that /proc is a UAPI that must > not change. Please elaborate what kind of

Re: [PATCH 0/2] mark some slabs as visible not mergeable

2018-02-25 Thread Stephen Hemminger
On Sat, 24 Feb 2018 11:04:52 -0800 Stephen Hemminger wrote: > This fixes an old bug in iproute2's ss command because it was > reading slabinfo to get statistics. There isn't a better API > to do this, and one can argue that /proc is a UAPI that must > not change. > > Therefore this patch set add

Re: [PATCH 0/2] mark some slabs as visible not mergeable

2018-02-24 Thread Matthew Wilcox
On Sat, Feb 24, 2018 at 11:04:52AM -0800, Stephen Hemminger wrote: > This fixes an old bug in iproute2's ss command because it was > reading slabinfo to get statistics. There isn't a better API > to do this, and one can argue that /proc is a UAPI that must > not change. > > Therefore this patch se

[PATCH 0/2] mark some slabs as visible not mergeable

2018-02-24 Thread Stephen Hemminger
This fixes an old bug in iproute2's ss command because it was reading slabinfo to get statistics. There isn't a better API to do this, and one can argue that /proc is a UAPI that must not change. Therefore this patch set adds a flag to slab to give another reason to prevent merging, and then uses