On 19.04.2021 09:04, Pkshih wrote:
-----Original Message-----
From: Larry Finger [mailto:larry.finger@xxxxxxxxx] On Behalf Of Larry Finger
Sent: Monday, April 19, 2021 9:23 AM
To: Pkshih; Maciej S. Szmigiero
Cc: linux-wireless@xxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
johannes@xxxxxxxxxxxxxxxx; kvalo@xxxxxxxxxxxxxx
Subject: Re: rtlwifi/rtl8192cu AP mode broken with PS STA
On 4/18/21 7:32 PM, Pkshih wrote:
-----Original Message-----
From: Maciej S. Szmigiero [mailto:mail@xxxxxxxxxxxxxxxxxxxxx]
Sent: Sunday, April 18, 2021 2:08 AM
To: Pkshih
Cc: linux-wireless@xxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
johannes@xxxxxxxxxxxxxxxx; kvalo@xxxxxxxxxxxxxx; Larry Finger
Subject: Re: rtlwifi/rtl8192cu AP mode broken with PS STA
On 08.04.2021 21:04, Maciej S. Szmigiero wrote:
On 08.04.2021 06:42, Pkshih wrote:
-----Original Message-----
From: Maciej S. Szmigiero [mailto:mail@xxxxxxxxxxxxxxxxxxxxx]
Sent: Thursday, April 08, 2021 4:53 AM
To: Larry Finger; Pkshih
Cc: linux-wireless@xxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
johannes@xxxxxxxxxxxxxxxx; kvalo@xxxxxxxxxxxxxx
Subject: Re: rtlwifi/rtl8192cu AP mode broken with PS STA
(...)
Maceij,
Does this patch fix the problem?
The beacon seems to be updating now and STAs no longer get stuck in PS
mode.
Although sometimes (every 2-3 minutes with continuous 1s interval pings)
there is around 5s delay in updating the transmitted beacon - don't know
why, maybe the NIC hardware still has the old version in queue?
Since USB device doesn't update every beacon, dtim_count isn't updated neither.
It leads STA doesn't awake properly. Please try to fix dtim_period=1 in
hostapd.conf, which tells STA awakes every beacon interval.
The situation is the same with dtim_period=1.
(...)
Ping-Ke,
are you going to submit your set_tim() patch so at least the AP mode is
usable with PS STAs or are you waiting for a solution to the delayed
beacon update issue?
I'm still trying to get a 8192cu, and then I can reproduce the symptom you
met. However, I'm busy now; maybe I have free time two weeks later.
Do you think I submit the set_tim() patch with your Reported-by and Tested-by first?
PK,
I would say yes. Get the fix in as soon as possible.
I have sent a patch that only 8192cu, which is the only one USB device supported by rtlwifi,
schedules a work to update beacon content to wifi card.
https://lore.kernel.org/linux-wireless/20210419065956.6085-1-pkshih@xxxxxxxxxxx/T/#u
Thanks, I have tested the patch and it seems to work as good as the previous one.
It definitely improves things.
However, it would be great to eventually fix the update delay issue, too.
It looks to me like possibly just a missing beacon queue flush.
--
Ping-Ke
Maciej
