diff options
author | Pravin B Shelar <pshelar@nicira.com> | 2013-05-13 08:22:34 -0700 |
---|---|---|
committer | Jesse Gross <jesse@nicira.com> | 2013-06-14 15:09:10 -0700 |
commit | 91b7514cdff406ad8f63d09b74f664c37bed2e01 (patch) | |
tree | e74b566546cfb0ec1b48baa9cc39acd647256bd9 /net/openvswitch/vport.c | |
parent | cbd531bebb02bc6c0fc3619a2cfc32f7d8843b18 (diff) | |
download | linux-stable-91b7514cdff406ad8f63d09b74f664c37bed2e01.tar.gz linux-stable-91b7514cdff406ad8f63d09b74f664c37bed2e01.tar.bz2 linux-stable-91b7514cdff406ad8f63d09b74f664c37bed2e01.zip |
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>
Diffstat (limited to 'net/openvswitch/vport.c')
-rw-r--r-- | net/openvswitch/vport.c | 9 |
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; } |