| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20151216115054.47dca715@griffin> Date: Wed, 16 Dec 2015 11:50:53 +0100 From: Jiri Benc <jbenc@...hat.com> To: Paolo Abeni <pabeni@...hat.com> Cc: shemming@...cade.com, netdev@...r.kernel.org, Roopa Prabhu <roopa@...ulusnetworks.com> Subject: Re: [PATCH iproute2 1/3] vxlan: add support for collect metadata flag On Tue, 15 Dec 2015 16:48:25 +0100, Paolo Abeni wrote: > This patch add support for IFLA_VXLAN_COLLECT_METADATA via the > 'collectmetadata' keyword to the vxlan link. Could we give this more understandable name? I've never liked the COLLECT_METADATA name as it describes the actual implementation, not the intention of the flag. The name should express that the vxlan interface is driven by an external control plane instead of the internal fdb. This could be ovs, a route, tc action, etc. Something like "external" would be better name, I think. Also, as a related remark, please note that IFLA_VXLAN_COLLECT_METADATA is mutually exclusive with many other flags. It's currently not enforced correctly at the kernel level (instead, you get weird behavior). I'll fix this on the kernel side but we may want to enforce this on the iproute2 side, too, to get a better error message. This is not to be addressed by this patch, though. Jiri -- Jiri Benc -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists