Re: [PATCH 0/8] ppc64: Restrict memory allocations for kernel and initrd

[Daniel Kiper]

On Fri, Oct 27, 2023 at 10:14:03AM -0400, Stefan Berger wrote:
> This series of patches converts the memory allocator for kernel and initrd
> to use the ppc64 regions_claim memory allocator that takes into account
> memory regions that are not allowed to be used, such as the gap between
> 640MB and 768MB as well as memory regions beyond an upper address in case
> an fadump is present. Otherwise those two could be loaded into those
> forbidden memory regions and either cause a crash or corruption of the
> fadump.
>
> I adjusted the kernel and initrd load in loader/powerpc/ieee1275 to use the
> new memory allocator. I am not aware of which other platforms besides
> PowerVM and PowerKVM, if any, may be using this code and so I did not
> touch the functions loading 32 bit kernels, which the targeted platform does
> not typically use.
>
> The last patch is for viewing where the memory allocations are going and
> is not to be applied, of course.

I am not entirely convinced we need this for the upcoming release. My
concern is that this patch set makes a lot of changes. OK, they are
mostly limited to the IEEE 1275 stuff but...

Anyway, quick glance through the patches revealed that most of them
missing explanation why they are really needed. At least... Please
fix these things...

Daniel