[ovs-dev] [PATCH] Strip down vport interface : OVS_VPORT_ATTR_MTU

[Jesse Gross]

On Mon, Sep 12, 2011 at 2:11 PM, Pravin Shelar <pshelar at nicira.com> wrote:
> diff --git a/include/openvswitch/datapath-protocol.h b/include/openvswitch/datapath-protocol.h
> index c1c9ef0..49a122c 100644
> --- a/include/openvswitch/datapath-protocol.h
> +++ b/include/openvswitch/datapath-protocol.h
> @@ -237,8 +235,8 @@ enum ovs_vport_cmd {
>  * %OVS_VPORT_ATTR_NAME attributes are required.  %OVS_VPORT_ATTR_PORT_NO is
>  * optional; if not specified a free port number is automatically selected.
>  * Whether %OVS_VPORT_ATTR_OPTIONS is required or optional depends on the type
> - * of vport.  %OVS_VPORT_ATTR_STATS, %OVS_VPORT_ATTR_ADDRESS, and
> - * %OVS_VPORT_ATTR_MTU are optional, and other attributes are ignored.
> + * of vport.  %OVS_VPORT_ATTR_STATS, %OVS_VPORT_ATTR_ADDRESS are optional,

There should be an "and" between those two OVS_VPORT_ATTR_ types.

Otherwise looks good:
Acked-by: Jesse Gross <jesse at nicira.com>