-
-
Notifications
You must be signed in to change notification settings - Fork 3
Sv boot test part1 #915
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
base: develop
Are you sure you want to change the base?
Sv boot test part1 #915
Conversation
Command used to run QEMU for these tests: Mounting USB disk image is not yet there so I had to specify the DTS image manually from the start. |
Logs from running the tests on revision 4a84254 |
4a84254
to
fce9dbc
Compare
Should it fix this run failure as well: Dasharo/coreboot#705 (comment) ? EDIT: OK, I think not, I can see that mostly the new suite is added here. |
No it won't. When wizard is enabled, the boot flow is different and no regular test will work. |
... settings reset. | ||
Skip If not ${TESTS_IN_FIRMWARE_SUPPORT} SVB001.002 not supported | ||
# FIXME: doesn't work on QEMU, start QEMU with DTS already mounted! | ||
# Mount USB Disk Image ${TEST_DATA_DIR}/dts/dts-base-image-v2.1.3.wic |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have a relevant issue to be linked here?
@philipanda just recently replaced UEFI shell with disk detection in boot manager tests here: https://github.com/Dasharo/open-source-firmware-validation/pull/859/files#diff-f69b8b427ce0b5b677a07ffe9a4c20976614e4b5975fecb2e8bdc9a61371097aR135
It even passes on your PR: https://github.com/Dasharo/open-source-firmware-validation/actions/runs/15976860721/job/45061516534#step:11:31
Though I can see a different kwd is used, so maybe the internals of Mount USB Disk Image
for QEMU needs fixing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Mount USB Disk Image
keyword first removes the drive from QEMU and only then mounts it. CI just mounts it right away. Maybe the keyword fails because there is nothing to remove. I can't really tell from the description and without the logs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This time I tried and it worked, but changed nothing in the code except uncommenting this line...
Also I started the QEMU with graphic firmware
instead of graphic os
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that I want to start QEMU again I get either:
access denied by acl file
qemu-system-x86_64: -netdev bridge,id=vmnic,br=br0: bridge helper failed
Or:
failed to get mtu of bridge `br0': No such device
qemu-system-x86_64: -netdev bridge,id=vmnic,br=br0: bridge helper failed
But it worked before...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've been experimenting with qemu-run.sh to conditionally use a bridged interface so that the machines can connect to our NFSes here https://github.com/Dasharo/open-source-firmware-validation/blob/qemu-run-install-windows/scripts/ci/qemu-run.sh#L169-L174
Seems it was partially merged by mistake in this commit fc455d8
You can either create a bridge named br0
, so that the QEMU VM will be connected to the local network, or revert the changes: https://github.com/Dasharo/open-source-firmware-validation/pull/923/files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess if the base already changed, everything may or may not work anymore...
fce9dbc
to
1f83eb7
Compare
Rebasing to see if/what would be picked up by the tests on HW. |
So if we look at the logs from run on HW there are a couple of problems: https://github.com/Dasharo/open-source-firmware-validation/actions/runs/16071833981/job/45358037556
@wiktormowinski @filipleple Is it from your latest tests?
Should this still be a problem @philipanda ?
|
If no one has modified the test to not depend on UEFI shell in that test, then yes. I've only removed the dependency from qemu self-tests.
That's sad, we might switch to the Odroid faster than I thought. It was working fine just yesterday.
Actually, there were three runs despite there being only two platforms. Both suites were run on both of them, but on MSI they were split into two runs... I'd say that's a problem - a bug, and it might cause serious issues as the runs are being executed in parallel, but only one can access the device at a time. I suspect this might the cause of MSI being "unresponsive". *Edit Ah, I see. The PiKVM of MSI was replaced and got a new IP address. I've updated the IP only in the config for modifying test cases. The config for modifying libraries was left intact. That's why the third run says there is no route to host. |
Signed-off-by: Michał Żygowski <[email protected]>
…hase 1 Signed-off-by: Michał Żygowski <[email protected]>
1f83eb7
to
7c4b66d
Compare
No description provided.