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

mt76: fixes for memory leak and kernels 6.7 and above #17662

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

namiltd
Copy link
Contributor

@namiltd namiltd commented Jan 19, 2025

  • Fix kernel 6.7 compatibility:
    The PP_FLAG_PAGE_FRAG constant was removed in kernel 6.7, so we are removing it from code for kernels 6.7 and above.
    This is needed for building the openwrt with the next LTS kernel 6.12.

  • Fix allocation check and missing memory freeing:
    Fixed allocation checking and freeing buffer memory inside mt76_eeprom_changes() function.

@namiltd
Copy link
Contributor Author

namiltd commented Jan 19, 2025

I tried to make these patches on the page https://github.com/openwrt/mt76 but the "Pull requests" tab on this page is dead, not verified by anyone.

@github-actions github-actions bot added the core packages pull request/issue for core (in-tree) packages label Jan 19, 2025
@rmandrad
Copy link

i believe you need to use upstream linux-wireless

@nbd168 ?

@neheb
Copy link
Contributor

neheb commented Jan 19, 2025

mt76 is provided by mac80211, not the kernel.

@Noltari
Copy link
Member

Noltari commented Jan 20, 2025

@namiltd I think that we should wait until openwrt/mt76#947 and openwrt/mt76#948 are merged in the mt76 repo and then bump the mt76 revision.

@namiltd
Copy link
Contributor Author

namiltd commented Jan 20, 2025

@Noltari One of those PRs was there for a month (as openwrt/mt76#938) and no one looked at it. I hope it will be faster here.

The PP_FLAG_PAGE_FRAG constant was removed in kernel 6.7,
so we are removing it from code for kernels 6.7 and above.

Signed-off-by: Mieczyslaw Nalewaj <namiltd@yahoo.com>
Fixed allocation checking and freeing buffer memory
inside mt76_eeprom_changes() function.

Signed-off-by: Mieczyslaw Nalewaj <namiltd@yahoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core packages pull request/issue for core (in-tree) packages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants