arp: make arp_hdr_len() return unsigned int
authorAlexey Dobriyan <adobriyan@gmail.com>
Tue, 26 Sep 2017 20:12:28 +0000 (23:12 +0300)
committerDavid S. Miller <davem@davemloft.net>
Thu, 28 Sep 2017 17:29:36 +0000 (10:29 -0700)
Negative ARP header length are not a thing.

Constify arguments while I'm at it.

Space savings:

add/remove: 0/0 grow/shrink: 0/1 up/down: 0/-3 (-3)
function                        old     new   delta
arpt_do_table                  1163    1160      -3

Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bond_main.c
include/linux/if_arp.h

index c99dc59d729b30dd3ffc4a673deae38630fb0a84..d2e94b8559f0f2667f716ad5a7c7bc75268453ab 100644 (file)
@@ -2491,7 +2491,8 @@ int bond_arp_rcv(const struct sk_buff *skb, struct bonding *bond,
        struct slave *curr_active_slave, *curr_arp_slave;
        unsigned char *arp_ptr;
        __be32 sip, tip;
-       int alen, is_arp = skb->protocol == __cpu_to_be16(ETH_P_ARP);
+       int is_arp = skb->protocol == __cpu_to_be16(ETH_P_ARP);
+       unsigned int alen;
 
        if (!slave_do_arp_validate(bond, slave)) {
                if ((slave_do_arp_validate_only(bond) && is_arp) ||
index 3355efc8978164cd56c0b77c862511bb11122a2e..6756fea18b69f6176bf73c749a977ba15e37d840 100644 (file)
@@ -31,7 +31,7 @@ static inline struct arphdr *arp_hdr(const struct sk_buff *skb)
        return (struct arphdr *)skb_network_header(skb);
 }
 
-static inline int arp_hdr_len(struct net_device *dev)
+static inline unsigned int arp_hdr_len(const struct net_device *dev)
 {
        switch (dev->type) {
 #if IS_ENABLED(CONFIG_FIREWIRE_NET)