openvswitch: Unify vport error stats handling.

Following patch changes vport->send return type so that vport
layer can do error accounting.

Signed-off-by: Pravin B Shelar <pshelar@nicira.com>
Signed-off-by: Jesse Gross <jesse@nicira.com>

1 files changed, 7 insertions, 2 deletions

diff --git a/net/openvswitch/vport.c b/net/openvswitch/vport.c
index 720623190eaa..7f20f6d1be94 100644
--- a/net/openvswitch/vport.c
+++ b/net/openvswitch/vport.c
@@ -351,7 +351,7 @@ int ovs_vport_send(struct vport *vport, struct sk_buff *skb)
 {
         int sent = vport->ops->send(vport, skb);
 
-        if (likely(sent)) {
+        if (likely(sent > 0)) {
                 struct pcpu_tstats *stats;
 
                 stats = this_cpu_ptr(vport->percpu_stats);
@@ -360,7 +360,12 @@ int ovs_vport_send(struct vport *vport, struct sk_buff *skb)
                 stats->tx_packets++;
                 stats->tx_bytes += sent;
                 u64_stats_update_end(&stats->syncp);
-        }
+        } else if (sent < 0) {
+                ovs_vport_record_error(vport, VPORT_E_TX_ERROR);
+                kfree_skb(skb);
+        } else
+                ovs_vport_record_error(vport, VPORT_E_TX_DROPPED);
+
         return sent;
 }