Commit graph

18 commits

Author SHA1 Message Date
Martino Fontana 1156f4d66e AMD: disable amdvlk install by default
Nobody recommends using amdvlk over RADV.
Also don't set environment variable that sets the use of RADV when installing amdvlk
2023-02-17 15:29:55 +01:00
bors[bot] 44ae00e02e
Merge #550
550: Add amdvlk and opencl options to amdgpu & Fix proton crash when Dual-Direct GFX enabled for lenovo legion 16ach6h r=Mic92 a=LostAttractor



Co-authored-by: ChaosAttractor <46527539+LostAttractor@users.noreply.github.com>
Co-authored-by: Jörg Thalheim <Mic92@users.noreply.github.com>
2023-02-09 09:06:46 +00:00
ChaosAttractor b45bd0353a Change lib.mkDefault to lib.mkOverride 990 for hardware.nvidia.prime.offload.enable
Fix conflict to 2a7130d13a
2023-02-09 12:41:39 +08:00
ChaosAttractor 5af4dc5d52 Add the option of whether to install the opencl environment and whether to use amdvlk for amdgpu 2023-01-30 15:23:11 +08:00
Jörg Thalheim 03c6d15152 treewide: apply deadnix and statix 2022-12-31 11:57:44 +01:00
ChaosAttractor a1a8723bf9 Add a option to add(or disadd) amdgpu to kernelModule
Add a option to add(or disadd) amdgpu to boot.initrd.kernelModule
2022-12-17 00:04:46 +08:00
Tristan Daniël Maat 7e60458d86
common/gpu/nvidia*: Migrate to common/gpu/nvidia/* and add non-prime
Fixes #338
2022-11-13 01:41:03 +00:00
SuperSamus 0efe78c51a
AMD: Add lib.mkDefault to AMD_VULKAN_ICD 2022-07-02 23:39:55 +02:00
Thiago Kenji Okada 9194b8e949 nvidia: remove -a flag from exec
The `-a` from exec sets the `$0` from the process. So `exec -a "$0"`
would set the name of the new process to `nvidia-offload` (the name of
the script), however this is causing issues with a few programs that try
to interpret `$0` in a special way. For example, see `wine`:

```
$ nvidia-offload /nix/store/zhv91s26gsrl1w8yn9800xd03a31r3wj-wine-osu-7.0/bin/wine .osu/drive_c/osu/osu\!.exe
/nix/store/zhv91s26gsrl1w8yn9800xd03a31r3wj-wine-osu-7.0/bin/nvidia-offload: could not open
```

What I think `wine` is doing here is trying to re-exec `wine` again, but
to do so it tries to figure out the original call of wine by readind `$0`,
and will fail in this case because the `$0` was changed because of the
`nvidia-offload` script using `-a` flag, as explained above.

Instead, let's simplify this. There is no good reason to rename the `$0`
from the script anyway (it just sets a few environment variables), so let's
just remove it. We may lose the ability to know if the command is being
offloaded, but I think having more commands to work is a good trade-off.
2022-06-24 23:45:31 +01:00
Bruno Bigras 53b470587f split CPU and GPU from common/cpu/intel
fix #388
2022-04-12 11:08:50 -04:00
Martino Fontana abd470874b AMD: Prefer RADV 2022-03-16 17:09:08 +01:00
Nicholas Sielicki 478cc4aa6e make comment more clear that it applies to all igpus, not just intel 2022-03-06 19:25:42 +00:00
Johannes Arnold a57fc74bc3 AMD: include 32-bit driver for Vulkan 2021-09-28 20:14:53 +02:00
Thiago Kenji Okada b0f31689ac Remove intel drivers from nvidia-disable
Two reasons for this change:
- `intel` drivers are not updated from quite a long time (since ~2019),
  and `modesetting` is the preferred one for Intel iGPUs.
- Technically you may want to disable NVIDIA GPUs on laptops with AMD
  processors too.

We don't want to pin `modesetting` here because it is already the default
since PR: https://github.com/NixOS/nixpkgs/pull/111551.
2021-06-11 17:04:10 -03:00
Johannes Arnold 40ddf363c2 Document driver choice 2021-04-14 16:18:05 +02:00
Johannes Arnold f2b2ae018d AMD: add configurations for graphics cards 2021-03-24 02:18:03 +01:00
Thiago Kenji Okada 36094450fd Add nvidia-disable profile
This is already used in XPS 15-9560, but instead of defining it in XPS
15-9560 let's create a separate profile and expose it for those who want
to disable their NVIDIA GPU.
2021-02-28 13:51:46 -03:00
bahaynes b7a5f1188f
Moved NVIDIA config to common, and updated README. 2020-07-25 23:53:47 -04:00