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

fix disable KVM on arch architecture #169

Closed
wants to merge 1 commit into from
Closed

Conversation

giper45
Copy link

@giper45 giper45 commented Dec 6, 2024

No description provided.

@kroese
Copy link
Contributor

kroese commented Dec 6, 2024

Why?

@giper45
Copy link
Author

giper45 commented Dec 6, 2024

Hi, I tried on arm64 (macOS M1) machine, it gives errors due to missing KVM device...
I though that this would be an error for arm-related machine that should disable kvm.
If kvm works for native arm, maybe this should be not performed. However, I know that you do not support macOS, but by disabling KVM (through this change, basically), it actually simply works for simple alpine setup.
If on arm Linux the behavior should be enable KVM, I could add an external KVM_DISABLE env to reach the purpose for macOS systems.

@kroese
Copy link
Contributor

kroese commented Dec 6, 2024

KVM works fine on ARM systems like the Raspberry Pi. It only doesnt work on Mac M1.

And there already exists an environment variable to disable it:

environment:
  KVM: "N"

It's even written in the error message that you can use that flag? And thats its not recommended because emulation is extremely slow that way.

@giper45
Copy link
Author

giper45 commented Dec 6, 2024

Ah ok sorry, yes it appeared in the log, did not figure out that was related to Docker external environment variables.
I know about performance issues, but I would like to try to integrate the acceleration of Mac hvf ecc. by taking information from the UTM virtualization system.
BTW great project!

@kroese kroese closed this Dec 7, 2024
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