Skip to content

bump isoalloc version to 1.2.7#257

Closed
struct wants to merge 2 commits intodaanx:masterfrom
struct:bump_isoalloc_ver_1.2.6
Closed

bump isoalloc version to 1.2.7#257
struct wants to merge 2 commits intodaanx:masterfrom
struct:bump_isoalloc_ver_1.2.6

Conversation

@struct
Copy link

@struct struct commented Mar 14, 2026

Bumps the current IsoAlloc version to 1.2.6 which includes significant performance improvements since late 2022

https://github.com/struct/isoalloc/releases/tag/1.2.6

Copy link
Collaborator

@mjp41 mjp41 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mjp41
Copy link
Collaborator

mjp41 commented Mar 14, 2026

@struct Looks like rocksdb is failing with an abort:

https://github.com/daanx/mimalloc-bench/actions/runs/23089629339/job/67075137496?pr=257#step:4:563

@struct
Copy link
Author

struct commented Mar 14, 2026

It's because IsoAlloc in newer versions abort's by default when a pointer is passed in that wasn't allocated by it.

https://github.com/struct/isoalloc/blob/1.2.6/src/iso_alloc.c#L1727

Likely something allocated by system malloc early in a RocksDB constructor. Commenting the IsoAlloc abort will successfully run the benchmark.

@struct
Copy link
Author

struct commented Mar 14, 2026

I've got some cleanups to my linker hooking, will likely require a new version which I will bump here once its landed.

@struct struct changed the title bump isoalloc version to 1.2.6 bump isoalloc version to 1.2.7 Mar 14, 2026
@struct
Copy link
Author

struct commented Mar 14, 2026

Going to revisit this after some new performance tweaks.

@struct struct closed this Mar 14, 2026
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