* ath9k broken [was: Linux 5.7.3] [not found] ` <CAEJqkgjV8p6LtBV8YUGbNb0vYzKOQt4-AMAvYw5mzFr3eicyTg@mail.gmail.com> @ 2020-06-25 4:57 ` Jiri Slaby 2020-06-25 10:48 ` Gabriel C 2020-06-27 10:13 ` Roman Mamedov 0 siblings, 2 replies; 13+ messages in thread From: Jiri Slaby @ 2020-06-25 4:57 UTC (permalink / raw) To: Gabriel C, Greg Kroah-Hartman Cc: LKML, Andrew Morton, Linus Torvalds, stable, lwn, angrypenguinpoland, Qiujun Huang, ath9k-devel, linux-wireless On 25. 06. 20, 0:05, Gabriel C wrote: > Am Mi., 17. Juni 2020 um 18:13 Uhr schrieb Greg Kroah-Hartman > <gregkh@linuxfoundation.org>: >> >> I'm announcing the release of the 5.7.3 kernel. >> > > Hello Greg, > >> Qiujun Huang (5): >> ath9k: Fix use-after-free Read in htc_connect_service >> ath9k: Fix use-after-free Read in ath9k_wmi_ctrl_rx >> ath9k: Fix use-after-free Write in ath9k_htc_rx_msg >> ath9x: Fix stack-out-of-bounds Write in ath9k_hif_usb_rx_cb >> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb >> > > We got a report on IRC about 5.7.3+ breaking a USB ath9k Wifi Dongle, > while working fine on <5.7.3. > > I don't have myself such HW, and the reported doesn't have any experience > in bisecting the kernel, so we build kernels, each with one of the > above commits reverted, > to find the bad commit. > > The winner is: > > commit 6602f080cb28745259e2fab1a4cf55eeb5894f93 > Author: Qiujun Huang <hqjagain@gmail.com> > Date: Sat Apr 4 12:18:38 2020 +0800 > > ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > commit 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 upstream. > ... > > Reverting this one fixed his problem. Obvious question: is 5.8-rc1 (containing the commit) broken too? I fail to see how the commit could cause an issue like this. Is this really reproducibly broken with the commit and irreproducible without it? As it looks like a USB/wiring problem: usb 1-2: USB disconnect, device number 2 ath: phy0: Reading Magic # failed ath: phy0: Unable to initialize hardware; initialization status: -5 ... usb 1-2: device descriptor read/64, error -110 usb 1-2: device descriptor read/64, error -71 Ccing ath9k maintainers too. > I don't have so much info about the HW, besides a dmesg showing the > phy breaking. > I also added the reporter to CC too. > > https://gist.github.com/AngryPenguinPL/1e545f0da3c2339e443b9e5044fcccea > > If you need more info, please let me know and I'll try my best to get > it as fast as possible for you. thanks, -- js ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: ath9k broken [was: Linux 5.7.3] 2020-06-25 4:57 ` ath9k broken [was: Linux 5.7.3] Jiri Slaby @ 2020-06-25 10:48 ` Gabriel C 2020-06-25 10:52 ` Gabriel C 2020-06-27 10:13 ` Roman Mamedov 1 sibling, 1 reply; 13+ messages in thread From: Gabriel C @ 2020-06-25 10:48 UTC (permalink / raw) To: Jiri Slaby Cc: Greg Kroah-Hartman, LKML, Andrew Morton, Linus Torvalds, stable, lwn, angrypenguinpoland, Qiujun Huang, ath9k-devel, linux-wireless Am Do., 25. Juni 2020 um 06:57 Uhr schrieb Jiri Slaby <jirislaby@kernel.org>: > > On 25. 06. 20, 0:05, Gabriel C wrote: > > Am Mi., 17. Juni 2020 um 18:13 Uhr schrieb Greg Kroah-Hartman > > <gregkh@linuxfoundation.org>: > >> > >> I'm announcing the release of the 5.7.3 kernel. > >> > > > > Hello Greg, > > > >> Qiujun Huang (5): > >> ath9k: Fix use-after-free Read in htc_connect_service > >> ath9k: Fix use-after-free Read in ath9k_wmi_ctrl_rx > >> ath9k: Fix use-after-free Write in ath9k_htc_rx_msg > >> ath9x: Fix stack-out-of-bounds Write in ath9k_hif_usb_rx_cb > >> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > >> > > > > We got a report on IRC about 5.7.3+ breaking a USB ath9k Wifi Dongle, > > while working fine on <5.7.3. > > > > I don't have myself such HW, and the reported doesn't have any experience > > in bisecting the kernel, so we build kernels, each with one of the > > above commits reverted, > > to find the bad commit. > > > > The winner is: > > > > commit 6602f080cb28745259e2fab1a4cf55eeb5894f93 > > Author: Qiujun Huang <hqjagain@gmail.com> > > Date: Sat Apr 4 12:18:38 2020 +0800 > > > > ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > > > commit 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 upstream. > > ... > > > > Reverting this one fixed his problem. > > Obvious question: is 5.8-rc1 (containing the commit) broken too? Yes, it does, just checked. git tag --contains 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 v5.8-rc1 v5.8-rc2 > > I fail to see how the commit could cause an issue like this. Is this > really reproducibly broken with the commit and irreproducible without > it? I can't see something obvious wrong either, but yes it's reproducible on his HW. Kernel with this commit breaks the dongle, with the commit reverted it works. >As it looks like a USB/wiring problem: > usb 1-2: USB disconnect, device number 2 > ath: phy0: Reading Magic # failed > ath: phy0: Unable to initialize hardware; initialization status: -5 > ... > usb 1-2: device descriptor read/64, error -110 > usb 1-2: device descriptor read/64, error -71 > > Ccing ath9k maintainers too. > > > I don't have so much info about the HW, besides a dmesg showing the > > phy breaking. > > I also added the reporter to CC too. > > > > https://gist.github.com/AngryPenguinPL/1e545f0da3c2339e443b9e5044fcccea > > > > If you need more info, please let me know and I'll try my best to get > > it as fast as possible for you. > Best Regards, Gabriel C ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: ath9k broken [was: Linux 5.7.3] 2020-06-25 10:48 ` Gabriel C @ 2020-06-25 10:52 ` Gabriel C 2020-06-26 11:48 ` Gabriel C 0 siblings, 1 reply; 13+ messages in thread From: Gabriel C @ 2020-06-25 10:52 UTC (permalink / raw) To: Jiri Slaby Cc: Greg Kroah-Hartman, LKML, Andrew Morton, Linus Torvalds, stable, lwn, angrypenguinpoland, Qiujun Huang, ath9k-devel, linux-wireless Am Do., 25. Juni 2020 um 12:48 Uhr schrieb Gabriel C <nix.or.die@googlemail.com>: > > Am Do., 25. Juni 2020 um 06:57 Uhr schrieb Jiri Slaby <jirislaby@kernel.org>: > > > > On 25. 06. 20, 0:05, Gabriel C wrote: > > > Am Mi., 17. Juni 2020 um 18:13 Uhr schrieb Greg Kroah-Hartman > > > <gregkh@linuxfoundation.org>: > > >> > > >> I'm announcing the release of the 5.7.3 kernel. > > >> > > > > > > Hello Greg, > > > > > >> Qiujun Huang (5): > > >> ath9k: Fix use-after-free Read in htc_connect_service > > >> ath9k: Fix use-after-free Read in ath9k_wmi_ctrl_rx > > >> ath9k: Fix use-after-free Write in ath9k_htc_rx_msg > > >> ath9x: Fix stack-out-of-bounds Write in ath9k_hif_usb_rx_cb > > >> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > >> > > > > > > We got a report on IRC about 5.7.3+ breaking a USB ath9k Wifi Dongle, > > > while working fine on <5.7.3. > > > > > > I don't have myself such HW, and the reported doesn't have any experience > > > in bisecting the kernel, so we build kernels, each with one of the > > > above commits reverted, > > > to find the bad commit. > > > > > > The winner is: > > > > > > commit 6602f080cb28745259e2fab1a4cf55eeb5894f93 > > > Author: Qiujun Huang <hqjagain@gmail.com> > > > Date: Sat Apr 4 12:18:38 2020 +0800 > > > > > > ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > > > > > commit 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 upstream. > > > ... > > > > > > Reverting this one fixed his problem. > > > > Obvious question: is 5.8-rc1 (containing the commit) broken too? > > Yes, it does, just checked. > > git tag --contains 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 > v5.8-rc1 > v5.8-rc2 > Sorry, I read the wrong, I just woke up. We didn't test 5.8-rc{1,2} yet but we will today and let you know. > > > > I fail to see how the commit could cause an issue like this. Is this > > really reproducibly broken with the commit and irreproducible without > > it? > > I can't see something obvious wrong either, but yes it's reproducible on his HW. > Kernel with this commit breaks the dongle, with the commit reverted it works. > > >As it looks like a USB/wiring problem: > > usb 1-2: USB disconnect, device number 2 > > ath: phy0: Reading Magic # failed > > ath: phy0: Unable to initialize hardware; initialization status: -5 > > ... > > usb 1-2: device descriptor read/64, error -110 > > usb 1-2: device descriptor read/64, error -71 > > > > Ccing ath9k maintainers too. > > > > > I don't have so much info about the HW, besides a dmesg showing the > > > phy breaking. > > > I also added the reporter to CC too. > > > > > > https://gist.github.com/AngryPenguinPL/1e545f0da3c2339e443b9e5044fcccea > > > > > > If you need more info, please let me know and I'll try my best to get > > > it as fast as possible for you. > > > > Best Regards, > > Gabriel C ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: ath9k broken [was: Linux 5.7.3] 2020-06-25 10:52 ` Gabriel C @ 2020-06-26 11:48 ` Gabriel C 2020-06-26 13:39 ` Greg Kroah-Hartman 0 siblings, 1 reply; 13+ messages in thread From: Gabriel C @ 2020-06-26 11:48 UTC (permalink / raw) To: Jiri Slaby Cc: Greg Kroah-Hartman, LKML, Andrew Morton, Linus Torvalds, stable, lwn, angrypenguinpoland, Qiujun Huang, ath9k-devel, linux-wireless Am Do., 25. Juni 2020 um 12:52 Uhr schrieb Gabriel C <nix.or.die@googlemail.com>: > > Am Do., 25. Juni 2020 um 12:48 Uhr schrieb Gabriel C > <nix.or.die@googlemail.com>: > > > > Am Do., 25. Juni 2020 um 06:57 Uhr schrieb Jiri Slaby <jirislaby@kernel.org>: > > > > > > On 25. 06. 20, 0:05, Gabriel C wrote: > > > > Am Mi., 17. Juni 2020 um 18:13 Uhr schrieb Greg Kroah-Hartman > > > > <gregkh@linuxfoundation.org>: > > > >> > > > >> I'm announcing the release of the 5.7.3 kernel. > > > >> > > > > > > > > Hello Greg, > > > > > > > >> Qiujun Huang (5): > > > >> ath9k: Fix use-after-free Read in htc_connect_service > > > >> ath9k: Fix use-after-free Read in ath9k_wmi_ctrl_rx > > > >> ath9k: Fix use-after-free Write in ath9k_htc_rx_msg > > > >> ath9x: Fix stack-out-of-bounds Write in ath9k_hif_usb_rx_cb > > > >> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > > >> > > > > > > > > We got a report on IRC about 5.7.3+ breaking a USB ath9k Wifi Dongle, > > > > while working fine on <5.7.3. > > > > > > > > I don't have myself such HW, and the reported doesn't have any experience > > > > in bisecting the kernel, so we build kernels, each with one of the > > > > above commits reverted, > > > > to find the bad commit. > > > > > > > > The winner is: > > > > > > > > commit 6602f080cb28745259e2fab1a4cf55eeb5894f93 > > > > Author: Qiujun Huang <hqjagain@gmail.com> > > > > Date: Sat Apr 4 12:18:38 2020 +0800 > > > > > > > > ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > > > > > > > commit 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 upstream. > > > > ... > > > > > > > > Reverting this one fixed his problem. > > > > > > Obvious question: is 5.8-rc1 (containing the commit) broken too? > > > > Yes, it does, just checked. > > > > git tag --contains 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 > > v5.8-rc1 > > v5.8-rc2 > > > > Sorry, I read the wrong, I just woke up. > > We didn't test 5.8-rc{1,2} yet but we will today and let you know. > We tested 5.8-rc2 and it is broken too. The exact HW name is: TP-link tl-wn722n (Atheros AR9271 chip) > > > > > > I fail to see how the commit could cause an issue like this. Is this > > > really reproducibly broken with the commit and irreproducible without > > > it? > > > > I can't see something obvious wrong either, but yes it's reproducible on his HW. > > Kernel with this commit breaks the dongle, with the commit reverted it works. > > > > >As it looks like a USB/wiring problem: > > > usb 1-2: USB disconnect, device number 2 > > > ath: phy0: Reading Magic # failed > > > ath: phy0: Unable to initialize hardware; initialization status: -5 > > > ... > > > usb 1-2: device descriptor read/64, error -110 > > > usb 1-2: device descriptor read/64, error -71 > > > > > > Ccing ath9k maintainers too. > > > > > > > I don't have so much info about the HW, besides a dmesg showing the > > > > phy breaking. > > > > I also added the reporter to CC too. > > > > > > > > https://gist.github.com/AngryPenguinPL/1e545f0da3c2339e443b9e5044fcccea > > > > > > > > If you need more info, please let me know and I'll try my best to get > > > > it as fast as possible for you. > > > Best Regards, Gabriel C ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: ath9k broken [was: Linux 5.7.3] 2020-06-26 11:48 ` Gabriel C @ 2020-06-26 13:39 ` Greg Kroah-Hartman 2020-06-26 13:51 ` Gabriel C 0 siblings, 1 reply; 13+ messages in thread From: Greg Kroah-Hartman @ 2020-06-26 13:39 UTC (permalink / raw) To: Gabriel C Cc: Jiri Slaby, LKML, Andrew Morton, Linus Torvalds, stable, lwn, angrypenguinpoland, Qiujun Huang, ath9k-devel, linux-wireless On Fri, Jun 26, 2020 at 01:48:59PM +0200, Gabriel C wrote: > Am Do., 25. Juni 2020 um 12:52 Uhr schrieb Gabriel C > <nix.or.die@googlemail.com>: > > > > Am Do., 25. Juni 2020 um 12:48 Uhr schrieb Gabriel C > > <nix.or.die@googlemail.com>: > > > > > > Am Do., 25. Juni 2020 um 06:57 Uhr schrieb Jiri Slaby <jirislaby@kernel.org>: > > > > > > > > On 25. 06. 20, 0:05, Gabriel C wrote: > > > > > Am Mi., 17. Juni 2020 um 18:13 Uhr schrieb Greg Kroah-Hartman > > > > > <gregkh@linuxfoundation.org>: > > > > >> > > > > >> I'm announcing the release of the 5.7.3 kernel. > > > > >> > > > > > > > > > > Hello Greg, > > > > > > > > > >> Qiujun Huang (5): > > > > >> ath9k: Fix use-after-free Read in htc_connect_service > > > > >> ath9k: Fix use-after-free Read in ath9k_wmi_ctrl_rx > > > > >> ath9k: Fix use-after-free Write in ath9k_htc_rx_msg > > > > >> ath9x: Fix stack-out-of-bounds Write in ath9k_hif_usb_rx_cb > > > > >> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > > > >> > > > > > > > > > > We got a report on IRC about 5.7.3+ breaking a USB ath9k Wifi Dongle, > > > > > while working fine on <5.7.3. > > > > > > > > > > I don't have myself such HW, and the reported doesn't have any experience > > > > > in bisecting the kernel, so we build kernels, each with one of the > > > > > above commits reverted, > > > > > to find the bad commit. > > > > > > > > > > The winner is: > > > > > > > > > > commit 6602f080cb28745259e2fab1a4cf55eeb5894f93 > > > > > Author: Qiujun Huang <hqjagain@gmail.com> > > > > > Date: Sat Apr 4 12:18:38 2020 +0800 > > > > > > > > > > ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > > > > > > > > > commit 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 upstream. > > > > > ... > > > > > > > > > > Reverting this one fixed his problem. > > > > > > > > Obvious question: is 5.8-rc1 (containing the commit) broken too? > > > > > > Yes, it does, just checked. > > > > > > git tag --contains 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 > > > v5.8-rc1 > > > v5.8-rc2 > > > > > > > Sorry, I read the wrong, I just woke up. > > > > We didn't test 5.8-rc{1,2} yet but we will today and let you know. > > > > We tested 5.8-rc2 and it is broken too. > > The exact HW name is: > > TP-link tl-wn722n (Atheros AR9271 chip) Great! Can you work with the developers to fix this in Linus's tree first? I bet they want to see the output of 'lsusb -v' for this device to see if the endpoint calculations are correct... thanks, greg k-h ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: ath9k broken [was: Linux 5.7.3] 2020-06-26 13:39 ` Greg Kroah-Hartman @ 2020-06-26 13:51 ` Gabriel C 2020-06-26 14:40 ` Gabriel C 0 siblings, 1 reply; 13+ messages in thread From: Gabriel C @ 2020-06-26 13:51 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Jiri Slaby, LKML, Andrew Morton, Linus Torvalds, stable, lwn, angrypenguinpoland, Qiujun Huang, ath9k-devel, linux-wireless Am Fr., 26. Juni 2020 um 15:40 Uhr schrieb Greg Kroah-Hartman <gregkh@linuxfoundation.org>: > > On Fri, Jun 26, 2020 at 01:48:59PM +0200, Gabriel C wrote: > > Am Do., 25. Juni 2020 um 12:52 Uhr schrieb Gabriel C > > <nix.or.die@googlemail.com>: > > > > > > Am Do., 25. Juni 2020 um 12:48 Uhr schrieb Gabriel C > > > <nix.or.die@googlemail.com>: > > > > > > > > Am Do., 25. Juni 2020 um 06:57 Uhr schrieb Jiri Slaby <jirislaby@kernel.org>: > > > > > > > > > > On 25. 06. 20, 0:05, Gabriel C wrote: > > > > > > Am Mi., 17. Juni 2020 um 18:13 Uhr schrieb Greg Kroah-Hartman > > > > > > <gregkh@linuxfoundation.org>: > > > > > >> > > > > > >> I'm announcing the release of the 5.7.3 kernel. > > > > > >> > > > > > > > > > > > > Hello Greg, > > > > > > > > > > > >> Qiujun Huang (5): > > > > > >> ath9k: Fix use-after-free Read in htc_connect_service > > > > > >> ath9k: Fix use-after-free Read in ath9k_wmi_ctrl_rx > > > > > >> ath9k: Fix use-after-free Write in ath9k_htc_rx_msg > > > > > >> ath9x: Fix stack-out-of-bounds Write in ath9k_hif_usb_rx_cb > > > > > >> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > > > > >> > > > > > > > > > > > > We got a report on IRC about 5.7.3+ breaking a USB ath9k Wifi Dongle, > > > > > > while working fine on <5.7.3. > > > > > > > > > > > > I don't have myself such HW, and the reported doesn't have any experience > > > > > > in bisecting the kernel, so we build kernels, each with one of the > > > > > > above commits reverted, > > > > > > to find the bad commit. > > > > > > > > > > > > The winner is: > > > > > > > > > > > > commit 6602f080cb28745259e2fab1a4cf55eeb5894f93 > > > > > > Author: Qiujun Huang <hqjagain@gmail.com> > > > > > > Date: Sat Apr 4 12:18:38 2020 +0800 > > > > > > > > > > > > ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > > > > > > > > > > > commit 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 upstream. > > > > > > ... > > > > > > > > > > > > Reverting this one fixed his problem. > > > > > > > > > > Obvious question: is 5.8-rc1 (containing the commit) broken too? > > > > > > > > Yes, it does, just checked. > > > > > > > > git tag --contains 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 > > > > v5.8-rc1 > > > > v5.8-rc2 > > > > > > > > > > Sorry, I read the wrong, I just woke up. > > > > > > We didn't test 5.8-rc{1,2} yet but we will today and let you know. > > > > > > > We tested 5.8-rc2 and it is broken too. > > > > The exact HW name is: > > > > TP-link tl-wn722n (Atheros AR9271 chip) > > Great! > > Can you work with the developers to fix this in Linus's tree first? I'm the man in the middle, but sure we will try patches or any suggestions from developers to identify and fix the problem. > > I bet they want to see the output of 'lsusb -v' for this device to see > if the endpoint calculations are correct... > Working on it. As soon the reporter gives me the output, I will post it here. I've told him to run it on a broken and one working kernel. > thanks, > > greg k-h Best Regards, Gabriel C ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: ath9k broken [was: Linux 5.7.3] 2020-06-26 13:51 ` Gabriel C @ 2020-06-26 14:40 ` Gabriel C 2020-07-07 14:11 ` Greg Kroah-Hartman 0 siblings, 1 reply; 13+ messages in thread From: Gabriel C @ 2020-06-26 14:40 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Jiri Slaby, LKML, Andrew Morton, Linus Torvalds, stable, lwn, angrypenguinpoland, Qiujun Huang, ath9k-devel, linux-wireless Am Fr., 26. Juni 2020 um 15:51 Uhr schrieb Gabriel C <nix.or.die@googlemail.com>: > > Am Fr., 26. Juni 2020 um 15:40 Uhr schrieb Greg Kroah-Hartman > <gregkh@linuxfoundation.org>: > > > > On Fri, Jun 26, 2020 at 01:48:59PM +0200, Gabriel C wrote: > > > Am Do., 25. Juni 2020 um 12:52 Uhr schrieb Gabriel C > > > <nix.or.die@googlemail.com>: > > > > > > > > Am Do., 25. Juni 2020 um 12:48 Uhr schrieb Gabriel C > > > > <nix.or.die@googlemail.com>: > > > > > > > > > > Am Do., 25. Juni 2020 um 06:57 Uhr schrieb Jiri Slaby <jirislaby@kernel.org>: > > > > > > > > > > > > On 25. 06. 20, 0:05, Gabriel C wrote: > > > > > > > Am Mi., 17. Juni 2020 um 18:13 Uhr schrieb Greg Kroah-Hartman > > > > > > > <gregkh@linuxfoundation.org>: > > > > > > >> > > > > > > >> I'm announcing the release of the 5.7.3 kernel. > > > > > > >> > > > > > > > > > > > > > > Hello Greg, > > > > > > > > > > > > > >> Qiujun Huang (5): > > > > > > >> ath9k: Fix use-after-free Read in htc_connect_service > > > > > > >> ath9k: Fix use-after-free Read in ath9k_wmi_ctrl_rx > > > > > > >> ath9k: Fix use-after-free Write in ath9k_htc_rx_msg > > > > > > >> ath9x: Fix stack-out-of-bounds Write in ath9k_hif_usb_rx_cb > > > > > > >> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > > > > > >> > > > > > > > > > > > > > > We got a report on IRC about 5.7.3+ breaking a USB ath9k Wifi Dongle, > > > > > > > while working fine on <5.7.3. > > > > > > > > > > > > > > I don't have myself such HW, and the reported doesn't have any experience > > > > > > > in bisecting the kernel, so we build kernels, each with one of the > > > > > > > above commits reverted, > > > > > > > to find the bad commit. > > > > > > > > > > > > > > The winner is: > > > > > > > > > > > > > > commit 6602f080cb28745259e2fab1a4cf55eeb5894f93 > > > > > > > Author: Qiujun Huang <hqjagain@gmail.com> > > > > > > > Date: Sat Apr 4 12:18:38 2020 +0800 > > > > > > > > > > > > > > ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > > > > > > > > > > > > > commit 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 upstream. > > > > > > > ... > > > > > > > > > > > > > > Reverting this one fixed his problem. > > > > > > > > > > > > Obvious question: is 5.8-rc1 (containing the commit) broken too? > > > > > > > > > > Yes, it does, just checked. > > > > > > > > > > git tag --contains 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 > > > > > v5.8-rc1 > > > > > v5.8-rc2 > > > > > > > > > > > > > Sorry, I read the wrong, I just woke up. > > > > > > > > We didn't test 5.8-rc{1,2} yet but we will today and let you know. > > > > > > > > > > We tested 5.8-rc2 and it is broken too. > > > > > > The exact HW name is: > > > > > > TP-link tl-wn722n (Atheros AR9271 chip) > > > > Great! > > > > Can you work with the developers to fix this in Linus's tree first? > > I'm the man in the middle, but sure we will try patches or any suggestions > from developers to identify and fix the problem. > > > > > I bet they want to see the output of 'lsusb -v' for this device to see > > if the endpoint calculations are correct... > > > > Working on it. As soon the reporter gives me the output, I will post it here. > I've told him to run it on a broken and one working kernel. That is from a good kernel with reverted commit https://gist.github.com/AngryPenguinPL/07c8e2abd3b103eaf8978a39ad8577d1 That is from the broken kernel without the commit reverted https://gist.github.com/AngryPenguinPL/5cdc0dd16ce5e59ff3c32c048e2f5111 This is from 5.7.5 kernel, I don't have yet a 5.8-rc2 package with the reverted commit. ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: ath9k broken [was: Linux 5.7.3] 2020-06-26 14:40 ` Gabriel C @ 2020-07-07 14:11 ` Greg Kroah-Hartman 2020-07-07 20:20 ` Viktor Jägersküpper 0 siblings, 1 reply; 13+ messages in thread From: Greg Kroah-Hartman @ 2020-07-07 14:11 UTC (permalink / raw) To: Gabriel C Cc: Jiri Slaby, LKML, Andrew Morton, Linus Torvalds, stable, lwn, angrypenguinpoland, Qiujun Huang, ath9k-devel, linux-wireless On Fri, Jun 26, 2020 at 04:40:18PM +0200, Gabriel C wrote: > Am Fr., 26. Juni 2020 um 15:51 Uhr schrieb Gabriel C > <nix.or.die@googlemail.com>: > > > > Am Fr., 26. Juni 2020 um 15:40 Uhr schrieb Greg Kroah-Hartman > > <gregkh@linuxfoundation.org>: > > > > > > On Fri, Jun 26, 2020 at 01:48:59PM +0200, Gabriel C wrote: > > > > Am Do., 25. Juni 2020 um 12:52 Uhr schrieb Gabriel C > > > > <nix.or.die@googlemail.com>: > > > > > > > > > > Am Do., 25. Juni 2020 um 12:48 Uhr schrieb Gabriel C > > > > > <nix.or.die@googlemail.com>: > > > > > > > > > > > > Am Do., 25. Juni 2020 um 06:57 Uhr schrieb Jiri Slaby <jirislaby@kernel.org>: > > > > > > > > > > > > > > On 25. 06. 20, 0:05, Gabriel C wrote: > > > > > > > > Am Mi., 17. Juni 2020 um 18:13 Uhr schrieb Greg Kroah-Hartman > > > > > > > > <gregkh@linuxfoundation.org>: > > > > > > > >> > > > > > > > >> I'm announcing the release of the 5.7.3 kernel. > > > > > > > >> > > > > > > > > > > > > > > > > Hello Greg, > > > > > > > > > > > > > > > >> Qiujun Huang (5): > > > > > > > >> ath9k: Fix use-after-free Read in htc_connect_service > > > > > > > >> ath9k: Fix use-after-free Read in ath9k_wmi_ctrl_rx > > > > > > > >> ath9k: Fix use-after-free Write in ath9k_htc_rx_msg > > > > > > > >> ath9x: Fix stack-out-of-bounds Write in ath9k_hif_usb_rx_cb > > > > > > > >> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > > > > > > >> > > > > > > > > > > > > > > > > We got a report on IRC about 5.7.3+ breaking a USB ath9k Wifi Dongle, > > > > > > > > while working fine on <5.7.3. > > > > > > > > > > > > > > > > I don't have myself such HW, and the reported doesn't have any experience > > > > > > > > in bisecting the kernel, so we build kernels, each with one of the > > > > > > > > above commits reverted, > > > > > > > > to find the bad commit. > > > > > > > > > > > > > > > > The winner is: > > > > > > > > > > > > > > > > commit 6602f080cb28745259e2fab1a4cf55eeb5894f93 > > > > > > > > Author: Qiujun Huang <hqjagain@gmail.com> > > > > > > > > Date: Sat Apr 4 12:18:38 2020 +0800 > > > > > > > > > > > > > > > > ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb > > > > > > > > > > > > > > > > commit 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 upstream. > > > > > > > > ... > > > > > > > > > > > > > > > > Reverting this one fixed his problem. > > > > > > > > > > > > > > Obvious question: is 5.8-rc1 (containing the commit) broken too? > > > > > > > > > > > > Yes, it does, just checked. > > > > > > > > > > > > git tag --contains 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 > > > > > > v5.8-rc1 > > > > > > v5.8-rc2 > > > > > > > > > > > > > > > > Sorry, I read the wrong, I just woke up. > > > > > > > > > > We didn't test 5.8-rc{1,2} yet but we will today and let you know. > > > > > > > > > > > > > We tested 5.8-rc2 and it is broken too. > > > > > > > > The exact HW name is: > > > > > > > > TP-link tl-wn722n (Atheros AR9271 chip) > > > > > > Great! > > > > > > Can you work with the developers to fix this in Linus's tree first? > > > > I'm the man in the middle, but sure we will try patches or any suggestions > > from developers to identify and fix the problem. > > > > > > > > I bet they want to see the output of 'lsusb -v' for this device to see > > > if the endpoint calculations are correct... > > > > > > > Working on it. As soon the reporter gives me the output, I will post it here. > > I've told him to run it on a broken and one working kernel. > > That is from a good kernel with reverted commit > https://gist.github.com/AngryPenguinPL/07c8e2abd3b103eaf8978a39ad8577d1 > > That is from the broken kernel without the commit reverted > https://gist.github.com/AngryPenguinPL/5cdc0dd16ce5e59ff3c32c048e2f5111 > > This is from 5.7.5 kernel, I don't have yet a 5.8-rc2 package with the > reverted commit. Did this ever get resolved? thanks, greg k-h ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: ath9k broken [was: Linux 5.7.3] 2020-07-07 14:11 ` Greg Kroah-Hartman @ 2020-07-07 20:20 ` Viktor Jägersküpper 2020-07-15 15:13 ` Kalle Valo 0 siblings, 1 reply; 13+ messages in thread From: Viktor Jägersküpper @ 2020-07-07 20:20 UTC (permalink / raw) To: Greg Kroah-Hartman, Gabriel C Cc: Jiri Slaby, LKML, Andrew Morton, Linus Torvalds, stable, lwn, angrypenguinpoland, Qiujun Huang, ath9k-devel, linux-wireless Greg Kroah-Hartman wrote: > On Fri, Jun 26, 2020 at 04:40:18PM +0200, Gabriel C wrote: >> Am Fr., 26. Juni 2020 um 15:51 Uhr schrieb Gabriel C >> <nix.or.die@googlemail.com>: >>> >>> Am Fr., 26. Juni 2020 um 15:40 Uhr schrieb Greg Kroah-Hartman >>> <gregkh@linuxfoundation.org>: >>>> >>>> On Fri, Jun 26, 2020 at 01:48:59PM +0200, Gabriel C wrote: >>>>> Am Do., 25. Juni 2020 um 12:52 Uhr schrieb Gabriel C >>>>> <nix.or.die@googlemail.com>: >>>>>> >>>>>> Am Do., 25. Juni 2020 um 12:48 Uhr schrieb Gabriel C >>>>>> <nix.or.die@googlemail.com>: >>>>>>> >>>>>>> Am Do., 25. Juni 2020 um 06:57 Uhr schrieb Jiri Slaby <jirislaby@kernel.org>: >>>>>>>> >>>>>>>> On 25. 06. 20, 0:05, Gabriel C wrote: >>>>>>>>> Am Mi., 17. Juni 2020 um 18:13 Uhr schrieb Greg Kroah-Hartman >>>>>>>>> <gregkh@linuxfoundation.org>: >>>>>>>>>> >>>>>>>>>> I'm announcing the release of the 5.7.3 kernel. >>>>>>>>>> >>>>>>>>> >>>>>>>>> Hello Greg, >>>>>>>>> >>>>>>>>>> Qiujun Huang (5): >>>>>>>>>> ath9k: Fix use-after-free Read in htc_connect_service >>>>>>>>>> ath9k: Fix use-after-free Read in ath9k_wmi_ctrl_rx >>>>>>>>>> ath9k: Fix use-after-free Write in ath9k_htc_rx_msg >>>>>>>>>> ath9x: Fix stack-out-of-bounds Write in ath9k_hif_usb_rx_cb >>>>>>>>>> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb >>>>>>>>>> >>>>>>>>> >>>>>>>>> We got a report on IRC about 5.7.3+ breaking a USB ath9k Wifi Dongle, >>>>>>>>> while working fine on <5.7.3. >>>>>>>>> >>>>>>>>> I don't have myself such HW, and the reported doesn't have any experience >>>>>>>>> in bisecting the kernel, so we build kernels, each with one of the >>>>>>>>> above commits reverted, >>>>>>>>> to find the bad commit. >>>>>>>>> >>>>>>>>> The winner is: >>>>>>>>> >>>>>>>>> commit 6602f080cb28745259e2fab1a4cf55eeb5894f93 >>>>>>>>> Author: Qiujun Huang <hqjagain@gmail.com> >>>>>>>>> Date: Sat Apr 4 12:18:38 2020 +0800 >>>>>>>>> >>>>>>>>> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb >>>>>>>>> >>>>>>>>> commit 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 upstream. >>>>>>>>> ... >>>>>>>>> >>>>>>>>> Reverting this one fixed his problem. >>>>>>>> >>>>>>>> Obvious question: is 5.8-rc1 (containing the commit) broken too? >>>>>>> >>>>>>> Yes, it does, just checked. >>>>>>> >>>>>>> git tag --contains 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 >>>>>>> v5.8-rc1 >>>>>>> v5.8-rc2 >>>>>>> >>>>>> >>>>>> Sorry, I read the wrong, I just woke up. >>>>>> >>>>>> We didn't test 5.8-rc{1,2} yet but we will today and let you know. >>>>>> >>>>> >>>>> We tested 5.8-rc2 and it is broken too. >>>>> >>>>> The exact HW name is: >>>>> >>>>> TP-link tl-wn722n (Atheros AR9271 chip) >>>> >>>> Great! >>>> >>>> Can you work with the developers to fix this in Linus's tree first? >>> >>> I'm the man in the middle, but sure we will try patches or any suggestions >>> from developers to identify and fix the problem. >>> >>>> >>>> I bet they want to see the output of 'lsusb -v' for this device to see >>>> if the endpoint calculations are correct... >>>> >>> >>> Working on it. As soon the reporter gives me the output, I will post it here. >>> I've told him to run it on a broken and one working kernel. >> >> That is from a good kernel with reverted commit >> https://gist.github.com/AngryPenguinPL/07c8e2abd3b103eaf8978a39ad8577d1 >> >> That is from the broken kernel without the commit reverted >> https://gist.github.com/AngryPenguinPL/5cdc0dd16ce5e59ff3c32c048e2f5111 >> >> This is from 5.7.5 kernel, I don't have yet a 5.8-rc2 package with the >> reverted commit. > > Did this ever get resolved? > > thanks, > > greg k-h > This bug was also reported on the thread where it had been posted originally: https://lore.kernel.org/linux-wireless/20200621020428.6417d6fb@natsu/ I am waiting for Kalle Valo to accept my patch (v2) which reverts the above mentioned commit and which looks correct according to him. He wrote that he would take a closer look at this as soon as he could. I don't know how busy Kalle is, especially under the current circumstances. I will remind him on Thursday (one week after his last e-mail). At the latest I want this to be fixed with the 5.8 release. I think that the patch has to land in several networking trees before it reaches the mainline kernel, so if I can do anything to speed things up, just tell me. Thanks, Viktor ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: ath9k broken [was: Linux 5.7.3] 2020-07-07 20:20 ` Viktor Jägersküpper @ 2020-07-15 15:13 ` Kalle Valo 2020-07-16 8:15 ` Viktor Jägersküpper 0 siblings, 1 reply; 13+ messages in thread From: Kalle Valo @ 2020-07-15 15:13 UTC (permalink / raw) To: Viktor Jägersküpper Cc: Greg Kroah-Hartman, Gabriel C, Jiri Slaby, LKML, Andrew Morton, Linus Torvalds, stable, lwn, angrypenguinpoland, Qiujun Huang, ath9k-devel, linux-wireless Viktor Jägersküpper <viktor_jaegerskuepper@freenet.de> writes: > Greg Kroah-Hartman wrote: >> On Fri, Jun 26, 2020 at 04:40:18PM +0200, Gabriel C wrote: >>> Am Fr., 26. Juni 2020 um 15:51 Uhr schrieb Gabriel C >>> <nix.or.die@googlemail.com>: >>>> >>>> Am Fr., 26. Juni 2020 um 15:40 Uhr schrieb Greg Kroah-Hartman >>>> <gregkh@linuxfoundation.org>: >>>>> >>>>> On Fri, Jun 26, 2020 at 01:48:59PM +0200, Gabriel C wrote: >>>>>> Am Do., 25. Juni 2020 um 12:52 Uhr schrieb Gabriel C >>>>>> <nix.or.die@googlemail.com>: >>>>>>> >>>>>>> Am Do., 25. Juni 2020 um 12:48 Uhr schrieb Gabriel C >>>>>>> <nix.or.die@googlemail.com>: >>>>>>>> >>>>>>>> Am Do., 25. Juni 2020 um 06:57 Uhr schrieb Jiri Slaby <jirislaby@kernel.org>: >>>>>>>>> >>>>>>>>> On 25. 06. 20, 0:05, Gabriel C wrote: >>>>>>>>>> Am Mi., 17. Juni 2020 um 18:13 Uhr schrieb Greg Kroah-Hartman >>>>>>>>>> <gregkh@linuxfoundation.org>: >>>>>>>>>>> >>>>>>>>>>> I'm announcing the release of the 5.7.3 kernel. >>>>>>>>>>> >>>>>>>>>> >>>>>>>>>> Hello Greg, >>>>>>>>>> >>>>>>>>>>> Qiujun Huang (5): >>>>>>>>>>> ath9k: Fix use-after-free Read in htc_connect_service >>>>>>>>>>> ath9k: Fix use-after-free Read in ath9k_wmi_ctrl_rx >>>>>>>>>>> ath9k: Fix use-after-free Write in ath9k_htc_rx_msg >>>>>>>>>>> ath9x: Fix stack-out-of-bounds Write in ath9k_hif_usb_rx_cb >>>>>>>>>>> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb >>>>>>>>>>> >>>>>>>>>> >>>>>>>>>> We got a report on IRC about 5.7.3+ breaking a USB ath9k Wifi Dongle, >>>>>>>>>> while working fine on <5.7.3. >>>>>>>>>> >>>>>>>>>> I don't have myself such HW, and the reported doesn't have any experience >>>>>>>>>> in bisecting the kernel, so we build kernels, each with one of the >>>>>>>>>> above commits reverted, >>>>>>>>>> to find the bad commit. >>>>>>>>>> >>>>>>>>>> The winner is: >>>>>>>>>> >>>>>>>>>> commit 6602f080cb28745259e2fab1a4cf55eeb5894f93 >>>>>>>>>> Author: Qiujun Huang <hqjagain@gmail.com> >>>>>>>>>> Date: Sat Apr 4 12:18:38 2020 +0800 >>>>>>>>>> >>>>>>>>>> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb >>>>>>>>>> >>>>>>>>>> commit 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 upstream. >>>>>>>>>> ... >>>>>>>>>> >>>>>>>>>> Reverting this one fixed his problem. >>>>>>>>> >>>>>>>>> Obvious question: is 5.8-rc1 (containing the commit) broken too? >>>>>>>> >>>>>>>> Yes, it does, just checked. >>>>>>>> >>>>>>>> git tag --contains 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 >>>>>>>> v5.8-rc1 >>>>>>>> v5.8-rc2 >>>>>>>> >>>>>>> >>>>>>> Sorry, I read the wrong, I just woke up. >>>>>>> >>>>>>> We didn't test 5.8-rc{1,2} yet but we will today and let you know. >>>>>>> >>>>>> >>>>>> We tested 5.8-rc2 and it is broken too. >>>>>> >>>>>> The exact HW name is: >>>>>> >>>>>> TP-link tl-wn722n (Atheros AR9271 chip) >>>>> >>>>> Great! >>>>> >>>>> Can you work with the developers to fix this in Linus's tree first? >>>> >>>> I'm the man in the middle, but sure we will try patches or any suggestions >>>> from developers to identify and fix the problem. >>>> >>>>> >>>>> I bet they want to see the output of 'lsusb -v' for this device to see >>>>> if the endpoint calculations are correct... >>>>> >>>> >>>> Working on it. As soon the reporter gives me the output, I will post it here. >>>> I've told him to run it on a broken and one working kernel. >>> >>> That is from a good kernel with reverted commit >>> https://gist.github.com/AngryPenguinPL/07c8e2abd3b103eaf8978a39ad8577d1 >>> >>> That is from the broken kernel without the commit reverted >>> https://gist.github.com/AngryPenguinPL/5cdc0dd16ce5e59ff3c32c048e2f5111 >>> >>> This is from 5.7.5 kernel, I don't have yet a 5.8-rc2 package with the >>> reverted commit. >> >> Did this ever get resolved? >> >> thanks, >> >> greg k-h >> > > This bug was also reported on the thread where it had been posted originally: > https://lore.kernel.org/linux-wireless/20200621020428.6417d6fb@natsu/ > > I am waiting for Kalle Valo to accept my patch (v2) which reverts the above > mentioned commit and which looks correct according to him. He wrote that he > would take a closer look at this as soon as he could. Mark posted a patch which I'm hoping to fix the issue: [1/1] ath9k: Fix regression with Atheros 9271 https://patchwork.kernel.org/patch/11657669/ Can someone confirm this, please? I would rather take Mark's fix than the revert. -- https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: ath9k broken [was: Linux 5.7.3] 2020-07-15 15:13 ` Kalle Valo @ 2020-07-16 8:15 ` Viktor Jägersküpper 2020-07-16 9:02 ` Kalle Valo 0 siblings, 1 reply; 13+ messages in thread From: Viktor Jägersküpper @ 2020-07-16 8:15 UTC (permalink / raw) To: Kalle Valo Cc: Greg Kroah-Hartman, Gabriel C, Jiri Slaby, LKML, Andrew Morton, Linus Torvalds, stable, lwn, angrypenguinpoland, Qiujun Huang, ath9k-devel, linux-wireless, Roman Mamedov [CC'ed Roman] Kalle Valo: > Viktor Jägersküpper <viktor_jaegerskuepper@freenet.de> writes: > >> Greg Kroah-Hartman wrote: >>> On Fri, Jun 26, 2020 at 04:40:18PM +0200, Gabriel C wrote: >>>> Am Fr., 26. Juni 2020 um 15:51 Uhr schrieb Gabriel C >>>> <nix.or.die@googlemail.com>: >>>>> >>>>> Am Fr., 26. Juni 2020 um 15:40 Uhr schrieb Greg Kroah-Hartman >>>>> <gregkh@linuxfoundation.org>: >>>>>> >>>>>> On Fri, Jun 26, 2020 at 01:48:59PM +0200, Gabriel C wrote: >>>>>>> Am Do., 25. Juni 2020 um 12:52 Uhr schrieb Gabriel C >>>>>>> <nix.or.die@googlemail.com>: >>>>>>>> >>>>>>>> Am Do., 25. Juni 2020 um 12:48 Uhr schrieb Gabriel C >>>>>>>> <nix.or.die@googlemail.com>: >>>>>>>>> >>>>>>>>> Am Do., 25. Juni 2020 um 06:57 Uhr schrieb Jiri Slaby <jirislaby@kernel.org>: >>>>>>>>>> >>>>>>>>>> On 25. 06. 20, 0:05, Gabriel C wrote: >>>>>>>>>>> Am Mi., 17. Juni 2020 um 18:13 Uhr schrieb Greg Kroah-Hartman >>>>>>>>>>> <gregkh@linuxfoundation.org>: >>>>>>>>>>>> >>>>>>>>>>>> I'm announcing the release of the 5.7.3 kernel. >>>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> Hello Greg, >>>>>>>>>>> >>>>>>>>>>>> Qiujun Huang (5): >>>>>>>>>>>> ath9k: Fix use-after-free Read in htc_connect_service >>>>>>>>>>>> ath9k: Fix use-after-free Read in ath9k_wmi_ctrl_rx >>>>>>>>>>>> ath9k: Fix use-after-free Write in ath9k_htc_rx_msg >>>>>>>>>>>> ath9x: Fix stack-out-of-bounds Write in ath9k_hif_usb_rx_cb >>>>>>>>>>>> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb >>>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> We got a report on IRC about 5.7.3+ breaking a USB ath9k Wifi Dongle, >>>>>>>>>>> while working fine on <5.7.3. >>>>>>>>>>> >>>>>>>>>>> I don't have myself such HW, and the reported doesn't have any experience >>>>>>>>>>> in bisecting the kernel, so we build kernels, each with one of the >>>>>>>>>>> above commits reverted, >>>>>>>>>>> to find the bad commit. >>>>>>>>>>> >>>>>>>>>>> The winner is: >>>>>>>>>>> >>>>>>>>>>> commit 6602f080cb28745259e2fab1a4cf55eeb5894f93 >>>>>>>>>>> Author: Qiujun Huang <hqjagain@gmail.com> >>>>>>>>>>> Date: Sat Apr 4 12:18:38 2020 +0800 >>>>>>>>>>> >>>>>>>>>>> ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb >>>>>>>>>>> >>>>>>>>>>> commit 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 upstream. >>>>>>>>>>> ... >>>>>>>>>>> >>>>>>>>>>> Reverting this one fixed his problem. >>>>>>>>>> >>>>>>>>>> Obvious question: is 5.8-rc1 (containing the commit) broken too? >>>>>>>>> >>>>>>>>> Yes, it does, just checked. >>>>>>>>> >>>>>>>>> git tag --contains 2bbcaaee1fcbd83272e29f31e2bb7e70d8c49e05 >>>>>>>>> v5.8-rc1 >>>>>>>>> v5.8-rc2 >>>>>>>>> >>>>>>>> >>>>>>>> Sorry, I read the wrong, I just woke up. >>>>>>>> >>>>>>>> We didn't test 5.8-rc{1,2} yet but we will today and let you know. >>>>>>>> >>>>>>> >>>>>>> We tested 5.8-rc2 and it is broken too. >>>>>>> >>>>>>> The exact HW name is: >>>>>>> >>>>>>> TP-link tl-wn722n (Atheros AR9271 chip) >>>>>> >>>>>> Great! >>>>>> >>>>>> Can you work with the developers to fix this in Linus's tree first? >>>>> >>>>> I'm the man in the middle, but sure we will try patches or any suggestions >>>>> from developers to identify and fix the problem. >>>>> >>>>>> >>>>>> I bet they want to see the output of 'lsusb -v' for this device to see >>>>>> if the endpoint calculations are correct... >>>>>> >>>>> >>>>> Working on it. As soon the reporter gives me the output, I will post it here. >>>>> I've told him to run it on a broken and one working kernel. >>>> >>>> That is from a good kernel with reverted commit >>>> https://gist.github.com/AngryPenguinPL/07c8e2abd3b103eaf8978a39ad8577d1 >>>> >>>> That is from the broken kernel without the commit reverted >>>> https://gist.github.com/AngryPenguinPL/5cdc0dd16ce5e59ff3c32c048e2f5111 >>>> >>>> This is from 5.7.5 kernel, I don't have yet a 5.8-rc2 package with the >>>> reverted commit. >>> >>> Did this ever get resolved? >>> >>> thanks, >>> >>> greg k-h >>> >> >> This bug was also reported on the thread where it had been posted originally: >> https://lore.kernel.org/linux-wireless/20200621020428.6417d6fb@natsu/ >> >> I am waiting for Kalle Valo to accept my patch (v2) which reverts the above >> mentioned commit and which looks correct according to him. He wrote that he >> would take a closer look at this as soon as he could. > > Mark posted a patch which I'm hoping to fix the issue: > > [1/1] ath9k: Fix regression with Atheros 9271 > > https://patchwork.kernel.org/patch/11657669/ > > Can someone confirm this, please? I would rather take Mark's fix than > the revert. > 12345678901234567890123456789012345678901234567890123456789012345678901234567890 This fixes the issue for me. Unfortunately the revert landed in 5.7.9 (it was also queued for the older releases, but I didn't check them) because Hans de Goede requested it since he didn't know about Mark's patch. So if you want to fix the stable and longterm kernels properly, we need another patch which includes a re-revert and Mark's patch. But Mark's patch should definitely land in the mainline kernel first. ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: ath9k broken [was: Linux 5.7.3] 2020-07-16 8:15 ` Viktor Jägersküpper @ 2020-07-16 9:02 ` Kalle Valo 0 siblings, 0 replies; 13+ messages in thread From: Kalle Valo @ 2020-07-16 9:02 UTC (permalink / raw) To: Viktor Jägersküpper Cc: Greg Kroah-Hartman, Gabriel C, Jiri Slaby, LKML, Andrew Morton, Linus Torvalds, stable, lwn, angrypenguinpoland, Qiujun Huang, ath9k-devel, linux-wireless, Roman Mamedov Viktor Jägersküpper <viktor_jaegerskuepper@freenet.de> writes: >>> This bug was also reported on the thread where it had been posted originally: >>> https://lore.kernel.org/linux-wireless/20200621020428.6417d6fb@natsu/ >>> >>> I am waiting for Kalle Valo to accept my patch (v2) which reverts the above >>> mentioned commit and which looks correct according to him. He wrote that he >>> would take a closer look at this as soon as he could. >> >> Mark posted a patch which I'm hoping to fix the issue: >> >> [1/1] ath9k: Fix regression with Atheros 9271 >> >> https://patchwork.kernel.org/patch/11657669/ >> >> Can someone confirm this, please? I would rather take Mark's fix than >> the revert. >> > > This fixes the issue for me. Great, thanks for testing. I'll wait at least another day for more comments and then queue it for v5.8. -- https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: ath9k broken [was: Linux 5.7.3] 2020-06-25 4:57 ` ath9k broken [was: Linux 5.7.3] Jiri Slaby 2020-06-25 10:48 ` Gabriel C @ 2020-06-27 10:13 ` Roman Mamedov 1 sibling, 0 replies; 13+ messages in thread From: Roman Mamedov @ 2020-06-27 10:13 UTC (permalink / raw) To: Jiri Slaby Cc: Gabriel C, Greg Kroah-Hartman, LKML, Andrew Morton, Linus Torvalds, stable, lwn, angrypenguinpoland, Qiujun Huang, ath9k-devel, linux-wireless On Thu, 25 Jun 2020 06:57:13 +0200 Jiri Slaby <jirislaby@kernel.org> wrote: > I fail to see how the commit could cause an issue like this. Is this > really reproducibly broken with the commit and irreproducible without > it? As it looks like a USB/wiring problem: > usb 1-2: USB disconnect, device number 2 > ath: phy0: Reading Magic # failed > ath: phy0: Unable to initialize hardware; initialization status: -5 > ... > usb 1-2: device descriptor read/64, error -110 > usb 1-2: device descriptor read/64, error -71 > > Ccing ath9k maintainers too. Note that this has been previously reported in: https://bugzilla.kernel.org/show_bug.cgi?id=208251 and confirmed by several people on various stable series and the mainline that the referenced commit is indeed causing the problem. I don't get the "device descriptor read" errors though, my dmesg is posted on the bug report, it just says "ath9k_htc: Failed to initialize the device". > > I don't have so much info about the HW, besides a dmesg showing the > > phy breaking. > > I also added the reporter to CC too. > > > > https://gist.github.com/AngryPenguinPL/1e545f0da3c2339e443b9e5044fcccea > > > > If you need more info, please let me know and I'll try my best to get > > it as fast as possible for you. -- With respect, Roman ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2020-07-16 9:02 UTC | newest] Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <1592410366125160@kroah.com> [not found] ` <CAEJqkgjV8p6LtBV8YUGbNb0vYzKOQt4-AMAvYw5mzFr3eicyTg@mail.gmail.com> 2020-06-25 4:57 ` ath9k broken [was: Linux 5.7.3] Jiri Slaby 2020-06-25 10:48 ` Gabriel C 2020-06-25 10:52 ` Gabriel C 2020-06-26 11:48 ` Gabriel C 2020-06-26 13:39 ` Greg Kroah-Hartman 2020-06-26 13:51 ` Gabriel C 2020-06-26 14:40 ` Gabriel C 2020-07-07 14:11 ` Greg Kroah-Hartman 2020-07-07 20:20 ` Viktor Jägersküpper 2020-07-15 15:13 ` Kalle Valo 2020-07-16 8:15 ` Viktor Jägersküpper 2020-07-16 9:02 ` Kalle Valo 2020-06-27 10:13 ` Roman Mamedov
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).