arp_tables: ifname_compare() can assume 16bit alignment
authorEric Dumazet <dada1@cosmosbay.com>
Tue, 24 Mar 2009 21:15:22 +0000 (14:15 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 24 Mar 2009 21:15:22 +0000 (14:15 -0700)
Arches without efficient unaligned access can still perform a loop
assuming 16bit alignment in ifname_compare()

Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/netfilter/arp_tables.c

index 64a7c6ce0b98a8d6dc57f6625c1c001d6dea923d..84b9c179df5145fb998e54963496fe7dfbc62573 100644 (file)
@@ -76,6 +76,7 @@ static inline int arp_devaddr_compare(const struct arpt_devaddr_info *ap,
 /*
  * Unfortunatly, _b and _mask are not aligned to an int (or long int)
  * Some arches dont care, unrolling the loop is a win on them.
+ * For other arches, we only have a 16bit alignement.
  */
 static unsigned long ifname_compare(const char *_a, const char *_b, const char *_mask)
 {
@@ -95,10 +96,13 @@ static unsigned long ifname_compare(const char *_a, const char *_b, const char *
        BUILD_BUG_ON(IFNAMSIZ > 4 * sizeof(unsigned long));
 #else
        unsigned long ret = 0;
+       const u16 *a = (const u16 *)_a;
+       const u16 *b = (const u16 *)_b;
+       const u16 *mask = (const u16 *)_mask;
        int i;
 
-       for (i = 0; i < IFNAMSIZ; i++)
-               ret |= (_a[i] ^ _b[i]) & _mask[i];
+       for (i = 0; i < IFNAMSIZ/sizeof(u16); i++)
+               ret |= (a[i] ^ b[i]) & mask[i];
 #endif
        return ret;
 }