Update udev rules and systemd service

khfeng commented 5 years ago

Make sure each rule is comma seperated.
Use DEVPATH=="/bus/pci/drivers/nvidia", which is added after nvidia driver probes and binds to the device, to prevent any race condition.
Use TAG+="systemd" and ENV{SYSTEMD_WANTS} to let systemd manage the service.
Now systemd knows the persistenced service depends on sys-bus-pci-drivers-nvidia.device, we can use StopWhenUnneeded=true to let systemd stop the service once sys-bus-pci-drivers-nvidia.device disappears.

tseliot commented 5 years ago

Thanks!

tseliot commented 5 years ago

I also added the following, or your change to debian/nvidia-compute-utils-430.nvidia-persistenced.service will be overwritten, when generated from debian/templates/nvidia-compute-utils-flavour.nvidia-persistenced.service.in:

index 5062a09..6ddf3b0 100644
--- a/debian/templates/nvidia-compute-utils-flavour.nvidia-persistenced.service.in
+++ b/debian/templates/nvidia-compute-utils-flavour.nvidia-persistenced.service.in
@@ -1,6 +1,7 @@
 [Unit]
 Description=NVIDIA Persistence Daemon
 Wants=syslog.target
+StopWhenUnneeded=true

 [Service]
 Type=forking

This is also part of the new "430" branch now.

khfeng commented 5 years ago

Didn't spot this, thanks!

canonical / nvidia-graphics-drivers

Update udev rules and systemd service #34