[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1327805865.2805.23.camel@edumazet-laptop>
Date: Sun, 29 Jan 2012 03:57:45 +0100
From: Eric Dumazet <eric.dumazet@...il.com>
To: Ryan Hajdaj <rhajdaj@...il.com>
Cc: "David S. Miller" <davem@...emloft.net>, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 001/001] net: Add tx_queue/rx_queue info to
/proc/net/unix
Le samedi 28 janvier 2012 à 21:37 -0500, Ryan Hajdaj a écrit :
> From: Ryan Hajdaj <rhajdaj@...il.com>
>
> Add tx_queue/rx_queue info to /proc/net/unix (unix_seq_show)
>
> Signed-off-by: Ryan Hajdaj <rhajdaj@...il.com>
> Tested-by: Ryan Hajdaj <rhajdaj@...il.com>
> ---
> tx_queue/rx_queue info is useful in systems with multiple processes
> using Unix Domain sockets to communicate, as it can be used to check
> process health "at a glance".
> Unit tested on Unix stream and datagram sockets.
> --- linux-3.3-rc1/net/unix/af_unix.c.orig 2012-01-28 20:36:33.000000000 +0000
> +++ linux-3.3-rc1/net/unix/af_unix.c 2012-01-28 19:09:18.000000000 +0000
> @@ -2286,13 +2286,14 @@ static int unix_seq_show(struct seq_file
>
> if (v == SEQ_START_TOKEN)
> seq_puts(seq, "Num RefCount Protocol Flags Type St "
> - "Inode Path\n");
> + "tx_queue rx_queue Inode Path\n");
> else {
> struct sock *s = v;
> struct unix_sock *u = unix_sk(s);
> unix_state_lock(s);
>
> - seq_printf(seq, "%pK: %08X %08X %08X %04X %02X %5lu",
> + seq_printf(seq,
> + "%pK: %08X %08X %08X %04X %02X %08lX %08lX %5lu",
> s,
> atomic_read(&s->sk_refcnt),
> 0,
> @@ -2301,6 +2302,8 @@ static int unix_seq_show(struct seq_file
> s->sk_socket ?
> (s->sk_state == TCP_ESTABLISHED ? SS_CONNECTED : SS_UNCONNECTED) :
> (s->sk_state == TCP_ESTABLISHED ? SS_CONNECTING : SS_DISCONNECTING),
> + unix_outq_len(s),
> + (unix_inq_len(s) < 0 ? 0 : unix_inq_len(s)),
> sock_i_ino(s));
>
> if (u->addr) {
You missed some discussion on netdev about this kind of patches.
/proc/net/unix format is frozen.
Pavel Emelyanov did the work to support unix_diag, where you can find
all you need.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists