[PATCH] lmb: Default to not-LMB_USE_MAX_REGIONS

Tom Rini trini at konsulko.com
Tue Feb 7 17:49:31 CET 2023


On Wed, Jan 25, 2023 at 06:08:23PM -0500, Tom Rini wrote:

> The LMB code allows for picking a hard limit on the number of regions it
> can know of, or to dynamically allocate these regions. The reason for
> this choice is to allow for the compiler to perform a size optimization
> in the common case. This optimization however, is very small, ranging
> from 196 bytes to 15 bytes saved, or in some cases, being larger. Now
> that we also have more regions covered by LMB (in order to protect
> various parts of our self at run time), the default of 8 is also much
> easier to hit and leads to non-obvious error messages (which imply that
> an area is protected, not that we're out of areas to add to the list).
> 
> Switch to the dynamic use as the default.
> 
> Signed-off-by: Tom Rini <trini at konsulko.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230207/f86e5076/attachment.sig>


More information about the U-Boot mailing list