提交 5645badd 编写于 作者: A Andrea Bolognani

gic: Remove VIR_GIC_VERSION_DEFAULT

The QEMU default is GICv2, and some of the code in libvirt
relies on the exact value. Stop pretending that's not the
case and use GICv2 explicitly where needed.
Signed-off-by: NAndrea Bolognani <abologna@redhat.com>
上级 bc07101a
...@@ -7403,9 +7403,9 @@ qemuBuildMachineCommandLine(virCommandPtr cmd, ...@@ -7403,9 +7403,9 @@ qemuBuildMachineCommandLine(virCommandPtr cmd,
goto cleanup; goto cleanup;
} }
/* The default GIC version should not be specified on the /* The default GIC version (GICv2) should not be specified on
* QEMU commandline for backwards compatibility reasons */ * the QEMU commandline for backwards compatibility reasons */
if (def->gic_version != VIR_GIC_VERSION_DEFAULT) { if (def->gic_version != VIR_GIC_VERSION_2) {
if (!virQEMUCapsGet(qemuCaps, if (!virQEMUCapsGet(qemuCaps,
QEMU_CAPS_MACH_VIRT_GIC_VERSION)) { QEMU_CAPS_MACH_VIRT_GIC_VERSION)) {
virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s", virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
......
...@@ -2596,12 +2596,11 @@ qemuDomainDefEnableDefaultFeatures(virDomainDefPtr def, ...@@ -2596,12 +2596,11 @@ qemuDomainDefEnableDefaultFeatures(virDomainDefPtr def,
def->features[VIR_DOMAIN_FEATURE_GIC] = VIR_TRISTATE_SWITCH_ON; def->features[VIR_DOMAIN_FEATURE_GIC] = VIR_TRISTATE_SWITCH_ON;
} }
/* Use the default GIC version if no version was specified */ /* Use the default GIC version (GICv2) if no version was specified */
if (def->features[VIR_DOMAIN_FEATURE_GIC] == VIR_TRISTATE_SWITCH_ON && if (def->features[VIR_DOMAIN_FEATURE_GIC] == VIR_TRISTATE_SWITCH_ON &&
def->gic_version == VIR_GIC_VERSION_NONE) { def->gic_version == VIR_GIC_VERSION_NONE) {
VIR_DEBUG("Using GIC version %s (default)", VIR_DEBUG("Using GIC version 2 (default)");
virGICVersionTypeToString(VIR_GIC_VERSION_DEFAULT)); def->gic_version = VIR_GIC_VERSION_2;
def->gic_version = VIR_GIC_VERSION_DEFAULT;
} }
} }
......
...@@ -35,9 +35,6 @@ typedef enum { ...@@ -35,9 +35,6 @@ typedef enum {
VIR_ENUM_DECL(virGICVersion); VIR_ENUM_DECL(virGICVersion);
/* Consider GIC v2 the default */
# define VIR_GIC_VERSION_DEFAULT VIR_GIC_VERSION_2
typedef enum { typedef enum {
VIR_GIC_IMPLEMENTATION_NONE = 0, VIR_GIC_IMPLEMENTATION_NONE = 0,
VIR_GIC_IMPLEMENTATION_KERNEL = (1 << 1), VIR_GIC_IMPLEMENTATION_KERNEL = (1 << 1),
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册