[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20081201093341.537bf03d@extreme>
Date: Mon, 1 Dec 2008 09:33:41 -0800
From: Stephen Hemminger <shemminger@...tta.com>
To: Benjamin Thery <benjamin.thery@...l.net>
Cc: Dave Miller <davem@...emloft.net>,
netdev hem <netdev@...r.kernel.org>
Subject: Re: [PATCH] net: fix /proc/net/ip_mr_cache display
On Mon, 01 Dec 2008 17:49:56 +0100
Benjamin Thery <benjamin.thery@...l.net> wrote:
> Argh, my patch breaks iproute2 command: ip mroute show
>
> iproute2 uses sscanf() to read /proc/net/ip_mr_cache and expects 6
> fields to be read. For the unresolved entries my patch only displays
> the first three fields. As a consequence, 'ip mroute show' skips the
> unresolved entries. :(
>
> So we can
> - either forget this patch and keep displaying garbage information
> in /proc/net/ip[6]_mr_cache for the unresolved entries. (No one
> complained before)
> - or may be we can just print '-' or 0 for the fields that have no
> data associated in the unresolved case.
>
> Tell me what you prefer.
>
> Benjamin
/proc file formats are part of the Linux ABI and can't change!
--
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
Powered by blists - more mailing lists