diff mbox series

[langdale] arm/fvpboot: Disable timing annotation by default

Message ID 20221216111941.1369079-1-peter.hoyes@arm.com
State New
Headers show
Series [langdale] arm/fvpboot: Disable timing annotation by default | expand

Commit Message

Peter Hoyes Dec. 16, 2022, 11:19 a.m. UTC 
From: Peter Hoyes <Peter.Hoyes@arm.com>

Timing Annotation is a feature of the model that enables high-level
performance estimations to be made [1]. It is not needed to demonstrate
a functioning software stack so set FASTSIM_DISABLE_TA to 1 in the model
environment to disable this feature. This also improves model
performance.

[1] https://developer.arm.com/documentation/100965/1119/Timing-Annotation

Signed-off-by: Peter Hoyes <Peter.Hoyes@arm.com>
---
 meta-arm/classes/fvpboot.bbclass | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Jon Mason Dec. 16, 2022, 2:12 p.m. UTC | #1 
On Fri, 16 Dec 2022 11:19:41 +0000, Peter Hoyes wrote:
> Timing Annotation is a feature of the model that enables high-level
> performance estimations to be made [1]. It is not needed to demonstrate
> a functioning software stack so set FASTSIM_DISABLE_TA to 1 in the model
> environment to disable this feature. This also improves model
> performance.
> 
> [1] https://developer.arm.com/documentation/100965/1119/Timing-Annotation

Applied, thanks!

[1/1] arm/fvpboot: Disable timing annotation by default
      commit: 025124814e8676e46d42ec5b07220283f1bdbcd0

Best regards,
diff mbox series

Patch

diff --git a/meta-arm/classes/fvpboot.bbclass b/meta-arm/classes/fvpboot.bbclass
index 78dabd73..508deb29 100644
--- a/meta-arm/classes/fvpboot.bbclass
+++ b/meta-arm/classes/fvpboot.bbclass
@@ -24,7 +24,9 @@  FVP_CONSOLES[default] ?= "${FVP_CONSOLE}"
 # Arbitrary extra arguments
 FVP_EXTRA_ARGS ?= ""
 # Bitbake variables to pass to the FVP environment
-FVP_ENV_PASSTHROUGH ?= ""
+FVP_ENV_PASSTHROUGH ?= "FASTSIM_DISABLE_TA"
+# Disable timing annotation by default
+FASTSIM_DISABLE_TA ?= "1"
 
 EXTRA_IMAGEDEPENDS += "${FVP_PROVIDER}"