chore(aquamarine): drop pcie_aspm=off

It was a precaution for the R9700 links, never an observed problem, and
on BIOS 1013 the FADT already disables ASPM ("ACPI FADT declares the
system doesn't support PCIe ASPM"), so the parameter did nothing.

Assisted-by: Claude Code (claude-opus-5-5)
This commit is contained in:
Ryan Yin authored and ryan4yin | 二花 committed 2026-11-01 10:51:51 -06:00
1 parent dd2bd44bc2
commit 32620ed9db
5 files changed
+4 -14

No files matched your search

+1 -2
View File
@@ -63,8 +63,7 @@ run on `youko`, not on this host, so opening a dashboard does not wake `aquamari
still need to be enabled in BIOS before this knob can save more power.
- Safe BIOS-side changes: enable C1E, Intel Speed Shift and CPU/package C-states (limit `Auto` or
`C10`), and enable iGPU Render Standby/RC6. Disable unused Wi-Fi/Bluetooth, onboard audio, SATA,
serial ports and RGB lighting. Do not change the storage mode, and do not enable PCIe ASPM while
the NVMe/GPU stability workarounds are in place.
serial ports and RGB lighting. Do not change the storage mode.
The iGPU carries display and video work: the kernel binds `i915`, and
[hardware-intel-igpu.nix](./hardware-intel-igpu.nix) installs `intel-media-driver` (VA-API),
+3 -5
View File
@@ -122,11 +122,9 @@ default `0xfff7bfff`; `0xfff7ffff` works). **verified**
`amdgpu.ppfeaturemask=0xffffffff` does not address the ATPX/SMU path; it comes from a community
tuning repository and no primary source ties it to it. **community-reported**
([r9700-setup](https://github.com/stew675/r9700-setup/blob/HEAD/README.md)). The host's own
`pcie_aspm=off` is unrelated to this path too — it is set for PCIe link retraining, with its reason
in [hardware-configuration.nix](./hardware-configuration.nix). CPU C-state depth is no longer
capped; the old `processor.max_cstate=2` trade-off was removed for idle power, so re-check inference
tail latency after that change.
([r9700-setup](https://github.com/stew675/r9700-setup/blob/HEAD/README.md)). CPU C-state depth is no
longer capped; the old `processor.max_cstate=2` trade-off was removed for idle power, so re-check
inference tail latency after that change.
## Kernel parameters and module options
@@ -16,11 +16,6 @@
(modulesPath + "/installer/scan/not-detected.nix")
];
boot.kernelParams = [
# Keep PCIe ASPM disabled. The R9700 links have retraining/timeout issues.
"pcie_aspm=off"
];
boot.loader.efi.canTouchEfiVariables = true;
boot.loader.efi.efiSysMountPoint = "/boot";
@@ -4,7 +4,6 @@
iommuPassthrough = true;
runtimePmDisabled = true;
gpuRecoveryEnabled = true;
aspmDisabled = true;
deepCStatesAllowed = true;
cpuEppConfigured = true;
nativeIpv6Preference = true;
@@ -22,7 +22,6 @@ in
# An unattended host recovers a hung card itself instead of waiting for a reboot.
gpuRecoveryEnabled = builtins.elem "amdgpu.gpu_recovery=1" cfg.boot.kernelParams;
aspmDisabled = builtins.elem "pcie_aspm=off" cfg.boot.kernelParams;
deepCStatesAllowed = !(builtins.elem "processor.max_cstate=2" cfg.boot.kernelParams);
cpuEppConfigured =
lib.attrByPath [