[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAKv+Gu930691BaDLuoVOU_gz0V8MtfWbfSTKcYuHxDvLz5hAiA@mail.gmail.com>
Date: Thu, 20 Feb 2020 10:21:29 +0100
From: Ard Biesheuvel <ardb@...nel.org>
To: Jing Xiangfeng <jingxiangfeng@...wei.com>
Cc: Thomas Gleixner <tglx@...utronix.de>,
Ingo Molnar <mingo@...hat.com>, Borislav Petkov <bp@...en8.de>,
"H. Peter Anvin" <hpa@...or.com>,
"the arch/x86 maintainers" <x86@...nel.org>,
Darren Hart <dvhart@...radead.org>,
Andy Shevchenko <andy@...radead.org>,
linux-efi <linux-efi@...r.kernel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
platform-driver-x86@...r.kernel.org, Linux-MM <linux-mm@...ck.org>,
Kefeng Wang <wangkefeng.wang@...wei.com>
Subject: Re: [PATCH 0/2] arm64: Support to find mirrored memory ranges
On Thu, 20 Feb 2020 at 04:43, Jing Xiangfeng <jingxiangfeng@...wei.com> wrote:
>
> This series enable finding mirrored memory ranges
> functionality on arm64 platform. This feature has been
> implemented on the x86 platform, so we move some
> functions from x86.
>
Hello Jing Xiangfeng,
Could you explain your use case a bit better? Usually, the firmware is
a better place to make modifications to the EFI memory map.
The reason I am asking is that currently, on ARM and arm64, we never
make *any* changes to the firmware provided tables (EFI system table,
EFI memory map, DT/ACPI/SMBIOS tables etc), in order to ensure that
kexec is idempotent, i.e., it will always see the exact same state as
far as the firmware is concerned. This is a bit different from x86,
where the memory map is already modified for various other reasons, so
using it for fake memory regions is not such a big deal.
Do you see a use case for this in production?
> Jing Xiangfeng (2):
> efi: allow EFI_FAKE_MEMMAP on arm64 platform
> arm64/efi: support to find mirrored memory ranges
>
> arch/x86/include/asm/efi.h | 5 -----
> arch/x86/platform/efi/efi.c | 39 ---------------------------------------
> drivers/firmware/efi/Kconfig | 2 +-
> drivers/firmware/efi/arm-init.c | 2 ++
> drivers/firmware/efi/efi.c | 23 +++++++++++++++++++++++
> drivers/firmware/efi/memmap.c | 16 ++++++++++++++++
> include/linux/efi.h | 5 +++++
> 7 files changed, 47 insertions(+), 45 deletions(-)
>
> --
> 1.8.3.1
>
Powered by blists - more mailing lists