]> Pileus Git - ~andy/linux/commitdiff
appletalk: Use ether_addr_copy
authorJoe Perches <joe@perches.com>
Mon, 20 Jan 2014 17:52:15 +0000 (09:52 -0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 22 Jan 2014 02:13:04 +0000 (18:13 -0800)
Use ether_addr_copy instead of memcpy(a, b, ETH_ALEN) to
save some cycles on arm and powerpc.

Convert struct aarp_entry.hwaddr[6] to hwaddr[ETH_ALEN].

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/appletalk/aarp.c

index 690356fa52b99a2d66a8522c448c51ba7eedba5f..d0b7be15dbb9dac631dd0da09789c1018ec67ed4 100644 (file)
@@ -67,7 +67,7 @@ struct aarp_entry {
        unsigned long           expires_at;
        struct atalk_addr       target_addr;
        struct net_device       *dev;
-       char                    hwaddr[6];
+       char                    hwaddr[ETH_ALEN];
        unsigned short          xmit_count;
        struct aarp_entry       *next;
 };
@@ -134,7 +134,7 @@ static void __aarp_send_query(struct aarp_entry *a)
        eah->pa_len      = AARP_PA_ALEN;
        eah->function    = htons(AARP_REQUEST);
 
-       memcpy(eah->hw_src, dev->dev_addr, ETH_ALEN);
+       ether_addr_copy(eah->hw_src, dev->dev_addr);
 
        eah->pa_src_zero = 0;
        eah->pa_src_net  = sat->s_net;
@@ -181,7 +181,7 @@ static void aarp_send_reply(struct net_device *dev, struct atalk_addr *us,
        eah->pa_len      = AARP_PA_ALEN;
        eah->function    = htons(AARP_REPLY);
 
-       memcpy(eah->hw_src, dev->dev_addr, ETH_ALEN);
+       ether_addr_copy(eah->hw_src, dev->dev_addr);
 
        eah->pa_src_zero = 0;
        eah->pa_src_net  = us->s_net;
@@ -190,7 +190,7 @@ static void aarp_send_reply(struct net_device *dev, struct atalk_addr *us,
        if (!sha)
                memset(eah->hw_dst, '\0', ETH_ALEN);
        else
-               memcpy(eah->hw_dst, sha, ETH_ALEN);
+               ether_addr_copy(eah->hw_dst, sha);
 
        eah->pa_dst_zero = 0;
        eah->pa_dst_net  = them->s_net;
@@ -232,7 +232,7 @@ static void aarp_send_probe(struct net_device *dev, struct atalk_addr *us)
        eah->pa_len      = AARP_PA_ALEN;
        eah->function    = htons(AARP_PROBE);
 
-       memcpy(eah->hw_src, dev->dev_addr, ETH_ALEN);
+       ether_addr_copy(eah->hw_src, dev->dev_addr);
 
        eah->pa_src_zero = 0;
        eah->pa_src_net  = us->s_net;
@@ -790,7 +790,7 @@ static int aarp_rcv(struct sk_buff *skb, struct net_device *dev,
                        break;
 
                /* We can fill one in - this is good. */
-               memcpy(a->hwaddr, ea->hw_src, ETH_ALEN);
+               ether_addr_copy(a->hwaddr, ea->hw_src);
                __aarp_resolved(&unresolved[hash], a, hash);
                if (!unresolved_count)
                        mod_timer(&aarp_timer,