On Fri, Sep 13, 2019 at 2:06 AM Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> wrote: > > On Fri, 30 Aug 2019 at 03:06, Dan Williams <dan.j.williams@xxxxxxxxx> wrote: > > > > In preparation for adding another EFI_MEMMAP dependent call that needs > > to occur before e820__memblock_setup() fixup the existing efi calls to > > check for EFI_MEMMAP internally. This ends up being cleaner than the > > alternative of checking EFI_MEMMAP multiple times in setup_arch(). > > > > Cc: <x86@xxxxxxxxxx> > > Cc: Ingo Molnar <mingo@xxxxxxxxxx> > > Cc: "H. Peter Anvin" <hpa@xxxxxxxxx> > > Cc: Andy Lutomirski <luto@xxxxxxxxxx> > > Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx> > > Cc: Peter Zijlstra <peterz@xxxxxxxxxxxxx> > > Reviewed-by: Dave Hansen <dave.hansen@xxxxxxxxxxxxxxx> > > Signed-off-by: Dan Williams <dan.j.williams@xxxxxxxxx> > > I'd prefer it if the spurious whitespace changes could be dropped, but > otherwise, this looks fine to me, so I am not going to obsess about > it. Fair point, I'll drop those when I resubmit after -rc1. > Reviewed-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Thanks!