-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[microTVM] Add wait to QEMU Setup #8236
Conversation
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.
added a couple of comments. can you add a test for the case where make fails to launch QEMU? e.g. just delete a file from the project between build and flash
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.
thanks for writing the test! a few minor things
1d9b45a
to
ae31405
Compare
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.
ok just two more last things here, this looks good otherwise
thanks @mehrdadh ! |
* add wait * address comments * address comments * test added * add suggestions * fadd message assert for test
* add wait * address comments * address comments * test added * add suggestions * fadd message assert for test
In some cases where the Zephyr project would rebuild itself, qemu would setup and wait for message from the microtvm target, however the microtvm target is not ready yet since these are happening in different processes. In this PR we add a wait to qemu to make sure they happen synchronously.
cc @areusch