fix incorrect alignment assumption in arena

This commit is contained in:
Ryan Fleury
2024-10-10 15:11:22 -07:00
parent 01eb655d19
commit 79b2bcf37a
2 changed files with 25 additions and 27 deletions
+2 -1
View File
@@ -105,7 +105,8 @@ arena_push(Arena *arena, U64 size, U64 align)
// rjf: commit new pages, if needed
if(current->cmt < pos_pst && !(current->flags & ArenaFlag_LargePages))
{
U64 cmt_pst_aligned = AlignPow2(pos_pst, current->cmt_size);
U64 cmt_pst_aligned = pos_pst + current->cmt_size-1;
cmt_pst_aligned -= cmt_pst_aligned%current->cmt_size;
U64 cmt_pst_clamped = ClampTop(cmt_pst_aligned, current->res);
U64 cmt_size = cmt_pst_clamped - current->cmt;
os_commit((U8 *)current + current->cmt, cmt_size);
+23 -26
View File
@@ -7,7 +7,6 @@
// [ ] remainder of @msgs pass:
// [ ] entity listers - kill-specific-process, etc.
// [ ] new universal ctx menu, hover, tooltips systems
// [ ] fix memory view
// [ ] `switch` replacement (recent files history)
// [ ] resolving name as file or #include
// [ ] new `restart processes` path
@@ -27,31 +26,6 @@
// because you're like "wait why did it stop" and then you later remember
// that's because there was a function breakpoint there.
// [ ] debugger readme pass
//
//
//
// [x] codebase readme pass
// [x] target editor
// [x] modules view
// [x] eval writing/committing
// [x] breakpoint hit count resetting
// [x] reset bp hit counts - not just on NewProcess, but on RUN! because
// we are now evaluating them on the control thread...
// [x] fix bug where text info is evicted, and switching back to a tab scrolls
// to the top
// [x] reset bp hit counts - not just on NewProcess, but on RUN! because
// we are now evaluating them on the control thread...
// [x] fix bug where text info is evicted, and switching back to a tab scrolls
// to the top
// [x] targets view
// [x] ensure launch controls parameterize commands correctly)
// [x] ensure ctrl+click
// [x] scheduler view
// [x] eval committing
// [x] fix registers
// [x] file overrides -> always pick most specific one! found with conflicting
// overrides, e.g. C:/devel/ -> D:/devel/, but also C:/devel/foo ->
// C:/devel/bar, etc.
////////////////////////////////
//~ rjf: post-0.9.12 TODO notes
@@ -525,6 +499,29 @@
// [x] file path map editor
// [x] file path map building
// [x] meta eval system
// [x] codebase readme pass
// [x] target editor
// [x] modules view
// [x] eval writing/committing
// [x] breakpoint hit count resetting
// [x] reset bp hit counts - not just on NewProcess, but on RUN! because
// we are now evaluating them on the control thread...
// [x] fix bug where text info is evicted, and switching back to a tab scrolls
// to the top
// [x] reset bp hit counts - not just on NewProcess, but on RUN! because
// we are now evaluating them on the control thread...
// [x] fix bug where text info is evicted, and switching back to a tab scrolls
// to the top
// [x] targets view
// [x] ensure launch controls parameterize commands correctly)
// [x] ensure ctrl+click
// [x] scheduler view
// [x] eval committing
// [x] fix registers
// [x] file overrides -> always pick most specific one! found with conflicting
// overrides, e.g. C:/devel/ -> D:/devel/, but also C:/devel/foo ->
// C:/devel/bar, etc.
// [x] fix memory view
////////////////////////////////
//~ rjf: Build Options