netdev - Re: [RFC vlan 03/03]: propagate transmission state

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4A2E8F27.3040202@gmail.com>
Date:	Tue, 09 Jun 2009 18:34:47 +0200
From:	Eric Dumazet <eric.dumazet@...il.com>
To:	Patrick McHardy <kaber@...sh.net>
CC:	netdev@...r.kernel.org
Subject: Re: [RFC vlan 03/03]: propagate transmission state

Patrick McHardy a écrit :
> commit b124eaaf852e982c7af9b130ee81cfc293ae30ff
> Author: Patrick McHardy <kaber@...sh.net>
> Date:   Tue Jun 9 18:01:11 2009 +0200
> 
>     vlan: propagate transmission state
>     
>     Propagate transmission state to upper layers and maintain error statistics.
>     
>     Signed-off-by: Patrick McHardy <kaber@...sh.net>
> 
> diff --git a/net/8021q/vlan_dev.c b/net/8021q/vlan_dev.c
> index 96bad8f..39a8e03 100644
> --- a/net/8021q/vlan_dev.c
> +++ b/net/8021q/vlan_dev.c
> @@ -292,6 +292,8 @@ static int vlan_dev_hard_start_xmit(struct sk_buff *skb, struct net_device *dev)
>  {
>  	struct netdev_queue *txq = netdev_get_tx_queue(dev, 0);
>  	struct vlan_ethhdr *veth = (struct vlan_ethhdr *)(skb->data);
> +	unsigned int len;
> +	int err;
>  
>  	/* Handle non-VLAN frames if they are sent to us, for example by DHCP.
>  	 *
> @@ -317,19 +319,25 @@ static int vlan_dev_hard_start_xmit(struct sk_buff *skb, struct net_device *dev)
>  			vlan_dev_info(dev)->cnt_inc_headroom_on_tx++;
>  	}
>  
> -	txq->tx_packets++;
> -	txq->tx_bytes += skb->len;
> -
>  	skb->dev = vlan_dev_info(dev)->real_dev;
> -	dev_queue_xmit(skb);
> -	return NETDEV_TX_OK;
> +	len = skb->len;
> +	err = dev_queue_xmit(skb);
> +	if (err == NET_XMIT_SUCCESS) {
> +		txq->tx_packets++;
> +		txq->tx_bytes += len;
> +	} else
> +		txq->tx_dropped++;
> +
> +	return err;
>  }
>  
>  static int vlan_dev_hwaccel_hard_start_xmit(struct sk_buff *skb,
>  					    struct net_device *dev)
>  {
>  	struct netdev_queue *txq = netdev_get_tx_queue(dev, 0);
> +	unsigned int len;
>  	u16 vlan_tci;
> +	int err;
>  
>  	vlan_tci = vlan_dev_info(dev)->vlan_id;
>  	vlan_tci |= vlan_dev_get_egress_qos_mask(dev, skb);
> @@ -339,8 +347,15 @@ static int vlan_dev_hwaccel_hard_start_xmit(struct sk_buff *skb,
>  	txq->tx_bytes += skb->len;

It seems you forgot to delete previous txq->tx_bytes += skb->len; && txq->tx_packets++; ?

>  
>  	skb->dev = vlan_dev_info(dev)->real_dev;
> -	dev_queue_xmit(skb);
> -	return NETDEV_TX_OK;
> +	len = skb->len;
> +	err = dev_queue_xmit(skb);
> +	if (err == NET_XMIT_SUCCESS) {
> +		txq->tx_packets++;
> +		txq->tx_bytes += len;
> +	} else
> +		txq->tx_dropped++;
> +
> +	return err;
>  }
>  
>  static int vlan_dev_change_mtu(struct net_device *dev, int new_mtu)
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html