Skip to content

kp: bump version_kp to v1.0.2 -> v1.1.0 - #270

Merged
mjp41 merged 1 commit into
daanx:masterfrom
northriv:kp-pin-v1.1.0
Sep 2, 2026
Merged

kp: bump version_kp to v1.0.2 -> v1.1.0#270
mjp41 merged 1 commit into
daanx:masterfrom
northriv:kp-pin-v1.1.0

Conversation

@northriv

@northriv northriv commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Thanks for merging #265. A follow-up: v1.1.0 was released during #265's review and the pin was not moved before the merge. Tags at or below v1.0.2 can hand the same block to two live users on Linux (and Windows): the cold free path's teardown check folded to a constant false anywhere but macOS, so a thread past its own allocator cleanup pushed into an already-destroyed cross-dealloc batch. macOS was never exposed -- which is precisely why this suite's Linux CI is the matrix that matters here.
Reproducer: 17/44 failures before, 0/44 after -- ours, not in this suite (it needs a preloaded program whose pthread_key destructor frees; no bench here has one, which is also why CI is green on either pin). It ships in kamepoolalloc as tests/alloc_tsd_exclusivity_test.cpp if you want to look.

Build integration is unchanged: v1.1.0's CMakeLists.txt is identical to v1.0.2's and the same cmake -B out / cmake --build out produces out/libkamepoolalloc$extso in the same place, so no other hunk is needed.

CI on my fork: kp is green on fedora and alpine. The reds in that run are the ones #269 is addressing (je / hm / tcg / mesh / sn, and the ubuntu base container); ubuntu's kp job did not run because its base container failed first.

Thanks for merging daanx#265.  A follow-up: v1.1.0 was released during daanx#265's review and the pin was not moved before the merge.  Tags at or below v1.0.2 can hand the same block to two live users on Linux (and Windows): the cold free path's teardown check folded to a constant false anywhere but macOS, so a thread past its own allocator cleanup pushed into an already-destroyed cross-dealloc batch.  macOS was never exposed --
which is precisely why this suite's Linux CI is the matrix that matters here.
Reproducer: 17/44 failures before, 0/44 after -- ours, not in this suite (it needs a preloaded program whose pthread_key destructor frees; no bench here has one, which is also why CI is green on either pin).  It ships in kamepoolalloc as tests/alloc_tsd_exclusivity_test.cpp if you want to look.
@mjp41
mjp41 merged commit fcb165f into daanx:master Sep 2, 2026
42 of 49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants