linux-kernel - Re: [PATCH v6 06/11] intel_sgx: driver for Intel Software Guard Extensions

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20171126190109.qfvjoq2sbz5x3ve7@linux.intel.com>
Date:   Sun, 26 Nov 2017 21:01:09 +0200
From:   Jarkko Sakkinen <jarkko.sakkinen@...ux.intel.com>
To:     platform-driver-x86@...r.kernel.org, x86@...nel.org
Cc:     linux-kernel@...r.kernel.org, Thomas Gleixner <tglx@...utronix.de>,
        Ingo Molnar <mingo@...hat.com>,
        "H. Peter Anvin" <hpa@...or.com>,
        Darren Hart <dvhart@...radead.org>,
        Andy Shevchenko <andy@...radead.org>
Subject: Re: [PATCH v6 06/11] intel_sgx: driver for Intel Software Guard
 Extensions

On Sun, Nov 26, 2017 at 08:46:27PM +0200, Jarkko Sakkinen wrote:
> On Sun, Nov 26, 2017 at 07:33:56PM +0200, Jarkko Sakkinen wrote:
> > > +struct sgx_encl_page {
> > > +	unsigned long addr;
> > > +	unsigned int flags;
> > > +	void *epc_page;
> > > +	struct sgx_va_page *va_page;
> > > +	unsigned int va_offset;
> > > +	struct list_head list;
> > > +};
> > 
> > This can be without major streches hoops packed way more. There are 12
> > bits free for use in the addr field. I can use low three bits for flags
> > and upper nine bits for va_offset.
> > 
> > I think I can also quite easily union epc_page and va_page fields.
> > 
> > With these changes the struct would shrink to:
> > 
> > struct sgx_encl_page {
> > 	unsigned long addr;
> > 	union {
> > 		void *epc_page;
> > 		struct sgx_va_page *va_page;
> > 	};
> > 	struct list_head list;
> > };
> 
> Once I started to thinking more about this I realized that this can be
> shrunk even further. I can drop the list field and use a tag for loaded
> pages in the radix tree where these are stored. The end result would be
> 
> struct sgx_encl_page {
> 	unsigned long addr;
>  	union {
>  		void *epc_page;
>  		struct sgx_va_page *va_page;
>  	};
> };

With this last size optimization there is an issue that it would be nice
to select the pages to be swapped out in FIFO order. That kind of means
that tag would not work quite well.

I could try an approach where I have a pointer array to present a queue
inside struct sgx_encl for loaded pages and in that way get rid of the
'list' field.

/Jarkko