Revert "ipv4: arp announce, arp_proxy and windows ip conflict verification"
commitded7afa853a7771eeae2067bc8253ccb427bffa9
authorEric W. Biederman <ebiederm@xmission.com>
Tue, 30 Jun 2009 16:27:17 +0000 (30 16:27 +0000)
committerGreg Kroah-Hartman <gregkh@suse.de>
Mon, 20 Jul 2009 03:38:54 +0000 (19 20:38 -0700)
treee4768132904cfbad9efbbccee83f8587d305b3c7
parent3c92900d9a4afb176d3de335dc0da0198660a244
Revert "ipv4: arp announce, arp_proxy and windows ip conflict verification"

commit f8a68e752bc4e39644843403168137663c984524 upstream.

This reverts commit 73ce7b01b4496a5fbf9caf63033c874be692333f.

After discovering that we don't listen to gratuitious arps in 2.6.30
I tracked the failure down to this commit.

The patch makes absolutely no sense.  RFC2131 RFC3927 and RFC5227.
are all in agreement that an arp request with sip == 0 should be used
for the probe (to prevent learning) and an arp request with sip == tip
should be used for the gratitous announcement that people can learn
from.

It appears the author of the broken patch got those two cases confused
and modified the code to drop all gratuitous arp traffic.  Ouch!

Signed-off-by: Eric W. Biederman <ebiederm@aristanetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
net/ipv4/arp.c