From: Patrick McHardy
Sent: 1/6/2006 12:52:34 PM

> Randy.Dunlap wrote:
> > On Fri, 6 Jan 2006, Patrick McHardy wrote:
> > 
> >>>--- x/net/atm/br2684.c     2006-01-02 21:21:10.000000000 -0600
> >>>+++ y/net/atm/br2684.c     2006-01-06 12:34:47.000000000 -0600
> >>>@@ -295,7 +295,7 @@ static inline __be16 br_type_trans(struc
> >>>   unsigned char *rawp;
> >>>   eth = eth_hdr(skb);
> >>>
> >>>-  if (*eth->h_dest & 1) {
> >>>+  if (is_multicast_ether_addr(eth->h_dest)) {
> >>
> >>This is not equivalent, is_multicast_ether_addr() ignores
> >>addresses starting with 0xff.
> > 
> > It used to.  Not today afaict.
> 
> You're right, Stephen changed it two days ago.

That's why I said the newer is_multicast_ether_addr(). Sorry for the confusion.

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to