So maybe I should add it as a separate version like the legacy versions below?

Yeah that sounds like a good idea

Never really did this in my PR since I assumed most people who would use it probably lie on amd64, though the assignment in videoDrivers should probably be checked (at least for nvidia) and defaulted to 3xx for 32bit.

https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/hardware/video/nvidia.nix#L14

I understand that v410 is now considered stable, so I would do 2 things:

• Choose a different version for the stable attribute depending on the platform (keep 390.87 for x86 users)
• Remove the beta version, since I understand there's no beta currently (be aware that there are references in the nvidia module and in all-packages)

In this way we keep a working default for both platforms

I have updated the commit:

Some packages reference the default nvidia_x11 package directly.
e.g. https://github.com/NixOS/nixpkgs/blob/master/pkgs/top-level/all-packages.nix#L921
and https://github.com/NixOS/nixpkgs/blob/master/pkgs/top-level/all-packages.nix#L11493

So nvidia_x11 should evaluate to the latest stable driver package for the platform I think.

@cpages I have set nvidia_x11_beta to be the same as nvidia_x11 and added a switch for nvidia_x11 depending on host architecture

The update works just fine with my 970.

This merge broke my system. I have a 64-bit system with an Nvidia GeForce GTX 560 Ti graphics card, which is only supported in the now legacy 390.x series. According to the changes, because I have a 64-bit system, the 410.x series would be used instead of the 390.x. However, there are no legacy_390 options to choose from, so I am stuck without a proper driver for the moment.

Please see #49679

Can we revert this until the 390.x series is packaged separately? Being stuck on an old revision of nixos-unstable is no fun. :(

@ben0x539 if I were you, I would test #51294 and try to merge this instead of reverting this one.

Hm, that PR hasn't even been reviewed yet, it seems like it getting merged and then making its way to nixos-unstable might be a while off. If I understand correctly, it also depends on #49703 to function as a workaround, which seems stuck. The prospect of all of that needing to land before I can update my system again is somewhat discouraging.

vvvvvvv ah cool thanks for clarifying

#51294 doesn't depend on #49703 though it should still work even if you merge both. Just adding nvidiaLegacy390 to videoDrivers should work just fine after merging #51294. Though I would prefer #49703 over mine.

@eadwu, can you explain why you prefer #49703? It seems that with #49703 you won't get automatic upgrades anymore?

With #51294, every time there needs to be a legacy package, the way most users use it would be through videoDrivers, so someone would have to add it there every single time. Though through #49703, you just need add nvidia to videoDrivers and then set hardware.nvidia.package to the desired package and it should give the same results. Also keeps the nvidia packages in its default.nix since the exposure is through config.boot.kernelPackages instead of searching and matching through videoDrivers.