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

Makefile: Workaround to fix QEMU's MSR errors #3316

Merged
merged 1 commit into from
Jul 6, 2023
Merged

Conversation

rene
Copy link
Contributor

@rene rene commented Jul 5, 2023

A QEMU crash has being experienced for some combinations of newer kernels and QEMU versions on the host when running EVE from QEMU (with make run-live). QEMU crashes when trying to access VMX_PROCBASED_CTLS2 MSR register and VMs/Containers are not initialized:

error: failed to set MSR 0x48b to 0x137bff00000000

This commit provides a workaround while the proper fix is not provided.

@rene rene requested review from eriknordmark and rvs as code owners July 5, 2023 14:49
@OhmSpectator
Copy link
Member

I've experienced the issue, and this workaround works for me.

But a proper solution should be found later.
Can you please put this node somewhere close to where this workaround is introduced?

@OhmSpectator
Copy link
Member

In general, LGTM

A QEMU crash has being experienced for some combinations of newer kernels
and QEMU versions on the host when running EVE from QEMU (with make
run-live). QEMU crashes when trying to access VMX_PROCBASED_CTLS2 MSR
register and VMs/Containers are not initialized:

error: failed to set MSR 0x48b to 0x137bff00000000

This commit provides a workaround while the proper fix is not provided.

Signed-off-by: Renê de Souza Pinto <rene@renesp.com.br>
@rene
Copy link
Contributor Author

rene commented Jul 5, 2023

I've experienced the issue, and this workaround works for me.

But a proper solution should be found later. Can you please put this node somewhere close to where this workaround is introduced?

it's done @OhmSpectator .

@OhmSpectator
Copy link
Member

it's done @OhmSpectator .

Great, thanks!

Copy link

@zedi-pramodh zedi-pramodh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@milan-zededa
Copy link
Contributor

I started experiencing this issue with eden as well and this patch fixes the issue. Rene, would you open PR with this workaround also for eden (this needs your change)?

@rouming rouming merged commit 668685a into lf-edge:master Jul 6, 2023
@rene rene deleted the fix-qemu branch November 2, 2023 09:40
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.

5 participants