Re: [PATCH 01/25] x86/sgx: Add shortlog descriptions to ENCLS wrappers

From: Reinette Chatre
Date: Mon Dec 13 2021 - 17:07:05 EST


Hi Jarkko,

On 12/10/2021 9:28 PM, Jarkko Sakkinen wrote:
On Mon, 2021-12-06 at 13:13 -0800, Reinette Chatre wrote:

...


__eadd():
/* Copy a source page from non-enclave memory into the EPC. */

Perhaps:

/*
* Associate an EPC page to an enclave either as a REG or TCS page
* populated with the provided data.
*/

This is more aligned with your description for __eremove().

I was trying to keep the descriptions as concise one-liners. I'll use the text you provide if you are ok with its line length being an exception.

...


__eremove():
/* Disassociate EPC page from its enclave and mark it as unused. */

...

Reinette