vlan: fix typo in vlan_dev_hard_start_xmit()

Wei Yongjun · davem330 · commit 307f73df2b98 · 2011-06-01T21:08:47.000-07:00
commit 4af429d (vlan: lockless transmit path) have a typo in vlan_dev_hard_start_xmit(), using u64_stats_update_begin() to end the stat update, it should be u64_stats_update_end(). Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com> Reviewed-by: WANG Cong <xiyou.wangcong@gmail.com> Acked-by: Eric Dumazet <eric.dumazet@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
diff --git a/net/8021q/vlan_dev.c b/net/8021q/vlan_dev.c
@@ -165,7 +165,7 @@ static netdev_tx_t vlan_dev_hard_start_xmit(struct sk_buff *skb,
 		u64_stats_update_begin(&stats->syncp);
 		stats->tx_packets++;
 		stats->tx_bytes += len;
-		u64_stats_update_begin(&stats->syncp);
+		u64_stats_update_end(&stats->syncp);
 	} else {
 		this_cpu_inc(vlan_dev_info(dev)->vlan_pcpu_stats->tx_dropped);
 	}

Original file line number	Diff line number	Diff line change
`@@ -165,7 +165,7 @@ static netdev_tx_t vlan_dev_hard_start_xmit(struct sk_buff *skb,`
`165`	`165`	`u64_stats_update_begin(&stats->syncp);`
`166`	`166`	`stats->tx_packets++;`
`167`	`167`	`stats->tx_bytes += len;`
`168`		`- u64_stats_update_begin(&stats->syncp);`
	`168`	`+ u64_stats_update_end(&stats->syncp);`
`169`	`169`	`} else {`
`170`	`170`	`this_cpu_inc(vlan_dev_info(dev)->vlan_pcpu_stats->tx_dropped);`
`171`	`171`	`}`