Re: [PATCH] arm64: efi: ignore EFI_MEMORY_XP attribute if RP and/or WP are set

2017-09-22 Thread Stephen Boyd
On 09/15, Ard Biesheuvel wrote: > On 15 September 2017 at 11:53, Stephen Boyd wrote: > > On 09/14, Ard Biesheuvel wrote: > >> The UEFI memory map is a bit vague about how to interpret the > >> EFI_MEMORY_XP attribute when it is combined with EFI_MEMORY_RP and/or > >> EFI_MEMORY_WP, which have retr

Re: [PATCH] arm64: efi: ignore EFI_MEMORY_XP attribute if RP and/or WP are set

2017-09-15 Thread Ard Biesheuvel
On 15 September 2017 at 11:53, Stephen Boyd wrote: > On 09/14, Ard Biesheuvel wrote: >> The UEFI memory map is a bit vague about how to interpret the >> EFI_MEMORY_XP attribute when it is combined with EFI_MEMORY_RP and/or >> EFI_MEMORY_WP, which have retroactively been redefined as cacheability >

Re: [PATCH] arm64: efi: ignore EFI_MEMORY_XP attribute if RP and/or WP are set

2017-09-15 Thread Stephen Boyd
On 09/14, Ard Biesheuvel wrote: > The UEFI memory map is a bit vague about how to interpret the > EFI_MEMORY_XP attribute when it is combined with EFI_MEMORY_RP and/or > EFI_MEMORY_WP, which have retroactively been redefined as cacheability > attributes rather than permission attributes. > > So le

Re: [PATCH] arm64: efi: ignore EFI_MEMORY_XP attribute if RP and/or WP are set

2017-09-14 Thread Ard Biesheuvel
On 14 September 2017 at 12:31, Ard Biesheuvel wrote: > The UEFI memory map *spec* > is a bit vague about how to interpret the > EFI_MEMORY_XP attribute when it is combined with EFI_MEMORY_RP and/or > EFI_MEMORY_WP, which have retroactively been redefined as cacheability > attributes rather than

[PATCH] arm64: efi: ignore EFI_MEMORY_XP attribute if RP and/or WP are set

2017-09-14 Thread Ard Biesheuvel
The UEFI memory map is a bit vague about how to interpret the EFI_MEMORY_XP attribute when it is combined with EFI_MEMORY_RP and/or EFI_MEMORY_WP, which have retroactively been redefined as cacheability attributes rather than permission attributes. So let's ignore EFI_MEMORY_XP if _RP and/or _WP a