[libvirt] [PATCH 01/13] conf: Define hvf domain type

Roman Bolshakov posted 13 patches 6 years, 6 months ago
[libvirt] [PATCH 01/13] conf: Define hvf domain type
Posted by Roman Bolshakov 6 years, 6 months ago
QEMU supports Hypervisor.framework as hvf accel since 2.12.

Signed-off-by: Roman Bolshakov <r.bolshakov@yadro.com>
---
 src/conf/domain_conf.c  | 4 +++-
 src/conf/domain_conf.h  | 1 +
 src/qemu/qemu_command.c | 4 ++++
 3 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index e8e0adc819..e5995fc1c8 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -119,7 +119,8 @@ VIR_ENUM_IMPL(virDomainVirt, VIR_DOMAIN_VIRT_LAST,
               "phyp",
               "parallels",
               "bhyve",
-              "vz")
+              "vz",
+              "hvf")
 
 VIR_ENUM_IMPL(virDomainOS, VIR_DOMAIN_OSTYPE_LAST,
               "hvm",
@@ -14977,6 +14978,7 @@ virDomainVideoDefaultType(const virDomainDef *def)
     case VIR_DOMAIN_VIRT_HYPERV:
     case VIR_DOMAIN_VIRT_PHYP:
     case VIR_DOMAIN_VIRT_NONE:
+    case VIR_DOMAIN_VIRT_HVF:
     case VIR_DOMAIN_VIRT_LAST:
     default:
         return VIR_DOMAIN_VIDEO_TYPE_DEFAULT;
diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
index e30a4b2fe7..dac39539df 100644
--- a/src/conf/domain_conf.h
+++ b/src/conf/domain_conf.h
@@ -245,6 +245,7 @@ typedef enum {
     VIR_DOMAIN_VIRT_PARALLELS,
     VIR_DOMAIN_VIRT_BHYVE,
     VIR_DOMAIN_VIRT_VZ,
+    VIR_DOMAIN_VIRT_HVF,
 
     VIR_DOMAIN_VIRT_LAST
 } virDomainVirtType;
diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index 1ff593c657..9150c690e5 100644
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -7130,6 +7130,10 @@ qemuBuildMachineCommandLine(virCommandPtr cmd,
         virBufferAddLit(&buf, ",accel=kvm");
         break;
 
+    case VIR_DOMAIN_VIRT_HVF:
+        virBufferAddLit(&buf, ",accel=hvf");
+        break;
+
     case VIR_DOMAIN_VIRT_KQEMU:
     case VIR_DOMAIN_VIRT_XEN:
     case VIR_DOMAIN_VIRT_LXC:
-- 
2.17.1 (Apple Git-112)

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 01/13] conf: Define hvf domain type
Posted by Daniel P. Berrangé 6 years, 6 months ago
On Sat, Oct 20, 2018 at 05:19:46PM +0300, Roman Bolshakov wrote:
> QEMU supports Hypervisor.framework as hvf accel since 2.12.
> 
> Signed-off-by: Roman Bolshakov <r.bolshakov@yadro.com>
> ---
>  src/conf/domain_conf.c  | 4 +++-
>  src/conf/domain_conf.h  | 1 +
>  src/qemu/qemu_command.c | 4 ++++
>  3 files changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
> index e8e0adc819..e5995fc1c8 100644
> --- a/src/conf/domain_conf.c
> +++ b/src/conf/domain_conf.c
> @@ -119,7 +119,8 @@ VIR_ENUM_IMPL(virDomainVirt, VIR_DOMAIN_VIRT_LAST,
>                "phyp",
>                "parallels",
>                "bhyve",
> -              "vz")
> +              "vz",
> +              "hvf")
>  
>  VIR_ENUM_IMPL(virDomainOS, VIR_DOMAIN_OSTYPE_LAST,
>                "hvm",
> @@ -14977,6 +14978,7 @@ virDomainVideoDefaultType(const virDomainDef *def)
>      case VIR_DOMAIN_VIRT_HYPERV:
>      case VIR_DOMAIN_VIRT_PHYP:
>      case VIR_DOMAIN_VIRT_NONE:
> +    case VIR_DOMAIN_VIRT_HVF:
>      case VIR_DOMAIN_VIRT_LAST:
>      default:
>          return VIR_DOMAIN_VIDEO_TYPE_DEFAULT;
> diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
> index e30a4b2fe7..dac39539df 100644
> --- a/src/conf/domain_conf.h
> +++ b/src/conf/domain_conf.h
> @@ -245,6 +245,7 @@ typedef enum {
>      VIR_DOMAIN_VIRT_PARALLELS,
>      VIR_DOMAIN_VIRT_BHYVE,
>      VIR_DOMAIN_VIRT_VZ,
> +    VIR_DOMAIN_VIRT_HVF,
>  
>      VIR_DOMAIN_VIRT_LAST
>  } virDomainVirtType;
> diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
> index 1ff593c657..9150c690e5 100644
> --- a/src/qemu/qemu_command.c
> +++ b/src/qemu/qemu_command.c
> @@ -7130,6 +7130,10 @@ qemuBuildMachineCommandLine(virCommandPtr cmd,
>          virBufferAddLit(&buf, ",accel=kvm");
>          break;
>  
> +    case VIR_DOMAIN_VIRT_HVF:
> +        virBufferAddLit(&buf, ",accel=hvf");
> +        break;
> +
>      case VIR_DOMAIN_VIRT_KQEMU:
>      case VIR_DOMAIN_VIRT_XEN:
>      case VIR_DOMAIN_VIRT_LXC:

Should also add to tests/qemuxml2argvtest.c to illustrate the hvf
example.


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list