* Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 [not found] <a8cddf24-ecfc-088e-27f4-98cbbb5fb67c@penguintechs.org> @ 2021-08-23 12:53 ` Kalle Valo 2021-08-23 16:02 ` Nicolas Schichan 2021-08-23 16:17 ` Wren Turkal 0 siblings, 2 replies; 15+ messages in thread From: Kalle Valo @ 2021-08-23 12:53 UTC (permalink / raw) To: Wren Turkal Cc: ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain (adding linux-wireless, regressions and some mhi folks) Wren Turkal <wt@penguintechs.org> writes: > I have no device for the wifi on my computer. 5.13 worked pretty well. > I also found others reporting similar issues: > https://forum.manjaro.org/t/kernel5-14-rc6-ath11k-on-qca6390-regression-mhi-probe-failure/79362 > > Here's info about the hardware on my laptop: > ➜ ~ sudo dmidecode -s system-product-name > XPS 13 9310 > > ➜ ~ sudo dmidecode -s bios-version > 3.0.4 Do you get the same mhi error as in the forum post? qcom_mhi_qrtr: probe of mhi0_IPCR failed with error -22 MHI folks, any ideas? I have XPS 13 9310 myself but I'm not able to test v5.14-rc6 kernel right now. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-23 12:53 ` 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 Kalle Valo @ 2021-08-23 16:02 ` Nicolas Schichan 2021-08-23 17:40 ` Kalle Valo 2021-08-23 16:17 ` Wren Turkal 1 sibling, 1 reply; 15+ messages in thread From: Nicolas Schichan @ 2021-08-23 16:02 UTC (permalink / raw) To: Kalle Valo, Wren Turkal Cc: ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain On 23/08/2021 14:53, Kalle Valo wrote: > (adding linux-wireless, regressions and some mhi folks) > > Wren Turkal <wt@penguintechs.org> writes: > >> I have no device for the wifi on my computer. 5.13 worked pretty well. >> I also found others reporting similar issues: >> https://forum.manjaro.org/t/kernel5-14-rc6-ath11k-on-qca6390-regression-mhi-probe-failure/79362 >> >> Here's info about the hardware on my laptop: >> ➜ ~ sudo dmidecode -s system-product-name >> XPS 13 9310 >> >> ➜ ~ sudo dmidecode -s bios-version >> 3.0.4 > > Do you get the same mhi error as in the forum post? > > qcom_mhi_qrtr: probe of mhi0_IPCR failed with error -22 > > MHI folks, any ideas? I have XPS 13 9310 myself but I'm not able to test > v5.14-rc6 kernel right now. Hello, I have hit the same issue on Linux 5.14-rc7: [ 14.780566] qcom_mhi_qrtr: probe of mhi0_IPCR failed with error -22 This looks to be caused by commit: ce78ffa3ef16 net: really fix the build... Reverting this commit results in ath11k being able to load without errors. Regards, -- Nicolas Schichan ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-23 16:02 ` Nicolas Schichan @ 2021-08-23 17:40 ` Kalle Valo 2021-08-26 14:25 ` [regression] " Kalle Valo 0 siblings, 1 reply; 15+ messages in thread From: Kalle Valo @ 2021-08-23 17:40 UTC (permalink / raw) To: Nicolas Schichan Cc: Wren Turkal, ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain, David S. Miller, Jakub Kicinski (Adding Dave and Jakub to CC) Nicolas Schichan <nschichan@freebox.fr> writes: > On 23/08/2021 14:53, Kalle Valo wrote: >> (adding linux-wireless, regressions and some mhi folks) >> >> Wren Turkal <wt@penguintechs.org> writes: >> >>> I have no device for the wifi on my computer. 5.13 worked pretty well. >>> I also found others reporting similar issues: >>> https://forum.manjaro.org/t/kernel5-14-rc6-ath11k-on-qca6390-regression-mhi-probe-failure/79362 >>> >>> Here's info about the hardware on my laptop: >>> ➜ ~ sudo dmidecode -s system-product-name >>> XPS 13 9310 >>> >>> ➜ ~ sudo dmidecode -s bios-version >>> 3.0.4 >> >> Do you get the same mhi error as in the forum post? >> >> qcom_mhi_qrtr: probe of mhi0_IPCR failed with error -22 >> >> MHI folks, any ideas? I have XPS 13 9310 myself but I'm not able to test >> v5.14-rc6 kernel right now. > > Hello, > > I have hit the same issue on Linux 5.14-rc7: > > [ 14.780566] qcom_mhi_qrtr: probe of mhi0_IPCR failed with error -22 > > This looks to be caused by commit: > > ce78ffa3ef16 net: really fix the build... > > Reverting this commit results in ath11k being able to load without errors. Thanks, this helps a lot. Commit ce78ffa3ef16 was introduced in v5.14-rc5 so I assume that's the first release which broke ath11k. Dave, what build issue did you fix with commit ce78ffa3ef16? Could we revert the commit? The problem is that, unless we fix this issue, with v5.14 WLAN will be broken on a quite few newer Dell laptops. Unfortunately I'm travelling right now and I'm not able to look at this in detail until Thursday or so. And I assume Linus will make the final v5.14 release on Sunday so we are on a tight schedule :/ -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 15+ messages in thread
* [regression] Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-23 17:40 ` Kalle Valo @ 2021-08-26 14:25 ` Kalle Valo 2021-08-26 14:38 ` Greg Kroah-Hartman 0 siblings, 1 reply; 15+ messages in thread From: Kalle Valo @ 2021-08-26 14:25 UTC (permalink / raw) To: David S. Miller, Jakub Kicinski, Linus Torvalds, Greg Kroah-Hartman Cc: Wren Turkal, ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain, Nicolas Schichan (Adding Linus and Greg) Kalle Valo <kvalo@codeaurora.org> writes: > (Adding Dave and Jakub to CC) > > Nicolas Schichan <nschichan@freebox.fr> writes: > >> On 23/08/2021 14:53, Kalle Valo wrote: >>> (adding linux-wireless, regressions and some mhi folks) >>> >>> Wren Turkal <wt@penguintechs.org> writes: >>> >>>> I have no device for the wifi on my computer. 5.13 worked pretty well. >>>> I also found others reporting similar issues: >>>> https://forum.manjaro.org/t/kernel5-14-rc6-ath11k-on-qca6390-regression-mhi-probe-failure/79362 >>>> >>>> Here's info about the hardware on my laptop: >>>> ➜ ~ sudo dmidecode -s system-product-name >>>> XPS 13 9310 >>>> >>>> ➜ ~ sudo dmidecode -s bios-version >>>> 3.0.4 >>> >>> Do you get the same mhi error as in the forum post? >>> >>> qcom_mhi_qrtr: probe of mhi0_IPCR failed with error -22 >>> >>> MHI folks, any ideas? I have XPS 13 9310 myself but I'm not able to test >>> v5.14-rc6 kernel right now. >> >> Hello, >> >> I have hit the same issue on Linux 5.14-rc7: >> >> [ 14.780566] qcom_mhi_qrtr: probe of mhi0_IPCR failed with error -22 >> >> This looks to be caused by commit: >> >> ce78ffa3ef16 net: really fix the build... >> >> Reverting this commit results in ath11k being able to load without errors. > > Thanks, this helps a lot. Commit ce78ffa3ef16 was introduced in > v5.14-rc5 so I assume that's the first release which broke ath11k. > > Dave, what build issue did you fix with commit ce78ffa3ef16? Could we > revert the commit? The problem is that, unless we fix this issue, with > v5.14 WLAN will be broken on a quite few newer Dell laptops. I did some investiation and I suspect that commit ce78ffa3ef16 ("net: really fix the build...")[1] is for handling a conflict between net-next and char-misc-next trees related to Loic's commit 0092a1e3f763 ("bus: mhi: Add inbound buffers allocation flag"). Greg mentions this in a commit[2]: commit 813272ed5238b37c81e448b302048e8008570121 Merge: de0534df9347 36a21d51725a Author: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Merge 5.14-rc5 into char-misc-next We need the fixes in here as well, and resolves some merge issues with the mhi codebase. If my assumption is correct when I propose that we revert commit ce78ffa3ef16 for v5.14, AFAICS it's not really needed for v5.14 (commit 0092a1e3f763 is in -next, not in Linus' tree yet) and commit ce78ffa3ef16 breaks ath11k. And the conflict between net-next and char-misc-next can be then later fixed during the merge window. Will this work for everyone? If no objections, I'll submit the revert to Linus later today. We are getting really close to final v5.14 release so not much time left to fix this. Just to reiterate why the urgency: commit ce78ffa3ef16 broke ath11k in v5.14-rc5, users have reported (and I have confirmed) that at least QCA6390 support is broken but I suspect all Qualcomm Wi-Fi 6 devices supported by ath11k are currently broken. [1] https://git.kernel.org/linus/ce78ffa3ef16 [2] https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git/commit/?h=char-misc-next&id=813272ed5238b37c81e448b302048e8008570121 -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [regression] Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-26 14:25 ` [regression] " Kalle Valo @ 2021-08-26 14:38 ` Greg Kroah-Hartman 2021-08-26 14:48 ` Jakub Kicinski 2021-08-27 12:46 ` Kalle Valo 0 siblings, 2 replies; 15+ messages in thread From: Greg Kroah-Hartman @ 2021-08-26 14:38 UTC (permalink / raw) To: Kalle Valo Cc: David S. Miller, Jakub Kicinski, Linus Torvalds, Wren Turkal, ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain, Nicolas Schichan On Thu, Aug 26, 2021 at 05:25:04PM +0300, Kalle Valo wrote: > (Adding Linus and Greg) > > Kalle Valo <kvalo@codeaurora.org> writes: > > > (Adding Dave and Jakub to CC) > > > > Nicolas Schichan <nschichan@freebox.fr> writes: > > > >> On 23/08/2021 14:53, Kalle Valo wrote: > >>> (adding linux-wireless, regressions and some mhi folks) > >>> > >>> Wren Turkal <wt@penguintechs.org> writes: > >>> > >>>> I have no device for the wifi on my computer. 5.13 worked pretty well. > >>>> I also found others reporting similar issues: > >>>> https://forum.manjaro.org/t/kernel5-14-rc6-ath11k-on-qca6390-regression-mhi-probe-failure/79362 > >>>> > >>>> Here's info about the hardware on my laptop: > >>>> ➜ ~ sudo dmidecode -s system-product-name > >>>> XPS 13 9310 > >>>> > >>>> ➜ ~ sudo dmidecode -s bios-version > >>>> 3.0.4 > >>> > >>> Do you get the same mhi error as in the forum post? > >>> > >>> qcom_mhi_qrtr: probe of mhi0_IPCR failed with error -22 > >>> > >>> MHI folks, any ideas? I have XPS 13 9310 myself but I'm not able to test > >>> v5.14-rc6 kernel right now. > >> > >> Hello, > >> > >> I have hit the same issue on Linux 5.14-rc7: > >> > >> [ 14.780566] qcom_mhi_qrtr: probe of mhi0_IPCR failed with error -22 > >> > >> This looks to be caused by commit: > >> > >> ce78ffa3ef16 net: really fix the build... > >> > >> Reverting this commit results in ath11k being able to load without errors. > > > > Thanks, this helps a lot. Commit ce78ffa3ef16 was introduced in > > v5.14-rc5 so I assume that's the first release which broke ath11k. > > > > Dave, what build issue did you fix with commit ce78ffa3ef16? Could we > > revert the commit? The problem is that, unless we fix this issue, with > > v5.14 WLAN will be broken on a quite few newer Dell laptops. > > I did some investiation and I suspect that commit ce78ffa3ef16 ("net: > really fix the build...")[1] is for handling a conflict between net-next > and char-misc-next trees related to Loic's commit 0092a1e3f763 ("bus: > mhi: Add inbound buffers allocation flag"). Greg mentions this in a > commit[2]: > > commit 813272ed5238b37c81e448b302048e8008570121 > Merge: de0534df9347 36a21d51725a > Author: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > Merge 5.14-rc5 into char-misc-next > > We need the fixes in here as well, and resolves some merge issues with > the mhi codebase. > > If my assumption is correct when I propose that we revert commit > ce78ffa3ef16 for v5.14, AFAICS it's not really needed for v5.14 (commit > 0092a1e3f763 is in -next, not in Linus' tree yet) and commit > ce78ffa3ef16 breaks ath11k. And the conflict between net-next and > char-misc-next can be then later fixed during the merge window. > > Will this work for everyone? If no objections, I'll submit the revert to > Linus later today. We are getting really close to final v5.14 release so > not much time left to fix this. > > Just to reiterate why the urgency: commit ce78ffa3ef16 broke ath11k in > v5.14-rc5, users have reported (and I have confirmed) that at least > QCA6390 support is broken but I suspect all Qualcomm Wi-Fi 6 devices > supported by ath11k are currently broken. No objection from me for reverting that, if it fixes the problems you are seeing for 5.14-final. The goal was for the mhi changes to go through the networking tree in the first place, I don't see how this got out of sync. If this is reverted, some help on how to resolve the merge issues it will cause would be appreciated. thanks, greg k-h ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [regression] Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-26 14:38 ` Greg Kroah-Hartman @ 2021-08-26 14:48 ` Jakub Kicinski 2021-08-26 15:00 ` Kalle Valo 2021-08-27 12:46 ` Kalle Valo 1 sibling, 1 reply; 15+ messages in thread From: Jakub Kicinski @ 2021-08-26 14:48 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Kalle Valo, David S. Miller, Linus Torvalds, Wren Turkal, ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain, Nicolas Schichan On Thu, 26 Aug 2021 16:38:35 +0200 Greg Kroah-Hartman wrote: > > I did some investiation and I suspect that commit ce78ffa3ef16 ("net: > > really fix the build...")[1] is for handling a conflict between net-next > > and char-misc-next trees related to Loic's commit 0092a1e3f763 ("bus: > > mhi: Add inbound buffers allocation flag"). Greg mentions this in a > > commit[2]: > > > > commit 813272ed5238b37c81e448b302048e8008570121 > > Merge: de0534df9347 36a21d51725a > > Author: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > > > Merge 5.14-rc5 into char-misc-next > > > > We need the fixes in here as well, and resolves some merge issues with > > the mhi codebase. > > > > If my assumption is correct when I propose that we revert commit > > ce78ffa3ef16 for v5.14, AFAICS it's not really needed for v5.14 (commit > > 0092a1e3f763 is in -next, not in Linus' tree yet) and commit > > ce78ffa3ef16 breaks ath11k. And the conflict between net-next and > > char-misc-next can be then later fixed during the merge window. > > > > Will this work for everyone? If no objections, I'll submit the revert to > > Linus later today. We are getting really close to final v5.14 release so > > not much time left to fix this. > > > > Just to reiterate why the urgency: commit ce78ffa3ef16 broke ath11k in > > v5.14-rc5, users have reported (and I have confirmed) that at least > > QCA6390 support is broken but I suspect all Qualcomm Wi-Fi 6 devices > > supported by ath11k are currently broken. > > No objection from me for reverting that, if it fixes the problems you > are seeing for 5.14-final. > > The goal was for the mhi changes to go through the networking tree in > the first place, I don't see how this got out of sync. > > If this is reverted, some help on how to resolve the merge issues it > will cause would be appreciated. Also no objections here. FWIW I'm about to send the last PR for networking, still waiting on BPF. You can send the revert to netdev, or directly to Linus as you prefer. LMK. ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [regression] Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-26 14:48 ` Jakub Kicinski @ 2021-08-26 15:00 ` Kalle Valo 2021-08-26 17:33 ` Kalle Valo 0 siblings, 1 reply; 15+ messages in thread From: Kalle Valo @ 2021-08-26 15:00 UTC (permalink / raw) To: Jakub Kicinski Cc: Greg Kroah-Hartman, David S. Miller, Linus Torvalds, Wren Turkal, ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain, Nicolas Schichan Jakub Kicinski <kuba@kernel.org> writes: > On Thu, 26 Aug 2021 16:38:35 +0200 Greg Kroah-Hartman wrote: >> > I did some investiation and I suspect that commit ce78ffa3ef16 ("net: >> > really fix the build...")[1] is for handling a conflict between net-next >> > and char-misc-next trees related to Loic's commit 0092a1e3f763 ("bus: >> > mhi: Add inbound buffers allocation flag"). Greg mentions this in a >> > commit[2]: >> > >> > commit 813272ed5238b37c81e448b302048e8008570121 >> > Merge: de0534df9347 36a21d51725a >> > Author: Greg Kroah-Hartman <gregkh@linuxfoundation.org> >> > >> > Merge 5.14-rc5 into char-misc-next >> > >> > We need the fixes in here as well, and resolves some merge issues with >> > the mhi codebase. >> > >> > If my assumption is correct when I propose that we revert commit >> > ce78ffa3ef16 for v5.14, AFAICS it's not really needed for v5.14 (commit >> > 0092a1e3f763 is in -next, not in Linus' tree yet) and commit >> > ce78ffa3ef16 breaks ath11k. And the conflict between net-next and >> > char-misc-next can be then later fixed during the merge window. >> > >> > Will this work for everyone? If no objections, I'll submit the revert to >> > Linus later today. We are getting really close to final v5.14 release so >> > not much time left to fix this. >> > >> > Just to reiterate why the urgency: commit ce78ffa3ef16 broke ath11k in >> > v5.14-rc5, users have reported (and I have confirmed) that at least >> > QCA6390 support is broken but I suspect all Qualcomm Wi-Fi 6 devices >> > supported by ath11k are currently broken. >> >> No objection from me for reverting that, if it fixes the problems you >> are seeing for 5.14-final. >> >> The goal was for the mhi changes to go through the networking tree in >> the first place, I don't see how this got out of sync. >> >> If this is reverted, some help on how to resolve the merge issues it >> will cause would be appreciated. > > Also no objections here. FWIW I'm about to send the last PR for > networking, still waiting on BPF. You can send the revert to netdev, > or directly to Linus as you prefer. LMK. I prefer take it via the net tree if possible, so if you can wait ~2h and I'll send it to you. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [regression] Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-26 15:00 ` Kalle Valo @ 2021-08-26 17:33 ` Kalle Valo 2021-08-26 18:12 ` Jakub Kicinski 0 siblings, 1 reply; 15+ messages in thread From: Kalle Valo @ 2021-08-26 17:33 UTC (permalink / raw) To: Jakub Kicinski Cc: Greg Kroah-Hartman, David S. Miller, Linus Torvalds, Wren Turkal, ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain, Nicolas Schichan Kalle Valo <kvalo@codeaurora.org> writes: > Jakub Kicinski <kuba@kernel.org> writes: > >> On Thu, 26 Aug 2021 16:38:35 +0200 Greg Kroah-Hartman wrote: >>> > I did some investiation and I suspect that commit ce78ffa3ef16 ("net: >>> > really fix the build...")[1] is for handling a conflict between net-next >>> > and char-misc-next trees related to Loic's commit 0092a1e3f763 ("bus: >>> > mhi: Add inbound buffers allocation flag"). Greg mentions this in a >>> > commit[2]: >>> > >>> > commit 813272ed5238b37c81e448b302048e8008570121 >>> > Merge: de0534df9347 36a21d51725a >>> > Author: Greg Kroah-Hartman <gregkh@linuxfoundation.org> >>> > >>> > Merge 5.14-rc5 into char-misc-next >>> > >>> > We need the fixes in here as well, and resolves some merge issues with >>> > the mhi codebase. >>> > >>> > If my assumption is correct when I propose that we revert commit >>> > ce78ffa3ef16 for v5.14, AFAICS it's not really needed for v5.14 (commit >>> > 0092a1e3f763 is in -next, not in Linus' tree yet) and commit >>> > ce78ffa3ef16 breaks ath11k. And the conflict between net-next and >>> > char-misc-next can be then later fixed during the merge window. >>> > >>> > Will this work for everyone? If no objections, I'll submit the revert to >>> > Linus later today. We are getting really close to final v5.14 release so >>> > not much time left to fix this. >>> > >>> > Just to reiterate why the urgency: commit ce78ffa3ef16 broke ath11k in >>> > v5.14-rc5, users have reported (and I have confirmed) that at least >>> > QCA6390 support is broken but I suspect all Qualcomm Wi-Fi 6 devices >>> > supported by ath11k are currently broken. >>> >>> No objection from me for reverting that, if it fixes the problems you >>> are seeing for 5.14-final. >>> >>> The goal was for the mhi changes to go through the networking tree in >>> the first place, I don't see how this got out of sync. >>> >>> If this is reverted, some help on how to resolve the merge issues it >>> will cause would be appreciated. >> >> Also no objections here. FWIW I'm about to send the last PR for >> networking, still waiting on BPF. You can send the revert to netdev, >> or directly to Linus as you prefer. LMK. > > I prefer take it via the net tree if possible, so if you can wait ~2h > and I'll send it to you. I now submitted the revert, please take it into the net tree if you still can: https://patchwork.kernel.org/project/netdevbpf/patch/20210826172816.24478-1-kvalo@codeaurora.org/ I also tested the build with various QRTR options and didn't see any build errors. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [regression] Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-26 17:33 ` Kalle Valo @ 2021-08-26 18:12 ` Jakub Kicinski 2021-08-27 12:41 ` Kalle Valo 0 siblings, 1 reply; 15+ messages in thread From: Jakub Kicinski @ 2021-08-26 18:12 UTC (permalink / raw) To: Kalle Valo Cc: Greg Kroah-Hartman, David S. Miller, Linus Torvalds, Wren Turkal, ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain, Nicolas Schichan On Thu, 26 Aug 2021 20:33:47 +0300 Kalle Valo wrote: > >> Also no objections here. FWIW I'm about to send the last PR for > >> networking, still waiting on BPF. You can send the revert to netdev, > >> or directly to Linus as you prefer. LMK. > > > > I prefer take it via the net tree if possible, so if you can wait ~2h > > and I'll send it to you. > > I now submitted the revert, please take it into the net tree if you > still can: > > https://patchwork.kernel.org/project/netdevbpf/patch/20210826172816.24478-1-kvalo@codeaurora.org/ > > I also tested the build with various QRTR options and didn't see any > build errors. Done, thank you! ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [regression] Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-26 18:12 ` Jakub Kicinski @ 2021-08-27 12:41 ` Kalle Valo 2021-08-30 16:04 ` Nicolas Schichan 0 siblings, 1 reply; 15+ messages in thread From: Kalle Valo @ 2021-08-27 12:41 UTC (permalink / raw) To: Jakub Kicinski Cc: Greg Kroah-Hartman, David S. Miller, Linus Torvalds, Wren Turkal, ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain, Nicolas Schichan Jakub Kicinski <kuba@kernel.org> writes: > On Thu, 26 Aug 2021 20:33:47 +0300 Kalle Valo wrote: >> >> Also no objections here. FWIW I'm about to send the last PR for >> >> networking, still waiting on BPF. You can send the revert to netdev, >> >> or directly to Linus as you prefer. LMK. >> > >> > I prefer take it via the net tree if possible, so if you can wait ~2h >> > and I'll send it to you. >> >> I now submitted the revert, please take it into the net tree if you >> still can: >> >> https://patchwork.kernel.org/project/netdevbpf/patch/20210826172816.24478-1-kvalo@codeaurora.org/ >> >> I also tested the build with various QRTR options and didn't see any >> build errors. > > Done, thank you! Thank you waiting for me :) I just tested commit 7e9965cddea8 from Linus' tree on Dell XPS 13 9310 laptop with QCA6390 and ath11k works without problems. So the issue should be solved in the final v5.14 release. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [regression] Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-27 12:41 ` Kalle Valo @ 2021-08-30 16:04 ` Nicolas Schichan 2021-08-31 10:37 ` Kalle Valo 0 siblings, 1 reply; 15+ messages in thread From: Nicolas Schichan @ 2021-08-30 16:04 UTC (permalink / raw) To: Kalle Valo Cc: Jakub Kicinski, Greg Kroah-Hartman, David S. Miller, Linus Torvalds, Wren Turkal, ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain On Fri, Aug 27, 2021 at 2:41 PM Kalle Valo <kvalo@codeaurora.org> wrote: > Jakub Kicinski <kuba@kernel.org> writes: > > Done, thank you! > > Thank you waiting for me :) > > I just tested commit 7e9965cddea8 from Linus' tree on Dell XPS 13 9310 > laptop with QCA6390 and ath11k works without problems. So the issue > should be solved in the final v5.14 release. > Good evening, I have tested loading ath11k on v5.14.0 and I can confirm that the driver loads fine now, with a phy and wlan interface being created as a result. Thanks and best regards, -- Nicolas Schichan ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [regression] Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-30 16:04 ` Nicolas Schichan @ 2021-08-31 10:37 ` Kalle Valo 0 siblings, 0 replies; 15+ messages in thread From: Kalle Valo @ 2021-08-31 10:37 UTC (permalink / raw) To: Nicolas Schichan Cc: Jakub Kicinski, Greg Kroah-Hartman, David S. Miller, Linus Torvalds, Wren Turkal, ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain Nicolas Schichan <nschichan@freebox.fr> writes: > On Fri, Aug 27, 2021 at 2:41 PM Kalle Valo <kvalo@codeaurora.org> wrote: >> Jakub Kicinski <kuba@kernel.org> writes: >> > Done, thank you! >> >> Thank you waiting for me :) >> >> I just tested commit 7e9965cddea8 from Linus' tree on Dell XPS 13 9310 >> laptop with QCA6390 and ath11k works without problems. So the issue >> should be solved in the final v5.14 release. >> > > Good evening, > > I have tested loading ath11k on v5.14.0 and I can confirm that the > driver loads fine now, with a phy and wlan interface being created as > a result. Great, thanks for testing and letting us know. And espeacially huge thanks for finding the commit which broke this! -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [regression] Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-26 14:38 ` Greg Kroah-Hartman 2021-08-26 14:48 ` Jakub Kicinski @ 2021-08-27 12:46 ` Kalle Valo 1 sibling, 0 replies; 15+ messages in thread From: Kalle Valo @ 2021-08-27 12:46 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: David S. Miller, Jakub Kicinski, Linus Torvalds, Wren Turkal, ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain, Nicolas Schichan Greg Kroah-Hartman <gregkh@linuxfoundation.org> writes: >> I did some investiation and I suspect that commit ce78ffa3ef16 ("net: >> really fix the build...")[1] is for handling a conflict between net-next >> and char-misc-next trees related to Loic's commit 0092a1e3f763 ("bus: >> mhi: Add inbound buffers allocation flag"). Greg mentions this in a >> commit[2]: >> >> commit 813272ed5238b37c81e448b302048e8008570121 >> Merge: de0534df9347 36a21d51725a >> Author: Greg Kroah-Hartman <gregkh@linuxfoundation.org> >> >> Merge 5.14-rc5 into char-misc-next >> >> We need the fixes in here as well, and resolves some merge issues with >> the mhi codebase. >> >> If my assumption is correct when I propose that we revert commit >> ce78ffa3ef16 for v5.14, AFAICS it's not really needed for v5.14 (commit >> 0092a1e3f763 is in -next, not in Linus' tree yet) and commit >> ce78ffa3ef16 breaks ath11k. And the conflict between net-next and >> char-misc-next can be then later fixed during the merge window. >> >> Will this work for everyone? If no objections, I'll submit the revert to >> Linus later today. We are getting really close to final v5.14 release so >> not much time left to fix this. >> >> Just to reiterate why the urgency: commit ce78ffa3ef16 broke ath11k in >> v5.14-rc5, users have reported (and I have confirmed) that at least >> QCA6390 support is broken but I suspect all Qualcomm Wi-Fi 6 devices >> supported by ath11k are currently broken. > > No objection from me for reverting that, if it fixes the problems you > are seeing for 5.14-final. Thanks! > The goal was for the mhi changes to go through the networking tree in > the first place, I don't see how this got out of sync. > > If this is reverted, some help on how to resolve the merge issues it > will cause would be appreciated. Yes, I'll help with this. I see that Stephen already reported the reappeared merge issue. It's just that I'm not familiar with inner workings of the MHI subsystem, but I can test code etc. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-23 12:53 ` 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 Kalle Valo 2021-08-23 16:02 ` Nicolas Schichan @ 2021-08-23 16:17 ` Wren Turkal 2021-08-27 12:43 ` Kalle Valo 1 sibling, 1 reply; 15+ messages in thread From: Wren Turkal @ 2021-08-23 16:17 UTC (permalink / raw) To: Kalle Valo Cc: ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain On 8/23/21 5:53 AM, Kalle Valo wrote: > Do you get the same mhi error as in the forum post? > > qcom_mhi_qrtr: probe of mhi0_IPCR failed with error -22 > > MHI folks, any ideas? I have XPS 13 9310 myself but I'm not able to test > v5.14-rc6 kernel right now. Yes, I get that same message in my logs. FWIW, ath11k_pci now does not get loaded on boot. I can manually load it, but it doesn't seem to do anything. wt -- You're more amazing than you think! ymatyt.com ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 2021-08-23 16:17 ` Wren Turkal @ 2021-08-27 12:43 ` Kalle Valo 0 siblings, 0 replies; 15+ messages in thread From: Kalle Valo @ 2021-08-27 12:43 UTC (permalink / raw) To: Wren Turkal Cc: ath11k, linux-wireless, regressions, Manivannan Sadhasivam, Hemant Kumar, Loic Poulain Wren Turkal <wt@penguintechs.org> writes: > On 8/23/21 5:53 AM, Kalle Valo wrote: >> Do you get the same mhi error as in the forum post? >> >> qcom_mhi_qrtr: probe of mhi0_IPCR failed with error -22 >> >> MHI folks, any ideas? I have XPS 13 9310 myself but I'm not able to test >> v5.14-rc6 kernel right now. > > Yes, I get that same message in my logs. > > FWIW, ath11k_pci now does not get loaded on boot. I can manually load > it, but it doesn't seem to do anything. Do you know if this automatic module loading is related to this qrtr regression or a separate problem? Can you try Linus' tree and report if the module load problem still exists? The qrtr regression is fixed now. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2021-08-31 10:37 UTC | newest] Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <a8cddf24-ecfc-088e-27f4-98cbbb5fb67c@penguintechs.org> 2021-08-23 12:53 ` 5.14 rc6 broken for QCA6390 on Dell XPS 13 9310 Kalle Valo 2021-08-23 16:02 ` Nicolas Schichan 2021-08-23 17:40 ` Kalle Valo 2021-08-26 14:25 ` [regression] " Kalle Valo 2021-08-26 14:38 ` Greg Kroah-Hartman 2021-08-26 14:48 ` Jakub Kicinski 2021-08-26 15:00 ` Kalle Valo 2021-08-26 17:33 ` Kalle Valo 2021-08-26 18:12 ` Jakub Kicinski 2021-08-27 12:41 ` Kalle Valo 2021-08-30 16:04 ` Nicolas Schichan 2021-08-31 10:37 ` Kalle Valo 2021-08-27 12:46 ` Kalle Valo 2021-08-23 16:17 ` Wren Turkal 2021-08-27 12:43 ` Kalle Valo
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).