Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 445c8a4

Browse files
evelikov-workevelikov
authored andcommittedJan 29, 2024
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: dell#386 Signed-off-by: Emil Velikov <[email protected]>
1 parent 2fbc986 commit 445c8a4

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed
 

‎dkms.in

+2-2
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)
Please sign in to comment.