Re: [libvirt] [PATCH 5/7] kvmtool: Add new domain type

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Nov 11, 2011 at 07:57:04PM +0800, Osier Yang wrote:
> It's named as "kvmtool".
> ---
>  src/conf/domain_conf.c |    4 +++-
>  src/conf/domain_conf.h |    1 +
>  2 files changed, 4 insertions(+), 1 deletions(-)
> 
> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
> index 58f4d0f..55121d8 100644
> --- a/src/conf/domain_conf.c
> +++ b/src/conf/domain_conf.c
> @@ -91,7 +91,8 @@ VIR_ENUM_IMPL(virDomainVirt, VIR_DOMAIN_VIRT_LAST,
>                "hyperv",
>                "vbox",
>                "one",
> -              "phyp")
> +              "phyp",
> +              "kvmtool")
>  
>  VIR_ENUM_IMPL(virDomainBoot, VIR_DOMAIN_BOOT_LAST,
>                "fd",
> @@ -4018,6 +4019,7 @@ virDomainChrDefParseXML(virCapsPtr caps,
>      if (type == NULL) {
>          def->source.type = VIR_DOMAIN_CHR_TYPE_PTY;
>      } else if ((def->source.type = virDomainChrTypeFromString(type)) < 0) {
> +        VIR_WARN("type = %s", type);
>          virDomainReportError(VIR_ERR_XML_ERROR,
>                               _("unknown type presented to host for character device: %s"),
>                               type);
> diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
> index a3cb834..001bc46 100644
> --- a/src/conf/domain_conf.h
> +++ b/src/conf/domain_conf.h
> @@ -59,6 +59,7 @@ enum virDomainVirtType {
>      VIR_DOMAIN_VIRT_VBOX,
>      VIR_DOMAIN_VIRT_ONE,
>      VIR_DOMAIN_VIRT_PHYP,
> +    VIR_DOMAIN_VIRT_KVMTOOL,
>  
>      VIR_DOMAIN_VIRT_LAST,
>  };

IMHO this patch is not required. The domain type is refering to the
hypervisor used for the domain, which is still 'kvm'. What is different
here is just the userspace device model.  If you look at the 3 different
Xen user spaces we support, all of them use <domain type='xen'> still.
So just use  <domain type='kvm'> here for kvmtool.


Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux