On Wed, Jul 10, 2019 at 09:21:32PM -0700, Cedric Xing wrote: > -#ifdef SGX_KERNEL_DOC Why is this removed? > + * int __vdso_sgx_enter_enclave(int leaf, void *tcs, > + * struct sgx_enclave_exinfo *exinfo, > + * sgx_callback callback) > + * { > + * while (leaf == EENTER || leaf == ERESUME) { > + * int rc; > + * try { > + * ENCLU[leaf]; > + * rc = 0; > + * if (exinfo) > + * exinfo->leaf = EEXIT; > + * } catch (exception) { > + * rc = -EFAULT; > + * if (exinfo) > + * *exinfo = exception; > + * } > + * > + * leaf = !callback ? rc: (*callback)(rdi, rsi, rdx, exinfo, > + * r8, r9, tcs, ursp); > + * } > + * > + * return leaf > 0 ? -EINVAL : leaf; > + * } > */ What is this? C++ and anyway there is already a source code. No need to duplicate with pseudo-code. Only adds maintenance burde. Please get rid of this. /Jarkko