Commit 550bce59 authored by Roopa Prabhu's avatar Roopa Prabhu Committed by David S. Miller

rtnetlink: rtnl_fill_stats: avoid an unnecssary stats copy

This patch passes netlink attr data ptr directly to dev_get_stats
thus elimiating a stats copy.
Suggested-by: default avatarDavid Miller <davem@davemloft.net>
Signed-off-by: default avatarRoopa Prabhu <roopa@cumulusnetworks.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent ccd37cff
...@@ -808,11 +808,6 @@ static void copy_rtnl_link_stats(struct rtnl_link_stats *a, ...@@ -808,11 +808,6 @@ static void copy_rtnl_link_stats(struct rtnl_link_stats *a,
a->rx_nohandler = b->rx_nohandler; a->rx_nohandler = b->rx_nohandler;
} }
static void copy_rtnl_link_stats64(void *v, const struct rtnl_link_stats64 *b)
{
memcpy(v, b, sizeof(*b));
}
/* All VF info */ /* All VF info */
static inline int rtnl_vfinfo_size(const struct net_device *dev, static inline int rtnl_vfinfo_size(const struct net_device *dev,
u32 ext_filter_mask) u32 ext_filter_mask)
...@@ -1054,25 +1049,23 @@ static int rtnl_phys_switch_id_fill(struct sk_buff *skb, struct net_device *dev) ...@@ -1054,25 +1049,23 @@ static int rtnl_phys_switch_id_fill(struct sk_buff *skb, struct net_device *dev)
static noinline_for_stack int rtnl_fill_stats(struct sk_buff *skb, static noinline_for_stack int rtnl_fill_stats(struct sk_buff *skb,
struct net_device *dev) struct net_device *dev)
{ {
const struct rtnl_link_stats64 *stats; struct rtnl_link_stats64 *sp;
struct rtnl_link_stats64 temp;
struct nlattr *attr; struct nlattr *attr;
stats = dev_get_stats(dev, &temp); attr = nla_reserve(skb, IFLA_STATS64,
sizeof(struct rtnl_link_stats64));
attr = nla_reserve(skb, IFLA_STATS,
sizeof(struct rtnl_link_stats));
if (!attr) if (!attr)
return -EMSGSIZE; return -EMSGSIZE;
copy_rtnl_link_stats(nla_data(attr), stats); sp = nla_data(attr);
dev_get_stats(dev, sp);
attr = nla_reserve(skb, IFLA_STATS64, attr = nla_reserve(skb, IFLA_STATS,
sizeof(struct rtnl_link_stats64)); sizeof(struct rtnl_link_stats));
if (!attr) if (!attr)
return -EMSGSIZE; return -EMSGSIZE;
copy_rtnl_link_stats64(nla_data(attr), stats); copy_rtnl_link_stats(nla_data(attr), sp);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment