From: Dave Chinner <david@fromorbit.com>
To: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: "Darrick J. Wong" <djwong@kernel.org>,
linux-xfs@vger.kernel.org, Stephen Zhang <starzhangzsd@gmail.com>,
Shida Zhang <zhangshida@kylinos.cn>,
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 commit in the xfs tree
Date: Tue, 4 Oct 2022 09:21:03 +1100 [thread overview]
Message-ID: <20221003222103.GM3600936@dread.disaster.area> (raw)
In-Reply-To: <20221004072302.345bfd4a@canb.auug.org.au>
On Tue, Oct 04, 2022 at 07:23:02AM +1100, Stephen Rothwell wrote:
> Hi all,
>
> Commits
>
> ece9d1c54c23 ("xfs: rearrange the logic and remove the broken comment for xfs_dir2_isxx")
> 7ee7a280ea9d ("xfs: trim the mapp array accordingly in xfs_da_grow_inode_int")
>
> are missing a Signed-off-by from their author.
Say what?
I just pulled them w/ b4 via their msg-ids. Have a look at the email
here:
https://lore.kernel.org/linux-xfs/20220918065026.1207016-1-zhangshida@kylinos.cn/
And the commit information from the XFS tree says:
author Stephen Zhang <starzhangzsd@gmail.com> 2022-09-26 10:36:11 +1000
committer Dave Chinner <david@fromorbit.com> 2022-09-26 10:36:11 +1000
commit ece9d1c54c23c316219c19c4c7091495007e149b (patch)
tree f4bc7747e2b604cf7718c584126e3e9bc8d5a51a
parent 7ee7a280ea9d3208c075151b06190630b8c20775 (diff)
download xfs-linux-ece9d1c54c23c316219c19c4c7091495007e149b.tar.gz
xfs: rearrange the logic and remove the broken comment for xfs_dir2_isxx
xfs_dir2_isleaf is used to see if the directory is a single-leaf
form directory instead, as commented right above the function.
Besides getting rid of the broken comment, we rearrange the logic by
converting everything over to standard formatting and conventions,
at the same time, to make it easier to understand and self
documenting.
Signed-off-by: Shida Zhang <zhangshida@kylinos.cn>
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Signed-off-by: Dave Chinner <david@fromorbit.com>
----
The commit matches exactly what was sent to the list. It's just
that the patch was sent from a personal email address with a
corporate signoff.
Since when has that been an issue? I -personally- have been doing
this for well over a decade and I'm pretty sure there are lots of
other people who also do this.
Hence if this is wrong, then we've got a tooling problem with b4.
Why does b4 allow this rather than warn/fail if it's not actually
allowed in the linux-next tree?
-Dave.
--
Dave Chinner
david@fromorbit.com
next prev parent reply other threads:[~2022-10-03 22:21 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-03 20:23 linux-next: Signed-off-by missing for commit in the xfs tree Stephen Rothwell
2022-10-03 22:21 ` Dave Chinner [this message]
2022-10-04 11:50 ` Stephen Rothwell
2022-10-04 15:57 ` Darrick J. Wong
2022-10-04 21:04 ` Dave Chinner
2022-10-05 2:52 ` Stephen Zhang
2022-10-05 3:01 ` Darrick J. Wong
2022-10-05 3:13 ` Willy Tarreau
2022-10-05 16:46 ` Konstantin Ryabitsev
-- strict thread matches above, loose matches on Subject: below --
2021-06-06 22:26 Stephen Rothwell
2020-07-26 21:57 Stephen Rothwell
2018-06-04 22:03 Stephen Rothwell
2018-06-04 22:17 ` Darrick J. Wong
2017-08-16 21:48 Stephen Rothwell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20221003222103.GM3600936@dread.disaster.area \
--to=david@fromorbit.com \
--cc=djwong@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-next@vger.kernel.org \
--cc=linux-xfs@vger.kernel.org \
--cc=sfr@canb.auug.org.au \
--cc=starzhangzsd@gmail.com \
--cc=zhangshida@kylinos.cn \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).