4 Commits

Author SHA1 Message Date
anth64 5b252d2b4e docs: update README and CHANGELOG for v1.0.0-pre.6 2026-03-07 14:52:26 +01:00
anth64 08c846d641 perf: batch stk_pending_add calls in stk_poll load loops
Replace per-module stk_pending_add() calls in the load holes loop and
append_modules loop with a single shared load_batch[load_count] buffer,
flushed via stk_pending_add_batch() once after both loops complete.
2026-03-07 14:51:10 +01:00
anth64 6ca46fe79f docs: update README and CHANGELOG for v1.0.0-pre.5 2026-03-07 14:41:20 +01:00
anth64 ba3a9dd163 perf: batch stk_init deferred module pending adds 2026-03-07 14:35:21 +01:00
3 changed files with 59 additions and 12 deletions
+13 -1
View File
@@ -7,6 +7,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
## [Unreleased] ## [Unreleased]
## [1.0.0-pre.6] - 2026-03-07
### Changed
- `stk_poll()`: replaced per-module `stk_pending_add()` calls inside the load holes and `append_modules` loops with batch collection followed by a single `stk_pending_add_batch()` call after both loops complete.
## [1.0.0-pre.5] - 2026-03-07
### Changed
- `stk_init()`: replaced per-module `stk_pending_add()` calls inside the deferred module loop with batch collection followed by a single `stk_pending_add_batch()` call after the loop.
## [1.0.0-pre.4] - 2026-03-07 ## [1.0.0-pre.4] - 2026-03-07
### Changed ### Changed
@@ -168,7 +178,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Dependency management and versioning not yet implemented - Dependency management and versioning not yet implemented
- API is unstable and subject to change in future releases - API is unstable and subject to change in future releases
[Unreleased]: https://github.com/anth64/stk/compare/v1.0.0-pre.4...HEAD [Unreleased]: https://github.com/anth64/stk/compare/v1.0.0-pre.6...HEAD
[1.0.0-pre.6]: https://github.com/anth64/stk/compare/v1.0.0-pre.5...v1.0.0-pre.6
[1.0.0-pre.5]: https://github.com/anth64/stk/compare/v1.0.0-pre.4...v1.0.0-pre.5
[1.0.0-pre.4]: https://github.com/anth64/stk/compare/v1.0.0-pre.3...v1.0.0-pre.4 [1.0.0-pre.4]: https://github.com/anth64/stk/compare/v1.0.0-pre.3...v1.0.0-pre.4
[1.0.0-pre.3]: https://github.com/anth64/stk/compare/v1.0.0-pre.2...v1.0.0-pre.3 [1.0.0-pre.3]: https://github.com/anth64/stk/compare/v1.0.0-pre.2...v1.0.0-pre.3
[1.0.0-pre.2]: https://github.com/anth64/stk/compare/v1.0.0-pre.1...v1.0.0-pre.2 [1.0.0-pre.2]: https://github.com/anth64/stk/compare/v1.0.0-pre.1...v1.0.0-pre.2
+1 -1
View File
@@ -234,7 +234,7 @@ stk_init();
## Project Status ## Project Status
**Current Version:** 1.0.0-pre.4 **Current Version:** 1.0.0-pre.6
### What Works ### What Works
- Cross-platform module loading and hot-reloading - Cross-platform module loading and hot-reloading
+45 -10
View File
@@ -136,10 +136,11 @@ unsigned char stk_init(void)
size_t index, test_count; size_t index, test_count;
char full_path[STK_PATH_MAX_OS]; char full_path[STK_PATH_MAX_OS];
char tmp_path[STK_PATH_MAX_OS]; char tmp_path[STK_PATH_MAX_OS];
char mod_tmp_path[STK_PATH_MAX_OS];
int load_result; int load_result;
unsigned char dep_result; unsigned char dep_result;
size_t *order = NULL; size_t *order = NULL;
char (*init_batch)[STK_PATH_MAX_OS] = NULL;
size_t init_batch_count = 0;
platform_mkdir(stk_mod_dir); platform_mkdir(stk_mod_dir);
build_path(stk_tmp_dir, sizeof(stk_tmp_dir), stk_mod_dir, stk_tmp_name); build_path(stk_tmp_dir, sizeof(stk_tmp_dir), stk_mod_dir, stk_tmp_name);
@@ -206,17 +207,25 @@ unsigned char stk_init(void)
stk_error_string(dep_result)); stk_error_string(dep_result));
} }
init_batch = malloc(module_count * sizeof(*init_batch));
for (j = 0; j < module_count; j++) { for (j = 0; j < module_count; j++) {
index = order ? order[j] : j; index = order ? order[j] : j;
dep_result = stk_validate_dependencies_single(index); dep_result = stk_validate_dependencies_single(index);
if (dep_result != STK_MOD_INIT_SUCCESS) { if (dep_result != STK_MOD_INIT_SUCCESS) {
stk_log_dependency_failures(index, "Deferring"); stk_log_dependency_failures(index, "Deferring");
build_path(mod_tmp_path, sizeof(mod_tmp_path), if (init_batch) {
stk_tmp_dir, stk_modules[index].id); build_path(init_batch[init_batch_count],
strncat(mod_tmp_path, STK_MODULE_EXT, sizeof(init_batch[init_batch_count]),
sizeof(mod_tmp_path) - strlen(mod_tmp_path) - stk_tmp_dir, stk_modules[index].id);
1); strncat(
stk_pending_add(mod_tmp_path); init_batch[init_batch_count],
STK_MODULE_EXT,
sizeof(init_batch[init_batch_count]) -
strlen(init_batch[init_batch_count]) -
1);
init_batch_count++;
}
stk_module_discard(index); stk_module_discard(index);
continue; continue;
} }
@@ -227,6 +236,14 @@ unsigned char stk_init(void)
} }
} }
if (init_batch_count > 0)
stk_pending_add_batch(
(const char (*)[STK_PATH_MAX_OS])init_batch,
init_batch_count);
free(init_batch);
init_batch = NULL;
if (order) { if (order) {
free(order); free(order);
order = NULL; order = NULL;
@@ -314,6 +331,8 @@ size_t stk_poll(void)
size_t dep_batch_count = 0; size_t dep_batch_count = 0;
char (*cascade_batch)[STK_PATH_MAX_OS] = NULL; char (*cascade_batch)[STK_PATH_MAX_OS] = NULL;
size_t cascade_batch_count = 0; size_t cascade_batch_count = 0;
char (*load_batch)[STK_PATH_MAX_OS] = NULL;
size_t load_batch_count = 0;
if (module_count > 0) { if (module_count > 0) {
module_ids = malloc(module_count * sizeof(*module_ids)); module_ids = malloc(module_count * sizeof(*module_ids));
@@ -509,6 +528,9 @@ begin_operations:
stk_sort_load_order(loaded_mod_indices, load_count, file_list, stk_sort_load_order(loaded_mod_indices, load_count, file_list,
stk_tmp_dir); stk_tmp_dir);
load_batch = malloc(load_count * sizeof(*load_batch));
load_batch_count = 0;
for (i = 0; i < holes_to_fill; ++i) { for (i = 0; i < holes_to_fill; ++i) {
target_index = unloaded_mod_indices[i]; target_index = unloaded_mod_indices[i];
file_index = loaded_mod_indices[i]; file_index = loaded_mod_indices[i];
@@ -519,7 +541,9 @@ begin_operations:
load_result = stk_module_load(tmp_path, target_index); load_result = stk_module_load(tmp_path, target_index);
if (load_result == STK_MOD_DEP_NOT_FOUND_ERROR || if (load_result == STK_MOD_DEP_NOT_FOUND_ERROR ||
load_result == STK_MOD_DEP_VERSION_MISMATCH_ERROR) { load_result == STK_MOD_DEP_VERSION_MISMATCH_ERROR) {
stk_pending_add(tmp_path); if (load_batch)
memcpy(load_batch[load_batch_count++], tmp_path,
STK_PATH_MAX_OS);
} else if (load_result != STK_MOD_INIT_SUCCESS) { } else if (load_result != STK_MOD_INIT_SUCCESS) {
stk_log(STK_LOG_ERROR, "Failed to load module %s: %s", stk_log(STK_LOG_ERROR, "Failed to load module %s: %s",
file_list[file_index], file_list[file_index],
@@ -532,7 +556,7 @@ begin_operations:
if (load_count > unload_count) if (load_count > unload_count)
goto append_modules; goto append_modules;
goto validate_deps; goto finish_loads;
append_modules: append_modules:
for (; i < load_count; ++i) { for (; i < load_count; ++i) {
@@ -545,7 +569,9 @@ append_modules:
successful_appends); successful_appends);
if (load_result == STK_MOD_DEP_NOT_FOUND_ERROR || if (load_result == STK_MOD_DEP_NOT_FOUND_ERROR ||
load_result == STK_MOD_DEP_VERSION_MISMATCH_ERROR) { load_result == STK_MOD_DEP_VERSION_MISMATCH_ERROR) {
stk_pending_add(tmp_path); if (load_batch)
memcpy(load_batch[load_batch_count++], tmp_path,
STK_PATH_MAX_OS);
} else if (load_result != STK_MOD_INIT_SUCCESS) { } else if (load_result != STK_MOD_INIT_SUCCESS) {
stk_log(STK_LOG_ERROR, "Failed to load module %s: %s", stk_log(STK_LOG_ERROR, "Failed to load module %s: %s",
file_list[file_index], file_list[file_index],
@@ -560,6 +586,15 @@ append_modules:
if (successful_appends < (load_count - holes_to_fill)) if (successful_appends < (load_count - holes_to_fill))
stk_module_realloc_memory(module_count); stk_module_realloc_memory(module_count);
finish_loads:
if (load_batch_count > 0)
stk_pending_add_batch(
(const char (*)[STK_PATH_MAX_OS])load_batch,
load_batch_count);
free(load_batch);
load_batch = NULL;
goto validate_deps; goto validate_deps;
validate_deps: validate_deps: