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

linux kernel: prefer zstd where possible #302300

Merged
merged 2 commits into from
May 8, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions nixos/doc/manual/release-notes/rl-2405.section.md
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,10 @@ Use `services.pipewire.extraConfig` or `services.pipewire.configPackages` for Pi
- A new option `system.etc.overlay.enable` was added. If enabled, `/etc` is
mounted via an overlayfs instead of being created by a custom perl script.

- For each supporting version of the Linux kernel firmware blobs and kernel modules
are compressed with zstd. For firmware blobs this means an increase of 4.4% in size, however
a significantly higher decompression speed.

- NixOS AMIs are now uploaded regularly to a new AWS Account.
Instructions on how to use them can be found on <https://nixos.github.io/amis>.
We are working on integration the data into the NixOS homepage.
Expand Down
14 changes: 10 additions & 4 deletions nixos/modules/services/hardware/udev.nix
Original file line number Diff line number Diff line change
Expand Up @@ -167,10 +167,16 @@ let
mv etc/udev/hwdb.bin $out
'';

compressFirmware = firmware: if (config.boot.kernelPackages.kernelAtLeast "5.3" && (firmware.compressFirmware or true)) then
pkgs.compressFirmwareXz firmware
else
id firmware;
compressFirmware = firmware:
let
inherit (config.boot.kernelPackages) kernelAtLeast;
in
if ! (firmware.compressFirmware or true) then
firmware
else
if kernelAtLeast "5.19" then pkgs.compressFirmwareZstd firmware
else if kernelAtLeast "5.3" then pkgs.compressFirmwareXz firmware
else firmware;

# Udev has a 512-character limit for ENV{PATH}, so create a symlink
# tree to work around this.
Expand Down
29 changes: 0 additions & 29 deletions pkgs/build-support/kernel/compress-firmware-xz.nix

This file was deleted.

43 changes: 43 additions & 0 deletions pkgs/build-support/kernel/compress-firmware.nix
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
{ runCommand, lib, type ? "zstd", zstd }:

firmware:

let
compressor = {
xz = {
ext = "xz";
nativeBuildInputs = [ ];
cmd = file: target: ''xz -9c -T1 -C crc32 --lzma2=dict=2MiB "${file}" > "${target}"'';
};
zstd = {
ext = "zst";
nativeBuildInputs = [ zstd ];
cmd = file: target: ''zstd -T1 -19 --long --check -f "${file}" -o "${target}"'';
};
}.${type} or (throw "Unsupported compressor type for firmware.");

args = {
allowedRequisites = [];
inherit (compressor) nativeBuildInputs;
} // lib.optionalAttrs (firmware ? meta) { inherit (firmware) meta; };
in

runCommand "${firmware.name}-${type}" args ''
mkdir -p $out/lib
(cd ${firmware} && find lib/firmware -type d -print0) |
(cd $out && xargs -0 mkdir -v --)
(cd ${firmware} && find lib/firmware -type f -print0) |
(cd $out && xargs -0rtP "$NIX_BUILD_CORES" -n1 \
sh -c '${compressor.cmd "${firmware}/$1" "$1.${compressor.ext}"}' --)
(cd ${firmware} && find lib/firmware -type l) | while read link; do
target="$(readlink "${firmware}/$link")"
if [ -f "${firmware}/$link" ]; then
ln -vs -- "''${target/^${firmware}/$out}.${compressor.ext}" "$out/$link.${compressor.ext}"
else
ln -vs -- "''${target/^${firmware}/$out}" "$out/$link"
fi
done

echo "Checking for broken symlinks:"
find -L $out -type l -print -execdir false -- '{}' '+'
''
2 changes: 1 addition & 1 deletion pkgs/build-support/kernel/modules-closure.sh
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ for module in $(< ~-/closure); do
# of its output.
modinfo -b $kernel --set-version "$version" -F firmware $module | grep -v '^name:' | while read -r i; do
echo "firmware for $module: $i"
for name in "$i" "$i.xz" ""; do
for name in "$i" "$i.xz" "$i.zst" ""; do
[ -z "$name" ] && echo "WARNING: missing firmware $i for module $module"
if cp -v --parents --no-preserve=mode lib/firmware/$name "$out" 2>/dev/null; then
Ma27 marked this conversation as resolved.
Show resolved Hide resolved
break
Expand Down
9 changes: 9 additions & 0 deletions pkgs/build-support/vm/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,14 @@ rec {
${pkgs.stdenv.cc.libc}/lib/libc.so.* \
${pkgs.stdenv.cc.libc}/lib/libm.so.* \
${pkgs.stdenv.cc.libc}/lib/libresolv.so.* \
${pkgs.stdenv.cc.libc}/lib/libpthread.so.* \
${pkgs.zstd.out}/lib/libzstd.so.* \
${pkgs.xz.out}/lib/liblzma.so.* \
$out/lib

# Copy BusyBox.
cp -pd ${pkgs.busybox}/bin/* $out/bin
cp -pd ${pkgs.kmod}/bin/* $out/bin
Ma27 marked this conversation as resolved.
Show resolved Hide resolved

# Run patchelf to make the programs refer to the copied libraries.
for i in $out/bin/* $out/lib/*; do if ! test -L $i; then nuke-refs $i; fi; done
Expand All @@ -54,6 +58,11 @@ rec {
patchelf --set-interpreter $out/lib/ld-*.so.? --set-rpath $out/lib $i || true
fi
done

find $out/lib -type f \! -name 'ld*.so.?' | while read i; do
echo "patching $i..."
patchelf --set-rpath $out/lib $i
done
''; # */


Expand Down
20 changes: 12 additions & 8 deletions pkgs/os-specific/linux/kernel/common-config.nix
Original file line number Diff line number Diff line change
Expand Up @@ -871,12 +871,14 @@ let
};

zram = {
ZRAM = module;
ZRAM_WRITEBACK = option yes;
ZRAM_MULTI_COMP = whenAtLeast "6.2" yes;
ZSWAP = option yes;
ZPOOL = yes;
ZBUD = option yes;
ZRAM = module;
ZRAM_WRITEBACK = option yes;
ZRAM_MULTI_COMP = whenAtLeast "6.2" yes;
ZRAM_DEF_COMP_ZSTD = whenAtLeast "5.11" yes;
ZSWAP = option yes;
ZSWAP_COMPRESSOR_DEFAULT_ZSTD = whenAtLeast "5.7" yes;
ZPOOL = yes;
ZSMALLOC = option yes;
};

brcmfmac = {
Expand Down Expand Up @@ -948,8 +950,9 @@ let
THRUSTMASTER_FF = yes;
ZEROPLUS_FF = yes;

MODULE_COMPRESS = whenOlder "5.13" yes;
MODULE_COMPRESS_XZ = yes;
MODULE_COMPRESS = whenOlder "5.13" yes;
MODULE_COMPRESS_XZ = whenOlder "5.13" yes;
MODULE_COMPRESS_ZSTD = whenAtLeast "5.13" yes;

SYSVIPC = yes; # System-V IPC

Expand Down Expand Up @@ -1124,6 +1127,7 @@ let
FW_LOADER_USER_HELPER_FALLBACK = option no;

FW_LOADER_COMPRESS = whenAtLeast "5.3" yes;
FW_LOADER_COMPRESS_ZSTD = whenAtLeast "5.19" yes;

HOTPLUG_PCI_ACPI = yes; # PCI hotplug using ACPI
HOTPLUG_PCI_PCIE = yes; # PCI-Expresscard hotplug support
Expand Down
4 changes: 3 additions & 1 deletion pkgs/top-level/all-packages.nix
Original file line number Diff line number Diff line change
Expand Up @@ -1314,7 +1314,9 @@ with pkgs;

makeBinaryWrapper = callPackage ../build-support/setup-hooks/make-binary-wrapper { };

compressFirmwareXz = callPackage ../build-support/kernel/compress-firmware-xz.nix { };
compressFirmwareXz = callPackage ../build-support/kernel/compress-firmware.nix { type = "xz"; };

compressFirmwareZstd = callPackage ../build-support/kernel/compress-firmware.nix { type = "zstd"; };

makeModulesClosure = { kernel, firmware, rootModules, allowMissing ? false }:
callPackage ../build-support/kernel/modules-closure.nix {
Expand Down