Re: [PATCH v5 11/11] intel_sgx: driver documentation

2017-11-28 Thread Jarkko Sakkinen
On Tue, Nov 28, 2017 at 10:37:48PM +0200, Jarkko Sakkinen wrote: > On Mon, Nov 27, 2017 at 09:03:39AM -0800, Sean Christopherson wrote: > > I have a branch based on Jarkko's patches (I believe it's up-to-date with > > v5) > > that implements what I described.  I'd be happy to send RFC patches if

Re: [PATCH v5 11/11] intel_sgx: driver documentation

2017-11-28 Thread Jarkko Sakkinen
On Mon, Nov 27, 2017 at 09:03:39AM -0800, Sean Christopherson wrote: > I have a branch based on Jarkko's patches (I believe it's up-to-date with v5) > that implements what I described.  I'd be happy to send RFC patches if that > would help. That would only slow things down. The code is easy to

Re: [PATCH v5 11/11] intel_sgx: driver documentation

2017-11-27 Thread Sean Christopherson
+ Cc: KVM, Paolo and Radim On Mon, 2017-11-27 at 09:03 -0800, Sean Christopherson wrote: > On Tue, 2017-11-21 at 01:08 +0200, Jarkko Sakkinen wrote: > > > > On Sat, Nov 18, 2017 at 12:34:33AM +0100, Thomas Gleixner wrote: > > > > > > > > > This is architecural. From the cursory read of that

Re: [PATCH v5 11/11] intel_sgx: driver documentation

2017-11-27 Thread Sean Christopherson
On Tue, 2017-11-21 at 01:08 +0200, Jarkko Sakkinen wrote: > On Sat, Nov 18, 2017 at 12:34:33AM +0100, Thomas Gleixner wrote: > > > > This is architecural. From the cursory read of that series it seems there > > are two parts to it: > > > >   1) The actual core handling, which should be in

Re: [PATCH v5 11/11] intel_sgx: driver documentation

2017-11-20 Thread Jarkko Sakkinen
On Fri, Nov 17, 2017 at 03:46:45PM -0800, Darren Hart wrote: > On Sat, Nov 18, 2017 at 12:34:33AM +0100, Thomas Gleixner wrote: > > On Fri, 17 Nov 2017, Darren Hart wrote: > > > > @intel: I removed intel-sgx-kernel-...@lists.01.org from CC because I can > > do without the silly moderation spam of

Re: [PATCH v5 11/11] intel_sgx: driver documentation

2017-11-20 Thread Jarkko Sakkinen
On Sat, Nov 18, 2017 at 12:34:33AM +0100, Thomas Gleixner wrote: > This is architecural. From the cursory read of that series it seems there > are two parts to it: > > 1) The actual core handling, which should be in arch/x86 because that > hardly qualifies as a 'platform' device driver. >

Re: [PATCH v5 11/11] intel_sgx: driver documentation

2017-11-17 Thread Darren Hart
On Sat, Nov 18, 2017 at 12:34:33AM +0100, Thomas Gleixner wrote: > On Fri, 17 Nov 2017, Darren Hart wrote: > > @intel: I removed intel-sgx-kernel-...@lists.01.org from CC because I can > do without the silly moderation spam of that list. Please disable that > nonsense. > > > On Mon, Nov 13, 2017

Re: [PATCH v5 11/11] intel_sgx: driver documentation

2017-11-17 Thread Thomas Gleixner
On Fri, 17 Nov 2017, Darren Hart wrote: @intel: I removed intel-sgx-kernel-...@lists.01.org from CC because I can do without the silly moderation spam of that list. Please disable that nonsense. > On Mon, Nov 13, 2017 at 09:45:28PM +0200, Jarkko Sakkinen wrote: > Is SGX considered architectural

Re: [PATCH v5 11/11] intel_sgx: driver documentation

2017-11-17 Thread Darren Hart
On Mon, Nov 13, 2017 at 09:45:28PM +0200, Jarkko Sakkinen wrote: Please do not submit patches to LKML without a commit message. There is *always* something you can provide to give the review additional context to aid in their review of your code. As Thomas has noted, the various maintainers have