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]> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
