-
Notifications
You must be signed in to change notification settings - Fork 134
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
feat(shim-sgx): mmap() with EACCEPTCOPY #1608
Conversation
WiP PR for EAUG and mmap(). |
Ignore this build failing for now. I just backup stuff while I fix the other pull request. |
Oops, converted by mistake to non-draft. |
Pull request was converted to draft
@npmccallum @haraldh there's a pile of commits but CI passes on all of them expect the very top-most. Do you see something in that commmit that looks odd? It's not very big because the patch set is cut down to fairly small pieces (and prepending pass the CI as said). |
Added Revert "feat: attestation checking for wasmldr" temporarily up until the main branch has been fixed so that I don't have to use CI as my main testing environment. |
With uild(shim-sgx): update sgx create to include SecInfo::accept_copy as the top-most commit:
With the tip commit:
|
A filtered (syscall names, clock_gettime removed) log of hello_wasi_snapshot1:
|
Looking at above log, can been seen that:
And AEX happens:
Two conclusions:
This discloses everything else expect thep |
You might hit a case, where wasmtime/cranelift installs guard pages and expects the signal handler for page fault to trigger |
|
|
So, the only missing thing here is gdb.rs |
yeah, sorry, my fault.... apologies |
Signed-off-by: Jarkko Sakkinen <jarkko@profian.com>
Signed-off-by: Jarkko Sakkinen <jarkko@profian.com>
I'll create issue for it. |
|
No description provided.