linux-kernel - Re: [PATCH v5 0/2] sh: Restructure setup code to reserve memory regions earlier

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <fbc596f7ad7ec4b08a5f50da8dcc0a1ce908793d.camel@physik.fu-berlin.de>
Date: Thu, 26 Sep 2024 14:41:50 +0200
From: John Paul Adrian Glaubitz <glaubitz@...sik.fu-berlin.de>
To: Oreoluwa Babatunde <quic_obabatun@...cinc.com>, dalias@...c.org, 
	ysato@...rs.sourceforge.jp
Cc: kernel@...cinc.com, linux-kernel@...r.kernel.org,
 linux-sh@...r.kernel.org,  robh+dt@...nel.org, Artur Rojek
 <contact@...ur-rojek.eu>, Geert Uytterhoeven <geert+renesas@...der.be>
Subject: Re: [PATCH v5 0/2] sh: Restructure setup code to reserve memory
 regions earlier

Hi Oreoluwa,

On Wed, 2024-07-17 at 19:18 -0700, Oreoluwa Babatunde wrote:
> The unflatten_devicetree() function allocates memory from memblock
> before the system gets a chance to set aside the regions of memory that
> are meant to be reserved.
> This means that there is a possibility for memblock to allocate from
> these regions, thereby preventing them from being reserved.
> 
> This series makes changes to the arch specific setup code to call the
> functions responsible for setting aside the reserved memory regions
> earlier in the init sequence.
> 
> Hence, by the time memblock starts being used to allocate memory, the
> memory regions that are meant to be carved out would already be set aside.
> 
> Oreoluwa Babatunde (2):
>   sh: Restructure call site for early_reserve_mem()
>   sh: Restructure setup code to reserve memory regions earlier
> 
>  arch/sh/include/asm/mmu.h |  1 +
>  arch/sh/kernel/setup.c    |  3 +++
>  arch/sh/mm/init.c         | 20 +++++++++-----------
>  3 files changed, 13 insertions(+), 11 deletions(-)

Apologies for the long silence. Both Artur and Geert (CC'ed) now also own a
J2 Turtleboard and I would like to have at least either of them test and
review your changes to make sure nothing breaks.

Thanks,
Adrian

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer
`. `'   Physicist
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913