Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixup for stale kickoff removal in wireless regulator API #186

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

prahal
Copy link

@prahal prahal commented Nov 15, 2023

Cope with the fix in stable 6.3.13 commit bf353116d1bf, 6.5-rc1 commte8c2af660ba0 and and 6.1.39 commit 132b7129c5fe LTS "wifi: cfg80211: fix regulatory disconnect with OCB/NAN". That is the removal of REGULATORY_IGNORE_STALE_KICKOFF from the wireless regulator internal API to fix any driver that allowed OCB/NAN.

I made this patch for the compilation error: wiphy->regulatory_flags |= REGULATORY_IGNORE_STALE_KICKOFF.
It was included and expanded in armbian https://github.com/armbian/build/blob/84e4a26bb09e9bc8401d3768c8158782ea601fc4/patch/misc/wireless-rtl88x2bu-wireless-ignore-stale-kickoff-removal.patch

The issue was this API change fixup was backported to stable/lts branches of the kernel.
This patch copes with this API change backports in al known branches it was included in, that
is in 6.1, 6.3, and 6.4 and above.

THis is for completeness as the current code which treat all kernel below 6.1.39 as including this API change will only break if one attempts to build an older tag of the 6.3 and 6.4 branches (or if one try to build the 6.2 branch). Most likely if one does regression testing. So the code already works in most cases.

Closes issue #165

@morrownr
Copy link
Owner

Hi @prahal

Thanks for this patch. It looks you did a good job. I'm sick right now. If I do not complete this within a week, can I get you post another message reminding me?

@morrownr

evgeny-boger added a commit to wirenboard/rtl8733bu that referenced this pull request Nov 19, 2023
@prahal
Copy link
Author

prahal commented Jan 2, 2024

Sorry I did not pinged you a week after your request to do so. Could this merge request be merged?

sikmir pushed a commit to wirenboard/rtl8733bu that referenced this pull request Apr 10, 2024
sikmir pushed a commit to wirenboard/rtl8733bu that referenced this pull request Apr 11, 2024
@prahal
Copy link
Author

prahal commented Sep 28, 2024

Hi @prahal

Thanks for this patch. It looks you did a good job. I'm sick right now. If I do not complete this within a week, can I get you post another message reminding me?

@morrownr

Pinging you for to merge this PR as requested.

@morrownr
Copy link
Owner

Hi @prahal

Sorry about the delay. I have had some real challenges over the last year. Let's get this taken care of.

Right now I see a conflict caused by a change that the RHEL maintainer made since you sent this patch:

#if (LINUX_VERSION_CODE >= KERNEL_VERSION(3, 19, 0)) && (LINUX_VERSION_CODE < KERNEL_VERSION(6, 1, 39)) && !defined(RHEL94)

Can I get you to look at this and propose a solution?

Cope with the fix in stable 6.3.13 commit bf353116d1bf,
6.5-rc1 commte8c2af660ba0 and and 6.1.39 commit 132b7129c5fe LTS
"wifi: cfg80211: fix regulatory disconnect with OCB/NAN".
That is the removal of REGULATORY_IGNORE_STALE_KICKOFF
from the wireless regulator internal API to fix any driver
that allowed OCB/NAN.
@prahal prahal force-pushed the complete-fixup-for-wifi-api-removal branch from aa3e383 to db69d76 Compare September 29, 2024 23:46
@prahal
Copy link
Author

prahal commented Sep 29, 2024

Done. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants