Hi, On 15 May 2018 at 23:46, Eugeniu Rosca <ero...@de.adit-jv.com> wrote: > Hi Simon, > > On Mon, May 14, 2018 at 01:52:09PM -0600, Simon Glass wrote: >> Hi, >> >> On 13 May 2018 at 16:45, Eugeniu Rosca <roscaeuge...@gmail.com> wrote: >> > Hi Simon, >> > >> > On Mon, May 14, 2018 at 08:00:40AM +1000, Simon Glass wrote: >> >> Hi, >> >> >> >> Can you please check that this does not adversely impact code size? >> > >> > Sure. Here are the sizes before and after this patch (sandbox): >> > >> > 1. ELF size comparison shows 96 bytes increase >> > $ du -b u-boot.before u-boot.after >> > 8055864 u-boot.before >> > 8055960 u-boot.after >> > >> > 2. BIN size comparison shows no change >> > $ du -b u-boot.bin.before u-boot.bin.after >> > 3377256 u-boot.bin.before >> > 3377256 u-boot.bin.after >> > >> > 3. "size" tool gives more details about ELF file size increase: >> > $ size u-boot.before u-boot.after >> > text data bss dec hex filename >> > 1199720 76952 291760 1568432 17eeb0 u-boot.before >> > 1199736 76952 291760 1568448 17eec0 u-boot.after >> >> Thanks for the info. That seems OK, but please check sandbox_spl (or >> better yet one of the Thumb2 SPL images that uses libfdt) since it is >> SPL that matters most. > > I am having issues bringing up buildman on my host, so I just went > through the same procedure of building before and after using make > and comparing the ELF/BIN size for: > 1. sandbox_spl_defconfig > 2. x600_defconfig > > There is no drastic change to previous results. > > 1.1 sandbox_spl_defconfig ELF size increase: 7394784-7394712 = 72 bytes. > 1.2 sandbox_spl_defconfig BIN size increase: 3298280-3298280 = 0 bytes. > > 2.1 x600_defconfig ELF size increase: 3766144-3766232 = 88 bytes. > 2.2 x600_defconfig BIN size increase: 439288-439240 = 48 bytes. >
OK Reviewed-by: Simon Glass <s...@chromium.org> >> > >> > 4. "bloaty" (diff mode) gives even more details (confirming the above) >> > $ bloaty -d compileunits u-boot.after -- u-boot.before >> > VM SIZE FILE SIZE >> > -------------- -------------- >> > +2.5% +30 lib/libfdt/fdt.c +96 +1.5% >> > [ = ] 0 lib/libfdt/fdt_sw.c +21 +0.1% >> > +2.6% +16 [LOAD [RX]] +16 +2.6% >> > [ = ] 0 lib/libfdt/fdt_overlay.c +16 +0.1% >> > [ = ] 0 [section .debug_str] +2 +0.0% >> > [ = ] 0 fs/ext4/ext4_common.c -5 -0.0% >> > [ = ] 0 common/fdt_support.c -11 -0.0% >> > -3.3% -14 [section .text] -14 -3.3% >> > [ = ] 0 [Unmapped] -25 -0.7% >> > +0.0% +32 TOTAL +96 +0.0% >> >> BTW buildman has a -B flag that shows function size. > > Thanks for the tip. The nice thing about bloaty is that it provides > both VM and FILE size numbers. I am not sure if buildman can do the > same. Anyway, this was helpful and is expected to save time on my end > when I manage to bring it up. I am not sure that the file size numbers are very useful though, since we mostly care about the text and data size. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot