Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-25 Thread Simon Glass
Hi Ilias, On Mon, 23 Sept 2024 at 12:03, Ilias Apalodimas wrote: > > On Fri, 20 Sept 2024 at 19:03, Simon Glass wrote: > > > > Hi Ilias, > > > > On Fri, 20 Sept 2024 at 14:10, Ilias Apalodimas > > wrote: > > > > > > Hi Simon > > > > > > A few more comments after looking into this a bit more > >

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-25 Thread Simon Glass
Hi Heinrich, On Mon, 23 Sept 2024 at 14:36, Heinrich Schuchardt wrote: > > On 12.09.24 02:59, Simon Glass wrote: > > Hi Sughosh, > > > > On Wed, 11 Sept 2024 at 00:50, Sughosh Ganu wrote: > >> > >> On Wed, 11 Sept 2024 at 00:14, Simon Glass wrote: > >>> > >>> Hi Sughosh, > >>> > >>> On Mon, 9 S

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-23 Thread Heinrich Schuchardt
On 12.09.24 02:59, Simon Glass wrote: Hi Sughosh, On Wed, 11 Sept 2024 at 00:50, Sughosh Ganu wrote: On Wed, 11 Sept 2024 at 00:14, Simon Glass wrote: Hi Sughosh, On Mon, 9 Sept 2024 at 01:44, Sughosh Ganu wrote: On Fri, 6 Sept 2024 at 18:31, Simon Glass wrote: Hi Sughosh, On Fri,

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-23 Thread Ilias Apalodimas
On Fri, 20 Sept 2024 at 19:03, Simon Glass wrote: > > Hi Ilias, > > On Fri, 20 Sept 2024 at 14:10, Ilias Apalodimas > wrote: > > > > Hi Simon > > > > A few more comments after looking into this a bit more > > > > On Mon, Sep 16, 2024 at 05:42:59PM +0200, Simon Glass wrote: > > > Hi Ilias, > > > >

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-20 Thread Simon Glass
Hi Ilias, On Fri, 20 Sept 2024 at 14:10, Ilias Apalodimas wrote: > > Hi Simon > > A few more comments after looking into this a bit more > > On Mon, Sep 16, 2024 at 05:42:59PM +0200, Simon Glass wrote: > > Hi Ilias, > > > > On Thu, 12 Sept 2024 at 08:37, Ilias Apalodimas > > wrote: > > > > > > H

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-20 Thread Ilias Apalodimas
Hi Simon A few more comments after looking into this a bit more On Mon, Sep 16, 2024 at 05:42:59PM +0200, Simon Glass wrote: > Hi Ilias, > > On Thu, 12 Sept 2024 at 08:37, Ilias Apalodimas > wrote: > > > > Hi Simon, > > > > On Fri, 6 Sept 2024 at 16:01, Simon Glass wrote: > > > > > > Hi Ilias,

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-16 Thread Simon Glass
Hi Ilias, On Thu, 12 Sept 2024 at 08:37, Ilias Apalodimas wrote: > > Hi Simon, > > On Fri, 6 Sept 2024 at 16:01, Simon Glass wrote: > > > > Hi Ilias, > > > > On Fri, 6 Sept 2024 at 01:13, Ilias Apalodimas > > wrote: > > > > > > Hi Simon, > > > > > > Apologies for the late reply, I was attending

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-11 Thread Ilias Apalodimas
Hi Simon, On Fri, 6 Sept 2024 at 16:01, Simon Glass wrote: > > Hi Ilias, > > On Fri, 6 Sept 2024 at 01:13, Ilias Apalodimas > wrote: > > > > Hi Simon, > > > > Apologies for the late reply, I was attending a conference. > > > > On Mon, 2 Sept 2024 at 01:23, Simon Glass wrote: > > > > > > This AP

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-11 Thread Simon Glass
Hi Sughosh, On Wed, 11 Sept 2024 at 00:50, Sughosh Ganu wrote: > > On Wed, 11 Sept 2024 at 00:14, Simon Glass wrote: > > > > Hi Sughosh, > > > > On Mon, 9 Sept 2024 at 01:44, Sughosh Ganu wrote: > > > > > > On Fri, 6 Sept 2024 at 18:31, Simon Glass wrote: > > > > > > > > Hi Sughosh, > > > > >

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-10 Thread Sughosh Ganu
On Wed, 11 Sept 2024 at 00:14, Simon Glass wrote: > > Hi Sughosh, > > On Mon, 9 Sept 2024 at 01:44, Sughosh Ganu wrote: > > > > On Fri, 6 Sept 2024 at 18:31, Simon Glass wrote: > > > > > > Hi Sughosh, > > > > > > On Fri, 6 Sept 2024 at 00:23, Sughosh Ganu > > > wrote: > > > > > > > > On Mon, 2

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-10 Thread Simon Glass
Hi Sughosh, On Mon, 9 Sept 2024 at 01:44, Sughosh Ganu wrote: > > On Fri, 6 Sept 2024 at 18:31, Simon Glass wrote: > > > > Hi Sughosh, > > > > On Fri, 6 Sept 2024 at 00:23, Sughosh Ganu wrote: > > > > > > On Mon, 2 Sept 2024 at 03:53, Simon Glass wrote: > > > > > > > > This API call is intende

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-09 Thread Sughosh Ganu
On Fri, 6 Sept 2024 at 18:31, Simon Glass wrote: > > Hi Sughosh, > > On Fri, 6 Sept 2024 at 00:23, Sughosh Ganu wrote: > > > > On Mon, 2 Sept 2024 at 03:53, Simon Glass wrote: > > > > > > This API call is intended for allocating small amounts of memory, > > > similar to malloc(). The current imp

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-06 Thread Simon Glass
Hi Ilias, On Fri, 6 Sept 2024 at 01:13, Ilias Apalodimas wrote: > > Hi Simon, > > Apologies for the late reply, I was attending a conference. > > On Mon, 2 Sept 2024 at 01:23, Simon Glass wrote: > > > > This API call is intended for allocating small amounts of memory, > > similar to malloc(). Th

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-06 Thread Simon Glass
Hi Sughosh, On Fri, 6 Sept 2024 at 00:23, Sughosh Ganu wrote: > > On Mon, 2 Sept 2024 at 03:53, Simon Glass wrote: > > > > This API call is intended for allocating small amounts of memory, > > similar to malloc(). The current implementation rounds up to whole pages > > which can waste large amou

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-06 Thread Ilias Apalodimas
Hi Simon, Apologies for the late reply, I was attending a conference. On Mon, 2 Sept 2024 at 01:23, Simon Glass wrote: > > This API call is intended for allocating small amounts of memory, > similar to malloc(). The current implementation rounds up to whole pages > which can waste large amounts

Re: [PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-05 Thread Sughosh Ganu
On Mon, 2 Sept 2024 at 03:53, Simon Glass wrote: > > This API call is intended for allocating small amounts of memory, > similar to malloc(). The current implementation rounds up to whole pages > which can waste large amounts of memory. It also implements its own > malloc()-style header on each bl

[PATCH v3 2/3] efi: Allow use of malloc() for the EFI pool

2024-09-01 Thread Simon Glass
This API call is intended for allocating small amounts of memory, similar to malloc(). The current implementation rounds up to whole pages which can waste large amounts of memory. It also implements its own malloc()-style header on each block. For certain allocations (those of type EFI_BOOT_SERVIC