On Thu, 2022-08-18 at 17:27 +0300, Jani Nikula wrote: On Thu, 18 Aug 2022, Jani Nikula > wrote: On Wed, 17 Aug 2022, "Colin King (gmail)" > wrote: On 17/08/2022 21:07, Vivi, Rodrigo wrote: On Tue, 2022-08-16 at 12:43 +0800, Zhenyu Wang wrote: On 2022.08.16 12:05:08 +0800, Zhenyu Wang wrote: On 2022.08.15 19:32:45 -0400, Rodrigo Vivi wrote: On Mon, Aug 15, 2022 at 10:38:55AM +0800, Zhenyu Wang wrote: oh, surprise! I just found Colin's email is actually defined in .mailmap, so all his commits in kernel are changed for @intel.com address as in mailmap... Colin, would you mind to get the Sign-off-by in the patches the same as your authorship so the tools don't get confused? (starting with modifying in tree this already merged patch) Since my patches are generally trivial janitorial fixed done in my spare time I'm going to get the .mailmap changed to use my gmail email address rather than my Intel one (since I don't do kernel work in my current role). This should clean up the confusion. Apologies. Everyone hold your horses. I think our tooling should handle the mailmap stuff. The commit *is* fine, it's just that the when we check it, we let mailmap alter it. We should check the commit without mailmap modifications. In this case, it's actually not about Colin's Signed-off-by or mailmap at all! Like the error message from dim says, "committer Signed-off-by missing". Committer, not author! $ git show -s tags/gvt-fixes-2022-08-15^ --pretty=fuller commit d6632370536d0b80be3bfc90dd67e1f693335a75 Author: Colin Ian King > AuthorDate: Tue Mar 15 20:24:49 2022 +0000 Commit: Zhenyu Wang > CommitDate: Mon Aug 15 10:51:15 2022 +0800 drm/i915/reg: Fix spelling mistake "Unsupport" -> "Unsupported" There is a spelling mistake in a gvt_vgpu_err error message. Fix it. Fixes: 695fbc08d80f ("drm/i915/gvt: replace the gvt_err with gvt_vgpu_err") Signed-off-by: Colin Ian King > Signed-off-by: Zhi Wang > Link: http://patchwork.freedesktop.org/patch/msgid/20220315202449.2952845-1-colin.i.king@gmail.com Reviewed-by: Zhi Wang > Committed by Zhenyu, Signed-off-by Zhi. Maybe caused by rebase by Zhenyu after being committed by Zhi? Probably easier if you could rebase it again signing it then? I could bypass dim here, but this will likely affect someone else later in the upstream chain as well.. BR, Jani.