diff options
| author | Kay Sievers <kay@vrfy.org> | 2014-04-12 22:35:50 -0700 |
|---|---|---|
| committer | Kay Sievers <kay@vrfy.org> | 2014-04-13 17:12:14 -0700 |
| commit | 9ea28c55a2488e6cd4a44ac5786f12b71ad5bc9f (patch) | |
| tree | 3aeccb80e1b72169b8bc4836b5a6d1a76130ebe3 /src/udev/udev-ctrl.c | |
| parent | a163b64c4b08e8a4ad39a9a295acf3d1634024a3 (diff) | |
udev: remove seqnum API and all assumptions about seqnums
The way the kernel namespaces have been implemented breaks assumptions
udev made regarding uevent sequence numbers. Creating devices in a
namespace "steals" uevents and its sequence numbers from the host. It
confuses the "udevadmin settle" logic, which might block until util a
timeout is reached, even when no uevent is pending.
Remove any assumptions about sequence numbers and deprecate libudev's
API exposing these numbers; none of that can reliably be used anymore
when namespaces are involved.
Diffstat (limited to 'src/udev/udev-ctrl.c')
| -rw-r--r-- | src/udev/udev-ctrl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/udev/udev-ctrl.c b/src/udev/udev-ctrl.c index 1e91ec20a..74bbd3a99 100644 --- a/src/udev/udev-ctrl.c +++ b/src/udev/udev-ctrl.c @@ -275,7 +275,7 @@ static int ctrl_send(struct udev_ctrl *uctrl, enum udev_ctrl_msg_type type, int pfd[0].fd = uctrl->sock; pfd[0].events = POLLIN; - r = poll(pfd, 1, timeout * 1000); + r = poll(pfd, 1, timeout * MSEC_PER_SEC); if (r < 0) { if (errno == EINTR) continue; |