Skip to content

kconfigcheck: Fix ZRAM checks

Minecrell requested to merge Minecrell/pmaports:fix-zram into master

main/postmarketos-base/rootfs-etc-conf.d-zram-init configures zstd as our default ZRAM configuration, but the kconfigcheck only checks for LZ4 at the moment. Because of this some of our kernels don't actually have ZSTD support enabled and end up using the default compression algorithm.

  • For Linux >=4.18 we should enable CONFIG_CRYPTO_ZSTD to enable ZSTD support for ZRAM.
  • In Linux >=6.12 the configuration was changed to CONFIG_ZRAM_BACKEND_ZSTD. This requires manual edits while upgrading, otherwise only LZO support will remain enabled.
  • Checking for >=3.14.0 additionally does not make any sense, because the kconfig check should not be enabled for kernels that cannot satisfy our minimum requirements.

Describe this properly in kconfigcheck.toml to ensure kernels are configured correctly.

The following kernels are currently missing the correct options:

config-htc-flounder.aarch64: CONFIG_CRYPTO_ZSTD should be set (category:zram)
config-postmarketos-amlogic.aarch64: CONFIG_CRYPTO_ZSTD should be set (category:zram)
config-postmarketos-grate.armv7: CONFIG_CRYPTO_ZSTD should be set (category:zram)
config-postmarketos-qcom-msm8974.armv7: CONFIG_CRYPTO_ZSTD should be set (category:zram)
config-postmarketos-qcom-sc7280.aarch64: CONFIG_CRYPTO_ZSTD should be set (category:zram)
config-postmarketos-qcom-sm6115.aarch64: CONFIG_CRYPTO_ZSTD should be set (category:zram)
config-postmarketos-qcom-sm6350.aarch64: CONFIG_CRYPTO_ZSTD should be set (category:zram)
config-purism-librem5.aarch64: CONFIG_CRYPTO_ZSTD should be set (category:zram)

The following kernels are too old to support CONFIG_CRYPTO_ZSTD:

config-samsung-jxelte.aarch64: CONFIG_CRYPTO_ZSTD should be set (category:zram)
config-volla-mimameid.aarch64: CONFIG_CRYPTO_ZSTD should be set (category:zram)

IMO we should drop pmb:kconfigcheck-zram for these since this is a configuration that we can't actively test or support.

Merge request reports