diff options
author | trasz <trasz@FreeBSD.org> | 2015-03-07 19:45:07 +0000 |
---|---|---|
committer | trasz <trasz@FreeBSD.org> | 2015-03-07 19:45:07 +0000 |
commit | cc83545ed3c82684e529df31a728138ed50fdd3c (patch) | |
tree | c71e8281f929e569a35a06b9bca1e519b159de68 /usr.sbin | |
parent | 9cde7d310db1684f4e7b4fbb98ebfd9accbd9b06 (diff) | |
download | FreeBSD-src-cc83545ed3c82684e529df31a728138ed50fdd3c.zip FreeBSD-src-cc83545ed3c82684e529df31a728138ed50fdd3c.tar.gz |
MFC r277834:
When there are no automounted filesystems, autounmountd(8) should wait
for filesystem event, instead of looping on a timeout.
Sponsored by: The FreeBSD Foundation
Diffstat (limited to 'usr.sbin')
-rw-r--r-- | usr.sbin/autofs/autounmountd.c | 36 |
1 files changed, 23 insertions, 13 deletions
diff --git a/usr.sbin/autofs/autounmountd.c b/usr.sbin/autofs/autounmountd.c index 0f52b66..b85f3ca 100644 --- a/usr.sbin/autofs/autounmountd.c +++ b/usr.sbin/autofs/autounmountd.c @@ -182,7 +182,7 @@ expire_automounted(double expiration_time) { struct automounted_fs *af, *tmpaf; time_t now; - double mounted_for, mounted_max = 0; + double mounted_for, mounted_max = -1.0; int error; now = time(NULL); @@ -231,21 +231,28 @@ do_wait(int kq, double sleep_time) { struct timespec timeout; struct kevent unused; - int error; - - assert(sleep_time > 0); - timeout.tv_sec = sleep_time; - timeout.tv_nsec = 0; - - log_debugx("waiting for filesystem event for %.0f seconds", sleep_time); - error = kevent(kq, NULL, 0, &unused, 1, &timeout); - if (error < 0) + int nevents; + + if (sleep_time != -1.0) { + assert(sleep_time > 0.0); + timeout.tv_sec = sleep_time; + timeout.tv_nsec = 0; + + log_debugx("waiting for filesystem event for %.0f seconds", sleep_time); + nevents = kevent(kq, NULL, 0, &unused, 1, &timeout); + } else { + log_debugx("waiting for filesystem event"); + nevents = kevent(kq, NULL, 0, &unused, 1, NULL); + } + if (nevents < 0) log_err(1, "kevent"); - if (error == 0) + if (nevents == 0) { log_debugx("timeout reached"); - else + assert(sleep_time > 0.0); + } else { log_debugx("got filesystem event"); + } } int @@ -324,7 +331,10 @@ main_autounmountd(int argc, char **argv) for (;;) { refresh_automounted(); mounted_max = expire_automounted(expiration_time); - if (mounted_max < expiration_time) { + if (mounted_max == -1.0) { + sleep_time = mounted_max; + log_debugx("no filesystems to expire"); + } else if (mounted_max < expiration_time) { sleep_time = difftime(expiration_time, mounted_max); log_debugx("some filesystems expire in %.0f seconds", sleep_time); |