Skip to content

Commit

Permalink
Revert "qemu_v8.mk: enable MEASURED_BOOT_FTPM by default"
Browse files Browse the repository at this point in the history
This reverts commit 9bf2bbe. CI has
become quite unstable since that commit was merged, see for instance
the multiple failures in [1] and the absence of failure in [2]. I do
not see any direct relationship betweem fTPM and the errors reported
in the CI runs, but in order to make CI useful again we need to have
it run reliably so disable MEASURED_BOOT_FTPM temporarily.

In my tests, 'make -j$(nproc) check-only XTEST_ARGS="-l 14"' always
failed with a TA panic when MEASURED_BOOT_FTPM=y, and never when it
is =n. The panic sometimes happens in xtest regression_4006, or
sometimes in pkcs11_1021 or pkcs11_1023.

Link: [1] OP-TEE/optee_os#7170
Link: [2] OP-TEE/optee_os#7175
Signed-off-by: Jerome Forissier <[email protected]>
Acked-by: Etienne Carriere <[email protected]>
Acked-by: Jens Wiklander <[email protected]>
  • Loading branch information
jforissier committed Dec 11, 2024
1 parent e537475 commit 9457d99
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions qemu_v8.mk
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,6 @@ ifeq ($(shell uname -m),x86_64)
RUST_ENABLE ?= y
endif

# Enable fTPM
MEASURED_BOOT_FTPM ?= y

include common.mk

DEBUG ?= 1
Expand Down

0 comments on commit 9457d99

Please sign in to comment.