[dm-devel] [PATCH 4/5] libmultipath: change directio get_events() timeout handling
Benjamin Marzinski
bmarzins at redhat.com
Wed Feb 19 06:54:32 UTC 2020
get_events() used a NULL or a zeroed timeout to mean "don't wait".
io_getevents() uses a NULL timeout to mean "wait forever" and a
zeroed timeout to mean "don't wait". Make get_events() work like
io_getevents().
Signed-off-by: Benjamin Marzinski <bmarzins at redhat.com>
---
libmultipath/checkers/directio.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/libmultipath/checkers/directio.c b/libmultipath/checkers/directio.c
index 6ad7c885..649961a4 100644
--- a/libmultipath/checkers/directio.c
+++ b/libmultipath/checkers/directio.c
@@ -262,7 +262,7 @@ get_events(struct aio_group *aio_grp, struct timespec *timeout)
struct io_event events[128];
int i, nr, got_events = 0;
struct timespec zero_timeout = {0};
- struct timespec *timep = (timeout)? timeout : &zero_timeout;
+ struct timespec *timep = timeout;
do {
errno = 0;
@@ -303,7 +303,6 @@ check_state(int fd, struct directio_context *ct, int sync, int timeout_secs)
int rc;
long r;
struct timespec currtime, endtime;
- struct timespec *timep = &timeout;
if (fstat(fd, &sb) == 0) {
LOG(4, "called for %x", (unsigned) sb.st_rdev);
@@ -339,18 +338,19 @@ check_state(int fd, struct directio_context *ct, int sync, int timeout_secs)
endtime.tv_nsec += timeout.tv_nsec;
normalize_timespec(&endtime);
while(1) {
- r = get_events(ct->aio_grp, timep);
+ r = get_events(ct->aio_grp, &timeout);
if (ct->req->state != PATH_PENDING) {
ct->running = 0;
return ct->req->state;
- } else if (r == 0 || !timep)
+ } else if (r == 0 ||
+ (timeout.tv_sec == 0 && timeout.tv_nsec == 0))
break;
get_monotonic_time(&currtime);
timespecsub(&endtime, &currtime, &timeout);
if (timeout.tv_sec < 0)
- timep = NULL;
+ timeout.tv_sec = timeout.tv_nsec = 0;
}
if (ct->running > timeout_secs || sync) {
struct io_event event;
--
2.17.2
More information about the dm-devel
mailing list