relative std::this_thread::sleep_for is more efficient than absolute std::this_thread::sleep_until. This is the opposite of std::condition_variable::wait_until/wait_for.

1 files changed, 2 insertions, 3 deletions

diff --git a/vespalib/src/vespa/vespalib/util/invokeserviceimpl.cpp b/vespalib/src/vespa/vespalib/util/invokeserviceimpl.cpp
index eac84568ec2..c10029b2f58 100644
--- a/vespalib/src/vespa/vespalib/util/invokeserviceimpl.cpp
+++ b/vespalib/src/vespa/vespalib/util/invokeserviceimpl.cpp
@@ -64,8 +64,7 @@ void
 InvokeServiceImpl::runLoop() {
     bool done = false;
     while ( ! done ) {
-        const steady_time now = steady_clock::now();
-        _now.store(now, std::memory_order_relaxed);
+        _now.store(steady_clock::now(), std::memory_order_relaxed);
         {
             std::lock_guard guard(_lock);
             for (auto & func: _toInvoke) {
@@ -74,7 +73,7 @@ InvokeServiceImpl::runLoop() {
             done = _closed;
         }
         if ( ! done) {
-            std::this_thread::sleep_until(now + _naptime);
+            std::this_thread::sleep_for(_naptime);
         }
     }