• Ard Biesheuvel's avatar
    efi/arm64: Don't apply MEMBLOCK_NOMAP to UEFI memory map mapping · 7cc8cbcf
    Ard Biesheuvel authored
    Commit 4dffbfc4 ("arm64/efi: mark UEFI reserved regions as
    MEMBLOCK_NOMAP") updated the mapping logic of both the RuntimeServices
    regions as well as the kernel's copy of the UEFI memory map to set the
    MEMBLOCK_NOMAP flag, which causes these regions to be omitted from the
    kernel direct mapping, and from being covered by a struct page.
    For the RuntimeServices regions, this is an obvious win, since the contents
    of these regions have significance to the firmware executable code itself,
    and are mapped in the EFI page tables using attributes that are described in
    the UEFI memory map, and which may differ from the attributes we use for
    mapping system RAM. It also prevents the contents from being modified
    inadvertently, since the EFI page tables are only live during runtime
    service invocations.
    None of these concerns apply to the allocation that covers the UEFI memory
    map, since it is entirely owned by the kernel. Setting the MEMBLOCK_NOMAP on
    the region did allow us to use ioremap_cache() to map it both on arm64 and
    on ARM, since the latter does not allow ioremap_cache() to be used on
    regions that are covered by a struct page.
    The ioremap_cache() on ARM restriction will be lifted in the v4.7 timeframe,
    but in the mean time, it has been reported that commit 4dffbfc4 causes
    a regression on 64k granule kernels. This is due to the fact that, given
    the 64 KB page size, the region that we end up removing from the kernel
    direct mapping is rounded up to 64 KB, and this 64 KB page frame may be
    shared with the initrd when booting via GRUB (which does not align its
    EFI_LOADER_DATA allocations to 64 KB like the stub does). This will crash
    the kernel as soon as it tries to access the initrd.
    Since the issue is specific to arm64, revert back to memblock_reserve()'ing
    the UEFI memory map when running on arm64. This is a temporary fix for v4.5
    and v4.6, and will be superseded in the v4.7 timeframe when we will be able
    to move back to memblock_reserve() unconditionally.
    Fixes: 4dffbfc4
     ("arm64/efi: mark UEFI reserved regions as MEMBLOCK_NOMAP")
    Reported-by: default avatarMark Salter <msalter@redhat.com>
    Signed-off-by: default avatarArd Biesheuvel <ard.biesheuvel@linaro.org>
    Acked-by: default avatarWill Deacon <will.deacon@arm.com>
    Cc: Leif Lindholm <leif.lindholm@linaro.org>
    Cc: Mark Rutland <mark.rutland@arm.com>
    Cc: Jeremy Linton <jeremy.linton@arm.com>
    Cc: Mark Langsdorf <mlangsdo@redhat.com>
    Cc: <stable@vger.kernel.org> # v4.5
    Signed-off-by: default avatarMatt Fleming <matt@codeblueprint.co.uk>