2017-11-13 23:44:41

by James Morris

[permalink] [raw]
Subject: Re: [PATCH v5 06/11] intel_sgx: driver for Intel Software Guard Extensions

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
<[email protected]>


From 1583981473850760573@xxx Mon Nov 13 19:47:29 +0000 2017
X-GM-THRID: 1583981473850760573
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread