On 2010-11-03 at 11:12:12 +0100, Sven Eckelmann <[email protected]> wrote:
> On Wednesday 03 November 2010 10:59:02 you wrote:
> > Replace custom ethernet address check functions by calls to the helpers
> > in linux/etherdevice.h
>
> Have you proven that the addresses are always two bytes aligned in memory?
> Afaik this function needs it.
I don't think they need to be two bytes aligned, but I might be wrong.
> > In one case where the address was tested for broadcast and multicast
> > address, the broadcast address check can be omitted as broadcast is also
> > a multicast address.
>
> We need to distinguish between these two types for different optimizations
> (research is currently done for multicast over mesh - but currently not part
> of batman-adv).
I was refering to the following part of the patch:
- if (is_bcast(ethhdr->h_dest) || is_mcast(ethhdr->h_dest)) {
+ if (is_multicast_ether_addr(ethhdr->h_dest)) {
I think this change should be legitimate as the same branch is done for
multicast and broadcast addresses.
Thanks
Tobias
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
