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

Groups > linux.kernel > #1203199

Re: [PATCH V10 4/5] arm64: apei: implement arch_apei_get_mem_attributes()

Show all headers | View raw

On 7 August 2015 at 20:57, Matt Fleming <matt@codeblueprint.co.uk> wrote:
> On Fri, 07 Aug, at 11:16:03AM, Zhang, Jonathan Zhixiong wrote:
>>
>> On some (future) arm64 platforms, APEI/GHES region may have full
>> coherent access by platform. In such case, the APEI/GHES region have
>> the same memory attributes as the rest of system RAM, such region
>> do not need to be advised by UEFI as separate entry, but as part of
>> system RAM memory region.
>> That being said, for arm64 platforms that do not have WB capability
>> for APEI/GHES region, such region should be mapped accordingly.
>
> OK, so what I need to know right now is whether I need to drop this
> entire series from my pull request or whether you can send a patch on
> top of the existing ones in the EFI 'next' branch to address the mapping
> heuristic in arch_apei_get_mem_attributes().
>
>> >>Jonathan, can you please provide the EFI memory map region attributes
>> >>for the GHES region that requires this series?
>> [Reserved           |   |  |  |  |   |  |  |  |UC]
>
> Assuming this memmap entry is indicative of most GHES region on arm64
> right now, I think it's worth taking this patch as-is and addressing the
> issue Ard raised as a separate patch.
>
> Does that work?
>

I think that is fine.

So we'll expect two patches on top of Matt's -next branch:
- one that removes the redundant __pgprot
- one that inverts the order in which the memory attributes are tested

It would be good to have these in the same release so that the
behavior does not change between releases.

Thanks,
Ard.
--
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

Thread

[PATCH V10 0/5] map GHES memory region according to EFI memory map "Jonathan (Zhixiong) Zhang" <zjzhang@codeaurora.org> - 2015-08-06 15:50 +0200
  [PATCH V10 1/5] efi: x86: rearrange efi_mem_attributes() "Jonathan (Zhixiong) Zhang" <zjzhang@codeaurora.org> - 2015-08-06 15:50 +0200
  [PATCH V10 4/5] arm64: apei: implement arch_apei_get_mem_attributes() "Jonathan (Zhixiong) Zhang" <zjzhang@codeaurora.org> - 2015-08-06 15:50 +0200
    Re: [PATCH V10 4/5] arm64: apei: implement arch_apei_get_mem_attributes() Ard Biesheuvel <ard.biesheuvel@linaro.org> - 2015-08-07 11:10 +0200
      Re: [PATCH V10 4/5] arm64: apei: implement  arch_apei_get_mem_attributes() Matt Fleming <matt@codeblueprint.co.uk> - 2015-08-07 11:40 +0200
        Re: [PATCH V10 4/5] arm64: apei: implement arch_apei_get_mem_attributes() Ard Biesheuvel <ard.biesheuvel@linaro.org> - 2015-08-07 12:00 +0200
          Re: [PATCH V10 4/5] arm64: apei: implement  arch_apei_get_mem_attributes() "Zhang, Jonathan Zhixiong" <zjzhang@codeaurora.org> - 2015-08-07 20:20 +0200
            Re: [PATCH V10 4/5] arm64: apei: implement  arch_apei_get_mem_attributes() Matt Fleming <matt@codeblueprint.co.uk> - 2015-08-07 21:00 +0200
              Re: [PATCH V10 4/5] arm64: apei: implement arch_apei_get_mem_attributes() Ard Biesheuvel <ard.biesheuvel@linaro.org> - 2015-08-08 10:20 +0200
                Re: [PATCH V10 4/5] arm64: apei: implement  arch_apei_get_mem_attributes() "Zhang, Jonathan Zhixiong" <zjzhang@codeaurora.org> - 2015-08-08 18:40 +0200
      Re: [PATCH V10 4/5] arm64: apei: implement  arch_apei_get_mem_attributes() "Zhang, Jonathan Zhixiong" <zjzhang@codeaurora.org> - 2015-08-07 19:50 +0200
        Re: [PATCH V10 4/5] arm64: apei: implement  arch_apei_get_mem_attributes() Matt Fleming <matt@codeblueprint.co.uk> - 2015-08-07 21:00 +0200

csiph-web