From: "Kris Katterjohn" <[EMAIL PROTECTED]>
Date: Fri, 6 Jan 2006 13:01:54 -0800
> 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.
Applied, thanks Kris.
-
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
