29185833209

Committed 12 Jul 2026 08:25AM UTC coverage: 51.235% (-1.5%) from 52.754%

Build # 29185833209

Build Type

Pull #96

github

Committed by

web-flow

Commit Message

Merge 0b9f07110 into 056c79287

Pull Request Pull Request #96: fix: consolidate stale-index rebuilds across consumers and fix multi-run breaks

Coverage Stats

33724 of 84486 branches covered (39.92%)

Branch coverage included in aggregate %.

140 of 147 new or added lines in 15 files covered. (95.24%)

5188 existing lines in 197 files now uncovered.

34547 of 48765 relevant lines covered (70.84%)

10791.85 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

60.62

/src/dftracer/utils/core/runtime.cpp

#include <dftracer/utils/core/common/error.h>
#include <dftracer/utils/core/common/logging.h>
#include <dftracer/utils/core/common/platform_compat.h>
#include <dftracer/utils/core/env.h>
#include <dftracer/utils/core/runtime.h>

#include <algorithm>
#include <cstdlib>
#include <stdexcept>
#include <string>
#include <string_view>
#include <thread>

namespace dftracer::utils {

namespace {
// Resolve the worker-thread count. DFTRACER_UTILS_THREADS overrides everything
// (a debugging lever: force a fixed count, e.g. 1 for a single-threaded async
// loop). Otherwise 0 means hardware_concurrency.
std::size_t resolve_threads(std::size_t requested) {
    if (auto env = Env::get<std::string_view>("DFTRACER_UTILS_THREADS");
        env.has_value()) {
        const long long n =
            std::strtoll(std::string(*env).c_str(), nullptr, 10);
        if (n > 0) return static_cast<std::size_t>(n);
    }
    return requested == 0 ? dftracer_utils_hardware_concurrency() : requested;
}
}  // namespace

Runtime::Runtime(std::size_t threads) : threads_(resolve_threads(threads)) {
    ExecutorConfig config;
    config.num_threads = threads_;
    executor_ = std::make_unique<Executor>(config);
    executor_->start();

    watchdog_ = std::make_unique<Watchdog>();
    watchdog_->set_executor(executor_.get());
}

Runtime::Runtime(const ExecutorConfig& config, bool enable_watchdog)
    : threads_(resolve_threads(config.num_threads)) {
    ExecutorConfig cfg = config;
    cfg.num_threads = threads_;
    executor_ = std::make_unique<Executor>(cfg);
    executor_->start();

    if (enable_watchdog) {
        watchdog_ = std::make_unique<Watchdog>();
        watchdog_->set_executor(executor_.get());
    }
}

Runtime::Runtime(const ExecutorConfig& config,
                 std::unique_ptr<Watchdog> watchdog)
    : threads_(resolve_threads(config.num_threads)) {
    ExecutorConfig cfg = config;
    cfg.num_threads = threads_;
    executor_ = std::make_unique<Executor>(cfg);
    executor_->start();

    watchdog_ = std::move(watchdog);
    if (watchdog_) {
        watchdog_->set_executor(executor_.get());
    }
}

Runtime::~Runtime() { shutdown(); }

TaskHandle Runtime::submit(coro::CoroTask<void> task, std::string name) {
    if (shutdown_called_.load(std::memory_order_acquire)) {
        throw DFTUtilsException(ErrorCode::PIPELINE, "Runtime is shut down");
    }
    if (name.empty()) {
        name = "task-" + std::to_string(task_name_counter_++);
    }

    auto promise = std::make_shared<std::promise<void>>();
    auto future = promise->get_future().share();
    auto tid = std::make_shared<std::atomic<TaskIndex>>(-1);

    auto wrapper =
        [](coro::CoroTask<void> t, std::shared_ptr<std::promise<void>> p,
           Executor* exec,
           std::shared_ptr<std::atomic<TaskIndex>> task_id) -> coro::Coro {
        try {
            co_await std::move(t);
            t = coro::CoroTask<void>{
                std::coroutine_handle<coro::CoroTask<void>::promise_type>{}};
            exec->mark_coro_completed(task_id->load(std::memory_order_acquire));
        } catch (...) {
            t = coro::CoroTask<void>{
                std::coroutine_handle<coro::CoroTask<void>::promise_type>{}};
            exec->mark_coro_completed(task_id->load(std::memory_order_acquire));
            p->set_exception(std::current_exception());
            co_return;
        }
        p->set_value();
    };

    // Set the executor on the task's promise so awaitables (e.g. channels)
    // that capture `get_root_promise()->get_executor()` can schedule
    // resumption. Without this, awaiters end up with executor=nullptr because
    // the wrapping `coro::Coro` doesn't extend PromiseBase and the
    // root-promise chain stops at the user's CoroTask.
    if (task.handle()) {
        task.handle().promise().set_executor(executor_.get());
    }
    auto coro = wrapper(std::move(task), promise, executor_.get(), tid);
    TaskIndex id = executor_->enqueue_tracked(std::move(coro), name, tid);

    {
        std::lock_guard<std::mutex> lock(futures_mutex_);
        cleanup_completed_futures();
        outstanding_futures_.push_back(future);
    }

    return TaskHandle{future, id, std::move(name)};
}

void Runtime::wait_all() {
    std::vector<std::shared_future<void>> futures;
    {
        std::lock_guard<std::mutex> lock(futures_mutex_);
        futures = std::move(outstanding_futures_);
        outstanding_futures_.clear();
    }
    for (auto& f : futures) {
        f.wait();
    }
}

void Runtime::cleanup_completed_futures() {
    outstanding_futures_.erase(
        std::remove_if(outstanding_futures_.begin(), outstanding_futures_.end(),
                       [](const std::shared_future<void>& f) {
                           return f.wait_for(std::chrono::seconds(0)) ==
                                  std::future_status::ready;
                       }),
        outstanding_futures_.end());
}

ExecutorProgress Runtime::get_progress() const {
    return executor_->get_progress();
}

bool Runtime::is_responsive() const { return executor_->is_responsive(); }

void Runtime::set_global_timeout(std::chrono::milliseconds timeout) {
    if (!watchdog_) {
        throw DFTUtilsException(
            ErrorCode::PIPELINE,
            "Cannot set timeout: Runtime created without watchdog");
    }
    watchdog_->set_global_timeout(timeout);
}

void Runtime::set_default_task_timeout(std::chrono::milliseconds timeout) {
    if (!watchdog_) {
        throw DFTUtilsException(
            ErrorCode::PIPELINE,
            "Cannot set timeout: Runtime created without watchdog");
    }
    watchdog_->set_default_task_timeout(timeout);
}

void Runtime::shutdown() {
    bool expected = false;
    if (!shutdown_called_.compare_exchange_strong(expected, true)) return;
    if (watchdog_) watchdog_->stop();
    if (executor_) executor_->shutdown();
}

std::size_t Runtime::threads() const { return threads_; }

std::size_t Runtime::io_threads() const {
    return executor_ ? executor_->get_io_pool_size() : 0;
}

}  // namespace dftracer::utils

1	#include <dftracer/utils/core/common/error.h>
2	#include <dftracer/utils/core/common/logging.h>
3	#include <dftracer/utils/core/common/platform_compat.h>
4	#include <dftracer/utils/core/env.h>
5	#include <dftracer/utils/core/runtime.h>
6
7	#include <algorithm>
8	#include <cstdlib>
9	#include <stdexcept>
10	#include <string>
11	#include <string_view>
12	#include <thread>
13
14	namespace dftracer::utils {
15
16	namespace {
17	// Resolve the worker-thread count. DFTRACER_UTILS_THREADS overrides everything
18	// (a debugging lever: force a fixed count, e.g. 1 for a single-threaded async
19	// loop). Otherwise 0 means hardware_concurrency.
20	std::size_t resolve_threads(std::size_t requested) {	459✔
21	if (auto env = Env::get<std::string_view>("DFTRACER_UTILS_THREADS");	918!
22	env.has_value()) {	459✔
UNCOV 23	const long long n =	×
24	std::strtoll(std::string(*env).c_str(), nullptr, 10);	×
25	if (n > 0) return static_cast<std::size_t>(n);	×
UNCOV 26	}	×
27	return requested == 0 ? dftracer_utils_hardware_concurrency() : requested;	459✔
28	}	459✔
29	} // namespace
30
31	Runtime::Runtime(std::size_t threads) : threads_(resolve_threads(threads)) {	292!
32	ExecutorConfig config;	146!
33	config.num_threads = threads_;	146✔
34	executor_ = std::make_unique<Executor>(config);	146!
35	executor_->start();	146!
36
37	watchdog_ = std::make_unique<Watchdog>();	146!
38	watchdog_->set_executor(executor_.get());	146!
39	}	292✔
40
41	Runtime::Runtime(const ExecutorConfig& config, bool enable_watchdog)	156✔
42	: threads_(resolve_threads(config.num_threads)) {	156!
43	ExecutorConfig cfg = config;	78✔
44	cfg.num_threads = threads_;	78✔
45	executor_ = std::make_unique<Executor>(cfg);	78!
46	executor_->start();	78!
47
48	if (enable_watchdog) {	78!
49	watchdog_ = std::make_unique<Watchdog>();	78!
50	watchdog_->set_executor(executor_.get());	78!
51	}	78✔
52	}	156✔
53
54	Runtime::Runtime(const ExecutorConfig& config,	470✔
55	std::unique_ptr<Watchdog> watchdog)
56	: threads_(resolve_threads(config.num_threads)) {	470!
57	ExecutorConfig cfg = config;	235✔
58	cfg.num_threads = threads_;	235✔
59	executor_ = std::make_unique<Executor>(cfg);	235!
60	executor_->start();	235!
61
62	watchdog_ = std::move(watchdog);	235✔
63	if (watchdog_) {	235✔
64	watchdog_->set_executor(executor_.get());	81!
65	}	81✔
66	}	470✔
67
68	Runtime::~Runtime() { shutdown(); }	918!
69
70	TaskHandle Runtime::submit(coro::CoroTask<void> task, std::string name) {	522✔
71	if (shutdown_called_.load(std::memory_order_acquire)) {	522✔
72	throw DFTUtilsException(ErrorCode::PIPELINE, "Runtime is shut down");	2!
73	}
74	if (name.empty()) {	520✔
75	name = "task-" + std::to_string(task_name_counter_++);	3!
76	}	3✔
77
78	auto promise = std::make_shared<std::promise<void>>();	520✔
79	auto future = promise->get_future().share();	520!
80	auto tid = std::make_shared<std::atomic<TaskIndex>>(-1);	520!
81
82	auto wrapper =	520✔
83	[](coro::CoroTask<void> t, std::shared_ptr<std::promise<void>> p,	3,634!
84	Executor* exec,
85	std::shared_ptr<std::atomic<TaskIndex>> task_id) -> coro::Coro {	520!
86	try {
87	co_await std::move(t);	2,594!
88	t = coro::CoroTask<void>{	1,034!
89	std::coroutine_handle<coro::CoroTask<void>::promise_type>{}};	517✔
90	exec->mark_coro_completed(task_id->load(std::memory_order_acquire));	517!
91	} catch (...) {	520✔
92	t = coro::CoroTask<void>{	6!
93	std::coroutine_handle<coro::CoroTask<void>::promise_type>{}};	3✔
94	exec->mark_coro_completed(task_id->load(std::memory_order_acquire));	3!
95	p->set_exception(std::current_exception());	3!
96	co_return;	3✔
97	}	3!
98	p->set_value();	517!
99	};	1,037✔
100
101	// Set the executor on the task's promise so awaitables (e.g. channels)
102	// that capture `get_root_promise()->get_executor()` can schedule
103	// resumption. Without this, awaiters end up with executor=nullptr because
104	// the wrapping `coro::Coro` doesn't extend PromiseBase and the
105	// root-promise chain stops at the user's CoroTask.
106	if (task.handle()) {	520!
107	task.handle().promise().set_executor(executor_.get());	520!
108	}	520✔
109	auto coro = wrapper(std::move(task), promise, executor_.get(), tid);	520!
110	TaskIndex id = executor_->enqueue_tracked(std::move(coro), name, tid);	520!
111
112	{
113	std::lock_guard<std::mutex> lock(futures_mutex_);	520!
114	cleanup_completed_futures();	520!
115	outstanding_futures_.push_back(future);	520!
116	}	520✔
117
118	return TaskHandle{future, id, std::move(name)};	520!
119	}	522✔
120
121	void Runtime::wait_all() {	100✔
122	std::vector<std::shared_future<void>> futures;	100✔
123	{
124	std::lock_guard<std::mutex> lock(futures_mutex_);	100!
125	futures = std::move(outstanding_futures_);	100✔
126	outstanding_futures_.clear();	100✔
127	}	100✔
128	for (auto& f : futures) {	139✔
129	f.wait();	39!
130	}
131	}	100✔
132
133	void Runtime::cleanup_completed_futures() {	606✔
134	outstanding_futures_.erase(	1,212✔
135	std::remove_if(outstanding_futures_.begin(), outstanding_futures_.end(),	606✔
136	[](const std::shared_future<void>& f) {	469✔
137	return f.wait_for(std::chrono::seconds(0)) ==	469✔
138	std::future_status::ready;
139	}),
140	outstanding_futures_.end());	606✔
141	}	606✔
142
143	ExecutorProgress Runtime::get_progress() const {	19✔
144	return executor_->get_progress();	19✔
145	}
146
147	bool Runtime::is_responsive() const { return executor_->is_responsive(); }	2✔
148
149	void Runtime::set_global_timeout(std::chrono::milliseconds timeout) {	×
150	if (!watchdog_) {	×
151	throw DFTUtilsException(	×
152	ErrorCode::PIPELINE,
153	"Cannot set timeout: Runtime created without watchdog");	×
154	}
155	watchdog_->set_global_timeout(timeout);	×
156	}	×
157
158	void Runtime::set_default_task_timeout(std::chrono::milliseconds timeout) {	×
159	if (!watchdog_) {	×
160	throw DFTUtilsException(	×
161	ErrorCode::PIPELINE,
162	"Cannot set timeout: Runtime created without watchdog");	×
163	}
164	watchdog_->set_default_task_timeout(timeout);	×
165	}	×
166
167	void Runtime::shutdown() {	869✔
168	bool expected = false;	869✔
169	if (!shutdown_called_.compare_exchange_strong(expected, true)) return;	869✔
170	if (watchdog_) watchdog_->stop();	459✔
171	if (executor_) executor_->shutdown();	459!
172	}	869✔
173
174	std::size_t Runtime::threads() const { return threads_; }	156✔
175
176	std::size_t Runtime::io_threads() const {	×
177	return executor_ ? executor_->get_io_pool_size() : 0;	×
178	}
179
180	} // namespace dftracer::utils

llnl / dftracer-utils / 29185833209

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous