Skip to content

Commit cda4025

Browse files
evelikov-workevelikov
authored andcommitted
dkms: use saner compression defaults
Currently our compression options vary from those used in upstream kernel. As of 6.8-rc2 upstream uses (ignoring force/rm input file etc): - gzip -9 - xz --check=crc32 --lzma2=dict=1MiB Note: some older kernels lacked crc32 and/or used 2MiB - zstd -T0 (aka -3) With the higher dictionary size (xz) and compression size (20+ zstd) more memory is required for decompression. Which may be a problem if the kernel itself is responsible for the decompression (instead of kmod), since it uses a smaller/limited ram amount. Reduce the numbers of be compatible with upstream. Closes: #386 Signed-off-by: Emil Velikov <[email protected]>
1 parent ef7dfa4 commit cda4025

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

dkms.in

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1154,9 +1154,9 @@ actual_build()
11541154
if [[ $module_compressed_suffix = .gz ]]; then
11551155
gzip -9f "$built_module" || compressed_module=""
11561156
elif [[ $module_compressed_suffix = .xz ]]; then
1157-
xz -f "$built_module" || compressed_module=""
1157+
xz --check=crc32 --lzma2=dict=1MiB -f "$built_module" || compressed_module=""
11581158
elif [[ $module_compressed_suffix = .zst ]]; then
1159-
zstd -q -f -T0 -20 --ultra "$built_module" || compressed_module=""
1159+
zstd -q -f -T0 -19 "$built_module" || compressed_module=""
11601160
fi
11611161
if [[ -n $compressed_module ]]; then
11621162
cp -f "$compressed_module" "$base_dir/module/${dest_module_name[$count]}$module_suffix" >/dev/null

0 commit comments

Comments
 (0)