incubator-teaclave-sgx-sdk icon indicating copy to clipboard operation
incubator-teaclave-sgx-sdk copied to clipboard

Published 20 hours ago • verified publisher icon apache

Use `std`-aware cargo instead of `sgx_tstd` / xargo

Open tarcieri opened this issue 5 years ago • 4 comments

This is mostly a heads up that design work is underway to allow parts of std to be swappable via cargo:

https://internals.rust-lang.org/t/pre-rfc-std-aware-cargo/9450

Would be good to make sure this feature meets the needs of rust-sgx-sdk

tarcieri avatar Feb 14 '19 22:02 tarcieri

agree!

seems to be something like "core-customizable xargo". am i correct? it'll be fantastic for embedded systems and TEEs!

dingelish avatar Feb 15 '19 02:02 dingelish

Merged :) https://github.com/rust-lang/cargo/pull/7216

Hopefully this can mean no longer maintaining any third party forks!

elichai avatar Nov 07 '19 23:11 elichai

@elichai Wow! Let me dig into it. My current thought is that let me upgrade to Intel SGX SDK v2.7 and provide threading support in the current style. And then I think we should move forward to the std-aware rust toolchain!

dingelish avatar Nov 07 '19 23:11 dingelish

The 3rd party problem still persists, because we may adjust something on file access such as tweaking std::untrusted::fs, or std::sgxfs. But I believe that some foundation crates would be fine without any modification such as serde and ring. For cpuid based feature detection, we do have a way to solve without triggering the unsupported instruction or any AEX.

dingelish avatar Nov 08 '19 00:11 dingelish