Skip to content

Conversation

@fwilhe
Copy link
Member

@fwilhe fwilhe commented Jan 20, 2025

@fwilhe fwilhe added this to the 2025-01 milestone Jan 20, 2025
@fwilhe fwilhe marked this pull request as ready for review January 20, 2025 16:32
@fwilhe fwilhe requested a review from a team January 20, 2025 16:32
Copy link
Contributor

@moritzkirchnersap moritzkirchnersap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Config options seems to be the same as in debian, looks good!

Apart from

CONFIG_ZRAM_DEF_COMP_ZSTD=y

Which according to

https://www.kernelconfig.io/CONFIG_ZRAM_DEF_COMP_ZSTD?q=CONFIG_ZRAM_DEF_COMP_ZSTD&kernelversion=6.12.10&arch=x86

sets the default zram compressor - is this needed here? What would it be in case this config option wouldn't be set? And why not set it to
CONFIG_ZRAM_DEF_COMP_LZ4
for example?

@moritzkirchnersap
Copy link
Contributor

@fwilhe fwilhe marked this pull request as draft January 21, 2025 11:25
@fwilhe fwilhe marked this pull request as ready for review January 21, 2025 11:27
@fwilhe fwilhe requested a review from gehoern January 21, 2025 11:30
@fwilhe
Copy link
Member Author

fwilhe commented Jan 21, 2025

@gehoern please review

@gehoern gehoern merged commit 86f97d1 into main Jan 21, 2025
4 checks passed
@gehoern gehoern deleted the zram-zstd branch January 21, 2025 12:59
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.

Enable zstd backend for zram

3 participants