Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-12 Thread Simon Glass
Hi Heinrich, On Wed, 12 Jun 2024 at 00:11, Heinrich Schuchardt wrote: > > > > Am 12. Juni 2024 04:42:00 MESZ schrieb Simon Glass : > >Hi Tom, > > > >On Tue, 11 Jun 2024 at 16:54, Tom Rini wrote: > >> > >> On Tue, Jun 11, 2024 at 04:22:25PM -0600, Simon Glass wrote: > >> > Hi Tom, > >> > > >> > O

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-12 Thread Simon Glass
Hi Ilias, On Tue, 11 Jun 2024 at 23:49, Ilias Apalodimas wrote: > > [...] > > > > > > > > > > + struct event_lmb_map_update *lmb_map = > > > > > > > > > &event->data.lmb_map; > > > > > > > > > + > > > > > > > > > + addr = (uintptr_t)map_sysmem(lmb_map->base, 0); > > > > > > > > > + pages =

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-12 Thread Sughosh Ganu
On Wed, 12 Jun 2024 at 12:16, Ilias Apalodimas wrote: > > On Mon, 10 Jun 2024 at 18:54, Simon Glass wrote: > > > > Hi, > > > > On Mon, 10 Jun 2024 at 09:42, Sughosh Ganu wrote: > > > > > > On Mon, 10 Jun 2024 at 20:47, Heinrich Schuchardt > > > wrote: > > > > > > > > On 07.06.24 20:52, Sughosh

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-11 Thread Ilias Apalodimas
On Mon, 10 Jun 2024 at 18:54, Simon Glass wrote: > > Hi, > > On Mon, 10 Jun 2024 at 09:42, Sughosh Ganu wrote: > > > > On Mon, 10 Jun 2024 at 20:47, Heinrich Schuchardt > > wrote: > > > > > > On 07.06.24 20:52, Sughosh Ganu wrote: > > > > There are events that would be used to notify other inte

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-11 Thread Sughosh Ganu
hi Ilias, On Wed, 12 Jun 2024 at 11:19, Ilias Apalodimas wrote: > > [...] > > > > > > > > > > + struct event_lmb_map_update *lmb_map = > > > > > > > > > &event->data.lmb_map; > > > > > > > > > + > > > > > > > > > + addr = (uintptr_t)map_sysmem(lmb_map->base, 0); > > > > > > > > > + pages =

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-11 Thread Heinrich Schuchardt
Am 12. Juni 2024 04:42:00 MESZ schrieb Simon Glass : >Hi Tom, > >On Tue, 11 Jun 2024 at 16:54, Tom Rini wrote: >> >> On Tue, Jun 11, 2024 at 04:22:25PM -0600, Simon Glass wrote: >> > Hi Tom, >> > >> > On Tue, 11 Jun 2024 at 15:01, Tom Rini wrote: >> > > >> > > On Tue, Jun 11, 2024 at 12:52:19P

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-11 Thread Ilias Apalodimas
[...] > > > > > > > > + struct event_lmb_map_update *lmb_map = &event->data.lmb_map; > > > > > > > > + > > > > > > > > + addr = (uintptr_t)map_sysmem(lmb_map->base, 0); > > > > > > > > + pages = efi_size_in_pages(lmb_map->size + (addr & > > > > > > > > EFI_PAGE_MASK)); > > > > > > > > + o

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-11 Thread Simon Glass
Hi Tom, On Tue, 11 Jun 2024 at 16:54, Tom Rini wrote: > > On Tue, Jun 11, 2024 at 04:22:25PM -0600, Simon Glass wrote: > > Hi Tom, > > > > On Tue, 11 Jun 2024 at 15:01, Tom Rini wrote: > > > > > > On Tue, Jun 11, 2024 at 12:52:19PM -0600, Simon Glass wrote: > > > > Hi, > > > > > > > > On Tue, 11

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-11 Thread Tom Rini
On Tue, Jun 11, 2024 at 04:22:25PM -0600, Simon Glass wrote: > Hi Tom, > > On Tue, 11 Jun 2024 at 15:01, Tom Rini wrote: > > > > On Tue, Jun 11, 2024 at 12:52:19PM -0600, Simon Glass wrote: > > > Hi, > > > > > > On Tue, 11 Jun 2024 at 08:36, Tom Rini wrote: > > > > > > > > On Tue, Jun 11, 2024 a

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-11 Thread Simon Glass
Hi Tom, On Tue, 11 Jun 2024 at 15:01, Tom Rini wrote: > > On Tue, Jun 11, 2024 at 12:52:19PM -0600, Simon Glass wrote: > > Hi, > > > > On Tue, 11 Jun 2024 at 08:36, Tom Rini wrote: > > > > > > On Tue, Jun 11, 2024 at 12:17:16PM +0200, Heinrich Schuchardt wrote: > > > > On 07.06.24 20:52, Sughosh

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-11 Thread Tom Rini
On Tue, Jun 11, 2024 at 12:52:19PM -0600, Simon Glass wrote: > Hi, > > On Tue, 11 Jun 2024 at 08:36, Tom Rini wrote: > > > > On Tue, Jun 11, 2024 at 12:17:16PM +0200, Heinrich Schuchardt wrote: > > > On 07.06.24 20:52, Sughosh Ganu wrote: > > > > There are events that would be used to notify othe

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-11 Thread Simon Glass
Hi, On Tue, 11 Jun 2024 at 08:36, Tom Rini wrote: > > On Tue, Jun 11, 2024 at 12:17:16PM +0200, Heinrich Schuchardt wrote: > > On 07.06.24 20:52, Sughosh Ganu wrote: > > > There are events that would be used to notify other interested modules > > > of any changes in available and occupied memory.

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-11 Thread Tom Rini
On Tue, Jun 11, 2024 at 12:17:16PM +0200, Heinrich Schuchardt wrote: > On 07.06.24 20:52, Sughosh Ganu wrote: > > There are events that would be used to notify other interested modules > > of any changes in available and occupied memory. This would happen > > when a module allocates or reserves mem

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-11 Thread Sughosh Ganu
On Tue, 11 Jun 2024 at 15:47, Heinrich Schuchardt wrote: > > On 07.06.24 20:52, Sughosh Ganu wrote: > > There are events that would be used to notify other interested modules > > of any changes in available and occupied memory. This would happen > > when a module allocates or reserves memory, or f

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-11 Thread Heinrich Schuchardt
On 07.06.24 20:52, Sughosh Ganu wrote: There are events that would be used to notify other interested modules of any changes in available and occupied memory. This would happen when a module allocates or reserves memory, or frees up memory. These changes in memory map should be notified to other

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-10 Thread Ilias Apalodimas
On Mon, Jun 10, 2024, 5:52 PM Sughosh Ganu wrote: > hi Ilias, > > On Mon, 10 Jun 2024 at 19:48, Ilias Apalodimas > wrote: > > > > Hi Sughosh > > > > On Mon, 10 Jun 2024 at 15:25, Sughosh Ganu > wrote: > > > > > > On Mon, 10 Jun 2024 at 17:40, Ilias Apalodimas > > > wrote: > > > > > > > > On Fr

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-10 Thread Simon Glass
Hi, On Mon, 10 Jun 2024 at 09:42, Sughosh Ganu wrote: > > On Mon, 10 Jun 2024 at 20:47, Heinrich Schuchardt wrote: > > > > On 07.06.24 20:52, Sughosh Ganu wrote: > > > There are events that would be used to notify other interested modules > > > of any changes in available and occupied memory. Th

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-10 Thread Sughosh Ganu
On Mon, 10 Jun 2024 at 20:47, Heinrich Schuchardt wrote: > > On 07.06.24 20:52, Sughosh Ganu wrote: > > There are events that would be used to notify other interested modules > > of any changes in available and occupied memory. This would happen > > when a module allocates or reserves memory, or f

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-10 Thread Heinrich Schuchardt
On 07.06.24 20:52, Sughosh Ganu wrote: There are events that would be used to notify other interested modules of any changes in available and occupied memory. This would happen when a module allocates or reserves memory, or frees up memory. These I am worried about the "frees up memory" case.

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-10 Thread Sughosh Ganu
On Mon, 10 Jun 2024 at 20:22, Sughosh Ganu wrote: > > hi Ilias, > > On Mon, 10 Jun 2024 at 19:48, Ilias Apalodimas > wrote: > > > > Hi Sughosh > > > > On Mon, 10 Jun 2024 at 15:25, Sughosh Ganu wrote: > > > > > > On Mon, 10 Jun 2024 at 17:40, Ilias Apalodimas > > > wrote: > > > > > > > > On Fri

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-10 Thread Sughosh Ganu
hi Ilias, On Mon, 10 Jun 2024 at 19:48, Ilias Apalodimas wrote: > > Hi Sughosh > > On Mon, 10 Jun 2024 at 15:25, Sughosh Ganu wrote: > > > > On Mon, 10 Jun 2024 at 17:40, Ilias Apalodimas > > wrote: > > > > > > On Fri, 7 Jun 2024 at 21:54, Sughosh Ganu wrote: > > > > > > > > There are events t

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-10 Thread Ilias Apalodimas
Hi Sughosh On Mon, 10 Jun 2024 at 15:25, Sughosh Ganu wrote: > > On Mon, 10 Jun 2024 at 17:40, Ilias Apalodimas > wrote: > > > > On Fri, 7 Jun 2024 at 21:54, Sughosh Ganu wrote: > > > > > > There are events that would be used to notify other interested modules > > > of any changes in available

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-10 Thread Sughosh Ganu
On Mon, 10 Jun 2024 at 17:40, Ilias Apalodimas wrote: > > On Fri, 7 Jun 2024 at 21:54, Sughosh Ganu wrote: > > > > There are events that would be used to notify other interested modules > > of any changes in available and occupied memory. This would happen > > when a module allocates or reserves

Re: [RFC PATCH 15/31] efi_memory: add an event handler to update memory map

2024-06-10 Thread Ilias Apalodimas
On Fri, 7 Jun 2024 at 21:54, Sughosh Ganu wrote: > > There are events that would be used to notify other interested modules > of any changes in available and occupied memory. This would happen > when a module allocates or reserves memory, or frees up memory. These > changes in memory map should be