Re: [PATCH 1/2] clone: Support passing tls argument via C rather than pt_regs magic

From: Josh Triplett
Date: Mon May 11 2015 - 15:31:35 EST


On Mon, May 11, 2015 at 04:00:43PM +0200, Ingo Molnar wrote:
>
> * Josh Triplett <josh@xxxxxxxxxxxxxxxx> wrote:
>
> > On Mon, May 11, 2015 at 12:13:13PM +0200, Ingo Molnar wrote:
> > >
> > > * josh@xxxxxxxxxxxxxxxx <josh@xxxxxxxxxxxxxxxx> wrote:
> > >
> > > > On Tue, May 05, 2015 at 08:53:03PM +0200, Thomas Gleixner wrote:
> > > > > On Tue, 21 Apr 2015, Josh Triplett wrote:
> > > > > >
> > > > > > Signed-off-by: Josh Triplett <josh@xxxxxxxxxxxxxxxx>
> > > > > > Signed-off-by: Thiago Macieira <thiago.macieira@xxxxxxxxx>
> > > > >
> > > > > Can you please clarify that SOB chain? It does not make any sense.
> > > >
> > > > Co-authored patch. We both worked on it together, and sadly git
> > > > doesn't support a commit with multiple authors, so this is the next
> > > > best thing.
> > >
> > > No, this is not a valid SOB chain.
> > >
> > > For 'co authored' patches you can add credits either to the file,
> > > as two copyright lines, or via the changelog, no need to mess up
> > > the SOB chain for that.
> >
> > Fine, I'll write it another way.
> >
> > Do you see any other issues with the patch other than the signoffs?
>
> Looks good to me, but I have not looked very deeply ...

I sent out a v2 with the co-author information moved from the signoffs
to the commit message. If it looks reasonable to you, can you take it
through the tip tree please?

- Josh Triplett
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/