Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Grant Likely
On 15/09/2020 17:00, Heinrich Schuchardt wrote: On 15.09.20 16:50, Grant Likely wrote: On 15/09/2020 15:36, Heinrich Schuchardt wrote: On 15.09.20 16:16, Ard Biesheuvel wrote: [...] EfiRuntimeServicesData is not covered by the linear map, but will map it in the EFI page tables for acces

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Heinrich Schuchardt
On 15.09.20 16:50, Grant Likely wrote: > > > On 15/09/2020 15:36, Heinrich Schuchardt wrote: >> On 15.09.20 16:16, Ard Biesheuvel wrote: >>> On Tue, 15 Sep 2020 at 17:05, Grant Likely wrote: On 15/09/2020 14:46, Leif Lindholm wrote: > On Tue, Sep 15, 2020 at 14:14:30 +0100,

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Grant Likely
On 15/09/2020 16:31, Leif Lindholm wrote: On Tue, Sep 15, 2020 at 15:04:44 +0100, Grant Likely wrote: On 15/09/2020 14:46, Leif Lindholm wrote: EfiBootServicesData: Memory still gets mapped in the linear map, but nothing protects it after ExitBootServices (would require leaving /memreserve/ in

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Leif Lindholm
On Tue, Sep 15, 2020 at 15:04:44 +0100, Grant Likely wrote: > On 15/09/2020 14:46, Leif Lindholm wrote: > > On Tue, Sep 15, 2020 at 14:14:30 +0100, Grant Likely wrote: > > > > The EFI stub in Linux removes /memreserve/ entries from the DT before > > > > handing it to the kernel proper. > > > > > >

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Grant Likely
On 15/09/2020 15:36, Heinrich Schuchardt wrote: On 15.09.20 16:16, Ard Biesheuvel wrote: On Tue, 15 Sep 2020 at 17:05, Grant Likely wrote: On 15/09/2020 14:46, Leif Lindholm wrote: On Tue, Sep 15, 2020 at 14:14:30 +0100, Grant Likely wrote: The EFI stub in Linux removes /memreserve/ ent

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Ard Biesheuvel
On Tue, 15 Sep 2020 at 17:33, Grant Likely wrote: > > > > On 15/09/2020 15:16, Ard Biesheuvel wrote: > > On Tue, 15 Sep 2020 at 17:05, Grant Likely wrote: > >> > >> > >> > >> On 15/09/2020 14:46, Leif Lindholm wrote: > >>> On Tue, Sep 15, 2020 at 14:14:30 +0100, Grant Likely wrote: > > The EF

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Heinrich Schuchardt
On 15.09.20 16:16, Ard Biesheuvel wrote: > On Tue, 15 Sep 2020 at 17:05, Grant Likely wrote: >> >> >> >> On 15/09/2020 14:46, Leif Lindholm wrote: >>> On Tue, Sep 15, 2020 at 14:14:30 +0100, Grant Likely wrote: > The EFI stub in Linux removes /memreserve/ entries from the DT before > handi

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Grant Likely
On 15/09/2020 15:16, Ard Biesheuvel wrote: On Tue, 15 Sep 2020 at 17:05, Grant Likely wrote: On 15/09/2020 14:46, Leif Lindholm wrote: On Tue, Sep 15, 2020 at 14:14:30 +0100, Grant Likely wrote: The EFI stub in Linux removes /memreserve/ entries from the DT before handing it to the kerne

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Ard Biesheuvel
On Tue, 15 Sep 2020 at 17:05, Grant Likely wrote: > > > > On 15/09/2020 14:46, Leif Lindholm wrote: > > On Tue, Sep 15, 2020 at 14:14:30 +0100, Grant Likely wrote: > >>> The EFI stub in Linux removes /memreserve/ entries from the DT before > >>> handing it to the kernel proper. > >>> > >>> commit

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Grant Likely
On 15/09/2020 14:59, Heinrich Schuchardt wrote: On 15.09.20 15:46, Leif Lindholm wrote: On Tue, Sep 15, 2020 at 14:14:30 +0100, Grant Likely wrote: +``/memory`` node and UEFI +~~ + +When booting via [UEFI]_, the system memory map is obtained via the +GetMemoryMap() UEF

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Grant Likely
On 15/09/2020 14:46, Leif Lindholm wrote: On Tue, Sep 15, 2020 at 14:14:30 +0100, Grant Likely wrote: The EFI stub in Linux removes /memreserve/ entries from the DT before handing it to the kernel proper. commit 0ceac9e094b065fe3fec19669740f338d3480498 Author: Mark Salter Date: Mon Sep 8 1

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Heinrich Schuchardt
On 15.09.20 15:49, Ard Biesheuvel wrote: > On Tue, 15 Sep 2020 at 16:15, Grant Likely wrote: >> >> On 15/09/2020 14:02, Ard Biesheuvel wrote: >>> On Tue, 15 Sep 2020 at 15:59, Grant Likely wrote: On 15/09/2020 13:35, Ard Biesheuvel wrote: > On Tue, 15 Sep 2020 at 15:34, Gr

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Heinrich Schuchardt
On 15.09.20 15:46, Leif Lindholm wrote: > On Tue, Sep 15, 2020 at 14:14:30 +0100, Grant Likely wrote: +``/memory`` node and UEFI +~~ + +When booting via [UEFI]_, the system memory map is obtained via the +GetMemoryMap() UEFI boot

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Ard Biesheuvel
On Tue, 15 Sep 2020 at 16:15, Grant Likely wrote: > > On 15/09/2020 14:02, Ard Biesheuvel wrote: > > On Tue, 15 Sep 2020 at 15:59, Grant Likely wrote: > >> > >> > >> > >> On 15/09/2020 13:35, Ard Biesheuvel wrote: > >>> On Tue, 15 Sep 2020 at 15:34, Grant Likely wrote: > > > > >>>

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Leif Lindholm
On Tue, Sep 15, 2020 at 14:14:30 +0100, Grant Likely wrote: > > > > > > > +``/memory`` node and UEFI > > > > > > > +~~ > > > > > > > + > > > > > > > +When booting via [UEFI]_, the system memory map is obtained via > > > > > > > the > > > > > > > +GetMemoryMap() UEFI boot ti

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Grant Likely
On 15/09/2020 14:02, Ard Biesheuvel wrote: On Tue, 15 Sep 2020 at 15:59, Grant Likely wrote: On 15/09/2020 13:35, Ard Biesheuvel wrote: On Tue, 15 Sep 2020 at 15:34, Grant Likely wrote: On 15/09/2020 13:25, Ard Biesheuvel wrote: On Tue, 15 Sep 2020 at 15:22, Grant Likely wrote: On

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Ard Biesheuvel
On Tue, 15 Sep 2020 at 15:59, Grant Likely wrote: > > > > On 15/09/2020 13:35, Ard Biesheuvel wrote: > > On Tue, 15 Sep 2020 at 15:34, Grant Likely wrote: > >> > >> > >> > >> On 15/09/2020 13:25, Ard Biesheuvel wrote: > >>> On Tue, 15 Sep 2020 at 15:22, Grant Likely wrote: > > On 15/09

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Grant Likely
On 15/09/2020 13:35, Ard Biesheuvel wrote: On Tue, 15 Sep 2020 at 15:34, Grant Likely wrote: On 15/09/2020 13:25, Ard Biesheuvel wrote: On Tue, 15 Sep 2020 at 15:22, Grant Likely wrote: On 15/09/2020 09:33, Heinrich Schuchardt wrote: Closes: #52 The no-map property of the /reserved-m

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Ard Biesheuvel
On Tue, 15 Sep 2020 at 15:34, Grant Likely wrote: > > > > On 15/09/2020 13:25, Ard Biesheuvel wrote: > > On Tue, 15 Sep 2020 at 15:22, Grant Likely wrote: > >> > >> On 15/09/2020 09:33, Heinrich Schuchardt wrote: > >>> Closes: #52 > >>> > >>> The no-map property of the /reserved-memory device tre

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Grant Likely
On 15/09/2020 13:25, Ard Biesheuvel wrote: On Tue, 15 Sep 2020 at 15:22, Grant Likely wrote: On 15/09/2020 09:33, Heinrich Schuchardt wrote: Closes: #52 The no-map property of the /reserved-memory device tree nodes is used to signal that a memory region shall not be accessed by the operati

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Ard Biesheuvel
On Tue, 15 Sep 2020 at 15:22, Grant Likely wrote: > > On 15/09/2020 09:33, Heinrich Schuchardt wrote: > > Closes: #52 > > > > The no-map property of the /reserved-memory device tree nodes is used to > > signal that a memory region shall not be accessed by the operating system, > > even not via spe

Re: [PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Grant Likely
On 15/09/2020 09:33, Heinrich Schuchardt wrote: Closes: #52 The no-map property of the /reserved-memory device tree nodes is used to signal that a memory region shall not be accessed by the operating system, even not via speculative access. /reserved-memory nodes without the no-map property des

[PATCH 1/1] GetMemoryMap(), handling of no-map DT property

2020-09-15 Thread Heinrich Schuchardt
Closes: #52 The no-map property of the /reserved-memory device tree nodes is used to signal that a memory region shall not be accessed by the operating system, even not via speculative access. /reserved-memory nodes without the no-map property describe memory that have special usage by the operat