diff options
author | Michele Guerini Rocco <rnhmjoj@users.noreply.github.com> | 2021-03-09 08:14:24 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-03-09 08:14:24 +0100 |
commit | af6037b338b5aec2f10d3b8afbdfb3116077497d (patch) | |
tree | 450dced966a901936054d666b710dfdec0b8e2d3 | |
parent | 29b0d4d0b600f8f5dd0b86e3362a33d4181938f9 (diff) | |
parent | e3d705e60162def9493eaff758b753143e09cd57 (diff) | |
download | nixpkgs-af6037b338b5aec2f10d3b8afbdfb3116077497d.tar nixpkgs-af6037b338b5aec2f10d3b8afbdfb3116077497d.tar.gz nixpkgs-af6037b338b5aec2f10d3b8afbdfb3116077497d.tar.bz2 nixpkgs-af6037b338b5aec2f10d3b8afbdfb3116077497d.tar.lz nixpkgs-af6037b338b5aec2f10d3b8afbdfb3116077497d.tar.xz nixpkgs-af6037b338b5aec2f10d3b8afbdfb3116077497d.tar.zst nixpkgs-af6037b338b5aec2f10d3b8afbdfb3116077497d.zip |
Merge pull request #111551 from xaverdh/xserver-modern-drivers
nixos/xserver: use modern video drivers
-rw-r--r-- | nixos/doc/manual/configuration/x-windows.xml | 16 | ||||
-rw-r--r-- | nixos/doc/manual/release-notes/rl-2105.xml | 5 | ||||
-rw-r--r-- | nixos/modules/services/x11/xserver.nix | 7 |
3 files changed, 13 insertions, 15 deletions
diff --git a/nixos/doc/manual/configuration/x-windows.xml b/nixos/doc/manual/configuration/x-windows.xml index dd879702d7d..a499f0080d7 100644 --- a/nixos/doc/manual/configuration/x-windows.xml +++ b/nixos/doc/manual/configuration/x-windows.xml @@ -150,7 +150,6 @@ <xref linkend="opt-services.xserver.videoDrivers"/> = [ "nvidiaLegacy390" ]; <xref linkend="opt-services.xserver.videoDrivers"/> = [ "nvidiaLegacy340" ]; <xref linkend="opt-services.xserver.videoDrivers"/> = [ "nvidiaLegacy304" ]; -<xref linkend="opt-services.xserver.videoDrivers"/> = [ "nvidiaLegacy173" ]; </programlisting> You may need to reboot after enabling this driver to prevent a clash with other kernel modules. @@ -159,21 +158,16 @@ <simplesect xml:id="sec-x11--graphics-cards-amd"> <title>Proprietary AMD drivers</title> <para> - AMD provides a proprietary driver for its graphics cards that has better 3D - performance than the X.org drivers. It is not enabled by default because - it’s not free software. You can enable it as follows: + AMD provides a proprietary driver for its graphics cards that is not + enabled by default because it’s not Free Software, is often broken + in nixpkgs and as of this writing doesn't offer more features or + performance. If you still want to use it anyway, you need to explicitly set: <programlisting> -<xref linkend="opt-services.xserver.videoDrivers"/> = [ "ati_unfree" ]; +<xref linkend="opt-services.xserver.videoDrivers"/> = [ "amdgpu-pro" ]; </programlisting> You will need to reboot after enabling this driver to prevent a clash with other kernel modules. </para> - <note> - <para> - For recent AMD GPUs you most likely want to keep either the defaults - or <literal>"amdgpu"</literal> (both free). - </para> - </note> </simplesect> <simplesect xml:id="sec-x11-touchpads"> <title>Touchpads</title> diff --git a/nixos/doc/manual/release-notes/rl-2105.xml b/nixos/doc/manual/release-notes/rl-2105.xml index 2f87869fbe3..2de32cd09ff 100644 --- a/nixos/doc/manual/release-notes/rl-2105.xml +++ b/nixos/doc/manual/release-notes/rl-2105.xml @@ -539,6 +539,11 @@ self: super: <package>imagemagick6Big</package> if you need the older version. </para> </listitem> + <listitem> + <para> + <xref linkend="opt-services.xserver.videoDrivers" /> no longer uses the deprecated <literal>cirrus</literal> and <literal>vesa</literal> device dependent X drivers by default. It also enables both <literal>amdgpu</literal> and <literal>nouveau</literal> drivers by default now. + </para> + </listitem> </itemizedlist> </section> diff --git a/nixos/modules/services/x11/xserver.nix b/nixos/modules/services/x11/xserver.nix index 8858559d8f2..35bd4dabb67 100644 --- a/nixos/modules/services/x11/xserver.nix +++ b/nixos/modules/services/x11/xserver.nix @@ -251,11 +251,10 @@ in videoDrivers = mkOption { type = types.listOf types.str; - # !!! We'd like "nv" here, but it segfaults the X server. - default = [ "radeon" "cirrus" "vesa" "modesetting" ]; + default = [ "amdgpu" "radeon" "nouveau" "modesetting" "fbdev" ]; example = [ - "ati_unfree" "amdgpu" "amdgpu-pro" - "nv" "nvidia" "nvidiaLegacy390" "nvidiaLegacy340" "nvidiaLegacy304" + "nvidia" "nvidiaLegacy390" "nvidiaLegacy340" "nvidiaLegacy304" + "amdgpu-pro" ]; # TODO(@oxij): think how to easily add the rest, like those nvidia things relatedPackages = concatLists |