> >> /**
> >> * enum ovs_action_attr - Action types.
> >> *
> >> @@ -633,6 +642,10 @@ enum ovs_action_attr {
> >> * data immediately followed by a
> mask.
> >> * The data must be zero for the
> unmasked
> >> * bits. */
> >> +#ifndef __KERNEL__
> >> + OVS_ACTION_ATTR_TUNNEL_PUSH,
> >> + OVS_ACTION_ATTR_TUNNEL_POP,
> >> +#endif
> >> __OVS_ACTION_ATTR_MAX
> >> };
> >
> > Should we remove the #ifndef so the action values are reserved in the
> > kernel datapath as well? We don't want conflicting actions types later on.
> >
> There is no plan for implementing this action for kernel datapath, so
> for now I do not want to reserve number for the action.
I can see an opportunity to support tunnel encapsulation through hardware
offload as well.
Dave
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
