[PATCH 2/4] lmb: handle scenario of of encompassing overlap

Simon Glass sjg at chromium.org
Tue Feb 18 12:38:09 CET 2025


Hi Sughosh,

On Thu, 13 Feb 2025 at 06:11, Sughosh Ganu <sughosh.ganu at linaro.org> wrote:
>
> The lmb_fix_over_lap_regions() function is called if the added region
> overlaps with an existing region. The function then fixes the overlap
> and removes the redundant region. However, it makes an assumption that
> the overlap would not encompass the existing region, and in such a
> scenario, it prints a message and returns without making the
> fix. Handle the case of an encompassing overlap also in the function.
>
> Signed-off-by: Sughosh Ganu <sughosh.ganu at linaro.org>
> Reported-by: Quentin Schulz <quentin.schulz at cherry.de>
> ---
>
> Note: To be applied after an A-b/R-b/T-b from the original author
> of the lmb_fix_over_lap_regions() function on this
>
>  lib/lmb.c | 8 +++-----
>  1 file changed, 3 insertions(+), 5 deletions(-)

Would it be possible to go back through your dozen-or-so lmb patches
and add some tests?

Regards,
Simon


More information about the U-Boot mailing list