Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752446AbcKNOCT (ORCPT ); Mon, 14 Nov 2016 09:02:19 -0500 Received: from mail-wm0-f54.google.com ([74.125.82.54]:38766 "EHLO mail-wm0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751496AbcKNOCP (ORCPT ); Mon, 14 Nov 2016 09:02:15 -0500 Date: Mon, 14 Nov 2016 14:01:58 +0000 From: Matt Fleming To: Ingo Molnar Cc: Thomas Gleixner , "H . Peter Anvin" , Ard Biesheuvel , linux-kernel@vger.kernel.org, linux-efi@vger.kernel.org, Kees Cook Subject: Re: [PATCH 5/9] efi/arm*: libstub: Invoke EFI_RNG_PROTOCOL to seed the UEFI RNG table Message-ID: <20161114140158.GD2373@codeblueprint.co.uk> References: <20161112213237.8804-1-matt@codeblueprint.co.uk> <20161112213237.8804-6-matt@codeblueprint.co.uk> <20161113071939.GA31483@gmail.com> <20161114132334.GB2373@codeblueprint.co.uk> <20161114135521.GA10151@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161114135521.GA10151@gmail.com> User-Agent: Mutt/1.5.24+41 (02bc14ed1569) (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1103 Lines: 28 On Mon, 14 Nov, at 02:55:22PM, Ingo Molnar wrote: > > The problem is not that Ard applied the patches, but that you subsequently rebased > the tree. For example: > > commit bf5d1f98c1d8be04a40eabb9dd6913347b1b3fc4 > Author: Ard Biesheuvel > AuthorDate: Thu Oct 20 12:21:26 2016 +0100 > Commit: Matt Fleming > CommitDate: Sat Nov 12 21:14:41 2016 +0000 > > efi/arm*: libstub: Invoke EFI_RNG_PROTOCOL to seed the UEFI RNG table > > Invoke the EFI_RNG_PROTOCOL protocol in the context of the stub and > install the Linux-specific RNG seed UEFI config table. This will be > picked up by the EFI routines in the core kernel to seed the kernel > entropy pool. > > Cc: Matt Fleming > Reviewed-by: Kees Cook > Signed-off-by: Ard Biesheuvel Ah yes, this is exactly what happened. > if you rebase it (with your co-maintainer's permission) then you need to add your > SoB tag. OK, will do so in future. Thanks Ingo.