96.43% Lines (243/252) 100.00% Functions (28/28)
TLA Baseline Branch
Line Hits Code Line Hits Code
1   // 1   //
2   // Copyright (c) 2025 Vinnie Falco (vinnie.falco@gmail.com) 2   // Copyright (c) 2025 Vinnie Falco (vinnie.falco@gmail.com)
3   // Copyright (c) 2026 Steve Gerbino 3   // Copyright (c) 2026 Steve Gerbino
4   // 4   //
5   // Distributed under the Boost Software License, Version 1.0. (See accompanying 5   // Distributed under the Boost Software License, Version 1.0. (See accompanying
6   // file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt) 6   // file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt)
7   // 7   //
8   // Official repository: https://github.com/cppalliance/corosio 8   // Official repository: https://github.com/cppalliance/corosio
9   // 9   //
10   10  
11   #ifndef BOOST_COROSIO_DETAIL_TIMER_SERVICE_HPP 11   #ifndef BOOST_COROSIO_DETAIL_TIMER_SERVICE_HPP
12   #define BOOST_COROSIO_DETAIL_TIMER_SERVICE_HPP 12   #define BOOST_COROSIO_DETAIL_TIMER_SERVICE_HPP
13   13  
14   #include <boost/corosio/detail/timer.hpp> 14   #include <boost/corosio/detail/timer.hpp>
15   #include <boost/corosio/detail/scheduler.hpp> 15   #include <boost/corosio/detail/scheduler.hpp>
16   #include <boost/corosio/detail/scheduler_op.hpp> 16   #include <boost/corosio/detail/scheduler_op.hpp>
17   #include <boost/corosio/detail/object_pool.hpp> 17   #include <boost/corosio/detail/object_pool.hpp>
18   #include <boost/corosio/detail/object_ref.hpp> 18   #include <boost/corosio/detail/object_ref.hpp>
19   #include <boost/corosio/detail/intrusive.hpp> 19   #include <boost/corosio/detail/intrusive.hpp>
20   #include <boost/corosio/detail/thread_local_ptr.hpp> 20   #include <boost/corosio/detail/thread_local_ptr.hpp>
21   #include <boost/capy/error.hpp> 21   #include <boost/capy/error.hpp>
22   #include <boost/capy/ex/execution_context.hpp> 22   #include <boost/capy/ex/execution_context.hpp>
23   #include <boost/capy/ex/abandon.hpp> 23   #include <boost/capy/ex/abandon.hpp>
24   #include <boost/capy/ex/executor_ref.hpp> 24   #include <boost/capy/ex/executor_ref.hpp>
25   #include <system_error> 25   #include <system_error>
26   26  
27   #include <atomic> 27   #include <atomic>
28   #include <chrono> 28   #include <chrono>
29   #include <coroutine> 29   #include <coroutine>
30   #include <cstddef> 30   #include <cstddef>
31   #include <limits> 31   #include <limits>
32   #include <mutex> 32   #include <mutex>
33   #include <thread> 33   #include <thread>
34   #include <stop_token> 34   #include <stop_token>
35   #include <utility> 35   #include <utility>
36   #include <vector> 36   #include <vector>
37   37  
38   namespace boost::corosio::detail { 38   namespace boost::corosio::detail {
39   39  
40   class scheduler; 40   class scheduler;
41   41  
42   /* 42   /*
43   Timer Service 43   Timer Service
44   ============= 44   =============
45   45  
46   Data Structures 46   Data Structures
47   --------------- 47   ---------------
48   waiter_node (defined in timer.hpp) holds per-waiter state: 48   waiter_node (defined in timer.hpp) holds per-waiter state:
49   coroutine handle, executor, error output, embedded 49   coroutine handle, executor, error output, embedded
50   completion_op. Each concurrent co_await t.wait() embeds one 50   completion_op. Each concurrent co_await t.wait() embeds one
51   waiter_node in the awaitable on the suspended coroutine's 51   waiter_node in the awaitable on the suspended coroutine's
52   frame — waits perform no allocation. 52   frame — waits perform no allocation.
53   53  
54   timer::implementation holds per-timer state: expiry, heap 54   timer::implementation holds per-timer state: expiry, heap
55   index, and the single published waiter. Each timer holds 55   index, and the single published waiter. Each timer holds
56   at most one waiter; process_expired's local cross-timer drain 56   at most one waiter; process_expired's local cross-timer drain
57   list still threads waiters through their intrusive hooks when 57   list still threads waiters through their intrusive hooks when
58   collecting several timers' waiters past the lock. 58   collecting several timers' waiters past the lock.
59   59  
60   timer_service owns a min-heap of active timers and recycles 60   timer_service owns a min-heap of active timers and recycles
61   retired impls through a per-service object_pool, fronted by a 61   retired impls through a per-service object_pool, fronted by a
62   thread-local single-slot cache. The heap is ordered by expiry 62   thread-local single-slot cache. The heap is ordered by expiry
63   time; the scheduler queries nearest_expiry() to set the 63   time; the scheduler queries nearest_expiry() to set the
64   epoll/timerfd timeout. 64   epoll/timerfd timeout.
65   65  
66   Optimization Strategy 66   Optimization Strategy
67   --------------------- 67   ---------------------
68   1. Deferred heap insertion — expires_after() stores the expiry 68   1. Deferred heap insertion — expires_after() stores the expiry
69   but does not insert into the heap. Insertion happens in wait(). 69   but does not insert into the heap. Insertion happens in wait().
70   2. Thread-local impl cache — single-slot per-thread cache, in 70   2. Thread-local impl cache — single-slot per-thread cache, in
71   front of the object_pool fallback; keeps a thread's hot impl 71   front of the object_pool fallback; keeps a thread's hot impl
72   out of the shared free list. 72   out of the shared free list.
73   3. Frame-resident waiter_node with embedded completion_op — 73   3. Frame-resident waiter_node with embedded completion_op —
74   eliminates heap allocation per wait/fire/cancel. 74   eliminates heap allocation per wait/fire/cancel.
75   4. Cached nearest expiry — atomic avoids mutex in nearest_expiry(). 75   4. Cached nearest expiry — atomic avoids mutex in nearest_expiry().
76   5. might_have_pending_waits_ flag — skips lock when no wait issued. 76   5. might_have_pending_waits_ flag — skips lock when no wait issued.
77   77  
78   Concurrency 78   Concurrency
79   ----------- 79   -----------
80   stop_token callbacks can fire from any thread. The impl_ 80   stop_token callbacks can fire from any thread. The impl_
81   pointer on waiter_node is used as a "still in list" marker. 81   pointer on waiter_node is used as a "still in list" marker.
82   A waiter_node's storage is the suspended coroutine's frame: 82   A waiter_node's storage is the suspended coroutine's frame:
83   every completion path must finish touching the node before 83   every completion path must finish touching the node before
84   dispatching the continuation or destroying the handle. 84   dispatching the continuation or destroying the handle.
85   */ 85   */
86   86  
87   inline void timer_service_invalidate_cache() noexcept; 87   inline void timer_service_invalidate_cache() noexcept;
88   88  
89   /** The timer service's recycling pool. 89   /** The timer service's recycling pool.
90   90  
91   Adds no behavior of its own — it exists only to re-expose 91   Adds no behavior of its own — it exists only to re-expose
92   `adopt()` and `remove()` as public, so the timer service's 92   `adopt()` and `remove()` as public, so the timer service's
93   thread-local cache can transfer a cached impl's `live_` tracking 93   thread-local cache can transfer a cached impl's `live_` tracking
94   on both sides of the handoff: `construct()` re-adopts a 94   on both sides of the handoff: `construct()` re-adopts a
95   TL-cached impl as live, `shutdown()` detaches one before its 95   TL-cached impl as live, `shutdown()` detaches one before its
96   direct delete. 96   direct delete.
97   */ 97   */
98   class timer_object_pool : public object_pool<timer::implementation> 98   class timer_object_pool : public object_pool<timer::implementation>
99   { 99   {
100   public: 100   public:
101   using object_pool::adopt; 101   using object_pool::adopt;
102   using object_pool::remove; 102   using object_pool::remove;
103   }; 103   };
104   104  
105   // timer_service class body — member function definitions are 105   // timer_service class body — member function definitions are
106   // out-of-class (after implementation and waiter_node are complete) 106   // out-of-class (after implementation and waiter_node are complete)
107   class BOOST_COROSIO_DECL timer_service final 107   class BOOST_COROSIO_DECL timer_service final
108   : public capy::execution_context::service 108   : public capy::execution_context::service
109   , public io_object::io_service 109   , public io_object::io_service
110   { 110   {
111   friend struct timer::implementation; 111   friend struct timer::implementation;
112   112  
113   public: 113   public:
114   using clock_type = std::chrono::steady_clock; 114   using clock_type = std::chrono::steady_clock;
115   using time_point = clock_type::time_point; 115   using time_point = clock_type::time_point;
116   116  
117   /// Type-erased callback for earliest-expiry-changed notifications. 117   /// Type-erased callback for earliest-expiry-changed notifications.
118   class callback 118   class callback
119   { 119   {
120   void* ctx_ = nullptr; 120   void* ctx_ = nullptr;
121   void (*fn_)(void*) = nullptr; 121   void (*fn_)(void*) = nullptr;
122   122  
123   public: 123   public:
124   /// Construct an empty callback. 124   /// Construct an empty callback.
HITCBC 125   932 callback() = default; 125   934 callback() = default;
126   126  
127   /// Construct a callback with the given context and function. 127   /// Construct a callback with the given context and function.
HITCBC 128   928 callback(void* ctx, void (*fn)(void*)) noexcept : ctx_(ctx), fn_(fn) {} 128   927 callback(void* ctx, void (*fn)(void*)) noexcept : ctx_(ctx), fn_(fn) {}
129   129  
130   /// Return true if the callback is non-empty. 130   /// Return true if the callback is non-empty.
131   explicit operator bool() const noexcept 131   explicit operator bool() const noexcept
132   { 132   {
133   return fn_ != nullptr; 133   return fn_ != nullptr;
134   } 134   }
135   135  
136   /// Invoke the callback. 136   /// Invoke the callback.
HITCBC 137   8065 void operator()() const 137   8045 void operator()() const
138   { 138   {
HITCBC 139   8065 if (fn_) 139   8045 if (fn_)
HITCBC 140   8065 fn_(ctx_); 140   8045 fn_(ctx_);
HITCBC 141   8065 } 141   8045 }
142   }; 142   };
143   143  
144   private: 144   private:
145   struct heap_entry 145   struct heap_entry
146   { 146   {
147   time_point time_; 147   time_point time_;
148   timer::implementation* timer_; 148   timer::implementation* timer_;
149   }; 149   };
150   150  
151   scheduler* sched_ = nullptr; 151   scheduler* sched_ = nullptr;
152   BOOST_COROSIO_MSVC_WARNING_PUSH 152   BOOST_COROSIO_MSVC_WARNING_PUSH
153   BOOST_COROSIO_MSVC_WARNING_DISABLE(4251) // std:: members, dll-interface 153   BOOST_COROSIO_MSVC_WARNING_DISABLE(4251) // std:: members, dll-interface
154   mutable std::mutex mutex_; 154   mutable std::mutex mutex_;
155   std::vector<heap_entry> heap_; 155   std::vector<heap_entry> heap_;
156   timer_object_pool pool_; 156   timer_object_pool pool_;
157   callback on_earliest_changed_; 157   callback on_earliest_changed_;
158   bool shutting_down_ = false; 158   bool shutting_down_ = false;
159   // Set once the scheduler waits on this service; read without the 159   // Set once the scheduler waits on this service; read without the
160   // mutex by every timer construction after the first. 160   // mutex by every timer construction after the first.
161   std::atomic<bool> attached_{false}; 161   std::atomic<bool> attached_{false};
162   162  
163   // Cancellers that claimed a waiter and have yet to post it; 163   // Cancellers that claimed a waiter and have yet to post it;
164   // shutdown waits them out so the scheduler's drain finds the post. 164   // shutdown waits them out so the scheduler's drain finds the post.
165   std::size_t cancels_posting_ = 0; // guarded by mutex_ 165   std::size_t cancels_posting_ = 0; // guarded by mutex_
166   // Avoids mutex in nearest_expiry() and empty() 166   // Avoids mutex in nearest_expiry() and empty()
167   mutable std::atomic<std::int64_t> cached_nearest_ns_{ 167   mutable std::atomic<std::int64_t> cached_nearest_ns_{
168   (std::numeric_limits<std::int64_t>::max)()}; 168   (std::numeric_limits<std::int64_t>::max)()};
169   BOOST_COROSIO_MSVC_WARNING_POP 169   BOOST_COROSIO_MSVC_WARNING_POP
170   170  
171   public: 171   public:
172   /** Construct the timer service for a context's scheduler. 172   /** Construct the timer service for a context's scheduler.
173   173  
174   Created with the context's first timer. It does not attach to 174   Created with the context's first timer. It does not attach to
175   the scheduler here: a concurrent first use may construct a 175   the scheduler here: a concurrent first use may construct a
176   second service that the context then discards. See 176   second service that the context then discards. See
177   @ref attach. 177   @ref attach.
178   178  
179   @throws std::logic_error if the context has no scheduler. 179   @throws std::logic_error if the context has no scheduler.
180   */ 180   */
HITCBC 181   932 inline explicit timer_service(capy::execution_context& ctx) 181   934 inline explicit timer_service(capy::execution_context& ctx)
HITCBC 182   932 : sched_(ctx.find_service<scheduler>()) 182   934 : sched_(ctx.find_service<scheduler>())
183   { 183   {
HITCBC 184   932 if (!sched_) 184   934 if (!sched_)
HITCBC 185   4 detail::throw_logic_error( 185   4 detail::throw_logic_error(
186   "timer_service: the context has no scheduler"); 186   "timer_service: the context has no scheduler");
HITCBC 187   944 } 187   946 }
188   188  
189   /// Return the associated scheduler. 189   /// Return the associated scheduler.
HITCBC 190   35521 inline scheduler& get_scheduler() noexcept 190   35375 inline scheduler& get_scheduler() noexcept
191   { 191   {
HITCBC 192   35521 return *sched_; 192   35375 return *sched_;
193   } 193   }
194   194  
195   /// Destroy the timer service, detaching it from the scheduler. 195   /// Destroy the timer service, detaching it from the scheduler.
HITCBC 196   1856 ~timer_service() override 196   1860 ~timer_service() override
HITCBC 197   928 { 197   930 {
HITCBC 198   928 if (attached_.load(std::memory_order_acquire)) 198   930 if (attached_.load(std::memory_order_acquire))
HITCBC 199   928 sched_->detach_timer_service(); 199   927 sched_->detach_timer_service();
HITCBC 200   1856 } 200   1860 }
201   201  
202   /** Attach this service to the scheduler, if not yet attached. 202   /** Attach this service to the scheduler, if not yet attached.
203   203  
204   Call once the context has settled which service it keeps; no 204   Call once the context has settled which service it keeps; no
205   timer of this service can wait before this returns. 205   timer of this service can wait before this returns.
206   206  
207   @par Exception Safety 207   @par Exception Safety
208   Strong guarantee: on failure the service stays detached and 208   Strong guarantee: on failure the service stays detached and
209   the next call retries. 209   the next call retries.
210   */ 210   */
HITCBC 211   20313 inline void attach() 211   20239 inline void attach()
212   { 212   {
HITCBC 213   20313 if (attached_.load(std::memory_order_acquire)) 213   20239 if (attached_.load(std::memory_order_acquire))
HITCBC 214   19385 return; 214   19312 return;
HITCBC 215   928 std::lock_guard lock(mutex_); 215   928 std::lock_guard lock(mutex_);
HITCBC 216   928 if (attached_.load(std::memory_order_relaxed)) 216   928 if (attached_.load(std::memory_order_relaxed))
HITGBC 217   ✗ return; 217   1 return;
HITCBC 218   928 sched_->attach_timer_service(*this); 218   927 sched_->attach_timer_service(*this);
HITCBC 219   928 attached_.store(true, std::memory_order_release); 219   927 attached_.store(true, std::memory_order_release);
HITCBC 220   928 } 220   928 }
221   221  
222   timer_service(timer_service const&) = delete; 222   timer_service(timer_service const&) = delete;
223   timer_service& operator=(timer_service const&) = delete; 223   timer_service& operator=(timer_service const&) = delete;
224   224  
225   /// Register a callback invoked when the earliest expiry changes. 225   /// Register a callback invoked when the earliest expiry changes.
HITCBC 226   928 inline void set_on_earliest_changed(callback cb) 226   927 inline void set_on_earliest_changed(callback cb)
227   { 227   {
HITCBC 228   928 on_earliest_changed_ = cb; 228   927 on_earliest_changed_ = cb;
HITCBC 229   928 } 229   927 }
230   230  
231   /// Return true if no timers are in the heap. 231   /// Return true if no timers are in the heap.
232   inline bool empty() const noexcept 232   inline bool empty() const noexcept
233   { 233   {
234   return cached_nearest_ns_.load(std::memory_order_acquire) == 234   return cached_nearest_ns_.load(std::memory_order_acquire) ==
235   (std::numeric_limits<std::int64_t>::max)(); 235   (std::numeric_limits<std::int64_t>::max)();
236   } 236   }
237   237  
238   /// Return the nearest timer expiry without acquiring the mutex. 238   /// Return the nearest timer expiry without acquiring the mutex.
HITCBC 239   18117 inline time_point nearest_expiry() const noexcept 239   17845 inline time_point nearest_expiry() const noexcept
240   { 240   {
HITCBC 241   18117 auto ns = cached_nearest_ns_.load(std::memory_order_acquire); 241   17845 auto ns = cached_nearest_ns_.load(std::memory_order_acquire);
HITCBC 242   18117 return time_point(time_point::duration(ns)); 242   17845 return time_point(time_point::duration(ns));
243   } 243   }
244   244  
245   /// Cancel all pending timers and free cached resources. 245   /// Cancel all pending timers and free cached resources.
246   inline void shutdown() override; 246   inline void shutdown() override;
247   247  
248   /// Construct a new timer implementation. 248   /// Construct a new timer implementation.
249   inline io_object::implementation* construct() override; 249   inline io_object::implementation* construct() override;
250   250  
251   /// Destroy a timer implementation, cancelling pending waiters. 251   /// Destroy a timer implementation, cancelling pending waiters.
252   inline void destroy(io_object::implementation* p) override; 252   inline void destroy(io_object::implementation* p) override;
253   253  
254   /// Cancel and recycle a timer implementation. 254   /// Cancel and recycle a timer implementation.
255   inline void destroy_impl(timer::implementation& impl); 255   inline void destroy_impl(timer::implementation& impl);
256   256  
257   /// Publish the timer's waiter and insert the timer into the heap. 257   /// Publish the timer's waiter and insert the timer into the heap.
258   inline void insert_waiter(timer::implementation& impl, waiter_node* w); 258   inline void insert_waiter(timer::implementation& impl, waiter_node* w);
259   259  
260   /// Cancel the timer's published waiter, if any. 260   /// Cancel the timer's published waiter, if any.
261   inline void cancel_timer(timer::implementation& impl); 261   inline void cancel_timer(timer::implementation& impl);
262   262  
263   /// Cancel one specific waiter ( stop_token callback path ). 263   /// Cancel one specific waiter ( stop_token callback path ).
264   inline void cancel_waiter(waiter_node* w); 264   inline void cancel_waiter(waiter_node* w);
265   265  
266   /// Complete all waiters whose timers have expired. 266   /// Complete all waiters whose timers have expired.
267   inline std::size_t process_expired(); 267   inline std::size_t process_expired();
268   268  
269   private: 269   private:
HITCBC 270   64002 inline void refresh_cached_nearest() noexcept 270   61243 inline void refresh_cached_nearest() noexcept
271   { 271   {
HITCBC 272   64002 auto ns = heap_.empty() ? (std::numeric_limits<std::int64_t>::max)() 272   61243 auto ns = heap_.empty() ? (std::numeric_limits<std::int64_t>::max)()
HITCBC 273   60454 : heap_[0].time_.time_since_epoch().count(); 273   57906 : heap_[0].time_.time_since_epoch().count();
HITCBC 274   64002 cached_nearest_ns_.store(ns, std::memory_order_release); 274   61243 cached_nearest_ns_.store(ns, std::memory_order_release);
HITCBC 275   64002 } 275   61243 }
276   276  
277   inline void remove_timer_impl(timer::implementation& impl); 277   inline void remove_timer_impl(timer::implementation& impl);
278   inline void up_heap(std::size_t index); 278   inline void up_heap(std::size_t index);
279   inline void down_heap(std::size_t index); 279   inline void down_heap(std::size_t index);
280   inline void swap_heap(std::size_t i1, std::size_t i2); 280   inline void swap_heap(std::size_t i1, std::size_t i2);
281   }; 281   };
282   282  
283   // Thread-local single-slot impl cache, validated by comparing svc_. 283   // Thread-local single-slot impl cache, validated by comparing svc_.
284   // Keeps a hot impl out of the shared free list; both sides still 284   // Keeps a hot impl out of the shared free list; both sides still
285   // take the pool lock to move it in and out of live tracking. 285   // take the pool lock to move it in and out of live tracking.
286   // Cleared by timer_service_invalidate_cache() during shutdown. 286   // Cleared by timer_service_invalidate_cache() during shutdown.
287   287  
288   inline thread_local_ptr<timer::implementation> tl_cached_impl; 288   inline thread_local_ptr<timer::implementation> tl_cached_impl;
289   289  
290   // The POD TLS slot above never runs destructors, so a short-lived 290   // The POD TLS slot above never runs destructors, so a short-lived
291   // run() thread would leak its cached impl. Each push arms this 291   // run() thread would leak its cached impl. Each push arms this
292   // owner, whose destructor frees the slot at thread exit. A cached 292   // owner, whose destructor frees the slot at thread exit. A cached
293   // entry is a quiescent heap object (nothing in the heap or free 293   // entry is a quiescent heap object (nothing in the heap or free
294   // list) and deletion touches no service state, so it is safe after 294   // list) and deletion touches no service state, so it is safe after
295   // the owning service is gone (the stale-entry path in 295   // the owning service is gone (the stale-entry path in
296   // try_pop_tl_cache deletes the same way). 296   // try_pop_tl_cache deletes the same way).
297   struct tl_cache_owner 297   struct tl_cache_owner
298   { 298   {
HITCBC 299   64 ~tl_cache_owner() 299   62 ~tl_cache_owner()
300   { 300   {
HITCBC 301   64 delete tl_cached_impl.get(); 301   62 delete tl_cached_impl.get();
HITCBC 302   64 tl_cached_impl.set(nullptr); 302   62 tl_cached_impl.set(nullptr);
HITCBC 303   64 } 303   62 }
304   }; 304   };
305   305  
306   inline void 306   inline void
HITCBC 307   14999 arm_tl_cache_cleanup() noexcept 307   15259 arm_tl_cache_cleanup() noexcept
308   { 308   {
HITCBC 309   14999 [[maybe_unused]] thread_local tl_cache_owner owner; 309   15259 [[maybe_unused]] thread_local tl_cache_owner owner;
HITCBC 310   14999 } 310   15259 }
311   311  
312   inline timer::implementation* 312   inline timer::implementation*
HITCBC 313   20313 try_pop_tl_cache(timer_service* svc) noexcept 313   20239 try_pop_tl_cache(timer_service* svc) noexcept
314   { 314   {
HITCBC 315   20313 auto* impl = tl_cached_impl.get(); 315   20239 auto* impl = tl_cached_impl.get();
HITCBC 316   20313 if (impl) 316   20239 if (impl)
317   { 317   {
HITCBC 318   14484 tl_cached_impl.set(nullptr); 318   14746 tl_cached_impl.set(nullptr);
HITCBC 319   14484 if (impl->svc_ == svc) 319   14746 if (impl->svc_ == svc)
HITCBC 320   14484 return impl; 320   14746 return impl;
321   // Stale impl from a destroyed service 321   // Stale impl from a destroyed service
MISUBC 322   ✗ delete impl; 322   ✗ delete impl;
323   } 323   }
HITCBC 324   5829 return nullptr; 324   5493 return nullptr;
325   } 325   }
326   326  
327   inline bool 327   inline bool
HITCBC 328   17062 try_push_tl_cache(timer::implementation* impl) noexcept 328   16988 try_push_tl_cache(timer::implementation* impl) noexcept
329   { 329   {
HITCBC 330   17062 if (!tl_cached_impl.get()) 330   16988 if (!tl_cached_impl.get())
331   { 331   {
HITCBC 332   14999 arm_tl_cache_cleanup(); 332   15259 arm_tl_cache_cleanup();
HITCBC 333   14999 tl_cached_impl.set(impl); 333   15259 tl_cached_impl.set(impl);
HITCBC 334   14999 return true; 334   15259 return true;
335   } 335   }
HITCBC 336   2063 return false; 336   1729 return false;
337   } 337   }
338   338  
339   inline void 339   inline void
HITCBC 340   928 timer_service_invalidate_cache() noexcept 340   927 timer_service_invalidate_cache() noexcept
341   { 341   {
HITCBC 342   928 delete tl_cached_impl.get(); 342   927 delete tl_cached_impl.get();
HITCBC 343   928 tl_cached_impl.set(nullptr); 343   927 tl_cached_impl.set(nullptr);
HITCBC 344   928 } 344   927 }
345   345  
346   // timer_service out-of-class member function definitions 346   // timer_service out-of-class member function definitions
347   347  
348   inline void 348   inline void
HITCBC 349   928 timer_service::shutdown() 349   927 timer_service::shutdown()
350   { 350   {
HITCBC 351   928 timer_service_invalidate_cache(); 351   927 timer_service_invalidate_cache();
HITCBC 352   928 shutting_down_ = true; 352   927 shutting_down_ = true;
353   // Flag only; the real drain walks the expiry heap below, not 353   // Flag only; the real drain walks the expiry heap below, not
354   // object_pool's live_ list, so the visit callback is a no-op. 354   // object_pool's live_ list, so the visit callback is a no-op.
HITCBC 355   928 pool_.shutdown([](timer::implementation*) {}); 355   927 pool_.shutdown([](timer::implementation*) {});
356   356  
357   // Snapshot impls and detach them from the heap so that 357   // Snapshot impls and detach them from the heap so that
358   // coroutine-owned timer destructors (run by the unwinds the 358   // coroutine-owned timer destructors (run by the unwinds the
359   // abandons below start) cannot re-enter remove_timer_impl() and 359   // abandons below start) cannot re-enter remove_timer_impl() and
360   // mutate the vector during iteration. A stop requested on another thread 360   // mutate the vector during iteration. A stop requested on another thread
361   // reaches cancel_waiter() under the same lock: whichever side 361   // reaches cancel_waiter() under the same lock: whichever side
362   // claims a waiter first owns it, and a waiter its canceller 362   // claims a waiter first owns it, and a waiter its canceller
363   // claimed is posted, so the scheduler's drain abandons it. 363   // claimed is posted, so the scheduler's drain abandons it.
HITCBC 364   928 std::vector<timer::implementation*> impls; 364   927 std::vector<timer::implementation*> impls;
HITCBC 365   928 std::vector<waiter_node*> waiters; 365   927 std::vector<waiter_node*> waiters;
366   { 366   {
HITCBC 367   928 std::lock_guard lock(mutex_); 367   927 std::lock_guard lock(mutex_);
HITCBC 368   928 impls.reserve(heap_.size()); 368   927 impls.reserve(heap_.size());
HITCBC 369   928 waiters.reserve(heap_.size()); 369   927 waiters.reserve(heap_.size());
HITCBC 370   4175 for (auto& entry : heap_) 370   4174 for (auto& entry : heap_)
371   { 371   {
HITCBC 372   3247 entry.timer_->heap_index_.store( 372   3247 entry.timer_->heap_index_.store(
373   (std::numeric_limits<std::size_t>::max)(), 373   (std::numeric_limits<std::size_t>::max)(),
374   std::memory_order_relaxed); 374   std::memory_order_relaxed);
HITCBC 375   3247 impls.push_back(entry.timer_); 375   3247 impls.push_back(entry.timer_);
HITCBC 376   3247 if (auto* w = std::exchange(entry.timer_->waiter_, nullptr)) 376   3247 if (auto* w = std::exchange(entry.timer_->waiter_, nullptr))
377   { 377   {
HITCBC 378   3247 w->impl_ = nullptr; 378   3247 w->impl_ = nullptr;
HITCBC 379   3247 waiters.push_back(w); 379   3247 waiters.push_back(w);
380   } 380   }
381   } 381   }
HITCBC 382   928 heap_.clear(); 382   927 heap_.clear();
HITCBC 383   928 cached_nearest_ns_.store( 383   927 cached_nearest_ns_.store(
384   (std::numeric_limits<std::int64_t>::max)(), 384   (std::numeric_limits<std::int64_t>::max)(),
385   std::memory_order_release); 385   std::memory_order_release);
HITCBC 386   928 } 386   927 }
387   387  
388   // A stop on another thread may have claimed its waiter before the 388   // A stop on another thread may have claimed its waiter before the
389   // pass above and still be posting it; the scheduler drains next, 389   // pass above and still be posting it; the scheduler drains next,
390   // and must find that post. 390   // and must find that post.
391   for (;;) 391   for (;;)
392   { 392   {
393   { 393   {
HITCBC 394   928 std::lock_guard lock(mutex_); 394   927 std::lock_guard lock(mutex_);
HITCBC 395   928 if (cancels_posting_ == 0) 395   927 if (cancels_posting_ == 0)
HITCBC 396   928 break; 396   927 break;
HITCBC 397   928 } 397   927 }
MISUBC 398   ✗ std::this_thread::yield(); 398   ✗ std::this_thread::yield();
MISUBC 399   ✗ } 399   ✗ }
400   400  
401   // Cancel waiting timers. Each waiter called work_started() 401   // Cancel waiting timers. Each waiter called work_started()
402   // in implementation::wait(); a waiter given up here never 402   // in implementation::wait(); a waiter given up here never
403   // completes, so its work is finished here to keep the count 403   // completes, so its work is finished here to keep the count
404   // balanced. Resetting the stop callback outside the lock 404   // balanced. Resetting the stop callback outside the lock
405   // waits out a canceller already running, which finds the waiter 405   // waits out a canceller already running, which finds the waiter
406   // claimed. 406   // claimed.
HITCBC 407   4175 for (auto* w : waiters) 407   4174 for (auto* w : waiters)
408   { 408   {
HITCBC 409   3247 w->reset_stop_cb(); 409   3247 w->reset_stop_cb();
HITCBC 410   3247 auto c = std::exchange(w->cont_, {}); 410   3247 auto c = std::exchange(w->cont_, {});
HITCBC 411   3247 sched_->work_finished(); 411   3247 sched_->work_finished();
412   // Unwinding the frame also ends the node's storage 412   // Unwinding the frame also ends the node's storage
HITCBC 413   3247 if (c.h) 413   3247 if (c.h)
HITCBC 414   3247 capy::abandon(c); 414   3247 capy::abandon(c);
415   } 415   }
416   416  
HITCBC 417   4175 for (auto* impl : impls) 417   4174 for (auto* impl : impls)
418   { 418   {
419   // Unlink from the pool's live_ list before the direct delete 419   // Unlink from the pool's live_ list before the direct delete
420   // below, or ~object_pool()'s unconditional sweep would delete 420   // below, or ~object_pool()'s unconditional sweep would delete
421   // this impl a second time. 421   // this impl a second time.
HITCBC 422   3247 [[maybe_unused]] bool const removed = pool_.remove(impl); 422   3247 [[maybe_unused]] bool const removed = pool_.remove(impl);
HITCBC 423   3247 BOOST_COROSIO_ASSERT(removed); 423   3247 BOOST_COROSIO_ASSERT(removed);
HITCBC 424   3247 delete impl; 424   3247 delete impl;
425   } 425   }
426   426  
427   // Anything still parked in pool_ (free-listed or live with no 427   // Anything still parked in pool_ (free-listed or live with no
428   // pending wait at shutdown) is freed by object_pool's own destructor. 428   // pending wait at shutdown) is freed by object_pool's own destructor.
HITCBC 429   928 } 429   927 }
430   430  
431   inline io_object::implementation* 431   inline io_object::implementation*
HITCBC 432   20313 timer_service::construct() 432   20239 timer_service::construct()
433   { 433   {
HITCBC 434   20313 timer::implementation* impl = try_pop_tl_cache(this); 434   20239 timer::implementation* impl = try_pop_tl_cache(this);
HITCBC 435   20313 if (impl) 435   20239 if (impl)
436   { 436   {
HITCBC 437   14484 impl->svc_ = this; 437   14746 impl->svc_ = this;
HITCBC 438   14484 impl->reuse(); 438   14746 impl->reuse();
HITCBC 439   14484 impl->refs_.store(1, std::memory_order_relaxed); 439   14746 impl->refs_.store(1, std::memory_order_relaxed);
440   // The TL push that parked impl here also removed it from 440   // The TL push that parked impl here also removed it from
441   // live_ (see destroy_impl) — re-adopt it now that it is live 441   // live_ (see destroy_impl) — re-adopt it now that it is live
442   // again. 442   // again.
HITCBC 443   14484 pool_.adopt(impl); 443   14746 pool_.adopt(impl);
HITCBC 444   14484 return impl; 444   14746 return impl;
445   } 445   }
446   446  
HITCBC 447   5829 return pool_.acquire(*this); 447   5493 return pool_.acquire(*this);
448   } 448   }
449   449  
450   inline void 450   inline void
HITCBC 451   20313 timer_service::destroy(io_object::implementation* p) 451   20239 timer_service::destroy(io_object::implementation* p)
452   { 452   {
453   // During shutdown the drain loop owns every impl and deletes 453   // During shutdown the drain loop owns every impl and deletes
454   // them directly. A frame destroyed by that loop can unwind a 454   // them directly. A frame destroyed by that loop can unwind a
455   // handle whose impl was freed in an earlier iteration (a 455   // handle whose impl was freed in an earlier iteration (a
456   // timeout's parent frame owns the timeout timer while 456   // timeout's parent frame owns the timeout timer while
457   // suspended on the inner delay's timer), so bail out before 457   // suspended on the inner delay's timer), so bail out before
458   // even downcasting the pointer. 458   // even downcasting the pointer.
HITCBC 459   20313 if (shutting_down_) 459   20239 if (shutting_down_)
HITCBC 460   3251 return; 460   3251 return;
HITCBC 461   17062 destroy_impl(static_cast<timer::implementation&>(*p)); 461   16988 destroy_impl(static_cast<timer::implementation&>(*p));
462   } 462   }
463   463  
464   inline void 464   inline void
HITCBC 465   17062 timer_service::destroy_impl(timer::implementation& impl) 465   16988 timer_service::destroy_impl(timer::implementation& impl)
466   { 466   {
467   // During shutdown the impl is owned by the shutdown loop. 467   // During shutdown the impl is owned by the shutdown loop.
468   // Re-entering here (from a coroutine-owned timer destructor 468   // Re-entering here (from a coroutine-owned timer destructor
469   // triggered by h.destroy()) must not modify the heap or 469   // triggered by h.destroy()) must not modify the heap or
470   // recycle the impl — shutdown deletes it directly. 470   // recycle the impl — shutdown deletes it directly.
HITCBC 471   17062 if (shutting_down_) 471   16988 if (shutting_down_)
MISUBC 472   ✗ return; 472   ✗ return;
473   473  
HITCBC 474   17062 cancel_timer(impl); 474   16988 cancel_timer(impl);
475   475  
HITCBC 476   34124 if (impl.heap_index_.load(std::memory_order_relaxed) != 476   33976 if (impl.heap_index_.load(std::memory_order_relaxed) !=
HITCBC 477   17062 (std::numeric_limits<std::size_t>::max)()) 477   16988 (std::numeric_limits<std::size_t>::max)())
478   { 478   {
MISUBC 479   ✗ std::lock_guard lock(mutex_); 479   ✗ std::lock_guard lock(mutex_);
MISUBC 480   ✗ remove_timer_impl(impl); 480   ✗ remove_timer_impl(impl);
MISUBC 481   ✗ refresh_cached_nearest(); 481   ✗ refresh_cached_nearest();
MISUBC 482   ✗ } 482   ✗ }
483   483  
HITCBC 484   17062 if (try_push_tl_cache(&impl)) 484   16988 if (try_push_tl_cache(&impl))
485   { 485   {
486   // The TL slot takes sole ownership: it is deleted directly by 486   // The TL slot takes sole ownership: it is deleted directly by
487   // tl_cache_owner / the stale-service path, never through this 487   // tl_cache_owner / the stale-service path, never through this
488   // pool, so it must leave live_ now or ~object_pool() would see 488   // pool, so it must leave live_ now or ~object_pool() would see
489   // it as abandoned (and, at the wrong moment, double-delete it 489   // it as abandoned (and, at the wrong moment, double-delete it
490   // after a later construct() re-adopts it). 490   // after a later construct() re-adopts it).
HITCBC 491   14999 [[maybe_unused]] bool const removed = pool_.remove(&impl); 491   15259 [[maybe_unused]] bool const removed = pool_.remove(&impl);
HITCBC 492   14999 BOOST_COROSIO_ASSERT(removed); 492   15259 BOOST_COROSIO_ASSERT(removed);
HITCBC 493   14999 return; 493   15259 return;
494   } 494   }
495   495  
496   // No op keepalives on a timer impl: this is the only reference, 496   // No op keepalives on a timer impl: this is the only reference,
497   // so release() always recycles immediately via retire(). 497   // so release() always recycles immediately via retire().
HITCBC 498   2063 release(&impl); 498   1729 release(&impl);
499   } 499   }
500   500  
501   inline void 501   inline void
HITCBC 502   25810 timer_service::insert_waiter(timer::implementation& impl, waiter_node* w) 502   24237 timer_service::insert_waiter(timer::implementation& impl, waiter_node* w)
503   { 503   {
HITCBC 504   25810 bool notify = false; 504   24237 bool notify = false;
HITCBC 505   25810 bool lost_cancel = false; 505   24237 bool lost_cancel = false;
506   { 506   {
HITCBC 507   25810 std::lock_guard lock(mutex_); 507   24237 std::lock_guard lock(mutex_);
508   // Grow before publishing anything, so the push_back below 508   // Grow before publishing anything, so the push_back below
509   // cannot throw: a failure here leaves the waiter untouched, 509   // cannot throw: a failure here leaves the waiter untouched,
510   // the strong guarantee rearm_wait's recovery relies on. 510   // the strong guarantee rearm_wait's recovery relies on.
HITCBC 511   25810 if (impl.heap_index_.load(std::memory_order_relaxed) == 511   24237 if (impl.heap_index_.load(std::memory_order_relaxed) ==
HITCBC 512   51620 (std::numeric_limits<std::size_t>::max)() && 512   48474 (std::numeric_limits<std::size_t>::max)() &&
HITCBC 513   25810 heap_.size() == heap_.capacity()) 513   24237 heap_.size() == heap_.capacity())
HITCBC 514   958 heap_.reserve(heap_.capacity() == 0 ? 16 : 2 * heap_.capacity()); 514   945 heap_.reserve(heap_.capacity() == 0 ? 16 : 2 * heap_.capacity());
515   // Publish: from here the waiter is visible to the fire path and 515   // Publish: from here the waiter is visible to the fire path and
516   // to its own stop callback (impl_ non-null enables cancel_waiter). 516   // to its own stop callback (impl_ non-null enables cancel_waiter).
HITCBC 517   25810 w->impl_ = &impl; 517   24237 w->impl_ = &impl;
HITCBC 518   51620 if (impl.heap_index_.load(std::memory_order_relaxed) == 518   48474 if (impl.heap_index_.load(std::memory_order_relaxed) ==
HITCBC 519   25810 (std::numeric_limits<std::size_t>::max)()) 519   24237 (std::numeric_limits<std::size_t>::max)())
520   { 520   {
HITCBC 521   25810 impl.heap_index_.store(heap_.size(), std::memory_order_relaxed); 521   24237 impl.heap_index_.store(heap_.size(), std::memory_order_relaxed);
HITCBC 522   25810 heap_.push_back({impl.expiry_, &impl}); 522   24237 heap_.push_back({impl.expiry_, &impl});
HITCBC 523   25810 up_heap(heap_.size() - 1); 523   24237 up_heap(heap_.size() - 1);
HITCBC 524   25810 notify = (impl.heap_index_.load(std::memory_order_relaxed) == 0); 524   24237 notify = (impl.heap_index_.load(std::memory_order_relaxed) == 0);
HITCBC 525   25810 refresh_cached_nearest(); 525   24237 refresh_cached_nearest();
526   } 526   }
HITCBC 527   25810 BOOST_COROSIO_ASSERT(impl.waiter_ == nullptr); 527   24237 BOOST_COROSIO_ASSERT(impl.waiter_ == nullptr);
HITCBC 528   25810 impl.waiter_ = w; 528   24237 impl.waiter_ = w;
529   529  
530   // Lost-cancel re-check: a stop requested after the canceller was 530   // Lost-cancel re-check: a stop requested after the canceller was
531   // armed in wait() but before this publication found impl_ null 531   // armed in wait() but before this publication found impl_ null
532   // and returned a no-op. Observe it now and undo the insertion. 532   // and returned a no-op. Observe it now and undo the insertion.
HITCBC 533   25810 if (w->token_->stop_requested()) 533   24237 if (w->token_->stop_requested())
534   { 534   {
HITCBC 535   389 w->impl_ = nullptr; 535   326 w->impl_ = nullptr;
HITCBC 536   389 impl.waiter_ = nullptr; 536   326 impl.waiter_ = nullptr;
HITCBC 537   389 remove_timer_impl(impl); 537   326 remove_timer_impl(impl);
HITCBC 538   389 impl.might_have_pending_waits_.store( 538   326 impl.might_have_pending_waits_.store(
539   false, std::memory_order_release); 539   false, std::memory_order_release);
HITCBC 540   389 refresh_cached_nearest(); 540   326 refresh_cached_nearest();
HITCBC 541   389 lost_cancel = true; 541   326 lost_cancel = true;
HITCBC 542   389 notify = false; // insertion undone; nearest unchanged 542   326 notify = false; // insertion undone; nearest unchanged
543   } 543   }
HITCBC 544   25810 } 544   24237 }
HITCBC 545   25810 if (notify) 545   24237 if (notify)
HITCBC 546   8065 on_earliest_changed_(); 546   8045 on_earliest_changed_();
HITCBC 547   25810 if (lost_cancel) 547   24237 if (lost_cancel)
548   { 548   {
HITCBC 549   389 w->ec_ = make_error_code(capy::error::canceled); 549   326 w->ec_ = make_error_code(capy::error::canceled);
HITCBC 550   389 sched_->post(&w->op_); 550   326 sched_->post(&w->op_);
551   } 551   }
HITCBC 552   25810 } 552   24237 }
553   553  
554   inline void 554   inline void
HITCBC 555   17062 timer_service::cancel_timer(timer::implementation& impl) 555   16988 timer_service::cancel_timer(timer::implementation& impl)
556   { 556   {
557   // Acquire pairs with the drains' release: a caller that skips the 557   // Acquire pairs with the drains' release: a caller that skips the
558   // lock may recycle the impl next, so the drain's writes to it must 558   // lock may recycle the impl next, so the drain's writes to it must
559   // happen before. 559   // happen before.
HITCBC 560   17062 if (!impl.might_have_pending_waits_.load(std::memory_order_acquire)) 560   16988 if (!impl.might_have_pending_waits_.load(std::memory_order_acquire))
HITCBC 561   17060 return; 561   16986 return;
562   562  
563   // No unlocked already-done fast-out here: it would need the 563   // No unlocked already-done fast-out here: it would need the
564   // non-atomic waiter_ (a race with concurrent drains), and an 564   // non-atomic waiter_ (a race with concurrent drains), and an
565   // index-only check is lifetime-unsafe because npos is stored 565   // index-only check is lifetime-unsafe because npos is stored
566   // before the drain finishes touching the impl. A stale-true 566   // before the drain finishes touching the impl. A stale-true
567   // flag is rare with the stateless API; the locked path below 567   // flag is rare with the stateless API; the locked path below
568   // re-validates. 568   // re-validates.
569   569  
HITCBC 570   2 waiter_node* canceled = nullptr; 570   2 waiter_node* canceled = nullptr;
571   571  
572   { 572   {
HITCBC 573   2 std::lock_guard lock(mutex_); 573   2 std::lock_guard lock(mutex_);
HITCBC 574   2 remove_timer_impl(impl); 574   2 remove_timer_impl(impl);
HITCBC 575   2 canceled = std::exchange(impl.waiter_, nullptr); 575   2 canceled = std::exchange(impl.waiter_, nullptr);
HITCBC 576   2 if (canceled) 576   2 if (canceled)
HITCBC 577   2 canceled->impl_ = nullptr; 577   2 canceled->impl_ = nullptr;
578   // Store false as the final touch of the impl under the lock so 578   // Store false as the final touch of the impl under the lock so
579   // a pre-lock false-flag check trusts it unqualified. 579   // a pre-lock false-flag check trusts it unqualified.
HITCBC 580   2 impl.might_have_pending_waits_.store(false, std::memory_order_release); 580   2 impl.might_have_pending_waits_.store(false, std::memory_order_release);
HITCBC 581   2 refresh_cached_nearest(); 581   2 refresh_cached_nearest();
HITCBC 582   2 } 582   2 }
583   583  
HITCBC 584   2 if (canceled) 584   2 if (canceled)
585   { 585   {
HITCBC 586   2 canceled->ec_ = make_error_code(capy::error::canceled); 586   2 canceled->ec_ = make_error_code(capy::error::canceled);
HITCBC 587   2 sched_->post(&canceled->op_); 587   2 sched_->post(&canceled->op_);
588   } 588   }
589   } 589   }
590   590  
591   inline void 591   inline void
HITCBC 592   2901 timer_service::cancel_waiter(waiter_node* w) 592   2394 timer_service::cancel_waiter(waiter_node* w)
593   { 593   {
594   { 594   {
HITCBC 595   2901 std::lock_guard lock(mutex_); 595   2394 std::lock_guard lock(mutex_);
596   // Already removed by another drain: cancel_timer, 596   // Already removed by another drain: cancel_timer,
597   // process_expired, or insert_waiter's lost-cancel recheck 597   // process_expired, or insert_waiter's lost-cancel recheck
HITCBC 598   2901 if (!w->impl_) 598   2394 if (!w->impl_)
HITCBC 599   780 return; 599   459 return;
HITCBC 600   2121 auto* impl = w->impl_; 600   1935 auto* impl = w->impl_;
HITCBC 601   2121 w->impl_ = nullptr; 601   1935 w->impl_ = nullptr;
HITCBC 602   2121 impl->waiter_ = nullptr; 602   1935 impl->waiter_ = nullptr;
HITCBC 603   2121 remove_timer_impl(*impl); 603   1935 remove_timer_impl(*impl);
HITCBC 604   2121 impl->might_have_pending_waits_.store(false, std::memory_order_release); 604   1935 impl->might_have_pending_waits_.store(false, std::memory_order_release);
HITCBC 605   2121 refresh_cached_nearest(); 605   1935 refresh_cached_nearest();
HITCBC 606   2121 ++cancels_posting_; 606   1935 ++cancels_posting_;
HITCBC 607   2901 } 607   2394 }
608   608  
HITCBC 609   2121 w->ec_ = make_error_code(capy::error::canceled); 609   1935 w->ec_ = make_error_code(capy::error::canceled);
HITCBC 610   2121 sched_->post(&w->op_); 610   1935 sched_->post(&w->op_);
611   611  
HITCBC 612   2121 std::lock_guard lock(mutex_); 612   1935 std::lock_guard lock(mutex_);
HITCBC 613   2121 --cancels_posting_; 613   1935 --cancels_posting_;
HITCBC 614   2121 } 614   1935 }
615   615  
616   inline std::size_t 616   inline std::size_t
HITCBC 617   35680 timer_service::process_expired() 617   34743 timer_service::process_expired()
618   { 618   {
HITCBC 619   35680 intrusive_list<waiter_node> expired; 619   34743 intrusive_list<waiter_node> expired;
620   620  
621   { 621   {
HITCBC 622   35680 std::lock_guard lock(mutex_); 622   34743 std::lock_guard lock(mutex_);
HITCBC 623   35680 auto now = clock_type::now(); 623   34743 auto now = clock_type::now();
624   624  
HITCBC 625   55731 while (!heap_.empty() && heap_[0].time_ <= now) 625   53470 while (!heap_.empty() && heap_[0].time_ <= now)
626   { 626   {
HITCBC 627   20051 timer::implementation* t = heap_[0].timer_; 627   18727 timer::implementation* t = heap_[0].timer_;
HITCBC 628   20051 remove_timer_impl(*t); 628   18727 remove_timer_impl(*t);
HITCBC 629   20051 if (auto* w = std::exchange(t->waiter_, nullptr)) 629   18727 if (auto* w = std::exchange(t->waiter_, nullptr))
630   { 630   {
HITCBC 631   20051 w->impl_ = nullptr; 631   18727 w->impl_ = nullptr;
HITCBC 632   20051 w->ec_ = {}; 632   18727 w->ec_ = {};
HITCBC 633   20051 expired.push_back(w); 633   18727 expired.push_back(w);
634   } 634   }
HITCBC 635   20051 t->might_have_pending_waits_.store( 635   18727 t->might_have_pending_waits_.store(
636   false, std::memory_order_release); 636   false, std::memory_order_release);
637   } 637   }
638   638  
HITCBC 639   35680 refresh_cached_nearest(); 639   34743 refresh_cached_nearest();
HITCBC 640   35680 } 640   34743 }
641   641  
HITCBC 642   35680 std::size_t count = 0; 642   34743 std::size_t count = 0;
HITCBC 643   55731 while (auto* w = expired.pop_front()) 643   53470 while (auto* w = expired.pop_front())
644   { 644   {
HITCBC 645   20051 sched_->post(&w->op_); 645   18727 sched_->post(&w->op_);
HITCBC 646   20051 ++count; 646   18727 ++count;
HITCBC 647   20051 } 647   18727 }
648   648  
HITCBC 649   35680 return count; 649   34743 return count;
650   } 650   }
651   651  
652   inline void 652   inline void
HITCBC 653   22563 timer_service::remove_timer_impl(timer::implementation& impl) 653   20990 timer_service::remove_timer_impl(timer::implementation& impl)
654   { 654   {
HITCBC 655   22563 std::size_t index = impl.heap_index_.load(std::memory_order_relaxed); 655   20990 std::size_t index = impl.heap_index_.load(std::memory_order_relaxed);
HITCBC 656   22563 if (index >= heap_.size()) 656   20990 if (index >= heap_.size())
MISUBC 657   ✗ return; // Not in heap 657   ✗ return; // Not in heap
658   658  
HITCBC 659   22563 if (index == heap_.size() - 1) 659   20990 if (index == heap_.size() - 1)
660   { 660   {
661   // Last element, just pop 661   // Last element, just pop
HITCBC 662   3360 impl.heap_index_.store( 662   3249 impl.heap_index_.store(
663   (std::numeric_limits<std::size_t>::max)(), 663   (std::numeric_limits<std::size_t>::max)(),
664   std::memory_order_relaxed); 664   std::memory_order_relaxed);
HITCBC 665   3360 heap_.pop_back(); 665   3249 heap_.pop_back();
666   } 666   }
667   else 667   else
668   { 668   {
669   // Swap with last and reheapify 669   // Swap with last and reheapify
HITCBC 670   19203 swap_heap(index, heap_.size() - 1); 670   17741 swap_heap(index, heap_.size() - 1);
HITCBC 671   19203 impl.heap_index_.store( 671   17741 impl.heap_index_.store(
672   (std::numeric_limits<std::size_t>::max)(), 672   (std::numeric_limits<std::size_t>::max)(),
673   std::memory_order_relaxed); 673   std::memory_order_relaxed);
HITCBC 674   19203 heap_.pop_back(); 674   17741 heap_.pop_back();
675   675  
HITCBC 676   19203 if (index > 0 && heap_[index].time_ < heap_[(index - 1) / 2].time_) 676   17741 if (index > 0 && heap_[index].time_ < heap_[(index - 1) / 2].time_)
HITCBC 677   2 up_heap(index); 677   1 up_heap(index);
678   else 678   else
HITCBC 679   19201 down_heap(index); 679   17740 down_heap(index);
680   } 680   }
681   } 681   }
682   682  
683   inline void 683   inline void
HITCBC 684   25812 timer_service::up_heap(std::size_t index) 684   24238 timer_service::up_heap(std::size_t index)
685   { 685   {
HITCBC 686   32799 while (index > 0) 686   30834 while (index > 0)
687   { 687   {
HITCBC 688   24438 std::size_t parent = (index - 1) / 2; 688   22582 std::size_t parent = (index - 1) / 2;
HITCBC 689   24438 if (!(heap_[index].time_ < heap_[parent].time_)) 689   22582 if (!(heap_[index].time_ < heap_[parent].time_))
HITCBC 690   17451 break; 690   15986 break;
HITCBC 691   6987 swap_heap(index, parent); 691   6596 swap_heap(index, parent);
HITCBC 692   6987 index = parent; 692   6596 index = parent;
693   } 693   }
HITCBC 694   25812 } 694   24238 }
695   695  
696   inline void 696   inline void
HITCBC 697   19201 timer_service::down_heap(std::size_t index) 697   17740 timer_service::down_heap(std::size_t index)
698   { 698   {
HITCBC 699   19201 std::size_t child = index * 2 + 1; 699   17740 std::size_t child = index * 2 + 1;
HITCBC 700   44033 while (child < heap_.size()) 700   39850 while (child < heap_.size())
701   { 701   {
HITCBC 702   26899 std::size_t min_child = (child + 1 == heap_.size() || 702   24065 std::size_t min_child = (child + 1 == heap_.size() ||
HITCBC 703   23408 heap_[child].time_ < heap_[child + 1].time_) 703   20548 heap_[child].time_ < heap_[child + 1].time_)
HITCBC 704   50307 ? child 704   44613 ? child
HITCBC 705   26899 : child + 1; 705   24065 : child + 1;
706   706  
HITCBC 707   26899 if (heap_[index].time_ < heap_[min_child].time_) 707   24065 if (heap_[index].time_ < heap_[min_child].time_)
HITCBC 708   2067 break; 708   1955 break;
709   709  
HITCBC 710   24832 swap_heap(index, min_child); 710   22110 swap_heap(index, min_child);
HITCBC 711   24832 index = min_child; 711   22110 index = min_child;
HITCBC 712   24832 child = index * 2 + 1; 712   22110 child = index * 2 + 1;
713   } 713   }
HITCBC 714   19201 } 714   17740 }
715   715  
716   inline void 716   inline void
HITCBC 717   51022 timer_service::swap_heap(std::size_t i1, std::size_t i2) 717   46447 timer_service::swap_heap(std::size_t i1, std::size_t i2)
718   { 718   {
HITCBC 719   51022 heap_entry tmp = heap_[i1]; 719   46447 heap_entry tmp = heap_[i1];
HITCBC 720   51022 heap_[i1] = heap_[i2]; 720   46447 heap_[i1] = heap_[i2];
HITCBC 721   51022 heap_[i2] = tmp; 721   46447 heap_[i2] = tmp;
HITCBC 722   51022 heap_[i1].timer_->heap_index_.store(i1, std::memory_order_relaxed); 722   46447 heap_[i1].timer_->heap_index_.store(i1, std::memory_order_relaxed);
HITCBC 723   51022 heap_[i2].timer_->heap_index_.store(i2, std::memory_order_relaxed); 723   46447 heap_[i2].timer_->heap_index_.store(i2, std::memory_order_relaxed);
HITCBC 724   51022 } 724   46447 }
725   725  
726   // waiter_node's completion_op and canceller members are defined in 726   // waiter_node's completion_op and canceller members are defined in
727   // timer.cpp alongside implementation::wait(), for the same reason 727   // timer.cpp alongside implementation::wait(), for the same reason
728   // wait() lives there (see below). 728   // wait() lives there (see below).
729   729  
730   // timer::implementation::wait() is defined in timer.cpp, not here. 730   // timer::implementation::wait() is defined in timer.cpp, not here.
731   // It must be a non-inline definition in a translation unit that is 731   // It must be a non-inline definition in a translation unit that is
732   // always pulled into the link whenever detail::timer is used (every 732   // always pulled into the link whenever detail::timer is used (every
733   // consumer needs timer's constructors from that same object file). 733   // consumer needs timer's constructors from that same object file).
734   // An inline definition in this header would only be emitted in 734   // An inline definition in this header would only be emitted in
735   // translation units that happen to also include this header, which 735   // translation units that happen to also include this header, which
736   // is not guaranteed for every caller of wait_awaitable::await_suspend 736   // is not guaranteed for every caller of wait_awaitable::await_suspend
737   // in timer.hpp (e.g. code that only reaches timer.hpp through 737   // in timer.hpp (e.g. code that only reaches timer.hpp through
738   // delay.hpp, without transitively including a scheduler header). 738   // delay.hpp, without transitively including a scheduler header).
739   739  
740   } // namespace boost::corosio::detail 740   } // namespace boost::corosio::detail
741   741  
742   #endif 742   #endif