diff options
| author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2020-07-24 15:54:31 +0300 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2020-12-02 23:35:58 +0000 |
| commit | 8c99c27df10f6c5a120e41ffb0948e357f5a2d20 (patch) | |
| tree | 710940c338b8b2a3ddebab60faf7856725ca37e2 /src/include/trace.h | |
| parent | 5e053af7a677d0c1f98e624057ba7409f8492596 (diff) | |
| download | coreboot-8c99c27df10f6c5a120e41ffb0948e357f5a2d20.tar.gz coreboot-8c99c27df10f6c5a120e41ffb0948e357f5a2d20.tar.bz2 coreboot-8c99c27df10f6c5a120e41ffb0948e357f5a2d20.zip | |
lib/trace: Remove TRACE support
Looks like the option is generally not compatible with
garbage collections.
Nothing gets inlined, for example is_smp_boot() no longer
evaluates to constant false and thus the symbols from
secondary.S would need to be present for the build to pass
even if we set SMP=n.
Also the addresses of relocatable ramstage are currently
not normalised on the logs, so util/genprof would be unable
dress those.
Change-Id: I0b6f310e15e6f4992cd054d288903fea8390e5cf
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/45757
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'src/include/trace.h')
| -rw-r--r-- | src/include/trace.h | 28 |
1 files changed, 0 insertions, 28 deletions
diff --git a/src/include/trace.h b/src/include/trace.h deleted file mode 100644 index ece1b2110c06..000000000000 --- a/src/include/trace.h +++ /dev/null @@ -1,28 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ - -#ifndef __TRACE_H -#define __TRACE_H - -#if !ENV_ROMSTAGE_OR_BEFORE && CONFIG(TRACE) - -void __cyg_profile_func_enter(void *, void *) - __attribute__((no_instrument_function)); - -void __cyg_profile_func_exit(void *, void *) - __attribute__((no_instrument_function)); - -extern volatile int trace_dis; - -#define DISABLE_TRACE do { trace_dis = 1; } while (0); -#define ENABLE_TRACE do { trace_dis = 0; } while (0); -#define DISABLE_TRACE_ON_FUNCTION __attribute__((no_instrument_function)); - -#else /* !CONFIG_TRACE */ - -#define DISABLE_TRACE -#define ENABLE_TRACE -#define DISABLE_TRACE_ON_FUNCTION - -#endif - -#endif |