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

Remove #include sgx_urts.h from sgx_uswitchless #309

Merged
merged 1 commit into from
Sep 17, 2018

Conversation

deeglaze
Copy link
Contributor

@deeglaze deeglaze commented Sep 5, 2018

The sgx_uswitchless.h header is used in both trusted and untrusted
contexts from transitive inclusions. The sgx_urts.h header is strictly
an untrusted domain header. Users that depend on the untrusted runtime
should not depend on the transitive include behavior to provide
sgx_urts.h.

Signed-off-by: Dionna Glaze dionnaglaze@google.com

The sgx_uswitchless.h header is used in both trusted and untrusted
contexts from transitive inclusions. The sgx_urts.h header is strictly
an untrusted domain header. Users that depend on the untrusted runtime
should not depend on the transitive include behavior to provide
sgx_urts.h.

Signed-off-by: Dionna Glaze <dionnaglaze@google.com>
@andyzyb
Copy link
Contributor

andyzyb commented Sep 7, 2018

Looks good. Will wait for a better timing for merging the change in the coming week.

@lzha101 lzha101 merged commit ef91b29 into intel:master Sep 17, 2018
llly added a commit to llly/linux-sgx that referenced this pull request Sep 28, 2018
These PRs are lost in Linux 2.3.1 Open Source Gold Release

Signed-off-by: Li, Xun <xun.li@intel.com>
andyzyb added a commit that referenced this pull request Sep 28, 2018
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.

3 participants