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

Finalize the "DCU004.001 Verify SMMSTORE changes" test case #541

Open
macpijan opened this issue Oct 18, 2024 · 4 comments
Open

Finalize the "DCU004.001 Verify SMMSTORE changes" test case #541

macpijan opened this issue Oct 18, 2024 · 4 comments
Assignees

Comments

@macpijan
Copy link
Contributor

macpijan commented Oct 18, 2024

Test environment version

Dasharo version (if applicable)

N/A

Dasharo variant (if applicable)

N/A

Affected component(s) or functionality (if applicable)

Brief summary

It seems this test has never been finalized: https://github.com/Dasharo/open-source-firmware-validation/blob/dcbfa25002de61b0ab812e6caacb51397cefe875/dasharo-compatibility/dcu.robot#L97C1-L97C35

Additional context

It may be that at the time of writing it, the DCU has not supported SMMSTORE changes yet. But now it does, and we could finalize this test and make sure it works correctly.

@philipandag
Copy link
Contributor

It seems that the test was expected to be run on a device which supports tests in the setup menu. The dcu utility allows to check for the values of variables, so I think it might be better to use it to check them.

After performing all the writes the fw from the chip could be read to a separate file and the variables could be checked in the file.

Although using the tested utility to test itself might lead to problems. I think I will create two implementations of the test. One in the setup menu via serial and one via ssh as a self-test of dcu.

@philipandag
Copy link
Contributor

Another issue here is that not every smmstore variable suggested in the comments is implemented in dcu. I think changing and verifying one variable should be enough to test if writing to the SMMSTORE works.

@philipandag
Copy link
Contributor

Tests added in #543

Logs:
dcu.robot_log_.zip

@krystian-hebel
Copy link
Contributor

Although using the tested utility to test itself might lead to problems. I think I will create two implementations of the test. One in the setup menu via serial and one via ssh as a self-test of dcu.

👍, but please mark in the documentation that the one used for self-tests can't be used on hardware, or we will have false-positive results in regression tests.

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

When branches are created from issues, their pull requests are automatically linked.

3 participants