Re: [PATCH] net/ipv4/arp.c, kernel 2.4.6 - PROXY_ARP bug on shaper device

David S. Miller (davem@redhat.com)
Sat, 28 Jul 2001 04:43:12 -0700 (PDT)


Roberto Arcomano writes:
> Solution: The patch investigates about the shaper device
> (it compares string with "shaper"): if yes, we
> search the real interface attached to shaper
> (we read value of priv->dev) and we use that to
> make the classic proxy arp devices compare.
>
> Problems: The patch uses routine "strncmp" which is not the
> best thing (I guess).

This is gross, make a device flag and have shaper set that
flag during shaper device init, then have ARP test it.

Make the name of the flag reflect what the attribute is that shaper
has (and potentially other devices could have) which makes this ARP
special case necessary. Ie. don't name the flag NETIF_F_SHAPER :-)

Later,
David S. Miller
davem@redhat.com
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/