[coreboot-gerrit] [S] Change in coreboot[master]: Append per-board ccache statistics in log

[Felix Singer (Code Review)]

Felix Singer has submitted this change. ( https://review.coreboot.org/c/coreboot/+/75087?usp=email )

Change subject: Append per-board ccache statistics in log
......................................................................

Append per-board ccache statistics in log

Starting with ccache 4.4 it is possible to collect statistics about
cache miss/hit rates in a separate file.

Add the info of the build at end of created make.log file or on stdout.

Change-Id: I1bab712712f4d6379ec6733fdc55b234e3845da7
Signed-off-by: Kyösti Mälkki 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/75087
Reviewed-by: Angel Pons 
Reviewed-by: Felix Singer 
Tested-by: build bot (Jenkins) 
---
M Makefile
M Makefile.inc
M toolchain.inc
3 files changed, 10 insertions(+), 2 deletions(-)

Approvals:
  Angel Pons: Looks good to me, but someone else must approve
  build bot (Jenkins): Verified
  Felix Singer: Looks good to me, approved

diff --git a/Makefile b/Makefile
index 459f724..d86eaf0 100644
--- a/Makefile
+++ b/Makefile
@@ -197,6 +197,10 @@
 strip_quotes = $(strip $(subst ",,$(subst ",,$(1))))
 # fix makefile syntax highlighting after strip macro " "))

+ifneq ($(NOCOMPILE),1)
+$(shell rm -f $(CCACHE_STATSLOG))
+endif
+
 # The primary target needs to be here before we include the
 # other files

diff --git a/Makefile.inc b/Makefile.inc
index abf47d4..b35ec86 100644
--- a/Makefile.inc
+++ b/Makefile.inc
@@ -66,8 +66,11 @@
 # targets after the build completes by creating a Makefile.inc in the
 # site-local directory with a target named 'build_complete::'
 build_complete:: coreboot
-	printf "\nBuilt %s (%s)\n" $(MAINBOARDDIR) \
-		$(CONFIG_MAINBOARD_PART_NUMBER)
+	printf "\nBuilt %s (%s)\n" $(MAINBOARDDIR) $(CONFIG_MAINBOARD_PART_NUMBER)
+	if [ -f "$(CCACHE_STATSLOG)" ]; then \
+		printf "\nccache statistics\n"; \
+		$(CCACHE) --show-log-stats -v; \
+	fi

# This target can be used to run rules after all files were added to CBFS,
 # for example to process FMAP regions or the entire image.
diff --git a/toolchain.inc b/toolchain.inc
index 47c5fe9..3165586 100644
--- a/toolchain.inc
+++ b/toolchain.inc
@@ -10,6 +10,7 @@

export CCACHE_COMPILERCHECK=content
 export CCACHE_BASEDIR=$(top)
+export CCACHE_STATSLOG=$(obj)/ccache.stats

$(foreach arch,$(ARCH_SUPPORTED), \
    $(eval CC_$(arch):=$(CCACHE) $(CC_$(arch))))