Re: [PATCH net-next] add DOVE extensions for VXLAN
From: Stephen Hemminger <hidden>
Date: 2012-11-13 22:42:52
On Tue, 13 Nov 2012 17:28:17 -0500 David Stevens [off-list ref] wrote:
Stephen Hemminger [off-list ref] wrote on 11/13/2012 04:28:42 PM:quoted
There are some issues with this. 1. DOVE flag is mixing multiple functions (arp and route) together, users may want one without the other.I can separate these.quoted
2. There is an implicit assumption that IP stack has valid IP address in the tenant network (vxlan). This is rarely the case. For security and other reasons, in my opinion the best practice is not to have the bridge as part of the tenant network.No, actually for testing I didn't set an IP address on the tunnel endpoint at all. The neighbor table entries must be in the domain, but they are only used within the domain when the tunnel endpoint is on a bridge and the host has no IP address on that interface.quoted
3. Misses might be common and this could easily be used to DoS the host from a malicious guest.Yes. The management piece can add forwarding table entries with "0.0.0.0" as the dst IP address to disable MAC misses, and neighbor table entries to disable IP misses, but it is our intention to have all reachable destinations with both forwarding table and neighbor table entries and no learning or multicast address (ie, no forwarding of anything that isn't in the forwarding table). And yes, we want a notification for every miss packet. Someone who doesn't want all of them shouldn't use this feature. If we're dropping the "dove" flag in favor of individual flags for each feature, then I could make this into "l2miss" and "l3miss" flags and they should default off, of course. +-DLS
Maybe a OVS style "here is the homeless packet" message is needed. That would allow for controller in user space to populate table on as needed basis.