Skip to content

compress: Replace kmap() with kmap_local() and friends #92

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

Closed
wants to merge 1 commit into from

Conversation

ethancedwards8
Copy link

kmap() is deprecated due to its slow speed and warned against by checkpatch.pl.

Compiled tested only.

kmap() is deprecated due to its slow speed and warned against by
checkpatch.pl.

Signed-off-by: Ethan Carter Edwards <[email protected]>
@eafer
Copy link
Member

eafer commented Mar 18, 2025

kmap_local was added in 5.11 apparently, so you'll need some version checks here.

@ethancedwards8
Copy link
Author

Ahhhh. Yes. Sorry. Still operating with mainline kernel in mind.

@eafer
Copy link
Member

eafer commented Apr 24, 2025

On second thought, it's probably not a good idea to add more version checks to handle checkpatch warnings. I guess I'll make this change in the future only if I drop support for kernels < 5.11. Thanks anyway.

@eafer eafer closed this Apr 24, 2025
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