Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1202003 > unrolled thread

[GIT PULL] EFI changes for v4.3

Back to article view | Back to linux.kernel

Contents

  [GIT PULL] EFI changes for v4.3 Matt Fleming <matt@codeblueprint.co.uk> - 2015-08-06 22:40 +0200
    Re: [PATCH 7/8] arm64: apei: implement arch_apei_get_mem_attributes() Matt Fleming <matt@codeblueprint.co.uk> - 2015-08-07 12:00 +0200
      Re: [PATCH 7/8] arm64: apei: implement arch_apei_get_mem_attributes() Ingo Molnar <mingo@kernel.org> - 2015-08-08 10:50 +0200
    Re: [PATCH 4/8] efi: x86: rearrange efi_mem_attributes() "Zhang, Jonathan Zhixiong" <zjzhang@codeaurora.org> - 2015-08-07 19:50 +0200
      Re: [PATCH 4/8] efi: x86: rearrange efi_mem_attributes() Matt Fleming <matt@codeblueprint.co.uk> - 2015-08-07 20:50 +0200

#1202003 — [GIT PULL] EFI changes for v4.3

Folks, please pull the following changes for v4.3. The biggest changes
are the memory map querying updates in the ACPI GHES driver and the
revert of commit aeffc4928ea2 ("x86/efi: Request desired alignment via
the PE/COFF headers") which was reportedly causing errors when users
attempt to sign their kernels with Signtool.

The GHES driver changes touch a mixture of ACPI, x86 and arm64 code but
all the patches have been ACKed and/or reviewed by relevant people.

The following changes since commit bc0195aad0daa2ad5b0d76cce22b167bc3435590:

  Linux 4.2-rc2 (2015-07-12 15:10:30 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/mfleming/efi.git tags/efi-next

for you to fetch changes up to 8229722e29da369a4e9512a5c18adb4028a7626b:

  acpi, apei: use appropriate pgprot_t to map GHES memory (2015-08-06 15:44:41 +0100)

----------------------------------------------------------------
 * Add support for the new EFI memory map attribute EFI_MEMORY_RO that
   was introduced in the UEFI 2.5 spec - Ard Biesheuvel

 * Lower the severity of the invalid BGRT message from pr_err()
   to pr_debug() since it's legitimate (per the spec) for the firmware
   to pass the kernel a BGRT with the invalid status bit set. The kernel
   just simply needs to ignore it.

 * Revert the optimisation of using CONFIG_PHYSICAL_ALIGN in the
   PE/COFF header SectionAlignment field of the EFI stub, since it
   violates the PE/COFF spec which says SizeOfImage must be a multiple
   of SectionAlignment, and because it was causing Signtool to fail.

 * Update the ACPI Platform Error Interface (APEI) GHES driver to query
   the EFI memory map for the correct cache attributes to use when
   mapping pages accessed by both the kernel and firmware. This prevents
   stale data being read by the kernel - Jonathan (Zhixiong) Zhang

----------------------------------------------------------------
Ard Biesheuvel (1):
      efi: Add support for EFI_MEMORY_RO attribute introduced by UEFIv2.5

Jonathan (Zhixiong) Zhang (5):
      efi: x86: rearrange efi_mem_attributes()
      x86: acpi: implement arch_apei_get_mem_attributes()
      arm64: mm: add PROT_DEVICE_nGnRnE and PROT_NORMAL_WT
      arm64: apei: implement arch_apei_get_mem_attributes()
      acpi, apei: use appropriate pgprot_t to map GHES memory

Matt Fleming (2):
      x86/efi-bgrt: Switch pr_err() to pr_debug() for invalid BGRT
      Revert "x86/efi: Request desired alignment via the PE/COFF headers"

 arch/arm64/include/asm/acpi.h    | 27 +++++++++++++++++++++++++++
 arch/arm64/include/asm/memory.h  |  1 +
 arch/arm64/include/asm/pgtable.h |  2 ++
 arch/arm64/mm/proc.S             |  4 +++-
 arch/x86/boot/header.S           |  2 +-
 arch/x86/include/asm/acpi.h      | 24 ++++++++++++++++++++++++
 arch/x86/platform/efi/efi-bgrt.c |  9 +++++++--
 arch/x86/platform/efi/efi.c      | 18 ------------------
 drivers/acpi/apei/ghes.c         |  6 ++++--
 drivers/firmware/efi/efi.c       | 39 ++++++++++++++++++++++++++++++++++++---
 include/linux/efi.h              |  1 +
 11 files changed, 106 insertions(+), 27 deletions(-)

-- 
Matt Fleming, Intel Open Source Technology Center
--
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/

[toc] | [next] | [standalone]

#1202529 — Re: [PATCH 7/8] arm64: apei: implement arch_apei_get_mem_attributes()

On Fri, 07 Aug, at 09:37:00AM, Matt Fleming wrote:
> From: "Jonathan (Zhixiong) Zhang" <zjzhang@codeaurora.org>
> 
> Table 8 of UEFI 2.5 section 2.3.6.1 defines mappings from EFI
> memory types to MAIR attribute encodings for arm64.
> 
> If the physical address has memory attributes defined by EFI
> memmap as EFI_MEMORY_[UC|WC|WT], return approprate page protection
> type according to the UEFI spec. Otherwise, return PAGE_KERNEL.
> 
> Reviewed-by: Catalin Marinas <catalin.marinas@arm.com>
> Acked-by: Hanjun Guo <hanjun.guo@linaro.org>
> Signed-off-by: Jonathan (Zhixiong) Zhang <zjzhang@codeaurora.org>
> Signed-off-by: Matt Fleming <matt.fleming@intel.com>
> ---
>  arch/arm64/include/asm/acpi.h | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)
> 

Ard Biesheuvel has brought up a concern with this patch and it's
currently under discussion here,

  https://lkml.kernel.org/r/CAKv+Gu8+g3dTaNzXjSD=5GxproaHM+Eu1hLmRWp7cVrQimzbHQ@mail.gmail.com

-- 
Matt Fleming, Intel Open Source Technology Center
--
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/

[toc] | [prev] | [next] | [standalone]

#1203224 — Re: [PATCH 7/8] arm64: apei: implement arch_apei_get_mem_attributes()

* Matt Fleming <matt@codeblueprint.co.uk> wrote:

> On Fri, 07 Aug, at 09:37:00AM, Matt Fleming wrote:
> > From: "Jonathan (Zhixiong) Zhang" <zjzhang@codeaurora.org>
> > 
> > Table 8 of UEFI 2.5 section 2.3.6.1 defines mappings from EFI
> > memory types to MAIR attribute encodings for arm64.
> > 
> > If the physical address has memory attributes defined by EFI
> > memmap as EFI_MEMORY_[UC|WC|WT], return approprate page protection
> > type according to the UEFI spec. Otherwise, return PAGE_KERNEL.
> > 
> > Reviewed-by: Catalin Marinas <catalin.marinas@arm.com>
> > Acked-by: Hanjun Guo <hanjun.guo@linaro.org>
> > Signed-off-by: Jonathan (Zhixiong) Zhang <zjzhang@codeaurora.org>
> > Signed-off-by: Matt Fleming <matt.fleming@intel.com>
> > ---
> >  arch/arm64/include/asm/acpi.h | 27 +++++++++++++++++++++++++++
> >  1 file changed, 27 insertions(+)
> > 
> 
> Ard Biesheuvel has brought up a concern with this patch and it's
> currently under discussion here,
> 
>   https://lkml.kernel.org/r/CAKv+Gu8+g3dTaNzXjSD=5GxproaHM+Eu1hLmRWp7cVrQimzbHQ@mail.gmail.com

Ok, I applied the patches up to this point, to tip:core/efi, to get the ball 
rolling - and will push them out after testing:

 8d446c8647c9 arm64/mm: Add PROT_DEVICE_nGnRnE and PROT_NORMAL_WT
 b40227fbfb1f acpi, x86: Implement arch_apei_get_mem_attributes()
 7bf793115dd9 efi, x86: Rearrange efi_mem_attributes()
 fa5c35011a8d Revert "x86/efi: Request desired alignment via the PE/COFF headers"
 248fbcd5aee0 x86/efi-bgrt: Switch pr_err() to pr_debug() for invalid BGRT
 87db73aebf55 efi: Add support for EFI_MEMORY_RO attribute introduced by UEFIv2.5

Let me know when the concerns are resolved or there are other patches to add.

Thanks,

	Ingo
--
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/

[toc] | [prev] | [next] | [standalone]

#1202954 — Re: [PATCH 4/8] efi: x86: rearrange efi_mem_attributes()

Thank you Matt for your help. I tried to find the cover letter
and patch 1, 2, 3 of the patch set. But I did not find them.
Probably the subject has typo? I appreciate that you re-post the
patch set to get more feed backs, I just wanted to be sure.

-- 
Jonathan (Zhixiong) Zhang
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project
--
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/

[toc] | [prev] | [next] | [standalone]

#1202995 — Re: [PATCH 4/8] efi: x86: rearrange efi_mem_attributes()

On Fri, 07 Aug, at 10:43:46AM, Zhang, Jonathan Zhixiong wrote:
> Thank you Matt for your help. I tried to find the cover letter
> and patch 1, 2, 3 of the patch set. But I did not find them.
> Probably the subject has typo? I appreciate that you re-post the
> patch set to get more feed backs, I just wanted to be sure.

I sent the patches as follow-up to my GIT PULL request here,

  https://lkml.kernel.org/r/20150806203232.GH4332@codeblueprint.co.uk

-- 
Matt Fleming, Intel Open Source Technology Center
--
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/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web