-
Notifications
You must be signed in to change notification settings - Fork 321
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
[BUG] vmh_test_single test fails #8629
Comments
@dabekjakub Can you check whether the calling sequence (i.e. the test) is correct? This seems to fail 100% and is causing "interesting" side-effects when failing (DSP panics as seen in #8621 ). |
Did this test pass when @lyakh submited it ? I am currently trying to setup unit testing for vmh. I will look at this one after I wrap up my tests. |
@dabekjakub @marc-hb I'd say it depends on what "this test" is referring to. If it's referring to the complete boot-test, then yes, Marc's answer is correct. If however it's referring to the VMH test, then no, it never passed. The boot-test passed on cAVS because VMH is disabled there (IIRC, or at least the vmh test was never enabled there) |
Ok, this is actually quite good (for ztest and we've caught an issue early) - it means the test results are being correctly captured. @kv2019i was this manual testing or automatic via CI ? |
Catching errors in firmware logs is still blocked because there are a few errors left in every run: |
@kv2019i |
Describe the bug
vmh_test_single() test in work/sof/zephyr/test/vmh.c fails.
To Reproduce
Reproduction Rate
100%
Expected behavior
Test should pass
Impact
The failing test caused DSP panics in #8621 . The relation is not fully understood yet, but potentially vmh error can have serious side-impacts that we need to understand
Environment
Screenshots or console output
#8621 (comment)
The text was updated successfully, but these errors were encountered: