From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
To: Sowmini Varadhan <sowmini.varadhan@oracle.com>
Cc: netdev@vger.kernel.org, davem@davemloft.net,
roopa@cumulusnetworks.com, eric.dumazet@gmail.com,
jose.marchesi@oracle.com
Subject: Re: [PATCHv2 RFC] RTEXT_FILTER_SKIP_STATS support to avoid dumping inet/inet6 stats
Date: Sat, 12 Sep 2015 00:22:44 +0530 [thread overview]
Message-ID: <55F322FC.1050509@linux.vnet.ibm.com> (raw)
In-Reply-To: <20150910213429.GH28585@oracle.com>
On 09/11/2015 03:04 AM, Sowmini Varadhan wrote:
>
> Many commonly used functions like getifaddrs() invoke RTM_GETLINK
> to dump the interface information, and do not need the
> the AF_INET6 statististics that are always returned by default
> from rtnl_fill_ifinfo().
>
> Computing the statistics can be an expensive operation that impacts
> scaling, so it is desirable to avoid this if the information is
> not needed.
>
> This patch adds a the RTEXT_FILTER_SKIP_STATS extended info flag that
> can be passed with netlink_request() to avoid statistics computation
> for the ifinfo path.
>
> Signed-off-by: Sowmini Varadhan <sowmini.varadhan@oracle.com>
> ---
> v2: David Miller comments: pass u32 ext_filter_mask down.
>
> include/net/rtnetlink.h | 3 ++-
> include/uapi/linux/rtnetlink.h | 1 +
> net/core/rtnetlink.c | 2 +-
> net/ipv4/devinet.c | 3 ++-
> net/ipv6/addrconf.c | 13 +++++++++----
> 5 files changed, 15 insertions(+), 7 deletions(-)
Sowmini, Thanks for the patch which is more cleaner way without
breaking current behaviour.
[ Though RTEXT_FILTER_NEED_STATS flag with reverse effect would have
helped immediately :)]
/me waits for the RTEXT_FILTER_SKIP_STATS to be supported in
gccgo/golang, so that it can be used in docker newNetlinkRequest() to
exploit this.
next prev parent reply other threads:[~2015-09-11 18:51 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-10 21:34 [PATCHv2 RFC] RTEXT_FILTER_SKIP_STATS support to avoid dumping inet/inet6 stats Sowmini Varadhan
2015-09-11 18:52 ` Raghavendra K T [this message]
2015-09-11 19:32 ` Sowmini Varadhan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55F322FC.1050509@linux.vnet.ibm.com \
--to=raghavendra.kt@linux.vnet.ibm.com \
--cc=davem@davemloft.net \
--cc=eric.dumazet@gmail.com \
--cc=jose.marchesi@oracle.com \
--cc=netdev@vger.kernel.org \
--cc=roopa@cumulusnetworks.com \
--cc=sowmini.varadhan@oracle.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.