Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43817 )
Change subject: [WIP] mb/emulation/qemu-i440fx: Remove TRACE=y from test build ......................................................................
Patch Set 1:
(1 comment)
See build er https://qa.coreboot.org/job/coreboot-gerrit/136133/testReport/junit/(root)/b...
https://review.coreboot.org/c/coreboot/+/43817/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/43817/1//COMMIT_MSG@10 PS1, Line 10: garbage collections.
I don't follow. […]
See the build error in parent that this fixes.
https://qa.coreboot.org/job/coreboot-gerrit/136133/testReport/junit/(root)/b...
Looks like compiler does not optimise is_smp_boot() to a constant false with TRACE=y, and copy_secondary_start_to_lowest_1M() is kept in the resulting object. We have CONFIG_SMP guard for secondary.S and those symbols now became unresolved.
OTH, if (ENV_xx) and if (CONFIG(x)) should not be affected by TRACE so it might be just a couple special cases where it fails like seen here.