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

Add exception handler to edger8r. #222

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kongoshuu
Copy link

Add a exception_handler to the edger8r. If any function in .edl file is marked
as "exception_handler", it should be set the "is_exception" field in the ecall
table. This is used for users to set their own exception, like signal
handler.

Add a exception_handler to the edger8r. If any function in .edl file is marked
as "exception_handler", it should be set the "is_exception" field in the ecall
table. This is used for users to set their own exception, like signal
handler.

Signed-off-by: Chong Cai<chongc@google.com>
@ghost
Copy link

ghost commented Mar 9, 2018

What is the motivation to high light the exception function in the edl file?

@kongoshuu
Copy link
Author

This is used when a user what to register their own exception handler. They can design their own exception handling that enters an enclave by highlighting the ecall as "exception_handler". Another PR has the changed code that deals with this flag (#223 ).

@ghost
Copy link

ghost commented Mar 15, 2018

The alternative solution is, when the application get the SIGCHLD, SIGUSR1, SIGRTMIN + n, etc, the application can use another free thread to handle this exception.

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