Re: [PATCH v22 02/24] x86/cpufeatures: x86/msr: Intel SGX Launch Control hardware bits

From: Jarkko Sakkinen
Date: Wed Sep 25 2019 - 10:39:07 EST


On Wed, Sep 25, 2019 at 05:10:58PM +0300, Jarkko Sakkinen wrote:
> On Wed, Sep 25, 2019 at 05:09:03PM +0300, Jarkko Sakkinen wrote:
> > > > [1] Intel SDM: 38.1.4 Intel SGX Launch Control Configuration
> > > >
> > > > Signed-off-by: Sean Christopherson <sean.j.christopherson@xxxxxxxxx>
> > > > Co-developed-by: Haim Cohen <haim.cohen@xxxxxxxxx>
> > > > Signed-off-by: Haim Cohen <haim.cohen@xxxxxxxxx>
> > > > Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@xxxxxxxxxxxxxxx>
> > >
> > > This time checkpatch is right:
> > >
> > > WARNING: Missing Signed-off-by: line by nominal patch author 'Kai Huang <kai.huang@xxxxxxxxxxxxxxx>'
> > >
> > > And looking at the SOB chain, sounds like people need to make up their
> > > mind about authorship...
> >
> > I'll make myself the sole author for this one as 98% of the effort in
> > this patch is really the commit message, which I rewrote for v22, and 2%
> > are the code changes (mechanical, peek at SDM). This patch was squashed
> > from three patches, all like one line changes, and Kai was author of one
> > of them.
> >
> > The next version will thus have only my SOB and author information will
> > be changed. I doubt anyone will complain if I do that.
>
> I'll take the same action also for "x86/cpufeatures: x86/msr: Add Intel
> SGX hardware bits"

I put to both:

Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@xxxxxxxxxxxxxxx>
Co-developed-by: Sean Christopherson <sean.j.christopherson@xxxxxxxxx>
Signed-off-by: Sean Christopherson <sean.j.christopherson@xxxxxxxxx>

And changed author to me.

/Jarkko