-
Notifications
You must be signed in to change notification settings - Fork 113
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
Reference process page table using the underlying PhysicalFrame #4957
base: main
Are you sure you want to change the base?
Conversation
more specifically the crash appears to occur here: https://github.com/jul-sh/oak/blob/back-port-frame-patch/oak_restricted_kernel/src/mm/bitmap_frame_allocator.rs#L190 |
specifically it fails to alloc 2MiB frames. A 4KiB one allocs without problems |
Kernel fails, but no panic is handled or logged. All that shows is:
mmap also causes a crash even if we merely request the number of allocated frames with |
output from qemu debug logs, obtained with
|
Change-Id: I95d097e7392ae54bc3abf2b4a4907abf7ca4b108
after discussing with @conradgrobler, it seems like the kernels allocator will need to be enlightened to support multiple proccesses. |
Backports the change outlined in #4918 (comment) onto main.
In principle this change should not change behavior. The only difference is that we directly store & use the physical frame that holds a processe's pml4, instead of identifying it anew when we start to execute.
However it does change behavior. Specifically fixes some issues explained in the comment linked above, but also leads to a panic when the enclave app (not orchestrator) allocates memory.