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

Pravin Shelar pshelar at nicira.com
Tue Sep 13 00:15:41 UTC 2011


On Mon, Sep 12, 2011 at 4:53 PM, Jesse Gross <jesse at nicira.com> wrote:
> 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>
>
pushed.

Thanks,
Pravin.



More information about the dev mailing list