[mpich-devel] [mpich-discuss] Use of Signed-off-by in MPICH
blocksom at us.ibm.com
Mon Jan 6 08:24:18 CST 2014
IBM uses the "signed off by" feature to enforce/track code reviews before
any code gets contributed to ANL. The intent is that the commit author !=
the commit signer when the code goes out. Often a developer at ANL will
also review the code before it gets integrated into the master branch (IBM
developers don't have push authority) and sign the commit again.
We use the text "Signed-off-by:xxxx <yyy at zzz>" simply because this is what
is supported by the git tools. You can use commands like `git cherry-pick
-s`, `git commit --amend -s`, and `git am -s` to ease the signing effort.
Parallel Environment MPI Middleware Team Lead, TCEM
POWER, x86, and Blue Gene HPC Messaging
blocksom at us.ibm.com
From: Pavan Balaji <balaji at mcs.anl.gov>
To: discuss at mpich.org,
Date: 01/04/2014 10:00 PM
Subject: Re: [mpich-discuss] Use of Signed-off-by in MPICH
Sent by: discuss-bounces at mpich.org
Please use devel at mpich.org for developer oriented discussion.
We use signoff as a way to get a second confirmation on the correctness of
the patch. In some cases, multiple confirmations. There’s also a "silent
signoff” done by the jenkins testing before things are merged into
On Jan 4, 2014, at 7:55 PM, Jed Brown <jedbrown at mcs.anl.gov> wrote:
> I'm confused by the current use of Signed-off-by tags in MPICH. Is this
> intended to be a developer's certificate of origin (DCO) following the
> model of the Linux kernel ? I can't tell if IBM is using it that
> way, where Michael Blocksome is the one authorized by IBM to assert on
> IBM's behalf that the work has clean IP and may be distributed under the
> MPICH license. But what does this mean?
> commit 87b20136ac20a0fbcffd8e29edcc500fcebbfc22
> Author: Pavan Balaji <balaji at mcs.anl.gov>
> AuthorDate: Wed Jan 1 19:33:36 2014 -0600
> Commit: Pavan Balaji <balaji at mcs.anl.gov>
> CommitDate: Thu Jan 2 15:39:07 2014 -0600
> Move default eager threshold setting to ch3.
> We were setting the default eager threshold inside nemesis, instead
> in ch3. This was causing other ch3 channels, notably sock, to get a
> garbage default value for the eager threshold.
> Signed-off-by: Wesley Bland <wbland at mcs.anl.gov>
> If this were a DCO, we would interpret it as meaning that Wesley is the
> one certifying the IP, but Pavan is the author.
> Or is Signed-off-by being used where Reviewed-by would be more
>  https://www.kernel.org/doc/Documentation/SubmittingPatches
> discuss mailing list discuss at mpich.org
> To manage subscription options or unsubscribe:
discuss mailing list discuss at mpich.org
To manage subscription options or unsubscribe:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the devel