Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1223955
| From | Ard Biesheuvel <ard.biesheuvel@linaro.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters |
| Date | 2015-09-14 11:10 +0200 |
| Message-ID | <q8xVV-1GG-29@gated-at.bofh.it> (permalink) |
| References | (7 earlier) <q7vWa-1Oz-21@gated-at.bofh.it> <q7wpc-2BZ-11@gated-at.bofh.it> <q7wIy-31c-13@gated-at.bofh.it> <q7yKl-5Yf-5@gated-at.bofh.it> <q8xMg-1gc-41@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On 14 September 2015 at 10:42, Shannon Zhao <zhaoshenglong@huawei.com> wrote:
[..]
>
> It only needs to apply following patch to fix a bug in Linux kernel when
> mapping EFI_MEMORY_RUNTIME memory.
>
Could you explain why you think efi_virtmap_init() should fail if
there are no EFI_MEMORY_RUNTIME regions?
The absence of such regions is allowed by the spec, so
efi_virtmap_init() is correct imo to return success.
If you are trying to work around the issue where Xen does not expose
any Runtime Services regions, there is simply no way to do that and be
still UEFI compliant. I have suggested before that we should perhaps
tolerate this anyway, by considering the case where the EFI System
Table has a NULL runtime services pointer. But rigging
efi_virtmap_init() like this is really not the way to achieve that.
--
Ard.
> Author: Shannon Zhao <shannon.zhao@linaro.org>
> Date: Thu Aug 20 14:54:58 2015 +0800
>
> arm64/efi: Fix a bug when no EFI_MEMORY_RUNTIME memory found
>
> Currently if the attribute type of all the EFI Memory Descriptors are
> not EFI_MEMORY_RUNTIME, efi_virtmap_init will return true. But at this
> case, it expect false as there are no EFI memory for RUNTIME. Fix it by
> introducing a status to show whether it finds EFI_MEMORY_RUNTIME.
>
> Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org>
>
> diff --git a/arch/arm64/kernel/efi.c b/arch/arm64/kernel/efi.c
> index e8ca6ea..bad7f87 100644
> --- a/arch/arm64/kernel/efi.c
> +++ b/arch/arm64/kernel/efi.c
> @@ -233,6 +233,7 @@ void __init efi_init(void)
> static bool __init efi_virtmap_init(void)
> {
> efi_memory_desc_t *md;
> + bool status = false;
>
> for_each_efi_memory_desc(&memmap, md) {
> u64 paddr, npages, size;
> @@ -264,8 +265,11 @@ static bool __init efi_virtmap_init(void)
> prot = PAGE_KERNEL;
>
> create_pgd_mapping(&efi_mm, paddr, md->virt_addr, size,
> prot);
> + status = true;
> }
> - return true;
> + if (status)
> + return true;
> + return false;
> }
>
> --
> Shannon
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Shannon Zhao <zhaoshenglong@huawei.com> - 2015-09-10 11:00 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Mark Rutland <mark.rutland@arm.com> - 2015-09-10 12:00 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Stefano Stabellini <stefano.stabellini@eu.citrix.com> - 2015-09-10 12:30 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Mark Rutland <mark.rutland@arm.com> - 2015-09-10 13:30 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Stefano Stabellini <stefano.stabellini@eu.citrix.com> - 2015-09-10 13:50 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Mark Rutland <mark.rutland@arm.com> - 2015-09-10 14:20 +0200
Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Ian Campbell <ian.campbell@citrix.com> - 2015-09-10 15:00 +0200
Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters "Jan Beulich" <JBeulich@suse.com> - 2015-09-10 15:10 +0200
Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Ian Campbell <ian.campbell@citrix.com> - 2015-09-10 15:40 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Stefano Stabellini <stefano.stabellini@eu.citrix.com> - 2015-09-10 16:00 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Leif Lindholm <leif.lindholm@linaro.org> - 2015-09-10 16:20 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Mark Rutland <mark.rutland@arm.com> - 2015-09-10 16:50 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Stefano Stabellini <stefano.stabellini@eu.citrix.com> - 2015-09-10 18:20 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Mark Rutland <mark.rutland@arm.com> - 2015-09-10 18:30 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Daniel Kiper <daniel.kiper@oracle.com> - 2015-09-11 14:50 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Stefano Stabellini <stefano.stabellini@eu.citrix.com> - 2015-09-11 15:20 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Ard Biesheuvel <ard.biesheuvel@linaro.org> - 2015-09-11 15:40 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Daniel Kiper <daniel.kiper@oracle.com> - 2015-09-11 17:50 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Shannon Zhao <zhaoshenglong@huawei.com> - 2015-09-14 11:00 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Ard Biesheuvel <ard.biesheuvel@linaro.org> - 2015-09-14 11:10 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Shannon Zhao <zhaoshenglong@huawei.com> - 2015-09-14 11:40 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Ard Biesheuvel <ard.biesheuvel@linaro.org> - 2015-09-14 11:40 +0200
Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters "Jan Beulich" <JBeulich@suse.com> - 2015-09-14 12:40 +0200
Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Ard Biesheuvel <ard.biesheuvel@linaro.org> - 2015-09-14 13:20 +0200
Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters "Jan Beulich" <JBeulich@suse.com> - 2015-09-14 13:40 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Mark Rutland <mark.rutland@arm.com> - 2015-09-11 18:40 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Mark Rutland <mark.rutland@arm.com> - 2015-09-11 18:30 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Daniel Kiper <daniel.kiper@oracle.com> - 2015-09-12 13:50 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Mark Rutland <mark.rutland@arm.com> - 2015-09-14 11:30 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Stefano Stabellini <stefano.stabellini@eu.citrix.com> - 2015-09-14 11:50 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Daniel Kiper <daniel.kiper@oracle.com> - 2015-09-14 14:30 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Ian Campbell <ian.campbell@citrix.com> - 2015-09-11 13:10 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters "Jan Beulich" <JBeulich@suse.com> - 2015-09-10 15:00 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Mark Rutland <mark.rutland@arm.com> - 2015-09-10 17:00 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters "Jan Beulich" <JBeulich@suse.com> - 2015-09-10 17:10 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Mark Rutland <mark.rutland@arm.com> - 2015-09-11 18:40 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Andrew Turner <andrew@fubar.geek.nz> - 2015-09-10 13:50 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Julien Grall <julien.grall@citrix.com> - 2015-09-10 13:50 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Roger Pau Monné <roger.pau@citrix.com> - 2015-09-10 14:10 +0200
Re: [Xen-devel] [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Julien Grall <julien.grall@citrix.com> - 2015-09-10 15:00 +0200
Re: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Shannon Zhao <zhaoshenglong@huawei.com> - 2015-09-17 14:00 +0200
csiph-web