[nixos-configs-is4] Extend nvidia role to include driver configuration #9

Merged
johno merged 1 commits from bead/nixos-configs-is4 into main 2026-01-10 14:39:46 -08:00
Owner

Summary

  • Extended roles/nvidia/default.nix with comprehensive driver configuration options
  • Updated machines/zix790prors/configuration.nix to use the new role instead of duplicated config
  • Updated machines/wixos/configuration.nix to use the new role with WSL-specific extraPackages

New Options

  • open - Use open source kernel driver (default: true)
  • modesetting - Enable kernel modesetting (default: true)
  • nvidiaSettings - Enable nvidia-settings GUI (default: true)
  • package - Driver package selection (stable/latest/beta/vulkan_beta/production)
  • powerManagement.enable and powerManagement.finegrained
  • graphics.enable, graphics.enable32Bit, graphics.extraPackages

Test plan

  • Verify zix790prors builds: nixos-rebuild build --flake .#zix790prors
  • Verify wixos builds: nixos-rebuild build --flake .#wixos

Bead ID: nixos-configs-is4

## Summary - Extended `roles/nvidia/default.nix` with comprehensive driver configuration options - Updated `machines/zix790prors/configuration.nix` to use the new role instead of duplicated config - Updated `machines/wixos/configuration.nix` to use the new role with WSL-specific extraPackages ## New Options - `open` - Use open source kernel driver (default: true) - `modesetting` - Enable kernel modesetting (default: true) - `nvidiaSettings` - Enable nvidia-settings GUI (default: true) - `package` - Driver package selection (stable/latest/beta/vulkan_beta/production) - `powerManagement.enable` and `powerManagement.finegrained` - `graphics.enable`, `graphics.enable32Bit`, `graphics.extraPackages` ## Test plan - [ ] Verify zix790prors builds: `nixos-rebuild build --flake .#zix790prors` - [ ] Verify wixos builds: `nixos-rebuild build --flake .#wixos` Bead ID: nixos-configs-is4
johno added 4 commits 2026-01-10 11:11:45 -08:00
The nvidia role now handles full driver configuration instead of just
packages. Added options for open driver, modesetting, power management,
graphics settings, and driver package selection.

Updated zix790prors and wixos machine configs to use the new role
options, removing duplicated hardware.nvidia configuration blocks.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Author
Owner

Does this still make sense to do now that we're removing wixos?

Does this still make sense to do now that we're removing wixos?
Author
Owner

Yes, the nvidia role extension still makes sense. The core value is the improved roles/nvidia/default.nix which benefits zix790prors.

If PR #20 (remove wixos) merges first, this PR will need a rebase to drop the wixos changes. Alternatively, merge this first and #20 will cleanly remove wixos afterward.

The wixos-specific changes here are incidental - the role itself is the main deliverable.

Yes, the nvidia role extension still makes sense. The core value is the improved `roles/nvidia/default.nix` which benefits zix790prors. If PR #20 (remove wixos) merges first, this PR will need a rebase to drop the wixos changes. Alternatively, merge this first and #20 will cleanly remove wixos afterward. The wixos-specific changes here are incidental - the role itself is the main deliverable.
johno merged commit 9973273b5e into main 2026-01-10 14:39:46 -08:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: johno/nixos-configs#9