On Wed, Apr 27, 2011 at 2:59 PM, Andrew Evans <[email protected]> wrote:
> +#define TNL_F_DF_DEFAULT (1 << 4) /* Set the DF bit if inherit off or
> not IP? */
I'm assuming that the comment wasn't meant to be a question...
> @@ -709,6 +709,14 @@ parse_tunnel_config(const char *name, const char *type,
> if (!strcmp(node->data, "true")) {
> flags |= TNL_F_CSUM;
> }
> + } else if (!strcmp(node->name, "df_inherit")) {
> + if (!strcmp(node->data, "true")) {
> + flags &= TNL_F_DF_INHERIT;
> + }
This should be |= to turn the flag on.
I think these three commits should really be one. This adds a
userspace interface that is broken and the next a feature that is not
documented so they're not really independent pieces.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
