linux-kernel - Re: [PATCH v9 14/18] nitro_enclaves: Add Kconfig for the Nitro Enclaves driver

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] [day] [month] [year] [list]

Date:   Tue, 22 Sep 2020 20:06:58 +0300
From:   "Paraschiv, Andra-Irina" <andraprs@...zon.com>
To:     Greg KH <gregkh@...uxfoundation.org>
CC:     linux-kernel <linux-kernel@...r.kernel.org>,
        Anthony Liguori <aliguori@...zon.com>,
        Benjamin Herrenschmidt <benh@...nel.crashing.org>,
        Colm MacCarthaigh <colmmacc@...zon.com>,
        David Duncan <davdunc@...zon.com>,
        Bjoern Doebel <doebel@...zon.de>,
        David Woodhouse <dwmw@...zon.co.uk>,
        "Frank van der Linden" <fllinden@...zon.com>,
        Alexander Graf <graf@...zon.de>,
        "Karen Noel" <knoel@...hat.com>,
        Martin Pohlack <mpohlack@...zon.de>,
        Matt Wilson <msw@...zon.com>,
        Paolo Bonzini <pbonzini@...hat.com>,
        Balbir Singh <sblbir@...zon.com>,
        Stefano Garzarella <sgarzare@...hat.com>,
        "Stefan Hajnoczi" <stefanha@...hat.com>,
        Stewart Smith <trawets@...zon.com>,
        "Uwe Dannowski" <uwed@...zon.de>,
        Vitaly Kuznetsov <vkuznets@...hat.com>,
        kvm <kvm@...r.kernel.org>,
        ne-devel-upstream <ne-devel-upstream@...zon.com>
Subject: Re: [PATCH v9 14/18] nitro_enclaves: Add Kconfig for the Nitro
 Enclaves driver



On 22/09/2020 19:20, Greg KH wrote:
> On Tue, Sep 22, 2020 at 05:13:02PM +0300, Paraschiv, Andra-Irina wrote:
>>
>> On 21/09/2020 15:34, Paraschiv, Andra-Irina wrote:
>>>
>>> On 14/09/2020 20:23, Paraschiv, Andra-Irina wrote:
>>>>
>>>> On 14/09/2020 18:59, Greg KH wrote:
>>>>> On Fri, Sep 11, 2020 at 05:11:37PM +0300, Andra Paraschiv wrote:
>>>>>> Signed-off-by: Andra Paraschiv <andraprs@...zon.com>
>>>>>> Reviewed-by: Alexander Graf <graf@...zon.com>
>>>>> I can't take patches without any changelog text at all, sorry.
>>>>>
>>>>> Same for a few other patches in this series :(
>>>>>
>>>> I can move the changelog text before the Sob tag(s) for all the
>>>> patches. I also can add a summary phrase in the commit message for
>>>> the commits like this one that have only the commit title and Sob &
>>>> Rb tags.
>>>>
>>>> Would these updates to the commit messages match the expectations?
>>>>
>>>> Let me know if remaining feedback to discuss and I should include as
>>>> updates in v10. Otherwise, I can send the new revision with the
>>>> updated commit messages.
>>>>
>>>> Thanks for review.
>>> Here we go, I published v10, including the updated commit messages and
>>> rebased on top of v5.9-rc6.
>>>
>>> https://lore.kernel.org/lkml/20200921121732.44291-1-andraprs@amazon.com/
>>>
>>> Any additional feedback, open to discuss.
>>>
>>> If all looks good, we can move forward as we've talked before, to have
>>> the patch series on the char-misc branch and target v5.10-rc1.
>> Thanks for merging the patch series on the char-misc-testing branch and for
>> the review sessions we've had.
>>
>> Let's see how all goes next; if anything in the meantime to be done (e.g.
>> and not coming via auto-generated mails), just let me know.
> Will do, thanks for sticking with this and cleaning it up to look a lot
> better than the original submission.

And this also came with a couple of lessons learnt that I've applied or 
will apply for other pieces of codebase as well, either for the Linux 
kernel or other projects.

>
> Now comes the real work, maintaining it for the next 10 years :)

I agree, maintenance is equally important. There is an ongoing process 
to make sure the whole ecosystem continues to work.

Andra



Amazon Development Center (Romania) S.R.L. registered office: 27A Sf. Lazar Street, UBC5, floor 2, Iasi, Iasi County, 700045, Romania. Registered in Romania. Registration number J22/2621/2005.