diff options
author | Pravin B Shelar <pshelar@nicira.com> | 2013-05-13 11:22:34 -0400 |
---|---|---|
committer | Jesse Gross <jesse@nicira.com> | 2013-06-14 18:09:10 -0400 |
commit | 91b7514cdff406ad8f63d09b74f664c37bed2e01 (patch) | |
tree | e74b566546cfb0ec1b48baa9cc39acd647256bd9 /net/openvswitch/vport-netdev.c | |
parent | cbd531bebb02bc6c0fc3619a2cfc32f7d8843b18 (diff) |
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-netdev.c')
-rw-r--r-- | net/openvswitch/vport-netdev.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/net/openvswitch/vport-netdev.c b/net/openvswitch/vport-netdev.c index 4f01c6d2ffa4..43712217a372 100644 --- a/net/openvswitch/vport-netdev.c +++ b/net/openvswitch/vport-netdev.c | |||
@@ -170,7 +170,7 @@ static int netdev_send(struct vport *vport, struct sk_buff *skb) | |||
170 | net_warn_ratelimited("%s: dropped over-mtu packet: %d > %d\n", | 170 | net_warn_ratelimited("%s: dropped over-mtu packet: %d > %d\n", |
171 | netdev_vport->dev->name, | 171 | netdev_vport->dev->name, |
172 | packet_length(skb), mtu); | 172 | packet_length(skb), mtu); |
173 | goto error; | 173 | goto drop; |
174 | } | 174 | } |
175 | 175 | ||
176 | skb->dev = netdev_vport->dev; | 176 | skb->dev = netdev_vport->dev; |
@@ -179,9 +179,8 @@ static int netdev_send(struct vport *vport, struct sk_buff *skb) | |||
179 | 179 | ||
180 | return len; | 180 | return len; |
181 | 181 | ||
182 | error: | 182 | drop: |
183 | kfree_skb(skb); | 183 | kfree_skb(skb); |
184 | ovs_vport_record_error(vport, VPORT_E_TX_DROPPED); | ||
185 | return 0; | 184 | return 0; |
186 | } | 185 | } |
187 | 186 | ||