mirror of
https://github.com/kata-containers/kata-containers.git
synced 2025-06-29 16:57:18 +00:00
vsock_module: add 'vhost_vsock' kernel module in kata-check
Since we prefer vsock over virtio serial port, we add 'vhost_vsock' in kernel mosules list. But vhost_vsock.ko shouldn't be the definitely required kernel modules, afterall, we could also use virtio serial port. if kata-env shows SupportsVSocks as false, users could run kata-check to manually load vhost_vsock.ko and get detailed info(errors) Fixes: #1512 Signed-off-by: Penny Zheng <penny.zheng@arm.com>
This commit is contained in:
parent
f21d5a37fe
commit
9b23d4f143
@ -35,6 +35,9 @@ type kernelModule struct {
|
||||
|
||||
// maps parameter names to values
|
||||
parameters map[string]string
|
||||
|
||||
// if it is definitely required
|
||||
required bool
|
||||
}
|
||||
|
||||
type vmContainerCapableDetails struct {
|
||||
@ -124,8 +127,11 @@ func haveKernelModule(module string) bool {
|
||||
// Now, check if the module is unloaded, but available.
|
||||
// And modprobe it if so.
|
||||
cmd := exec.Command(modProbeCmd, module)
|
||||
err := cmd.Run()
|
||||
return err == nil
|
||||
if output, err := cmd.CombinedOutput(); err != nil {
|
||||
kataLog.WithField("module", module).WithError(err).Warnf("modprobe insert module failed: %s", string(output))
|
||||
return false
|
||||
}
|
||||
return true
|
||||
}
|
||||
|
||||
// checkCPU checks all required CPU attributes modules and returns a count of
|
||||
@ -200,7 +206,9 @@ func checkKernelModules(modules map[string]kernelModule, handler kernelParamHand
|
||||
|
||||
if !haveKernelModule(module) {
|
||||
kataLog.WithFields(fields).Error("kernel property not found")
|
||||
count++
|
||||
if details.required {
|
||||
count++
|
||||
}
|
||||
continue
|
||||
}
|
||||
|
||||
|
@ -15,14 +15,15 @@ import (
|
||||
)
|
||||
|
||||
const (
|
||||
cpuFlagsTag = genericCPUFlagsTag
|
||||
archCPUVendorField = genericCPUVendorField
|
||||
archCPUModelField = genericCPUModelField
|
||||
archGenuineIntel = "GenuineIntel"
|
||||
archAuthenticAMD = "AuthenticAMD"
|
||||
msgKernelVM = "Kernel-based Virtual Machine"
|
||||
msgKernelVirtio = "Host kernel accelerator for virtio"
|
||||
msgKernelVirtioNet = "Host kernel accelerator for virtio network"
|
||||
cpuFlagsTag = genericCPUFlagsTag
|
||||
archCPUVendorField = genericCPUVendorField
|
||||
archCPUModelField = genericCPUModelField
|
||||
archGenuineIntel = "GenuineIntel"
|
||||
archAuthenticAMD = "AuthenticAMD"
|
||||
msgKernelVM = "Kernel-based Virtual Machine"
|
||||
msgKernelVirtio = "Host kernel accelerator for virtio"
|
||||
msgKernelVirtioNet = "Host kernel accelerator for virtio network"
|
||||
msgKernelVirtioVhostVsock = "Host Support for Linux VM Sockets"
|
||||
)
|
||||
|
||||
// CPU types
|
||||
@ -75,17 +76,25 @@ func setCPUtype() error {
|
||||
}
|
||||
archRequiredKernelModules = map[string]kernelModule{
|
||||
"kvm": {
|
||||
desc: msgKernelVM,
|
||||
desc: msgKernelVM,
|
||||
required: true,
|
||||
},
|
||||
"kvm_intel": {
|
||||
desc: "Intel KVM",
|
||||
parameters: kvmIntelParams,
|
||||
required: true,
|
||||
},
|
||||
"vhost": {
|
||||
desc: msgKernelVirtio,
|
||||
desc: msgKernelVirtio,
|
||||
required: true,
|
||||
},
|
||||
"vhost_net": {
|
||||
desc: msgKernelVirtioNet,
|
||||
desc: msgKernelVirtioNet,
|
||||
required: true,
|
||||
},
|
||||
"vhost_vsock": {
|
||||
desc: msgKernelVirtioVhostVsock,
|
||||
required: false,
|
||||
},
|
||||
}
|
||||
} else if cpuType == cpuTypeAMD {
|
||||
@ -99,16 +108,24 @@ func setCPUtype() error {
|
||||
}
|
||||
archRequiredKernelModules = map[string]kernelModule{
|
||||
"kvm": {
|
||||
desc: msgKernelVM,
|
||||
desc: msgKernelVM,
|
||||
required: true,
|
||||
},
|
||||
"kvm_amd": {
|
||||
desc: "AMD KVM",
|
||||
desc: "AMD KVM",
|
||||
required: true,
|
||||
},
|
||||
"vhost": {
|
||||
desc: msgKernelVirtio,
|
||||
desc: msgKernelVirtio,
|
||||
required: true,
|
||||
},
|
||||
"vhost_net": {
|
||||
desc: msgKernelVirtioNet,
|
||||
desc: msgKernelVirtioNet,
|
||||
required: true,
|
||||
},
|
||||
"vhost_vsock": {
|
||||
desc: msgKernelVirtioVhostVsock,
|
||||
required: false,
|
||||
},
|
||||
}
|
||||
}
|
||||
|
@ -30,13 +30,20 @@ var archRequiredCPUAttribs = map[string]string{}
|
||||
// required module parameters.
|
||||
var archRequiredKernelModules = map[string]kernelModule{
|
||||
"kvm": {
|
||||
desc: "Kernel-based Virtual Machine",
|
||||
desc: "Kernel-based Virtual Machine",
|
||||
required: true,
|
||||
},
|
||||
"vhost": {
|
||||
desc: "Host kernel accelerator for virtio",
|
||||
desc: "Host kernel accelerator for virtio",
|
||||
required: true,
|
||||
},
|
||||
"vhost_net": {
|
||||
desc: "Host kernel accelerator for virtio network",
|
||||
desc: "Host kernel accelerator for virtio network",
|
||||
required: true,
|
||||
},
|
||||
"vhost_vsock": {
|
||||
desc: "Host Support for Linux VM Sockets",
|
||||
required: false,
|
||||
},
|
||||
}
|
||||
|
||||
|
@ -42,10 +42,16 @@ var archRequiredCPUAttribs = map[string]string{}
|
||||
// required module parameters.
|
||||
var archRequiredKernelModules = map[string]kernelModule{
|
||||
"kvm": {
|
||||
desc: "Kernel-based Virtual Machine",
|
||||
desc: "Kernel-based Virtual Machine",
|
||||
required: true,
|
||||
},
|
||||
"kvm_hv": {
|
||||
desc: "Kernel-based Virtual Machine hardware virtualization",
|
||||
desc: "Kernel-based Virtual Machine hardware virtualization",
|
||||
required: true,
|
||||
},
|
||||
"vhost_vsock": {
|
||||
desc: "Host Support for Linux VM Sockets",
|
||||
required: false,
|
||||
},
|
||||
}
|
||||
|
||||
|
@ -33,7 +33,12 @@ var archRequiredCPUAttribs = map[string]string{}
|
||||
// required module parameters.
|
||||
var archRequiredKernelModules = map[string]kernelModule{
|
||||
"kvm": {
|
||||
desc: "Kernel-based Virtual Machine",
|
||||
desc: "Kernel-based Virtual Machine",
|
||||
required: true,
|
||||
},
|
||||
"vhost_vsock": {
|
||||
desc: "Host Support for Linux VM Sockets",
|
||||
required: false,
|
||||
},
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user