On Mon, Sep 19, 2011 at 4:21 PM, Pravin Shelar <[email protected]> wrote: > On Mon, Sep 19, 2011 at 4:14 PM, Jesse Gross <[email protected]> wrote: >> The 'u' in uint64_t apparently got clipped off of the tx_dropped >> member of struct vport_stats in between review and push, incorrectly >> making this a signed type. >> >> CC: Pravin Shelar <[email protected]> >> Signed-off-by: Jesse Gross <[email protected]> >> --- >> include/openvswitch/datapath-protocol.h | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/include/openvswitch/datapath-protocol.h >> b/include/openvswitch/datapath-protocol.h >> index 8ed1c7f..fc7cc1f 100644 >> --- a/include/openvswitch/datapath-protocol.h >> +++ b/include/openvswitch/datapath-protocol.h >> @@ -143,7 +143,7 @@ struct ovs_vport_stats { >> uint64_t rx_errors; /* bad packets received */ >> uint64_t tx_errors; /* packet transmit problems */ >> uint64_t rx_dropped; /* no space in linux buffers */ >> - int64_t tx_dropped; /* no space available in linux */ >> + uint64_t tx_dropped; /* no space available in linux */ >> }; >> >> /* Logical ports. */ >> -- >> 1.7.4.1 >> > > looks good. > > Acked-by: Pravin Shelar <[email protected]>
Thanks, pushed. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
