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

[contrib][linux] Add zstd_common module #3292

Merged
merged 2 commits into from
Oct 22, 2022

Conversation

terrelln
Copy link
Contributor

The zstd_common module was added upstream in commit torvalds/linux@637a642.

But the kernel specific code was inlined into the library. This commit switches it to use the out of line method that we use for the other modules.

@terrelln terrelln force-pushed the 2022-10-17-kernel-fixes branch 2 times, most recently from d4f5eb4 to 2c6da54 Compare October 17, 2022 21:25
The zstd_common module was added upstream in commit
torvalds/linux@637a642.

But the kernel specific code was inlined into the library. This commit
switches it to use the out of line method that we use for the other
modules.
Disable ASM in the kernel for now. It requires a few changes & setup to
get working. Instead of doing it in a zstd version update, I'd prefer to
package that change as a single patch, and propose it separately from
the version update. This makes the version update easier, and reduces
some risk.
@terrelln terrelln merged commit 43de2aa into facebook:dev Oct 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants