[pve-devel] [PATCH pve-nvidia-vgpu-helper v2 4/4] debian: add and install nvidia-vgpu systemd template unit file

[Thomas Lamprecht]

Am 21.01.25 um 11:36 schrieb Dominik Csapak:
> On 1/21/25 11:04, Hannes Duerr wrote:
>> diff --git a/debian/nvidia-vgpud at .service b/debian/nvidia-vgpud at .service
>> new file mode 100644
>> index 0000000..b3c1220
>> --- /dev/null
>> +++ b/debian/nvidia-vgpud at .service
>
> mhmm not sure if it's so good to reuse the exiting name of the nvidia service
> for this. also not sure how systemd likes it if there is a service 'foo.service'
> and a template 'foo at .service' (though my guess would be that it's not that big of an issue)
> 
> i'd like to seee a distinct name for this (maybe even with pve prefix) like:
> 
> pve-nvidia-sriov at .service
> 
> Then it's clear where it comes from and what it's for (vgpud is an nvidia daemon
> that does not really has anything to do with what this does)
> Or do you have any objections to that?
> 
> also, would it make sense to add a
> 
> ---
> ConditionPathExists=/usr/lib/nvidia/sriov-manage
> ---
> 
> too?
> 
> otherwise users that enable it accidentally or too early run into an ugly error
> (though that can be desired too)

An option might be to replace the ExecStartPre "sleep 5" with an actual pre-start
command provided by the helper that ensures that everything is setup correctly
and prints telling errors otherwise. That can then also handle any sleep that is
required, maybe in a more time-efficient way – like loop and check for the required
condition with shorter sleeps in-between, the wiki also mentions that this might
need to be adapted depending on the setup/HW.

Anyway, it would be good to get an actual commit message, it can be short but
could describe what this does and why a sleep is required in the first place,
also mentioning that this was taken over from the wiki [0] could be nice to
know.

https://pve.proxmox.com/mediawiki/index.php?title=NVIDIA_vGPU_on_Proxmox_VE&oldid=12125#Enabling_SR-IOV