From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Goldschmidt Date: Thu, 23 May 2019 22:08:54 +0200 Subject: [U-Boot] [PATCH v2] spl: add overall SPL size check In-Reply-To: References: <20190422202721.15892-1-simon.k.r.goldschmidt@gmail.com> <20190511015553.GJ6312@bill-the-cat> Message-ID: <4d51152d-1236-ee77-5aa0-ca60bd3cfe7b@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Am 13.05.2019 um 14:45 schrieb Simon Goldschmidt: > On Sat, May 11, 2019 at 3:55 AM Tom Rini wrote: >> >> On Mon, Apr 22, 2019 at 10:27:21PM +0200, Simon Goldschmidt wrote: >> >>> This adds a size check for SPL that can dynamically check generated >>> SPL binaries (including devicetree) for a size limit that ensures >>> this image plus global data, heap and stack fit in initial SRAM. >>> >>> Since some of these sizes are not available to make, a new host tool >>> 'spl_size_limit' is added that dumps the resulting maximum size for >>> an SPL binary to stdout. This tool is used in toplevel Makefile to >>> implement the size check on SPL binaries. >>> >>> Signed-off-by: Simon Goldschmidt >> >> OK, this has a race / dependency problem: >> https://travis-ci.org/trini/u-boot/jobs/530803338 > > Hmm, let me check that. OK, so as I see it, this is not a race but a dependency problem: the "tools-only" target just does not generate the "generic-asm-offsets.h" file. However, just adding a dependency to include/generated/generic-asm-offsets.h leads to an error (no rule to build that file) since that file is generated by Kbuild magic that I don't really get... Any idea how to make this depend on /Kbuild being run? Regards, Simon