make more readable

This commit is contained in:
J. Nick Koston 2025-07-19 10:40:21 -10:00
parent 5ed589fc97
commit acbcc5f9b8
No known key found for this signature in database

View File

@ -589,11 +589,12 @@ uint64_t Scheduler::millis_64_(uint32_t now) {
#ifdef ESPHOME_MULTI_CORE_ATOMICS #ifdef ESPHOME_MULTI_CORE_ATOMICS
// This is the multi core with atomics implementation. // This is the multi core with atomics implementation.
// //
// The implementation handles the 32-bit rollover (every 49.7 days) by: // Uses atomic operations with acquire/release semantics to ensure coherent
// 1. Using a lock when detecting rollover to ensure atomic update // reads of millis_major_ and last_millis_ across cores. Features:
// 2. Restricting normal updates to forward movement within the same epoch // 1. Epoch-coherency retry loop to handle concurrent updates
// This prevents race conditions at the rollover boundary without requiring // 2. Lock only taken for actual rollover detection and update
// 64-bit atomics or locking on every call. // 3. Lock-free CAS updates for normal forward time progression
// 4. Memory ordering ensures cores see consistent time values
for (;;) { for (;;) {
uint16_t major = this->millis_major_.load(std::memory_order_acquire); uint16_t major = this->millis_major_.load(std::memory_order_acquire);