Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Tue, 22 Jun 2021 07:27:19 +1000
From:   Stephen Rothwell <sfr@canb.auug.org.au>
To:     "Darrick J. Wong" <djwong@kernel.org>
Cc:     David Chinner <david@fromorbit.com>, linux-xfs@vger.kernel.org,
        Allison Henderson <allison.henderson@oracle.com>,
        Chandan Babu R <chandanrlinux@gmail.com>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux Next Mailing List <linux-next@vger.kernel.org>
Subject: Re: linux-next: Signed-off-by missing for commits in the xfs tree
Message-ID: <20210622072719.1d312bf0@canb.auug.org.au>
In-Reply-To: <20210621171208.GD3619569@locust>
References: <20210621082656.59cae0d8@canb.auug.org.au>
        <20210621171208.GD3619569@locust>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="Sig_/RW4H6sPo8Y9vSIfwqAeW8lk";
 protocol="application/pgp-signature"; micalg=pgp-sha256
Precedence: bulk

--Sig_/RW4H6sPo8Y9vSIfwqAeW8lk
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable

Hi Darrick,

On Mon, 21 Jun 2021 10:12:08 -0700 "Darrick J. Wong" <djwong@kernel.org> wr=
ote:
>
> On Mon, Jun 21, 2021 at 08:26:56AM +1000, Stephen Rothwell wrote:
> >=20
> > Commits
> >=20
> >   742140d2a486 ("xfs: xfs_log_force_lsn isn't passed a LSN")
> >   e30fbb337045 ("xfs: Fix CIL throttle hang when CIL space used going b=
ackwards")
> >   feb616896031 ("xfs: journal IO cache flush reductions")
> >   6a5c6f5ef0a4 ("xfs: remove need_start_rec parameter from xlog_write()=
")
> >   d7693a7f4ef9 ("xfs: CIL checkpoint flushes caches unconditionally")
> >   e45cc747a6fd ("xfs: async blkdev cache flush")
> >   9b845604a4d5 ("xfs: remove xfs_blkdev_issue_flush")
> >   25f25648e57c ("xfs: separate CIL commit record IO")
> >   a6a65fef5ef8 ("xfs: log stripe roundoff is a property of the log")
> >=20
> > are missing a Signed-off-by from their committers. =20
>=20
> <sigh> Ok, I'll rebase the branch again to fix the paperwork errors.
>=20
> For future reference, if I want to continue accepting pull requests from
> other XFS developers, what are the applicable standards for adding the
> tree maintainer's (aka my) S-o-B tags?  I can't add my own S-o-Bs after
> the fact without rewriting the branch history and changing the commit
> ids (which would lose the signed tag), so I guess that means the person
> sending the pull request has to add my S-o-B for me?  Which also doesn't
> make sense?

If you want to take a pull request, then use "git pull" (or "git fetch"
followed by "git merge") which will create a merge commit committed by
you.  The above commits were applied to your tree by you as patches (or
rebased) and so need your sign off.  The commits in a branch that you
just merge into your tree only need the SOBs for their author(s) and
committer.

If you then rebase your tree (with merge commits in it), you need to
use "git rebase -r" to preserve the merge commits.  alternatively, you
can rebase the commits you applied as patches and then redo the
pulls/merges manually.  You generally should not rebase other's work.

Of course, you should not really rebase a published tree at all (unless
vitally necessary) - see Documentation/maintainer/rebasing-and-merging.rst

--=20
Cheers,
Stephen Rothwell

--Sig_/RW4H6sPo8Y9vSIfwqAeW8lk
Content-Type: application/pgp-signature
Content-Description: OpenPGP digital signature

-----BEGIN PGP SIGNATURE-----

iQEzBAEBCAAdFiEENIC96giZ81tWdLgKAVBC80lX0GwFAmDRBDcACgkQAVBC80lX
0GzIlQf/S0VOECaLF1wYu8PKzSojZvRfec8gBUCFJB7sfphjhY+S8AEYUi6+tqb3
dUj85ErLygvwy50Pqjj8oWtGlEtzJsBEhMzA4DEgtDtSNUXfyOvw/LfPW75Cs0Lo
Z0lcBDfb/02GFD3llaPiGy7lOVpMDwYYEb24TbRbsmrwDXUsDmcB/sf4exUVxPTb
iq9PKPYSzLQKpfg0d6ZyIcILD8fPev8+WwduTVRHveKYRI1F5ISykP790p1OUhlU
iq8u1tpErCYbwSNFwe07u9FgkMUGoRojwWgi9Tbtd7feo1Gf1bSYteysqS1S4TI7
pUvlP9Uofp7VWqQlYlqG2dBzBLqWqg==
=OqX0
-----END PGP SIGNATURE-----

--Sig_/RW4H6sPo8Y9vSIfwqAeW8lk--