chiark / gitweb /
Properly handle device aliases used as dependencies
authorEelco Dolstra <eelco.dolstra@logicblox.com>
Wed, 10 Oct 2012 20:43:10 +0000 (16:43 -0400)
committerDave Reisner <dreisner@archlinux.org>
Wed, 17 Oct 2012 13:54:53 +0000 (09:54 -0400)
commit22349cee29b31cde7ef8582b4882475f553af97a
tree0f44a8984abf2b92def3014cdb237759c8e483e7
parent5e6870eab5aa00675e4cd2f914fd1c25642fc964
Properly handle device aliases used as dependencies

If a device unit has aliases defined in udev rules, and there are
other units that depend on that alias, as in

  BindTo=sys-subsystem-net-devices-eth0.device

then systemd will fail the start the alias, and any dependent units
will time out.  See

  https://bugs.freedesktop.org/show_bug.cgi?id=52580

This is because unit_add_name() in device_add_escaped_name() will
return EEXIST.

The solution taken here is to call device_update_unit() on the alias
name.  Thus if a unit with the alias name already exists, we reuse it;
otherwise a new unit is created.  Creating multiple units for a single
device is perhaps suboptimal, but it's consistent with the treatment
of udev symlinks in device_process_new_device().
src/core/device.c