On Thu, Dec 15, 2005 at 06:54:21PM -0600, Michael Ellerman wrote:

> +     /* XXX Why are we checking for 0xff here ? */
> +     return (addr[0] == 0xff) || (!is_multicast_ether_addr(addr)
> +             && !is_zero_ether_addr(addr));

That is a good question...what does a MAC address like
FF:xx:xx:xx:xx:xx signify?  Anyone know?

I can understand why you made that change (compatibility), but if noone
has a good answer as to what it's for, I'd be tempted to chuck it...

John
-- 
John W. Linville
[EMAIL PROTECTED]
-
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