Skip to content

updating QEMU parameters and default path for the firmware#1058

Open
AlissonMachadoEmbitel wants to merge 1 commit intolf-edge:masterfrom
AlissonMachadoEmbitel:qemu-fix-macos
Open

updating QEMU parameters and default path for the firmware#1058
AlissonMachadoEmbitel wants to merge 1 commit intolf-edge:masterfrom
AlissonMachadoEmbitel:qemu-fix-macos

Conversation

@AlissonMachadoEmbitel
Copy link
Copy Markdown

Hello,

I was facing a couple of issues when trying to run it on MacOS, then I made these fixes.

It was the same problem related on this issue: #1028

@uncleDecart
Copy link
Copy Markdown
Member

Hey @AlissonMachadoEmbitel thank you very much for your contribution! Couple of things:

  1. We use commit signing in, so all of our commits are signed by authors, you can easily do that by running
git commit --amend  --signoff
  1. Could you please tell me what qemu version are you using?
    You can do it by running if you installed it with brew or you have qemu binary in your PATH
qemu-system-x86_64 --version

Signed-off-by: AlissonMachadoEmbitel <alisson.machado@diconium.com>
@AlissonMachadoEmbitel
Copy link
Copy Markdown
Author

Hi @uncleDecart ,

This is my current version:

➜ ~ qemu-system-x86_64 --version
QEMU emulator version 9.2.2
Copyright (c) 2003-2024 Fabrice Bellard and the QEMU Project developers

@europaul
Copy link
Copy Markdown
Contributor

I think it's a valid concern and I've seen this issue before, but it works on my combo QEMU 10.0.2 on Linux.
@shjala did you face something similar on macOS?

@shjala
Copy link
Copy Markdown
Member

shjala commented Aug 15, 2025

@europaul I run things on a remote linux machine.

Comment thread pkg/defaults/defaults.go

DefaultQemuAccelDarwin = "-machine q35,accel=hvf -cpu kvm64,kvmclock=off "
DefaultQemuAccelDarwinArm64 = "-machine virt,accel=hvf,usb=off,dump-guest-core=off -cpu host "
DefaultQemuAccelDarwinArm64 = "-machine virt,accel=hvf,usb=off,dump-guest-core=on -cpu host "
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This option is not needed for macOS. This just includes the guest memory in a core dump. The default is already on, but let's keep it off to be consistent with the options for Linux.

@rene
Copy link
Copy Markdown
Contributor

rene commented Apr 30, 2026

I tested on macOS, it works. @AlissonMachadoEmbitel , I know it's been a year from this PR, but if you are still interested, I just have one comment to be addressed and we are good to merge....

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