[PATCH] buildman: Include symbols in the read-only data section

Tom Rini trini at konsulko.com
Mon Oct 23 15:40:12 CEST 2023


On Mon, Oct 23, 2023 at 12:52:43AM -0700, Simon Glass wrote:

> When symbols switch between the inited data section and the read-only
> data section their visbility changes, at present, with the -B option.
> 
> This is confusing, since adding 'const' to a variable declaration can
> make it look like a significant improvement in bloat. But in fact
> nothing has changed.
> 
> Add 'r' to the list of symbols types that are recorded, to correct this
> problem. Add a constant to make it easier to find this code next time.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>

Thanks for fixing this.

Reported-by: Tom Rini <trini at konsulko.com>
Reviewed-by: Tom Rini <trini at konsulko.com>
Link: https://source.denx.de/u-boot/custodians/u-boot-dm/-/issues/6

-- 
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/20231023/e2c2bf4b/attachment.sig>


More information about the U-Boot mailing list