On Thu, 11 Oct 2001, Luigi Rizzo wrote:

> I guess it might be marginally more efficient to
> call bcmp() (and rely on it being optimized), or do 3
> comparisons with unsigned short *
>
> > +#define      IS_ETHER_BROADCAST(a) ( \
> > +     ((unsigned short *)(a))[0] == 0xffff && \
> > +     ((unsigned short *)(a))[1] == 0xffff && \
> > +     ((unsigned short *)(a))[2] == 0xffff)

You can't assume a short * is aligned either.

BTW- this also applies to sparc.

>
> Apart from this, have you actually tested bridging on
> the alpha ? The purpose of the warning was also to
> say "look, this code might not work on architectures
> not supporting unaligned accesses, so you might need to
> hack on the code yourself"

Nope. But a better place to say "Bridging is not tested" is to look in a
kernel config file.



To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-net" in the body of the message

Reply via email to