linux-kernel - Re: [PATCH v5] Introduce support for Systems Management Driver over WMI for Dell Systems

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <797a25c1-03f2-9a1d-62ba-86fd39c09a3a@redhat.com>
Date:   Fri, 9 Oct 2020 09:56:05 +0200
From:   Hans de Goede <hdegoede@...hat.com>
To:     "Bharathi, Divya" <Divya.Bharathi@...l.com>,
        Divya Bharathi <divya27392@...il.com>,
        "dvhart@...radead.org" <dvhart@...radead.org>
Cc:     LKML <linux-kernel@...r.kernel.org>,
        "platform-driver-x86@...r.kernel.org" 
        <platform-driver-x86@...r.kernel.org>,
        Andy Shevchenko <andy.shevchenko@...il.com>,
        mark gross <mgross@...ux.intel.com>,
        "Limonciello, Mario" <Mario.Limonciello@...l.com>,
        "Ksr, Prasanth" <Prasanth.Ksr@...l.com>
Subject: Re: [PATCH v5] Introduce support for Systems Management Driver over
 WMI for Dell Systems

Hi,

On 10/6/20 10:37 AM, Bharathi, Divya wrote:
> <snip>
> 
>>
>> Hmm, checkpatch is saying:
>>
>> WARNING: Missing Signed-off-by: line by nominal patch author 'Divya
>> Bharathi <divya27392@...il.com>'
>>
>> I assume the dell address is the one you want to use ?
>>
>> If so try setting the following in ~/.gitconfig:
>>
>> [user]
>>           email = divya.bharathi@...l.com
>>
>> And then do:
>>
>> git commit --amend --reset-author
>>
>> To change the author which git is using for the patch.
>>
>>> ---
>>>
> 
> Thanks, I used --reset-author while creating patch-v6.
> 
> And I had added my dell address in gitconfig for previous patches
> as well. Also, local checkpatch.pl script did not throw me this warning.
> 
> Hope, next patch will not hit this warning.

I'm afraid v6 still has this issue. I guess this has something todo
with how you are sending the emails. I assume you are not directly
sending them from your Dell email because that leading to mangled
patches.

For now I can fix up the git author to match the s-o-b after I've
applied the patch to my tree, but it would be nice if you can
get this fixed eventually.

The usual way to deal with this is to use git send-email. There are
several howtos out there on how to use git send-email in combination
with a gmail account.

Note I'm not 100% sure if using git send-email will actually help,
I hope it is smart enough to see the gmail email address used is
not the one from the author field of the commit and then add an
extra "From: Divya Bharathi <divya.bharathi@...l.com>" header, which
git am will then use as the author when I apply the patch.

I know it will add the extra "From:"if I submit patches which
were originally by someone else, but then the author field of
the commit does not match the [user] email setting from my
config...

Anyways this is not an urgent thing to fix, I can fix things
up for now. Would be nice to get this sorted out eventually though.

Regards,

Hans