Re: [PATCH v4 09/14] of: reserved-memory: Fix using wrong number of cells to get property 'alignment'

From: Zijun Hu
Date: Thu Feb 27 2025 - 06:56:23 EST


On 2025/2/27 05:30, Rob Herring wrote:
> Every time this code is touched, it breaks. This is not even the only
> breakage right now[1].
>
>> 2) IMO, the spec may be right.
>> The type of size is enough to express any alignment wanted.
>> For several kernel allocators. type of 'alignment' should be the type
>> of 'size', NOT the type of 'address'
> As I said previously, it can be argued either way.
>
> Rob
>
> [1] https://lore.kernel.org/all/20250226115044.zw44p5dxlhy5eoni@xxxxxxxxxxxxxx/

this issue may be a downstream issue as well.

based on comments. they expects the CMA area is within 4G range.

but the wrong alloc-ranges config make the range (0x42000000 +
0xc0000000) > 4G across 4G boundary.