Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gp: disable test ce-7b-17 (gp_40178) #511

Merged
merged 1 commit into from
Apr 28, 2021

Conversation

jforissier
Copy link
Contributor

Add a patch to exclude ce-7b-17 from the test suite since OP-TEE does
not provide the necessary external command to erase TA persistent time.

Note that skipping a test will shift the numbering in the "gp" suite,
so after this patch gp_40178 will be the same as gp_40179 prior to the
patch.

Fixes: #510
Signed-off-by: Jerome Forissier jerome@forissier.org

@jenswi-linaro
Copy link
Contributor

Since this is a bug in OP-TEE we should keep track of this so it can be fixed eventually.

@jenswi-linaro
Copy link
Contributor

Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>

@jforissier
Copy link
Contributor Author

Since this is a bug in OP-TEE we should keep track of this so it can be fixed eventually.

Not exactly a bug, because the spec does not mention there should be any mean to discard the persistent time once it has been set (see the state machine in the description of TEE_GetTAPersistentTime()). It is only needed for testability. That would be a special "test mode" enabled at compile time perhaps.

Anyway, I will create an issue.

Add a patch to exclude ce-7b-17 from the test suite since OP-TEE does
not provide the necessary external command to erase TA persistent time.

Note that skipping a test will shift the numbering in the "gp" suite,
so after this patch gp_40178 will be the same as gp_40179 prior to the
patch.

Fixes: OP-TEE#510
Signed-off-by: Jerome Forissier <jerome@forissier.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
@jforissier jforissier merged commit 159cd4a into OP-TEE:master Apr 28, 2021
@jforissier jforissier deleted the disable-ce-7b-17 branch April 28, 2021 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

gp_40178 random failures
2 participants