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

Parameterised compartments #37

Merged
merged 1 commit into from
Aug 30, 2024
Merged

Conversation

0152la
Copy link
Contributor

@0152la 0152la commented Aug 30, 2024

Add a CompConfig struct, which contains parameterisable values for compartment properties. Currently this includes only the heap and stack sizes, as well as entry points. The parameters are taken from the .comp TOML config file, similar to how entry points are determined.

Enable lua_suite_some test, which runs a few hand-picked tests from the Lua test suite, just to see if we can find any other parts of Lua that need further handling in the manager.

NOTE gc.lua should work, but it seems to take 10 hours on a virtualised CHERI instance. I think it might be due to the unoptimised memory allocator implementation, but that would need further benchmarking.

Additionally refactor some old code, including getting rid of CompEntryPoint in favour of the singular CompEntryPointDef struct.

Add a `CompConfig` struct, which contains parameterisable values for
compartment properties. Currently this includes only the heap and stack
sizes, as well as entry points. The parameters are taken from the
`.comp` TOML config file, similar to how entry points are determined.

Enable `lua_suite_some` test, which runs a few hand-picked tests from
the Lua test suite, just to see if we can find any other parts of Lua
that need further handling in the manager.

NOTE `gc.lua` *should* work, but it seems to take 10 hours on a
virtualised CHERI instance. I think it might be due to the unoptimised
memory allocator implementation, but that would need further
benchmarking.

Additionally refactor some old code, including getting rid of
`CompEntryPoint` in favour of the singular `CompEntryPointDef` struct.
@0152la 0152la requested a review from ltratt August 30, 2024 13:40
@ltratt ltratt added this pull request to the merge queue Aug 30, 2024
Merged via the queue into capablevms:master with commit 290b6c6 Aug 30, 2024
2 checks passed
@0152la 0152la deleted the parameterise branch September 2, 2024 11:13
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