On Mon, 13 Nov 2017, Jarkko Sakkinen wrote:

> +
> +     secs_epc = sgx_alloc_page(0);

Use SGX_ALLOC_ATOMIC instead of 0 ?

> +     if (IS_ERR(secs_epc)) {
> +             ret = PTR_ERR(secs_epc);
> +             goto out;
> +     }

> +out:
> +     if (encl)
> +             kref_put(&encl->refcount, sgx_encl_release);
> +     return ret;
> +}

Don't you need an sgx_free_page() somewhere here?



-- 
James Morris
<james.l.mor...@oracle.com>

Reply via email to