Closed sarrchri closed 11 months ago
I took another look at this and from what I see via https://elixir.bootlin.com/, the change is backported to pretty much every 6.x kernel < 6.5 at some point. I think these should be taken into account, so I'll close the PR for now and re-do it, this time on a separate branch even (accidentally did the commit to my main branch and PR'd it).
I'll make the changes this evening after work.
Maybe was backported to all TLS?
Could be. I re-created the PR from another branch, although ended up doing the same change in the end, since I noticed Ubuntu has included the change in 6.2, while debian skipped 6.2 and 6.3, jumping straight to 6.4 and now to 6.5. If you think they should still be included, I can add them. Sorry about the hassle.
This was broken due to kernel changes backported from 6.5. I found the fix for this here, so credit goes to them: https://github.com/morrownr/8812au-20210629/commit/b5f4e6e894eca8fea38661e2fc22a2570e0274ad Breaking commit is: https://github.com/torvalds/linux/commit/e8c2af660ba0790afd14d5cbc2fd05c6dc85e207
I tested this change by building the module on Debian kernel 6.1.52-1 and 6.1.38-4.