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

upgrade SDK and rust compiler urgently needed #360

Open
brenzi opened this issue Sep 13, 2021 · 17 comments
Open

upgrade SDK and rust compiler urgently needed #360

brenzi opened this issue Sep 13, 2021 · 17 comments

Comments

@brenzi
Copy link

brenzi commented Sep 13, 2021

This SDK is behind Intel's SDK for quite a while already and the supported rust compiler is very old.

We currently face a deadlock because dtolnay/proc-macro2#296 breaks our build. We can't downgrade, because other dependencies request proc-macro2-1.0.29. But we can't upgrade because teaclave's rust compiler support is outdated.

Please upgrade ASAP

@mssun
Copy link
Member

mssun commented Sep 13, 2021

Thanks for the report. I'll link a related PR here: #349

@mssun
Copy link
Member

mssun commented Sep 14, 2021

@brenzi, as discussed with @volcano0dr, he's working on a new release recently. You'll see the changes in this month.

In the meantime, I suggest to pin to a specific version (through vendoring) to avoid such problem.

@PiDelport
Copy link
Contributor

@mssun, @volcano0dr: Do you have any status update about the new release? Can SDK users or contributors do anything to help?

@volcano0dr
Copy link
Contributor

@PiDelport sorry for the late upgrade.
I 'v upgraded the SDK to support rust-nightly-2021-09-13, and the updated vesion is 1.1.4. The latest SDK has been pushed to v1.1.4-testing branch.
Any respondes will be greatly welcome.

@volcano0dr
Copy link
Contributor

@brenzi SDK has been upgraded.

@sbellem
Copy link

sbellem commented Oct 26, 2021

A few questions & observations:

Thank you.

@volcano0dr
Copy link
Contributor

@sbellem

  • The master branch fully supports Intel sgx sdk 2.14. The rust-toolchain is nightly-2020-10-25.(The master branch can also be compiled with rust-nightly-2020-03-30)
  • The v1.1.4-testing branch fully supports Intel sgx sdk 2.15. The rust-toolchain is nightly-2021-09-13.
  • After testing, the v1.1.4-testing branch will be merged into the master branch.
  • If you plan to use Intel sgx sdk 2.14 now, it is best to use the master branch.

@sbellem
Copy link

sbellem commented Oct 26, 2021

Thanks for the detailed answer @volcano0dr!

@haerdib
Copy link

haerdib commented Nov 3, 2021

We were able to test it and it seems to work perfectly. proc-macro was updated to v1.0.32 and rust version to nightly-2021-10-01, which successfully built & run. (integritee-network/worker#486)
Currently, not very nice patching to a teaclave fork is necessary for the mesalock libraries to work, since cargo does not allow patches to the same github repo. Other than that, nothing to complain about. Thanks a lot for your effort!

@brenzi
Copy link
Author

brenzi commented Nov 5, 2021

when can we expect the release of 1.1.4?

@haerdib
Copy link

haerdib commented Nov 16, 2021

Update: Upgrade to rust-toolchain nightly-2021-10-30 and later is not possible, because of the following error:

error[E0522]: definition of an unknown language item: `begin_panic_fmt`
   --> /.cargo/git/checkouts/incubator-teaclave-sgx-sdk-f208006cdf70a903/19355dc/sgx_tstd/src/panicking.rs:406:1
    |
406 | #[lang = "begin_panic_fmt"]
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ definition of unknown language item `begin_panic_fmt`

@volcano0dr
Copy link
Contributor

@haerdib I will upgrade rust-toolchain again recently to support rust edition 2021.

@PiDelport
Copy link
Contributor

Upstream release: Intel SGX SDK 2.15.1 Release Notes (12 Nov 2021)

Changes in version 2.15.1:

  • Upgraded to OpenSSL 1.1.1l.

Does v1.1.4-testing need any update for this?

Is anything else still blocking it from being merged?

@volcano0dr
Copy link
Contributor

volcano0dr commented Nov 26, 2021

@PiDelport
v1.1.4-testing can support intel sgx sdk 2.15.1 and dcap 1.12.1
v1.1.4-testing has been merged into master

@PiDelport
Copy link
Contributor

Awesome!

Does that resolve this issue?

@volcano0dr
Copy link
Contributor

@haerdib I have upgraded rust-toolchain to nightly-2021-11-01.

@volcano0dr
Copy link
Contributor

@PiDelport This issue has been resolved.

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

No branches or pull requests

6 participants