Re: [PATCH v4 3/6] cacheinfo: Check 'cache-unified' property to count cache leaves

From: Krzysztof Kozlowski
Date: Mon Jan 09 2023 - 11:30:30 EST


On 04/01/2023 19:30, Pierre Gondois wrote:
> The DeviceTree Specification v0.3 specifies that the cache node
> '[d-|i-|]cache-size' property is required. The 'cache-unified'
> property is specifies whether the cache level is separate
> or unified.
>
> If the cache-size property is missing, no cache leaves is accounted.
> This can lead to a 'BUG: KASAN: slab-out-of-bounds' [1] bug.
>
> Check 'cache-unified' property and always account for at least
> one cache leave when parsing the device tree.
>
> [1] https://lore.kernel.org/all/0f19cb3f-d6cf-4032-66d2-dedc9d09a0e3@xxxxxxxxxx/
>
> Reported-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> Signed-off-by: Pierre Gondois <pierre.gondois@xxxxxxx>

Thanks, solves the issue:

Tested-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>

Best regards,
Krzysztof