From: Jes Sorensen <[email protected]>
Hi,
This patch works around the issue with 8192eu and 8723bu devices not
being able to reconnect. It is still not clear why this fails, and
looking at rtlwifi it does issue the command for 8192ee and 8723be
devices.
Until we have a better idea of what is going on, I have commented out
the offending code. I prefer not deleting it until we have a better
understanding of why this is causing issues.
Credits to Barry Day for tracking down and isolating the issue.
Kalle, not sure if this can make it to 4.9, but if it's possible it
would be ideal since this is causing real issues in the field.
Cheers,
Jes
Jes Sorensen (1):
rtl8xxxu: Work around issue with 8192eu and 8723bu devices not
reconnecting
drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c | 8 ++++++++
1 file changed, 8 insertions(+)
--
2.7.4
[email protected] writes:
> From: Jes Sorensen <[email protected]>
>
> Hi,
>
> This patch works around the issue with 8192eu and 8723bu devices not
> being able to reconnect. It is still not clear why this fails, and
> looking at rtlwifi it does issue the command for 8192ee and 8723be
> devices.
>
> Until we have a better idea of what is going on, I have commented out
> the offending code. I prefer not deleting it until we have a better
> understanding of why this is causing issues.
>
> Credits to Barry Day for tracking down and isolating the issue.
>
> Kalle, not sure if this can make it to 4.9, but if it's possible it
> would be ideal since this is causing real issues in the field.
For 4.9 this is too late, sorry. I'll queue this to 4.10.
--
Kalle Valo
On Wed, 2016-11-30 at 10:46 +0200, Kalle Valo wrote:
> [email protected] writes:
>
> > From: Jes Sorensen <[email protected]>
> >
> > Hi,
> >
> > This patch works around the issue with 8192eu and 8723bu devices not
> > being able to reconnect. It is still not clear why this fails, and
> > looking at rtlwifi it does issue the command for 8192ee and 8723be
> > devices.
> >
> > Until we have a better idea of what is going on, I have commented out
> > the offending code. I prefer not deleting it until we have a better
> > understanding of why this is causing issues.
> >
> > Credits to Barry Day for tracking down and isolating the issue.
> >
> > Kalle, not sure if this can make it to 4.9, but if it's possible it
> > would be ideal since this is causing real issues in the field.
>
> For 4.9 this is too late, sorry. I'll queue this to 4.10.
You should then CC stable so it gets into 4.9.x when it reaches 4.10.
--
Luca.
Luca Coelho <[email protected]> writes:
> On Wed, 2016-11-30 at 10:46 +0200, Kalle Valo wrote:
>> [email protected] writes:
>>
>> > From: Jes Sorensen <[email protected]>
>> >
>> > Hi,
>> >
>> > This patch works around the issue with 8192eu and 8723bu devices not
>> > being able to reconnect. It is still not clear why this fails, and
>> > looking at rtlwifi it does issue the command for 8192ee and 8723be
>> > devices.
>> >
>> > Until we have a better idea of what is going on, I have commented out
>> > the offending code. I prefer not deleting it until we have a better
>> > understanding of why this is causing issues.
>> >
>> > Credits to Barry Day for tracking down and isolating the issue.
>> >
>> > Kalle, not sure if this can make it to 4.9, but if it's possible it
>> > would be ideal since this is causing real issues in the field.
>>
>> For 4.9 this is too late, sorry. I'll queue this to 4.10.
>
> You should then CC stable so it gets into 4.9.x when it reaches 4.10.
Yeah good idea - Kalle do you want me to repost it with the CC line or
are you happy to add it?
Cheers,
Jes
Jes Sorensen <[email protected]> writes:
> Luca Coelho <[email protected]> writes:
>> On Wed, 2016-11-30 at 10:46 +0200, Kalle Valo wrote:
>>> [email protected] writes:
>>>
>>> > From: Jes Sorensen <[email protected]>
>>> >
>>> > Hi,
>>> >
>>> > This patch works around the issue with 8192eu and 8723bu devices not
>>> > being able to reconnect. It is still not clear why this fails, and
>>> > looking at rtlwifi it does issue the command for 8192ee and 8723be
>>> > devices.
>>> >
>>> > Until we have a better idea of what is going on, I have commented out
>>> > the offending code. I prefer not deleting it until we have a better
>>> > understanding of why this is causing issues.
>>> >
>>> > Credits to Barry Day for tracking down and isolating the issue.
>>> >
>>> > Kalle, not sure if this can make it to 4.9, but if it's possible it
>>> > would be ideal since this is causing real issues in the field.
>>>
>>> For 4.9 this is too late, sorry. I'll queue this to 4.10.
>>
>> You should then CC stable so it gets into 4.9.x when it reaches 4.10.
>
> Yeah good idea - Kalle do you want me to repost it with the CC line or
> are you happy to add it?
I can add it. To which stable releases should this go to?
--
Kalle Valo
Kalle Valo <[email protected]> writes:
> Jes Sorensen <[email protected]> writes:
>
>> Luca Coelho <[email protected]> writes:
>>> On Wed, 2016-11-30 at 10:46 +0200, Kalle Valo wrote:
>>>> [email protected] writes:
>>>>
>>>> > From: Jes Sorensen <[email protected]>
>>>> >
>>>> > Hi,
>>>> >
>>>> > This patch works around the issue with 8192eu and 8723bu devices not
>>>> > being able to reconnect. It is still not clear why this fails, and
>>>> > looking at rtlwifi it does issue the command for 8192ee and 8723be
>>>> > devices.
>>>> >
>>>> > Until we have a better idea of what is going on, I have commented out
>>>> > the offending code. I prefer not deleting it until we have a better
>>>> > understanding of why this is causing issues.
>>>> >
>>>> > Credits to Barry Day for tracking down and isolating the issue.
>>>> >
>>>> > Kalle, not sure if this can make it to 4.9, but if it's possible it
>>>> > would be ideal since this is causing real issues in the field.
>>>>
>>>> For 4.9 this is too late, sorry. I'll queue this to 4.10.
>>>
>>> You should then CC stable so it gets into 4.9.x when it reaches 4.10.
>>
>> Yeah good idea - Kalle do you want me to repost it with the CC line or
>> are you happy to add it?
>
> I can add it. To which stable releases should this go to?
Lets target 4.8 and 4.9 - I just verified it applies against 4.8.
Thanks,
Jes
From: Jes Sorensen <[email protected]>
The H2C MEDIA_STATUS_RPT command for some reason causes 8192eu and
8723bu devices not being able to reconnect.
Reported-by: Barry Day <[email protected]>
Signed-off-by: Jes Sorensen <[email protected]>
---
drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c
index a9137ab..3a86675 100644
--- a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c
+++ b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c
@@ -4372,6 +4372,13 @@ void rtl8xxxu_gen1_report_connect(struct rtl8xxxu_priv *priv,
void rtl8xxxu_gen2_report_connect(struct rtl8xxxu_priv *priv,
u8 macid, bool connect)
{
+#ifdef RTL8XXXU_GEN2_REPORT_CONNECT
+ /*
+ * Barry Day reports this causes issues with 8192eu and 8723bu
+ * devices reconnecting. The reason for this is unclear, but
+ * until it is better understood, leave the code in place but
+ * disabled, so it is not lost.
+ */
struct h2c_cmd h2c;
memset(&h2c, 0, sizeof(struct h2c_cmd));
@@ -4383,6 +4390,7 @@ void rtl8xxxu_gen2_report_connect(struct rtl8xxxu_priv *priv,
h2c.media_status_rpt.parm &= ~BIT(0);
rtl8xxxu_gen2_h2c_cmd(priv, &h2c, sizeof(h2c.media_status_rpt));
+#endif
}
void rtl8xxxu_gen1_init_aggregation(struct rtl8xxxu_priv *priv)
--
2.7.4
Jes Sorensen <[email protected]> wrote:
> From: Jes Sorensen <[email protected]>
>
> The H2C MEDIA_STATUS_RPT command for some reason causes 8192eu and
> 8723bu devices not being able to reconnect.
>
> Reported-by: Barry Day <[email protected]>
> Cc: <[email protected]> #4.8+
> Signed-off-by: Jes Sorensen <[email protected]>
Patch applied to wireless-drivers-next.git, thanks.
c59f13bbead4 rtl8xxxu: Work around issue with 8192eu and 8723bu devices not reconnecting
--
https://patchwork.kernel.org/patch/9453251/
Documentation about submitting wireless patches and checking status
from patchwork:
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
Jes Sorensen <[email protected]> wrote:
> From: Jes Sorensen <[email protected]>
>
> The H2C MEDIA_STATUS_RPT command for some reason causes 8192eu and
> 8723bu devices not being able to reconnect.
>
> Reported-by: Barry Day <[email protected]>
> Signed-off-by: Jes Sorensen <[email protected]>
Like discussed I'll add:
Cc: <[email protected]> #4.8+
--
https://patchwork.kernel.org/patch/9453251/
Documentation about submitting wireless patches and checking status
from patchwork:
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches