Skip to content

xtask: Improve QEMU boot time #805

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

Merged

Conversation

nicholasbishop
Copy link
Member

On aarch64 it sits for a while on the initial screen and shows a progress bar to give the user time to enter the boot menu. We don't want to do that in our tests, so set the timeout to zero.

Even though we're skipping the boot menu, we have to also enable it here, otherwise the timeout value is ignored and a hardcoded platform default of 3 seconds is used.

Checklist

  • Sensible git history (for example, squash "typo" or "fix" commits). See the Rewriting History guide for help.
  • Update the changelog (if necessary)

@nicholasbishop nicholasbishop marked this pull request as ready for review May 14, 2023 14:49
@phip1611 phip1611 force-pushed the bishop-speed-up-qemu-boot branch from 37c95be to ff06d3b Compare May 17, 2023 19:52
On aarch64 it sits for a while on the initial screen and shows a progress bar to
give the user time to enter the boot menu. We don't want to do that in our
tests, so set the timeout to zero.

Even though we're skipping the boot menu, we have to also enable it here,
otherwise the timeout value is ignored and a hardcoded platform default of 3
seconds is used.
@nicholasbishop nicholasbishop force-pushed the bishop-speed-up-qemu-boot branch from ff06d3b to 60b23be Compare May 17, 2023 20:04
@nicholasbishop nicholasbishop merged commit 539a7f1 into rust-osdev:main May 17, 2023
@nicholasbishop nicholasbishop deleted the bishop-speed-up-qemu-boot branch May 17, 2023 20:20
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.

2 participants