Re: [PATCH v2 2/3] efi: Convert device_path allocation to use malloc()

Hi Ilias, On Thu, 29 Aug 2024 at 08:14, Simon Glass wrote: > > Hi Ilias, > > On Thu, 29 Aug 2024 at 06:39, Ilias Apalodimas > wrote: > > > > On Thu, 29 Aug 2024 at 15:17, Simon Glass wrote: > > > > > > Hi Ilias, > > > > > > On Wed, 28 Aug 2024 at 01:40, Ilias Apalodimas > > > wrote: > > > > >

Re: [PATCH v2 2/3] efi: Convert device_path allocation to use malloc()

Hi Ilias, On Thu, 29 Aug 2024 at 06:39, Ilias Apalodimas wrote: > > On Thu, 29 Aug 2024 at 15:17, Simon Glass wrote: > > > > Hi Ilias, > > > > On Wed, 28 Aug 2024 at 01:40, Ilias Apalodimas > > wrote: > > > > > > Hi Simon, > > > > > > [...] > > > > > > > > > > > > --- > > > > > > > > > > > > >

Re: [PATCH v2 2/3] efi: Convert device_path allocation to use malloc()

On Thu, 29 Aug 2024 at 15:17, Simon Glass wrote: > > Hi Ilias, > > On Wed, 28 Aug 2024 at 01:40, Ilias Apalodimas > wrote: > > > > Hi Simon, > > > > [...] > > > > > > > > > > --- > > > > > > > > > > > > > > > > (no changes since v1) > > > > > > > > > > > > > > What about > > > > > > > https://lo

Re: [PATCH v2 2/3] efi: Convert device_path allocation to use malloc()

Hi Ilias, On Wed, 28 Aug 2024 at 01:40, Ilias Apalodimas wrote: > > Hi Simon, > > [...] > > > > > > > > --- > > > > > > > > > > > > > > (no changes since v1) > > > > > > > > > > > > What about > > > > > > https://lore.kernel.org/u-boot/caflsztjlakayk_jlxj7z571l-qmtoiqe-oxhcrs186dz2qo...@mail.gma

Re: [PATCH v2 2/3] efi: Convert device_path allocation to use malloc()

Hi Simon, [...] > > > > > > --- > > > > > > > > > > > > (no changes since v1) > > > > > > > > > > What about > > > > > https://lore.kernel.org/u-boot/caflsztjlakayk_jlxj7z571l-qmtoiqe-oxhcrs186dz2qo...@mail.gmail.com/ > > > > > ? > > > > > > > > Yes, I reordered the patches in this series. > >

Re: [PATCH v2 2/3] efi: Convert device_path allocation to use malloc()

Hi Simon, On Thu, 15 Aug 2024 at 23:35, Simon Glass wrote: > > Hi Ilias, > > On Wed, 14 Aug 2024 at 07:07, Ilias Apalodimas > wrote: > > > > Hi Simon > > > > On Wed, 14 Aug 2024 at 15:40, Simon Glass wrote: > > > > > > Hi Ilias, > > > > > > On Wed, 14 Aug 2024 at 05:03, Ilias Apalodimas > > >

Re: [PATCH v2 2/3] efi: Convert device_path allocation to use malloc()

Hi Ilias, On Wed, 14 Aug 2024 at 07:07, Ilias Apalodimas wrote: > > Hi Simon > > On Wed, 14 Aug 2024 at 15:40, Simon Glass wrote: > > > > Hi Ilias, > > > > On Wed, 14 Aug 2024 at 05:03, Ilias Apalodimas > > wrote: > > > > > > Hi Simon, > > > > > > On Thu, 1 Aug 2024 at 20:36, Simon Glass wrote

Re: [PATCH v2 2/3] efi: Convert device_path allocation to use malloc()

Hi Simon On Wed, 14 Aug 2024 at 15:40, Simon Glass wrote: > > Hi Ilias, > > On Wed, 14 Aug 2024 at 05:03, Ilias Apalodimas > wrote: > > > > Hi Simon, > > > > On Thu, 1 Aug 2024 at 20:36, Simon Glass wrote: > > > > > > Currently this calls efi_alloc() which reserves a page for each > > > allocat

Re: [PATCH v2 2/3] efi: Convert device_path allocation to use malloc()

Hi Ilias, On Wed, 14 Aug 2024 at 05:03, Ilias Apalodimas wrote: > > Hi Simon, > > On Thu, 1 Aug 2024 at 20:36, Simon Glass wrote: > > > > Currently this calls efi_alloc() which reserves a page for each > > allocation and this can overwrite memory that will be used for reading > > images. > > > >

Re: [PATCH v2 2/3] efi: Convert device_path allocation to use malloc()

Hi Simon, On Thu, 1 Aug 2024 at 20:36, Simon Glass wrote: > > Currently this calls efi_alloc() which reserves a page for each > allocation and this can overwrite memory that will be used for reading > images. > > Switch the code to use malloc(), as with other parts of EFI, such as > efi_add_proto

[PATCH v2 2/3] efi: Convert device_path allocation to use malloc()

Currently this calls efi_alloc() which reserves a page for each allocation and this can overwrite memory that will be used for reading images. Switch the code to use malloc(), as with other parts of EFI, such as efi_add_protocol(). Signed-off-by: Simon Glass --- (no changes since v1) lib/efi_