I believe this should be optional with default being turned off, reason being its when you have NVidia GPU's that are used for machine learning so you want the Nvidia GPU's to always be online even though your display is being run through a completely different GPU (i.e. Intel integrated GPU) or not at all (server with a terminal interface over network).

@CMCDragonkai Can you confirm this?

Yes by default it should be off.

OK.

This requires a module option and a mkIf that checks the enable option before writing the config.

This requires a module option and a mkIf that checks the enable option before writing the config.

Done changes to make this optional and default to false.
@CMCDragonkai @mdedetrich

Looks fine to me, @CMCDragonkai ?

Works for me: nvidia-smi consistently reports both GPUs available

Tue Jan 21 17:46:14 2020       
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 440.44       Driver Version: 440.44       CUDA Version: 10.2     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  Tesla K10.G1.8GB    On   | 00000000:05:00.0 Off |                  Off |
| N/A   29C    P8    17W / 117W |      0MiB /  4037MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
|   1  Tesla K10.G1.8GB    On   | 00000000:06:00.0 Off |                  Off |
| N/A   37C    P8    17W / 117W |      0MiB /  4037MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
                                                                               
+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
|  No running processes found                                                 |
+-----------------------------------------------------------------------------+

Your machine is different. This is needed headless servers with embedded intel gpu and 4 or more gpus.
…
On 22 January 2020 12:47:07 GMT+11:00, coreyoconnor @.***> wrote: Works for me: nvidia-smi consistently reports both GPUs available ~~~ Tue Jan 21 17:46:14 2020 +-----------------------------------------------------------------------------+ | NVIDIA-SMI 440.44 Driver Version: 440.44 CUDA Version: 10.2 | |-------------------------------+----------------------+----------------------+ | GPU Name Persistence-M| Bus-Id Disp.A | Volatile Uncorr. ECC | | Fan Temp Perf Pwr:Usage/Cap| Memory-Usage | GPU-Util Compute M. | |===============================+======================+======================| | 0 Tesla K10.G1.8GB On | 00000000:05:00.0 Off | Off | | N/A 29C P8 17W / 117W | 0MiB / 4037MiB | 0% Default | +-------------------------------+----------------------+----------------------+ | 1 Tesla K10.G1.8GB On | 00000000:06:00.0 Off | Off | | N/A 37C P8 17W / 117W | 0MiB / 4037MiB | 0% Default | +-------------------------------+----------------------+----------------------+ +-----------------------------------------------------------------------------+ | Processes: GPU Memory | | GPU PID Type Process name Usage | |=============================================================================| | No running processes found | +-----------------------------------------------------------------------------+ ~~~ -- You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: #77054 (comment)
-- Sent from my Android device with K-9 Mail. Please excuse my brevity.

I think you are confused. This is useful on any system using a nvidia GPU in a headless mode. As that is my system and this definitely resolves issues the requirements of "embedded intel gpu and 4 or more gpus" is incorrect. That would also be inconsistent with the documentation of nvidia-persistenced.

Great, then this works for any headless system then.

This is ready to be merged. Anybody?

@Rakesh4G Can you clean up the commits, and squash them into 1, and abide by the commit standard.

Hi @CMCDragonkai , I have cleaned up the commits, and squash them into 1. Thanks.

You have to resolve the conflicts in the MR right now.

But it's unfortunate that this MR has been left without any one attempting to push it in.

This is ready to be merged.

Anyone?

Someone please merge this before this falls out of scope for maintenance!

I have no permission to merge, I think you will have to ping a maintainer.

Who's the maintainer for this module?

Judging by #66601 @romildo seems to merge NVidia related PR's