On Thu, Sep 15, 2011 at 6:17 PM, Pravin Shelar <[email protected]> wrote: > diff --git a/include/openvswitch/datapath-protocol.h > b/include/openvswitch/datapath-protocol.h > index 5d765b1..3c79c0f 100644 > --- a/include/openvswitch/datapath-protocol.h > +++ b/include/openvswitch/datapath-protocol.h > struct ovs_vport_stats { > - uint64_t rx_packets; /* total packets received */ > - uint64_t tx_packets; /* total packets transmitted */ > - uint64_t rx_bytes; /* total bytes received */ > - uint64_t tx_bytes; /* total bytes transmitted */ > - uint64_t rx_errors; /* bad packets received */ > - uint64_t tx_errors; /* packet transmit problems */ > - uint64_t rx_dropped; /* no space in linux buffers */ > - uint64_t tx_dropped; /* no space available in linux */ > + uint64_t rx_packets; /* total packets received */ > + uint64_t tx_packets; /* total packets transmitted */ > + uint64_t rx_bytes; /* total bytes received */ > + uint64_t tx_bytes; /* total bytes transmitted */ > + uint64_t rx_errors; /* bad packets received */ > + uint64_t tx_errors; /* packet transmit problems */ > + uint64_t rx_dropped; /* no space in linux buffers */ > + uint64_t tx_dropped; /* no space available in linux */ > };
I was actually referring to the spaces before the type when I mentioned tabs. (I know that the file isn't consistent but it's supposed to be tabs.) Other than that, this looks good. I'm assuming that you'll squash it with the previous patch. The combined result: Acked-by: Jesse Gross <[email protected]> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
