On Wed, Oct 5, 2011 at 6:45 PM, Jesse Gross <[email protected]> wrote:
> On Wed, Oct 5, 2011 at 2:21 PM, Pravin B Shelar <[email protected]> wrote:
>> diff --git a/include/openvswitch/datapath-protocol.h
>> b/include/openvswitch/datapath-protocol.h
>> index 3db960a..80a76d6 100644
>> --- a/include/openvswitch/datapath-protocol.h
>> +++ b/include/openvswitch/datapath-protocol.h
>> @@ -230,14 +230,11 @@ enum ovs_vport_cmd {
>> * @OVS_VPORT_ATTR_STATS: A &struct ovs_vport_stats giving statistics for
>> * packets sent or received through the vport.
>> * @OVS_VPORT_ATTR_ADDRESS: A 6-byte Ethernet address for the vport.
>> - * @OVS_VPORT_ATTR_IFINDEX: ifindex of the underlying network device, if
>> any.
>> *
>> * These attributes follow the &struct ovs_header within the Generic Netlink
>> * payload for %OVS_VPORT_* commands.
>> *
>> * All attributes applicable to a given port are present in notifications.
>> - * This means that, for example, a vport that has no corresponding network
>> - * device would omit %OVS_VPORT_ATTR_IFINDEX.
>
> You can remove the sentence above this as well, since all attributes
> are now included all the time.
>
> Otherwise looks good:
> Acked-by: Jesse Gross <[email protected]>
>
Thanks, pushed to master.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
