From 76077e59ec238bb7816af2d81e96189cdb7e562e Mon Sep 17 00:00:00 2001 From: Dongsheng Yang Date: Sun, 17 Mar 2019 23:04:52 -0400 Subject: [PATCH] rbd: krbd: return -ETIMEDOUT in polling We don't want to wait on uevent forever, but the return value of polling in timeout is 0 rather than a negative value. Fixes: http://tracker.ceph.com/issues/38792 Signed-off-by: Dongsheng Yang (cherry picked from commit ab833b823bf7dc5750f281a81f31ed8f708df73d) --- src/krbd.cc | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/src/krbd.cc b/src/krbd.cc index 97ba92ac3a75d..9192174f7aa44 100644 --- a/src/krbd.cc +++ b/src/krbd.cc @@ -193,12 +193,17 @@ static int wait_for_udev_add(struct udev_monitor *mon, const char *pool, for (;;) { struct pollfd fds[1]; struct udev_device *dev; + int r; fds[0].fd = udev_monitor_get_fd(mon); fds[0].events = POLLIN; - if (poll(fds, 1, POLL_TIMEOUT) < 0) + r = poll(fds, 1, POLL_TIMEOUT); + if (r < 0) return -errno; + if (r == 0) + return -ETIMEDOUT; + dev = udev_monitor_receive_device(mon); if (!dev) continue; @@ -489,12 +494,17 @@ static int wait_for_udev_remove(struct udev_monitor *mon, dev_t devno) for (;;) { struct pollfd fds[1]; struct udev_device *dev; + int r; fds[0].fd = udev_monitor_get_fd(mon); fds[0].events = POLLIN; - if (poll(fds, 1, POLL_TIMEOUT) < 0) + r = poll(fds, 1, POLL_TIMEOUT); + if (r < 0) return -errno; + if (r == 0) + return -ETIMEDOUT; + dev = udev_monitor_receive_device(mon); if (!dev) continue; -- 2.39.5