Re: VLAN I/F's and TX queue.

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Eric Dumazet
Date: Friday, May 7, 2010 - 1:53 am

Le vendredi 07 mai 2010 à 10:04 +0200, Joakim Tjernlund a écrit :

dropped on RX path or TX path ?


Congestion notifications can be stacked since commit cbbef5e183079
(vlan/macvlan: propagate transmission state to upper layers)


You try to send packets on eth0.XXX, some are dropped, and accounted for
on eth0.XXX stats. What is wrong with this ?

If you want to avoid this, just add queues to your vlans

ip link add link eth0 eth0.103 txqueuelen 100 type vlan id 103

Patrick what do you think of special casing NET_XMIT_CN ?


diff --git a/net/8021q/vlan_dev.c b/net/8021q/vlan_dev.c
index b5249c5..c671b1a 100644
--- a/net/8021q/vlan_dev.c
+++ b/net/8021q/vlan_dev.c
@@ -327,6 +327,8 @@ static netdev_tx_t vlan_dev_hard_start_xmit(struct sk_buff *skb,
 	len = skb->len;
 	ret = dev_queue_xmit(skb);
 
+	ret = net_xmit_eval(ret);
+
 	if (likely(ret == NET_XMIT_SUCCESS)) {
 		txq->tx_packets++;
 		txq->tx_bytes += len;
@@ -353,6 +355,8 @@ static netdev_tx_t vlan_dev_hwaccel_hard_start_xmit(struct sk_buff *skb,
 	len = skb->len;
 	ret = dev_queue_xmit(skb);
 
+	ret = net_xmit_eval(ret);
+
 	if (likely(ret == NET_XMIT_SUCCESS)) {
 		txq->tx_packets++;
 		txq->tx_bytes += len;


--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: VLAN I/F's and TX queue., Joakim Tjernlund, (Fri May 7, 1:04 am)
Re: VLAN I/F's and TX queue., Eric Dumazet, (Fri May 7, 1:53 am)
Re: VLAN I/F's and TX queue., Joakim Tjernlund, (Fri May 7, 2:29 am)
Re: VLAN I/F's and TX queue., Patrick McHardy, (Mon May 10, 7:26 am)
Re: VLAN I/F's and TX queue., Patrick McHardy, (Mon May 10, 7:33 am)
Re: VLAN I/F's and TX queue., Eric Dumazet, (Mon May 10, 7:36 am)
Re: VLAN I/F's and TX queue., Patrick McHardy, (Mon May 10, 7:41 am)
Re: VLAN I/F's and TX queue., Joakim Tjernlund, (Mon May 10, 7:50 am)
Re: VLAN I/F's and TX queue., Eric Dumazet, (Mon May 10, 7:51 am)
Re: VLAN I/F's and TX queue., Joakim Tjernlund, (Mon May 10, 7:54 am)
Re: VLAN I/F's and TX queue., Eric Dumazet, (Mon May 10, 8:14 am)
Re: VLAN I/F's and TX queue., David Miller, (Sun May 16, 12:40 am)
Re: VLAN I/F's and TX queue., Joakim Tjernlund, (Sun May 16, 7:22 am)