Re: throughput problem/bisect with rtl8187B

[Larry Finger <Larry.Finger@lwfinger.net>]

Herton Ronaldo Krzesinski wrote:
> Ok, Looks like sifs or eifs setting then expect values in another format or
> have a different meaning may be (or just cause some hw bug), it's strange,
> I suspect sifs value is the culprit. In the patch I just reverted to values vendor
> driver uses, in it difs remains the same, just eifs isn't changed for short slot
> case and kept with a default value (along with what is supposed to be ack
> timeout register), and sifs set to 0x22. Please try just the following change
> to isolate that the SIFS setting caused the throughput issue:
> 
> diff --git a/drivers/net/wireless/rtl818x/rtl8187_dev.c b/drivers/net/wireless/rtl818x/rtl8187_dev.c
> index d49f2a7..c0392e4 100644
> --- a/drivers/net/wireless/rtl818x/rtl8187_dev.c
> +++ b/drivers/net/wireless/rtl818x/rtl8187_dev.c
> @@ -924,7 +924,7 @@ static void rtl8187_conf_erp(struct rtl8187_priv *priv, bool use_short_slot,
>  			difs = 0x32;
>  			eifs = 0x5b;
>  		}
> -		rtl818x_iowrite8(priv, &priv->map->SIFS, 0xa);
> +		rtl818x_iowrite8(priv, &priv->map->SIFS, 0x22);
>  		rtl818x_iowrite8(priv, &priv->map->SLOT, slot_time);
>  		rtl818x_iowrite8(priv, &priv->map->DIFS, difs);

This patch gives me the best performance yet. I get about 1.0 MB/s download and
580 KB/s upload with a 25 MB file using sftp.

Larry