* [PATCH] efi/arm64: store Runtime Services revision @ 2014-08-14 14:55 Semen Protsenko 2014-08-15 6:10 ` Ard Biesheuvel 2014-08-26 9:05 ` Will Deacon 0 siblings, 2 replies; 6+ messages in thread From: Semen Protsenko @ 2014-08-14 14:55 UTC (permalink / raw) To: linux-arm-kernel "efi" global data structure contains "runtime_version" field which must be assigned in order to use it later in Runtime Services virtual calls (virt_efi_* functions). Before this patch "runtime_version" was unassigned (0), so each Runtime Service virtual call that checks revision would fail. Runtime Services revision being passed in UEFI system table from UEFI to kernel (see efi_entry() function). In UEFI it's being stored at MdePkg/Include/Uefi/UefiSpec.h as EFI_RUNTIME_SERVICES_REVISION. Signed-off-by: Semen Protsenko <semen.protsenko@linaro.org> --- arch/arm64/kernel/efi.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm64/kernel/efi.c b/arch/arm64/kernel/efi.c index e72f310..5dbb7bd 100644 --- a/arch/arm64/kernel/efi.c +++ b/arch/arm64/kernel/efi.c @@ -463,6 +463,8 @@ static int __init arm64_enter_virtual_mode(void) efi_native_runtime_setup(); set_bit(EFI_RUNTIME_SERVICES, &efi.flags); + efi.runtime_version = efi.systab->hdr.revision; + return 0; err_unmap: -- 2.0.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] efi/arm64: store Runtime Services revision 2014-08-14 14:55 [PATCH] efi/arm64: store Runtime Services revision Semen Protsenko @ 2014-08-15 6:10 ` Ard Biesheuvel 2014-08-26 9:05 ` Will Deacon 1 sibling, 0 replies; 6+ messages in thread From: Ard Biesheuvel @ 2014-08-15 6:10 UTC (permalink / raw) To: linux-arm-kernel On 14 August 2014 16:55, Semen Protsenko <semen.protsenko@linaro.org> wrote: > "efi" global data structure contains "runtime_version" field which must > be assigned in order to use it later in Runtime Services virtual calls > (virt_efi_* functions). > > Before this patch "runtime_version" was unassigned (0), so each > Runtime Service virtual call that checks revision would fail. > > Runtime Services revision being passed in UEFI system table from UEFI > to kernel (see efi_entry() function). In UEFI it's being stored at > MdePkg/Include/Uefi/UefiSpec.h as EFI_RUNTIME_SERVICES_REVISION. > Could you drop the last line with the Tianocore path? Tianocore is just one of the many UEFI implementations the kernel should be compatible with, so where it happens to store this value is not entirely relevant. > Signed-off-by: Semen Protsenko <semen.protsenko@linaro.org> Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> > --- > arch/arm64/kernel/efi.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/arm64/kernel/efi.c b/arch/arm64/kernel/efi.c > index e72f310..5dbb7bd 100644 > --- a/arch/arm64/kernel/efi.c > +++ b/arch/arm64/kernel/efi.c > @@ -463,6 +463,8 @@ static int __init arm64_enter_virtual_mode(void) > efi_native_runtime_setup(); > set_bit(EFI_RUNTIME_SERVICES, &efi.flags); > > + efi.runtime_version = efi.systab->hdr.revision; > + > return 0; > > err_unmap: > -- > 2.0.1 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] efi/arm64: store Runtime Services revision 2014-08-14 14:55 [PATCH] efi/arm64: store Runtime Services revision Semen Protsenko 2014-08-15 6:10 ` Ard Biesheuvel @ 2014-08-26 9:05 ` Will Deacon 2014-08-26 18:24 ` Matt Fleming 2014-10-13 16:14 ` Sam Protsenko 1 sibling, 2 replies; 6+ messages in thread From: Will Deacon @ 2014-08-26 9:05 UTC (permalink / raw) To: linux-arm-kernel On Thu, Aug 14, 2014 at 03:55:22PM +0100, Semen Protsenko wrote: > "efi" global data structure contains "runtime_version" field which must > be assigned in order to use it later in Runtime Services virtual calls > (virt_efi_* functions). > > Before this patch "runtime_version" was unassigned (0), so each > Runtime Service virtual call that checks revision would fail. > > Runtime Services revision being passed in UEFI system table from UEFI > to kernel (see efi_entry() function). In UEFI it's being stored at > MdePkg/Include/Uefi/UefiSpec.h as EFI_RUNTIME_SERVICES_REVISION. > > Signed-off-by: Semen Protsenko <semen.protsenko@linaro.org> > --- > arch/arm64/kernel/efi.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/arm64/kernel/efi.c b/arch/arm64/kernel/efi.c > index e72f310..5dbb7bd 100644 > --- a/arch/arm64/kernel/efi.c > +++ b/arch/arm64/kernel/efi.c > @@ -463,6 +463,8 @@ static int __init arm64_enter_virtual_mode(void) > efi_native_runtime_setup(); > set_bit(EFI_RUNTIME_SERVICES, &efi.flags); > > + efi.runtime_version = efi.systab->hdr.revision; > + > return 0; I doubt it makes any difference, but can we assign this before we call efi_native_runtime_setup please? Will ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] efi/arm64: store Runtime Services revision 2014-08-26 9:05 ` Will Deacon @ 2014-08-26 18:24 ` Matt Fleming 2014-08-26 18:27 ` Will Deacon 2014-10-13 16:14 ` Sam Protsenko 1 sibling, 1 reply; 6+ messages in thread From: Matt Fleming @ 2014-08-26 18:24 UTC (permalink / raw) To: linux-arm-kernel On Tue, 26 Aug, at 10:05:33AM, Will Deacon wrote: > > I doubt it makes any difference, but can we assign this before we call > efi_native_runtime_setup please? This would have to be a follow up patch, v2 of this patch is already in Linus' tree as commit 6a7519e81321. -- Matt Fleming, Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] efi/arm64: store Runtime Services revision 2014-08-26 18:24 ` Matt Fleming @ 2014-08-26 18:27 ` Will Deacon 0 siblings, 0 replies; 6+ messages in thread From: Will Deacon @ 2014-08-26 18:27 UTC (permalink / raw) To: linux-arm-kernel On Tue, Aug 26, 2014 at 07:24:50PM +0100, Matt Fleming wrote: > On Tue, 26 Aug, at 10:05:33AM, Will Deacon wrote: > > > > I doubt it makes any difference, but can we assign this before we call > > efi_native_runtime_setup please? > > This would have to be a follow up patch, v2 of this patch is already in > Linus' tree as commit 6a7519e81321. Ah, ok, I wouldn't worry about it then. I was just trawling the list for arm64 patches without me on CC in case I missed anything during the kernel summit. Thanks for picking it up! Will ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] efi/arm64: store Runtime Services revision 2014-08-26 9:05 ` Will Deacon 2014-08-26 18:24 ` Matt Fleming @ 2014-10-13 16:14 ` Sam Protsenko 1 sibling, 0 replies; 6+ messages in thread From: Sam Protsenko @ 2014-10-13 16:14 UTC (permalink / raw) To: linux-arm-kernel > I doubt it makes any difference It really does. The whole reason I made this patch is that I was unable to call UpdateCapsule() runtime service, because when you try to call it, kernel code first tests if we have UEFI with certain Runtime Services revision. In this code (drivers/firmware/efi/runtime.c): static efi_status_t virt_efi_update_capsule(efi_capsule_header_t **capsules, unsigned long count, unsigned long sg_list) { if (efi.runtime_version < EFI_2_00_SYSTEM_TABLE_REVISION) return EFI_UNSUPPORTED; return efi_call_virt(update_capsule, capsules, count, sg_list); } Condition is false, because efi.runtime_version == 0. The same stuff was happening about some other Runtime Services. So this patch just fixes this. > but can we assign this before we call efi_native_runtime_setup please? Yeah, it's quite late for this as patch is already upstreamed :) Anyway, if you could explain why it's essential -- I can make another patch moving this code to correct place. On 26 August 2014 12:05, Will Deacon <will.deacon@arm.com> wrote: > On Thu, Aug 14, 2014 at 03:55:22PM +0100, Semen Protsenko wrote: >> "efi" global data structure contains "runtime_version" field which must >> be assigned in order to use it later in Runtime Services virtual calls >> (virt_efi_* functions). >> >> Before this patch "runtime_version" was unassigned (0), so each >> Runtime Service virtual call that checks revision would fail. >> >> Runtime Services revision being passed in UEFI system table from UEFI >> to kernel (see efi_entry() function). In UEFI it's being stored at >> MdePkg/Include/Uefi/UefiSpec.h as EFI_RUNTIME_SERVICES_REVISION. >> >> Signed-off-by: Semen Protsenko <semen.protsenko@linaro.org> >> --- >> arch/arm64/kernel/efi.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/arch/arm64/kernel/efi.c b/arch/arm64/kernel/efi.c >> index e72f310..5dbb7bd 100644 >> --- a/arch/arm64/kernel/efi.c >> +++ b/arch/arm64/kernel/efi.c >> @@ -463,6 +463,8 @@ static int __init arm64_enter_virtual_mode(void) >> efi_native_runtime_setup(); >> set_bit(EFI_RUNTIME_SERVICES, &efi.flags); >> >> + efi.runtime_version = efi.systab->hdr.revision; >> + >> return 0; > > I doubt it makes any difference, but can we assign this before we call > efi_native_runtime_setup please? > > Will ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-10-13 16:14 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-08-14 14:55 [PATCH] efi/arm64: store Runtime Services revision Semen Protsenko 2014-08-15 6:10 ` Ard Biesheuvel 2014-08-26 9:05 ` Will Deacon 2014-08-26 18:24 ` Matt Fleming 2014-08-26 18:27 ` Will Deacon 2014-10-13 16:14 ` Sam Protsenko
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).