diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2021-03-14 22:00:42 +0100 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2021-03-15 11:13:24 +0100 |
commit | 1a451207b8ddbd15519ff9556c59287023ab50fa (patch) | |
tree | 99ae31e4a4c86a48b69db60447d00149341dfa6f /tools/polldaemon.c | |
parent | 5edb353062b26bbe93002717d77aa5d1707f1f70 (diff) | |
download | lvm2-1a451207b8ddbd15519ff9556c59287023ab50fa.tar.gz |
pooldaemon: increase min polling interval
Although we support '0' interval - it's highly inefficent to
do so many scans in busy-loop.
So ATM raise minimal rescan time to 100ms.
TODO: revisit whole timing logic here as it does have some sideeffect
hiddent impact and can considerably eat CPU in some cases.
Diffstat (limited to 'tools/polldaemon.c')
-rw-r--r-- | tools/polldaemon.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tools/polldaemon.c b/tools/polldaemon.c index d4e0be13e..32733b8ea 100644 --- a/tools/polldaemon.c +++ b/tools/polldaemon.c @@ -21,7 +21,7 @@ #include <time.h> -#define WAIT_AT_LEAST_NANOSECS 100000 +#define WAIT_AT_LEAST_NANOSECS 100000000 progress_t poll_mirror_progress(struct cmd_context *cmd, struct logical_volume *lv, const char *name, @@ -127,7 +127,7 @@ static void _nanosleep(unsigned secs, unsigned allow_zero_time) static void _sleep_and_rescan_devices(struct cmd_context *cmd, struct daemon_parms *parms) { - if (parms->interval && !parms->aborting) { + if (!parms->aborting) { /* * FIXME: do we really need to drop everything and then rescan * everything between each iteration? What change exactly does @@ -136,7 +136,7 @@ static void _sleep_and_rescan_devices(struct cmd_context *cmd, struct daemon_par */ lvmcache_destroy(cmd, 1, 0); label_scan_destroy(cmd); - _nanosleep(parms->interval, 1); + _nanosleep(parms->interval, 0); lvmcache_label_scan(cmd); } } |