[Patch 2.5.26] ewrk3.c - unaligend access

From: Martin Brulisauer (bruli@uceb.org)
Date: Fri Jul 19 2002 - 10:09:40 EST


Hi,

Here follows a very small patch to the Etherworks3 driver.
On alpha architecture all memory accesses must be done naturally
aligned. The element dev_addr of the struct net_device is of
type unsigned char*. So the cast to 32bit long int for a faster
compare will result in a unaligend trap on the alpha processor.
The current implementation will compare the destination address
of the incoming packet with two instructions. My patch uses
three instructions. The trap on alpha is much more expensive
then one additional integer instruction.

--- linux-2.5.26/drivers/net/ewrk3.c Wed Jul 17 01:49:32 2002
+++ linux/drivers/net/ewrk3.c Fri Jul 19 17:02:11 2002
@@ -133,6 +133,7 @@
    0.42 22-Apr-96 Fix alloc_device() bug <jari@markkus2.fimr.fi>
    0.43 16-Aug-96 Update alloc_device() to conform to de4x5.c
    0.44 08-Nov-01 use library crc32 functions <Matt_Domsch@dell.com>
+ 0.45 19-Jul-02 fix unaligend access on alpha <martin@bruli.net>
 
    =========================================================================
  */
@@ -1008,12 +1009,13 @@
                                                 }
                                                 p = skb->data; /* Look at the dest addr */
                                                 if (p[0] & 0x01) { /* Multicast/Broadcast */
- if ((*(s32 *) & p[0] == -1) && (*(s16 *) & p[4] == -1)) {
+ if ((*(s16 *) & p[0] == -1) && (*(s16 *) & p[2] == -1) && (*(s16 *) & p[4] == -1)) {
                                                                 lp->pktStats.broadcast++;
                                                         } else {
                                                                 lp->pktStats.multicast++;
                                                         }
- } else if ((*(s32 *) & p[0] == *(s32 *) & dev->dev_addr[0]) &&
+ } else if ((*(s16 *) & p[0] == *(s16 *) & dev->dev_addr[0]) &&
+ (*(s16 *) & p[2] == *(s16 *) & dev->dev_addr[2]) &&
                                                            (*(s16 *) & p[4] == *(s16 *) & dev->dev_addr[4])) {
                                                         lp->pktStats.unicast++;
                                                 }
-
Greetings,
Martin
------------------------------------
Martin Brulisauer <martin@bruli.net>
http://www.bruli.net
------------------------------------
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Tue Jul 23 2002 - 22:00:30 EST