summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorDavid S. Miller <davem@davemloft.net>2016-04-19 19:49:29 -0400
committerDavid S. Miller <davem@davemloft.net>2016-04-19 19:49:29 -0400
commit35c5845957c7982dac1f525ff3412f8acf0a0385 (patch)
treee27bce362485677d849a5924e9072a09692e4b83 /net
parent18402843bf88c2e9674e1a3a05c73b7d9b09ee05 (diff)
downloadlinux-35c5845957c7982dac1f525ff3412f8acf0a0385.tar.bz2
net: Add helpers for 64-bit aligning netlink attributes.
Suggested-by: Eric Dumazet <eric.dumazet@gmail.com> Suggested-by: Nicolas Dichtel <nicolas.dichtel@6wind.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/core/rtnetlink.c24
1 files changed, 5 insertions, 19 deletions
diff --git a/net/core/rtnetlink.c b/net/core/rtnetlink.c
index 198ca2c99510..d3694a13c85a 100644
--- a/net/core/rtnetlink.c
+++ b/net/core/rtnetlink.c
@@ -878,10 +878,7 @@ static noinline size_t if_nlmsg_size(const struct net_device *dev,
+ nla_total_size(IFNAMSIZ) /* IFLA_QDISC */
+ nla_total_size(sizeof(struct rtnl_link_ifmap))
+ nla_total_size(sizeof(struct rtnl_link_stats))
-#ifndef HAVE_EFFICIENT_UNALIGNED_ACCESS
- + nla_total_size(0) /* IFLA_PAD */
-#endif
- + nla_total_size(sizeof(struct rtnl_link_stats64))
+ + nla_total_size_64bit(sizeof(struct rtnl_link_stats64))
+ nla_total_size(MAX_ADDR_LEN) /* IFLA_ADDRESS */
+ nla_total_size(MAX_ADDR_LEN) /* IFLA_BROADCAST */
+ nla_total_size(4) /* IFLA_TXQLEN */
@@ -1054,22 +1051,11 @@ static noinline_for_stack int rtnl_fill_stats(struct sk_buff *skb,
{
struct rtnl_link_stats64 *sp;
struct nlattr *attr;
+ int err;
-#ifndef HAVE_EFFICIENT_UNALIGNED_ACCESS
- /* IF necessary, add a zero length NOP attribute so that the
- * nla_data() of the IFLA_STATS64 will be 64-bit aligned.
- *
- * The nlattr header is 4 bytes in size, that's why we test
- * if the skb->data _is_ aligned. This NOP attribute, plus
- * nlattr header for IFLA_STATS64, will make nla_data() 8-byte
- * aligned.
- */
- if (IS_ALIGNED((unsigned long)skb->data, 8)) {
- attr = nla_reserve(skb, IFLA_PAD, 0);
- if (!attr)
- return -EMSGSIZE;
- }
-#endif
+ err = nla_align_64bit(skb, IFLA_PAD);
+ if (err)
+ return err;
attr = nla_reserve(skb, IFLA_STATS64,
sizeof(struct rtnl_link_stats64));