On Tue, Aug 5, 2014 at 4:38 PM, Jarno Rajahalme <jrajaha...@nicira.com> wrote:
> diff --git a/datapath/linux/compat/include/linux/openvswitch.h 
> b/datapath/linux/compat/include/linux/openvswitch.h
> index 271a14e..64761b4 100644
> --- a/datapath/linux/compat/include/linux/openvswitch.h
> +++ b/datapath/linux/compat/include/linux/openvswitch.h
> @@ -590,6 +590,8 @@ struct ovs_action_hash {
>   * indicate the new packet contents. This could potentially still be
>   * %ETH_P_MPLS if the resulting MPLS label stack is not empty.  If there
>   * is no MPLS label stack, as determined by ethertype, no action is taken.
> + * @OVS_ACTION_ATTR_METER: Run packet through a meter, which may drop the
> + * packet, or modify the packet (e.g., change the DSCP field).
>   *
>   * Only a single header can be set with a single %OVS_ACTION_ATTR_SET.  Not 
> all
>   * fields within a header are modifiable, e.g. the IPv4 protocol and fragment
> @@ -608,6 +610,7 @@ enum ovs_action_attr {
>         OVS_ACTION_ATTR_HASH,         /* struct ovs_action_hash. */
>         OVS_ACTION_ATTR_PUSH_MPLS,    /* struct ovs_action_push_mpls. */
>         OVS_ACTION_ATTR_POP_MPLS,     /* __be16 ethertype. */
> +       OVS_ACTION_ATTR_METER,        /* u32 meter number. */

One problem with sharing a single interface file across platforms is
that we can't do this unless there is a Linux implementation. We can
pick a high number like before and change it later but that only works
for tightly coupled interfaces like dpif-netdev. It wouldn't really be
OK if you had an implementation on say, Windows but not Linux.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to