Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Extended size classes #13

Open
wants to merge 59 commits into
base: main
Choose a base branch
from
Open

Conversation

cmovcc
Copy link
Collaborator

@cmovcc cmovcc commented Jun 26, 2024

This is mostly a large rebase of a previous branch.

  • cherry-pick files
  • merge initialization
  • fix sizeclass selection
  • remove admits/assumes

@cmovcc cmovcc changed the title WIP: extended size classes WIP: Extended size classes Jun 26, 2024
@cmovcc
Copy link
Collaborator Author

cmovcc commented Jul 1, 2024

Current goal is for CI to pass, before fixing all of the admits/assume. src/LargeAlloc.fst and lib_avl_mono/Impl.Trees.Types.fst are the only remaining files to cause some issues.

Antonin Reitz added 4 commits July 3, 2024 18:07
remaining one is very light,
question is more about easiness of reconfiguring the allocator
@cmovcc
Copy link
Collaborator Author

cmovcc commented Jul 4, 2024

CI fails but verification now works. Many things yet to be fixed, though...

Antonin Reitz added 30 commits July 16, 2024 15:09
src/Main.fst fix
perf tweak to match hardened_malloc default settings
- Main: 2 sladmits remaining
- SlabsCommon also still WIP...
- Main: 1 sladmit remaining
- SlabsCommon also still WIP
SlabsCommon still has 3
sc/sc_ex pattern still yet to prove
fix Quarantine2 correspondingly,
as madvise is now unused
Main.fst: no admit anymore
only a bunch of assumes left
+ sizes layout proof
+ fix upstream build breakage
only NLA-related assumes remaining
+ 1 admit
mostly NLA,
one admit remaining
still a few verif issues
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.

1 participant