From 741761ad92d9b65246666bd338030622c1d1117e Mon Sep 17 00:00:00 2001 From: Kenichi Kamiya Date: Sun, 1 Dec 2024 23:10:48 +0900 Subject: [PATCH] Introduce memreserver to fix unrecoverable suspend on NixOS 24.11 and AMD APU --- flake.nix | 1 + nixos/hosts/algae/default.nix | 3 ++ nixos/hosts/moss/default.nix | 3 ++ nixos/modules/hardware/memreserver.nix | 36 ++++++++++++++++++++++++ pkgs/default.nix | 2 ++ pkgs/memreserver/default.nix | 38 ++++++++++++++++++++++++++ 6 files changed, 83 insertions(+) create mode 100644 nixos/modules/hardware/memreserver.nix create mode 100644 pkgs/memreserver/default.nix diff --git a/flake.nix b/flake.nix index db545bbf..69039320 100644 --- a/flake.nix +++ b/flake.nix @@ -159,6 +159,7 @@ pkgs = mkPkgs system; in { + memreserver = pkgs.my.memreserver; cozette = pkgs.my.cozette; micro-kdl = pkgs.my.micro-kdl; micro-nordcolors = pkgs.my.micro-nordcolors; diff --git a/nixos/hosts/algae/default.nix b/nixos/hosts/algae/default.nix index 54f5c417..dfbb8976 100644 --- a/nixos/hosts/algae/default.nix +++ b/nixos/hosts/algae/default.nix @@ -37,4 +37,7 @@ # Enable keyboard - "ThinkPad Compact USB Keyboard with TrackPoint" wakeup ACTION=="add", SUBSYSTEM=="usb", DRIVERS=="usb", ATTRS{idVendor}=="17ef", ATTRS{idProduct}=="6047", ATTR{power/wakeup}="enabled" ''; + + # GH-959 + hardware.memreserver.enable = true; } diff --git a/nixos/hosts/moss/default.nix b/nixos/hosts/moss/default.nix index 89fcd544..f37f0f11 100644 --- a/nixos/hosts/moss/default.nix +++ b/nixos/hosts/moss/default.nix @@ -27,4 +27,7 @@ evdev:name:AT Translated Set 2 keyboard:* KEYBOARD_KEY_3a=leftctrl # original: capslock ''; + + # GH-959 + hardware.memreserver.enable = true; } diff --git a/nixos/modules/hardware/memreserver.nix b/nixos/modules/hardware/memreserver.nix new file mode 100644 index 00000000..decb8cf4 --- /dev/null +++ b/nixos/modules/hardware/memreserver.nix @@ -0,0 +1,36 @@ +# Imported from https://github.com/NixOS/nixpkgs/pull/225819 to fix GH-959 + +{ + config, + lib, + pkgs, + ... +}: + +with lib; +{ + options = { + hardware.memreserver = { + enable = mkOption { + default = false; + type = types.bool; + description = lib.mdDoc '' + Enable memreserver sleep hook. + Makes sure that GPUs with dedicated VRAM can suspend correctly. + ''; + }; + }; + }; + + config = mkIf config.hardware.memreserver.enable { + systemd.services.memreserver = { + description = "Sleep hook which frees up RAM needed to evacuate GPU VRAM into"; + before = [ "sleep.target" ]; + wantedBy = [ "sleep.target" ]; + serviceConfig = { + Type = "oneshot"; + ExecStart = "${pkgs.my.memreserver}/bin/memreserver"; + }; + }; + }; +} diff --git a/pkgs/default.nix b/pkgs/default.nix index 847d7dc3..ca36988b 100644 --- a/pkgs/default.nix +++ b/pkgs/default.nix @@ -64,4 +64,6 @@ git-resolve-conflict = pkgs.callPackage ./git-resolve-conflict { }; renmark = pkgs.callPackage ./renmark { }; preview = pkgs.callPackage ./preview { }; + + memreserver = pkgs.callPackage ./memreserver { }; } diff --git a/pkgs/memreserver/default.nix b/pkgs/memreserver/default.nix new file mode 100644 index 00000000..24a2f628 --- /dev/null +++ b/pkgs/memreserver/default.nix @@ -0,0 +1,38 @@ +# Imported from https://github.com/NixOS/nixpkgs/pull/225819 to fix GH-959 + +{ + lib, + stdenv, + fetchFromGitLab, + meson, + ninja, + pkg-config, + libdrm, +}: + +stdenv.mkDerivation rec { + pname = "memreserver"; + version = "unstable-2023-04-01"; + + src = fetchFromGitLab { + domain = "git.dolansoft.org"; + owner = "lorenz"; + repo = pname; + rev = "480253e565dab935df1d1c4e615ebc8a8dc81ba4"; + hash = "sha256-HjcrH98hH2zKdsHolYCFugL39sT1VjroVhRf8a8dpIA="; + }; + + nativeBuildInputs = [ + meson + ninja + pkg-config + ]; + buildInputs = [ libdrm ]; + + meta = with lib; { + homepage = "https://git.dolansoft.org/lorenz/memreserver"; + description = "Sleep hook which frees up RAM needed to evacuate GPU VRAM into"; + license = licenses.mit; + maintainers = with maintainers; [ lorenz ]; + }; +}