From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Gibson Subject: Re: Size growth? Date: Mon, 19 Oct 2020 12:42:13 +1100 Message-ID: <20201019014213.GA11625@yekko.fritz.box> References: Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="DocE+STaALJfprDB" Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gibson.dropbear.id.au; s=201602; t=1603079020; bh=G0LP1dNAsc2T/SxL/fBK27lgYED73al0tofxrRz17Zo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=WKQtSBwNp1pH44ZrF0t84adw867WZkCAK4mq+xfwSmmCIN/W88zL5qdKCmF+kG7mM ZWOyXN1Yq9PYbaQWLvXP22EiDCoAPmbO+8AY28ZsqNbViseNMJFySTf4e3j+UoCYHd IpgM3dylH5z4Kp0Hn99pzraX2wuXUbG6tpvcF6kg= Content-Disposition: inline In-Reply-To: List-ID: To: Simon Glass Cc: Tom Rini , Devicetree Compiler --DocE+STaALJfprDB Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Oct 16, 2020 at 03:46:41PM -0600, Simon Glass wrote: > Hi Tom, >=20 > On Fri, 16 Oct 2020 at 13:36, Tom Rini wrote: > > > > Hey all, > > > > With my U-Boot hat on, I'm trying to move us up from 84e414b0b5bc in dtc > > to cbca977ea121. I'm seeing some rather worrying size increases > > however. For example, on an aarch64 board such as leez-rk3399 we have: > > leez-rk3399 : all +2696 spl/u-boot-spl:all +1116 spl/u-boot-spl= :text +1116 text +2696 > > u-boot: add: 0/0, grow: 42/0 bytes: 2696/0 (2696) > > function old new delta > > do_fdt 3992 4308 +316 > > fdt_check_header 276 492 +216 > > fdt_add_property_ 388 556 +168 > > fdt_ro_probe_ 128 252 +124 > > fdt_packblocks_ 176 296 +120 > > fdt_open_into 392 512 +120 > > fdt_blocks_misordered_ 96 216 +120 > > fdt_get_mem_rsv 108 220 +112 > > fdt_offset_ptr 104 208 +104 > > fdt_get_string 288 388 +100 > > fdt_splice_ 148 228 +80 > > fdt_valid 204 276 +72 > > fdt_getprop_by_offset 184 256 +72 > > fdt_splice_mem_rsv_ 96 152 +56 > > fdt_num_mem_rsv 60 116 +56 > > fdt_splice_struct_ 96 144 +48 > > fdt_shrink_to_minimum 220 268 +48 > > fdt_rw_probe_ 108 156 +48 > > fdt_mem_rsv 60 108 +48 > > fdt_getprop_namelen 100 148 +48 > > fdt_get_property_by_offset_ 100 148 +48 > > fdt_get_name 164 212 +48 > > efi_install_fdt 964 1012 +48 > > boot_get_fdt 888 936 +48 > > fdt_move 80 116 +36 > > reserve_fdt 72 96 +24 > > reloc_fdt 76 100 +24 > > image_setup_libfdt 284 308 +24 > > fit_image_load 1608 1632 +24 > > fit_image_get_data_and_size 172 196 +24 > > fit_get_end 16 40 +24 > > fdt_next_tag 256 280 +24 > > fdt_header_size 12 36 +24 > > fdt_get_property_namelen_ 212 236 +24 > > fdt_get_property_namelen 44 68 +24 > > fdt_get_phandle 120 144 +24 > > fdt_del_node 96 120 +24 > > fdt_del_mem_rsv 112 136 +24 > > fdt_add_subnode_namelen 284 308 +24 > > fdt_add_mem_rsv 124 148 +24 > > common_diskboot 680 704 +24 > > fdt_next_subnode 80 88 +8 > > spl-u-boot-spl: add: 1/-1, grow: 25/-2 bytes: 1176/-60 (1116) > > function old new delta > > fdt_get_mem_rsv 52 236 +184 > > fdt_add_property_ 340 484 +144 > > fdt_get_name 68 152 +84 > > fdt_splice_ 148 228 +80 > > fdt_num_mem_rsv 64 128 +64 > > fdt_splice_mem_rsv_ 96 152 +56 > > fdt_offset_ptr 52 104 +52 > > fdt_splice_struct_ 96 144 +48 > > fdt_shrink_to_minimum 220 268 +48 > > fdt_get_property_by_offset_ 36 84 +48 > > fdt_ro_probe_ - 36 +36 > > fdt_subnode_offset_namelen 200 232 +32 > > spl_load_simple_fit 848 872 +24 > > spl_fit_append_fdt 196 220 +24 > > fdt_getprop_by_offset 80 104 +24 > > fdt_get_string 64 88 +24 > > fdt_get_property_namelen_ 200 224 +24 > > fdt_get_phandle 120 144 +24 > > fdt_del_mem_rsv 104 128 +24 > > fdt_check_header 28 52 +24 > > fdt_add_subnode_namelen 260 284 +24 > > fdt_add_mem_rsv 112 136 +24 > > fdt_next_tag 192 212 +20 > > fdt_path_offset_namelen 240 256 +16 > > fdt_supernode_atdepth_offset 160 172 +12 > > fdt_node_offset_by_phandle 120 132 +12 > > fdt_mem_rsv 20 - -20 > > fdt_check_prop_offset_ 64 44 -20 > > fdt_check_node_offset_ 64 44 -20 > > > > And note that for the SPL case we're already setting ASSUME_MASK to > > 0xff so there's maximum savings already being done there. Does anyone > > have ideas on where / how to further tweak code size? Thanks! >=20 > +David Gibson >=20 > I suspect there are more checks that need to be made conditional. Seems likely. Though, as I've opined before, from what I understand the SPL is *so* restricted an environment, I'm not really convinced DTs are the right tool for the job there. --=20 David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson --DocE+STaALJfprDB Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIyBAEBCAAdFiEEdfRlhq5hpmzETofcbDjKyiDZs5IFAl+M7vAACgkQbDjKyiDZ s5LH4Q/4iHNITSEjhOwUjgv5buCGjKISB+/vo/3U78IDPlWIB7vKKsWh7euGrBW5 20V9t1cHclgQj14i5x6np7AeHD3bQxX1ga3+O90BBf0U5r9G1Qu65koy2Yyz5rc9 kk0GDzIOubr3q2Gz2nLhhCWfrMv6pRIi6nfhXsKMNrbm9NgZTo0rqELHtS8IgHMM +0AUrrvdpXGBq7/FVdpsmmKdexCsTLjAxtVTXmj4KSF9nbQgNpYmfO7NYpsT4jz6 LbyDMtHtxPm2xZnUNvJGvCM3au2y+8VrdW/+ph4QKVpj02E/V2PEhuRH3ZwFgL3E 16VxvcoPPfDdwHPzJbvrt31TseZKB+eLZFbvtXju5LWI+up2GhYpTPs8m0Pon49Y 96zYnAPfhpudQa/Po5MOSd8Qg1GMtIvEY66OgGh/pdxHs7RfiQ/SQ8LcfFAz2Xdt ahsZkX55UfUatZAeZFg605aRt7kNSHIReUAJ4aaQ4uxYbbXqt/PcJZydx62W8vXR QuivyqqExwYbbo7bdD3p8oR5/dLbsO0A9s6rMuJYPIYZBpcbknpCley+t7JZ3NWP sKvI+81vMWuXPZ9v3lyM7EyAWKUO+jUpeVcImP/yoTzwE13YagvokmChNCLuaabj z1Ddsmq3K3cFqxzpqR9a7WyLQmK4G1OvlX6+RZTFZv1sT3JQNg== =n0zm -----END PGP SIGNATURE----- --DocE+STaALJfprDB--