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

[Feature]: Add QEMU Supported Version Check/Process #839

Closed
makubacki opened this issue Feb 5, 2024 · 4 comments
Closed

[Feature]: Add QEMU Supported Version Check/Process #839

makubacki opened this issue Feb 5, 2024 · 4 comments
Assignees
Labels
state:stale Has not been updated in a long time type:feature-request A new feature proposal urgency:low Little to no impact

Comments

@makubacki
Copy link
Member

makubacki commented Feb 5, 2024

Feature Overview

Older QEMU versions do not work with the latest FW code. Newer QEMU version sometimes have regressions and MMIs fail, etc. The QEMU version dictates whether firmware from the repo will succeed or fail.

A common solution to matching a supported version of QEMU was to simply use whatever version CI is using. However, this might lead to inconvenient updates, make it difficult to find matching versions in source history, and some people don't know how to find the version used in CI.

Solution Overview

  • Inform the user if they are using an incompatible QEMU version.
  • Manage the version in source control, so at any given point in source history a compatible QEMU version for that point is known.
  • Ideally, make the QEMU version available in an external dependency, so the binaries for a given point in time are immediately known.

Alternatives Considered

No response

Urgency

Low

Are you going to implement the feature request?

Someone else needs to implement the feature

Do you need maintainer feedback?

No maintainer feedback needed

Anything else?

No response

@makubacki makubacki added state:needs-triage Needs to triaged to determine next steps type:feature-request A new feature proposal labels Feb 5, 2024
@github-actions github-actions bot added urgency:low Little to no impact state:needs-owner Needs an issue owner to be assigned labels Feb 5, 2024
@makubacki
Copy link
Member Author

@Javagedes mentioned he might be interested. Please take it up if you still are and have time.

@makubacki makubacki removed the state:needs-triage Needs to triaged to determine next steps label Feb 5, 2024
@Javagedes Javagedes self-assigned this Feb 5, 2024
@github-actions github-actions bot removed the state:needs-owner Needs an issue owner to be assigned label Feb 5, 2024
@Javagedes
Copy link
Contributor

#852 will add a workflow to generate an external dependency containing qemu for Windows and Linux, that will be used by default when compiling platforms in this repository.

#851 will add the logic to consume the external dependency.

Copy link

This issue has been automatically marked as stale because it has not had activity in 45 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions.

@github-actions github-actions bot added the state:stale Has not been updated in a long time label Mar 24, 2024
Copy link

This issue has been automatically been closed because it did not have any activity in 45 days and no follow up within 7 days after being marked stale. Thank you for your contributions.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state:stale Has not been updated in a long time type:feature-request A new feature proposal urgency:low Little to no impact
Projects
None yet
Development

No branches or pull requests

2 participants