netdev - Re: [PATCH iproute2] update obsolete fields in slabstat

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAC2pzGe3Yd6sjeQG4UYcf9jyK4txjVR-CADzHEtkevMhWDTCOA@mail.gmail.com>
Date:	Fri, 16 Jan 2015 10:27:11 +0800
From:	Bryton Lee <brytonlee01@...il.com>
To:	netdev@...r.kernel.org, stephen@...workplumber.org
Cc:	Bryton Lee <brytonlee01@...il.com>
Subject: Re: [PATCH iproute2] update obsolete fields in slabstat_ids[]

Hi Stephen:

This is the first time I submit a patch to iproute2, I forgot CC to a
iproute2's maintainer. It's my fault. Could you please review my
patch? :)

this patch can help some old system get TCP info correctly.  such as
Red Hat 5u7,  although  is not perfect.  ss searching  tcp_bind_bucket
and request_sock_TCP in /proc/slabinfo will fail on current upstream
kernel,
because on current upstream kernel slab tcp_bind_bucket and
request_sock_TCP have merged into kmalloc-xx when they are creating.
so  in /proc/slabinfo  we cannot see them any more.

I have submitted other patch to mm subsystem to prevent such merging,
but it still not accepted. (Yep, finally, I know my patch has problem,
it doesn't make any sense, but may be we should still think about how
to prevent slab merging in slab layer. )

On Mon, Dec 29, 2014 at 8:16 PM, Bryton Lee <brytonlee01@...il.com> wrote:
> From: Bryton Lee <brytonlee01@...il.com>
> Date: Mon, 29 Dec 2014 19:56:23 +0800
> Subject: [PATCH] tcp_open_request and tcp_tw_bucket are obsolete in current
>  stable kernel version, replace by request_sock_TCP and tw_sock_TCP
>
> Signed-off-by: Bryton Lee <brytonlee01@...il.com>
> ---
>  misc/ss.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/misc/ss.c b/misc/ss.c
> index f0c7b34..24d58c7 100644
> --- a/misc/ss.c
> +++ b/misc/ss.c
> @@ -506,8 +506,8 @@ static const char *slabstat_ids[] =
>  {
>      "sock",
>      "tcp_bind_bucket",
> -    "tcp_tw_bucket",
> -    "tcp_open_request",
> +    "tw_sock_TCP",
> +    "request_sock_TCP",
>      "skbuff_head_cache",
>  };
>
> --
> 2.0.5
>
> --
> Best Regards
>
> Bryton.Lee
>

-- 
Best Regards

Bryton.Lee
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html