[DO NOT MERGE] ci: QEMUv8: set MEASURED_BOOT_FTPM=n #7175
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Disable fTPM to check if this un-breaks CI. Signed-off-by: Jerome Forissier <[email protected]>
jforissier
added a commit
to jforissier/optee_repo_build_files
that referenced
this pull request
Dec 11, 2024
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. Link: [1] OP-TEE/optee_os#7170 Link: [2] OP-TEE/optee_os#7175 Signed-off-by: Jerome Forissier <[email protected]>
jforissier
added a commit
to jforissier/optee_repo_build_files
that referenced
this pull request
Dec 11, 2024
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]>
jforissier
added a commit
to jforissier/optee_repo_build_files
that referenced
this pull request
Dec 11, 2024
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]>
jforissier
added a commit
to OP-TEE/build
that referenced
this pull request
Dec 11, 2024
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]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Disable fTPM to check if this un-breaks CI.