how reload udev rules and systemd on F18

[Lennart Poettering]

On Tue, 29.01.13 00:36, Sérgio Basto (sergio at serjux.com) wrote:

> On Ter, 2013-01-29 at 00:50 +0100, Kay Sievers wrote: 
> > On Sat, Jan 26, 2013 at 6:14 AM, Sérgio Basto <sergio at serjux.com> wrote:
> > > how I should do the old command udevadm: control --reload-rules ?
> > > is correct : systemctl restart udev.service ?
> > > I don't find any in documentation that guarantee this .
> > >
> > > BTW in F18: udev.service change the name to systemd-udevd.service.
> > > So put in package spec, is this correct way ?
> > >
> > > %if 0%{?fedora} < 18
> > >     systemctl restart udev.service
> > >     systemctl restart udev-trigger.service
> > >     systemctl restart udev-settle.service
> > > %else
> > >     systemctl restart systemd-udevd.service
> > >     systemctl restart systemd-udev-trigger.service
> > >     systemctl restart systemd-udev-settle.service
> > > %endif
> > 
> > Please get rid of all of that, none of it is necessary, udev will
> > notice that on its own.
> > 
> > It is completely wrong to ever do that and to restart udev or other
> > essential services from packages. No package besides udev itself is
> > allowed to do restart these services.
> 
> 
> Hi, 
> Even when package adds some rules to /etc/udev/rules.d ? 

Yes, even then. udev will notice rules dropped there.

> like : 
> cat /etc/udev/rules.d/90-vboxdrv.rules 
> SUBSYSTEM=="usb_device", ACTION=="add", RUN+="VBoxCreateUSBNode.sh
> $major $minor $attr{bDeviceClass} vboxusers"
> SUBSYSTEM=="usb", ACTION=="add", ENV{DEVTYPE}=="usb_device", RUN
> +="VBoxCreateUSBNode.sh $major $minor $attr{bDeviceClass} vboxusers"
> SUBSYSTEM=="usb_device", ACTION=="remove", RUN+="VBoxCreateUSBNode.sh
> --remove $major $minor"
> SUBSYSTEM=="usb", ACTION=="remove", ENV{DEVTYPE}=="usb_device", RUN
> +="VBoxCreateUSBNode.sh --remove $major $minor"

Ahum, you are aware that this will run your script for any USB device
plugged in? Which Fedora package includes this? This code really looks
really not right.

Lennart

-- 
Lennart Poettering - Red Hat, Inc.