Re: Warnings with ASUS PCE-N15 (RTL8188CE)

[Vincent Brillault <gentoo@lerya.net>]

[-- Attachment #1: Type: text/plain, Size: 799 bytes --]

Hi again :)

On Mon 25.Mar'13 at 15:05:35 -0500, Larry Finger wrote:
> I will let you change that G to a B and test some more. If all goes well, then I 
> will not need to set up that test bed, but at least I have the necessary details 
> if ever required.

I changed it from G to B. Now the rate (in iwconfig) is fixed to 11
Mb/s, but that's not a problem as you said it's not the real rate. I
didn't see any warning on module loading since that modification :)

I still have the deconnection/reconnection issue. After looking on the
reason, I think I have found a bug. Please look at the patched included
in this mail: I think the rtlpriv->link_info.num_rx_inperiod is set to 0
too early, as the '/* <4> roaming */' part uses it to see if the AP is
still alive. What do you think ?

Best,
Vincent

[-- Attachment #2: rtlwifi-watchdog-rx_inperiod.patch --]
[-- Type: text/x-diff, Size: 687 bytes --]

diff --git a/drivers/net/wireless/rtlwifi/base.c b/drivers/net/wireless/rtlwifi/base.c
index cac1fa9..d540938 100644
--- a/drivers/net/wireless/rtlwifi/base.c
+++ b/drivers/net/wireless/rtlwifi/base.c
@@ -1411,7 +1411,6 @@ void rtl_watchdog_wq_callback(void *data)
 		schedule_work(&rtlpriv->works.lps_change_work);
 	}
 
-	rtlpriv->link_info.num_rx_inperiod = 0;
 	rtlpriv->link_info.num_tx_inperiod = 0;
 	for (tid = 0; tid <= 7; tid++)
 		rtlpriv->link_info.tidtx_inperiod[tid] = 0;
@@ -1448,6 +1447,7 @@ void rtl_watchdog_wq_callback(void *data)
 			rtlpriv->link_info.roam_times = 0;
 		}
 	}
+	rtlpriv->link_info.num_rx_inperiod = 0;
 	rtlpriv->link_info.bcn_rx_inperiod = 0;
 }