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.

Sign up for GitHub

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

Jump to bottom

Enable CONFIG_VHOST_VSOCK on rockchip64 #7512

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Enable CONFIG_VHOST_VSOCK on rockchip64 #7512

wants to merge 1 commit into from
+2 −2

Conversation

halfa
Copy link

@halfa halfa commented Nov 23, 2024 •
edited
Loading

Description

The Incus (LXD fork after Canonical takeover) container and VM management tools require CONFIG_VHOST_VSOCK to be enabled to use VMs, and some targets are missing that option. I'm not sure of the implications, but they seem to be null as this is linked to the virtio subsystem.

Documentation summary for feature / change

  • short description (copy / paste of PR title)

How Has This Been Tested?

I did test this on my helios64 with kernel 6.6.47 and I was able to start a VM with Incus.

Checklist:

Please delete options that are not relevant.

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings

@github-actions github-actions bot added size/small PR with less then 50 lines Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... labels Nov 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amazingfate amazingfate amazingfate approved these changes

@Manouchehri Manouchehri Awaiting requested review from Manouchehri Manouchehri is a code owner

@TRSx80 TRSx80 Awaiting requested review from TRSx80 TRSx80 is a code owner

@TheSnowfield TheSnowfield Awaiting requested review from TheSnowfield TheSnowfield is a code owner

@Tonymac32 Tonymac32 Awaiting requested review from Tonymac32 Tonymac32 is a code owner

@ahoneybun ahoneybun Awaiting requested review from ahoneybun ahoneybun is a code owner

@brentr brentr Awaiting requested review from brentr brentr is a code owner

@catalinii catalinii Awaiting requested review from catalinii catalinii is a code owner

@clee clee Awaiting requested review from clee clee is a code owner

@igorpecovnik igorpecovnik Awaiting requested review from igorpecovnik igorpecovnik is a code owner

@joekhoobyar joekhoobyar Awaiting requested review from joekhoobyar joekhoobyar is a code owner

@krachlatte krachlatte Awaiting requested review from krachlatte krachlatte is a code owner

@paolosabatino paolosabatino Awaiting requested review from paolosabatino paolosabatino is a code owner

@rpardini rpardini Awaiting requested review from rpardini rpardini is a code owner

Assignees
No one assigned
Labels
Hardware Hardware related like kernel, U-Boot, ... Needs review Seeking for review size/small PR with less then 50 lines
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@halfa @amazingfate