[PATCH] libertas: tweak tx path debugging

Dan Williams dcbw at redhat.com
Wed Nov 28 09:56:03 EST 2007


On Wed, 2007-11-28 at 14:08 +0100, Holger Schurig wrote:
> Make two functions in the TX packet path emit
> their debug messages with LBS_DEB_TX, not LBS_DEB_MAIN.
> 
> Signed-off-by: Holger Schurig <hs4233 at mail.mn-solutions.de>

Acked-by: Dan Williams <dcbw at redhat.com>

> ---
> Before this, any TXed packed did emit some messages with
> "lbsdebug +enter +main", which was a bit annoying.
> 
> Index: wireless-2.6/drivers/net/wireless/libertas/main.c
> ===================================================================
> --- wireless-2.6.orig/drivers/net/wireless/libertas/main.c	2007-11-28 14:35:45.000000000 +0100
> +++ wireless-2.6/drivers/net/wireless/libertas/main.c	2007-11-28 14:37:13.000000000 +0100
> @@ -514,7 +514,7 @@ static int lbs_hard_start_xmit(struct sk
>  	int ret = 0;
>  	struct lbs_private *priv = dev->priv;
>  
> -	lbs_deb_enter(LBS_DEB_NET);
> +	lbs_deb_enter(LBS_DEB_TX);
>  
>  	if (priv->dnld_sent || priv->adapter->TxLockFlag) {
>  		priv->stats.tx_dropped++;
> @@ -528,7 +528,7 @@ static int lbs_hard_start_xmit(struct sk
>  	if (lbs_process_tx(priv, skb) == 0)
>  		dev->trans_start = jiffies;
>  done:
> -	lbs_deb_leave_args(LBS_DEB_NET, "ret %d", ret);
> +	lbs_deb_leave_args(LBS_DEB_TX, "ret %d", ret);
>  	return ret;
>  }
>  
> @@ -564,7 +564,7 @@ static int lbs_pre_start_xmit(struct sk_
>  	struct lbs_private *priv = dev->priv;
>  	int ret;
>  
> -	lbs_deb_enter(LBS_DEB_NET);
> +	lbs_deb_enter(LBS_DEB_TX);
>  
>  	if (priv->adapter->monitormode != LBS_MONITOR_OFF) {
>  		netif_stop_queue(dev);
> @@ -574,7 +574,7 @@ static int lbs_pre_start_xmit(struct sk_
>  	UNSET_MESH_FRAME(skb);
>  
>  	ret = lbs_hard_start_xmit(skb, dev);
> -	lbs_deb_leave_args(LBS_DEB_NET, "ret %d", ret);
> +	lbs_deb_leave_args(LBS_DEB_TX, "ret %d", ret);
>  	return ret;
>  }
>  




More information about the libertas-dev mailing list