WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH 1 of 6 V2] libxl: Give the HVM domain type the na

To: Ian Campbell <ian.campbell@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH 1 of 6 V2] libxl: Give the HVM domain type the name "HVM"
From: Wei LIU <liuw@xxxxxxxxx>
Date: Mon, 18 Jul 2011 22:56:51 +0800
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Delivery-date: Mon, 18 Jul 2011 07:57:56 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <90e2ae994ebbc37ba627.1310997430@xxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <patchbomb.1310997429@xxxxxxxxxxxxxxxxxxxxx> <90e2ae994ebbc37ba627.1310997430@xxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Mon, 2011-07-18 at 14:57 +0100, Ian Campbell wrote:
> # HG changeset patch
> # User Ian Campbell <ian.campbell@xxxxxxxxxx>
> # Date 1310997149 -3600
> # Node ID 90e2ae994ebbc37ba6279b9071ab2e76c2ebeec2
> # Parent  b18728227bf9e45058a4cc22425d1d35317e2c8d
> libxl: Give the HVM domain type the name "HVM"
> 
> This is generally used in the Xen universe, rather than "FV" which is
> not used elsewhere.
> 
> Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> 
> diff -r b18728227bf9 -r 90e2ae994ebb tools/libxl/libxl.idl
> --- a/tools/libxl/libxl.idl   Mon Jul 18 14:52:29 2011 +0100
> +++ b/tools/libxl/libxl.idl   Mon Jul 18 14:52:29 2011 +0100
> @@ -21,7 +21,7 @@ libxl_hwcap = Builtin("hwcap")
>  #
>  
>  libxl_domain_type = Enumeration("domain_type", [
> -    (1, "FV"),
> +    (1, "HVM"),
>      (2, "PV"),
>      ])
>  
> diff -r b18728227bf9 -r 90e2ae994ebb tools/libxl/libxl_dm.c
> --- a/tools/libxl/libxl_dm.c  Mon Jul 18 14:52:29 2011 +0100
> +++ b/tools/libxl/libxl_dm.c  Mon Jul 18 14:52:29 2011 +0100
> @@ -144,7 +144,7 @@ static char ** libxl__build_device_model
>      if (info->serial) {
>          flexarray_vappend(dm_args, "-serial", info->serial, NULL);
>      }
> -    if (info->type == LIBXL_DOMAIN_TYPE_FV) {
> +    if (info->type == LIBXL_DOMAIN_TYPE_HVM) {
>          int ioemu_vifs = 0;
>  
>          if (info->videoram) {
> @@ -211,7 +211,7 @@ static char ** libxl__build_device_model
>      case LIBXL_DOMAIN_TYPE_PV:
>          flexarray_append(dm_args, "xenpv");
>          break;
> -    case LIBXL_DOMAIN_TYPE_FV:
> +    case LIBXL_DOMAIN_TYPE_HVM:
>          flexarray_append(dm_args, "xenfv");
>          break;
>      }
> @@ -336,7 +336,7 @@ static char ** libxl__build_device_model
>      if (info->serial) {
>          flexarray_vappend(dm_args, "-serial", info->serial, NULL);
>      }
> -    if (info->type == LIBXL_DOMAIN_TYPE_FV) {
> +    if (info->type == LIBXL_DOMAIN_TYPE_HVM) {
>          int ioemu_vifs = 0;
>  
>          if (info->stdvga) {
> @@ -408,7 +408,7 @@ static char ** libxl__build_device_model
>      case LIBXL_DOMAIN_TYPE_PV:
>          flexarray_append(dm_args, "xenpv");
>          break;
> -    case LIBXL_DOMAIN_TYPE_FV:
> +    case LIBXL_DOMAIN_TYPE_HVM:
>          flexarray_append(dm_args, "xenfv");

QEMU uses the term "xenfv", this is a terminology mismatch between QEMU
and Xen...

Wei.

>          break;
>      }
> @@ -417,7 +417,7 @@ static char ** libxl__build_device_model
>      flexarray_append(dm_args, "-m");
>      flexarray_append(dm_args, libxl__sprintf(gc, "%d", info->target_ram));
>  
> -    if (info->type == LIBXL_DOMAIN_TYPE_FV) {
> +    if (info->type == LIBXL_DOMAIN_TYPE_HVM) {
>          for (i = 0; i < num_disks; i++) {
>              int disk, part;
>              int dev_number =
> diff -r b18728227bf9 -r 90e2ae994ebb tools/libxl/xl_cmdimpl.c
> --- a/tools/libxl/xl_cmdimpl.c        Mon Jul 18 14:52:29 2011 +0100
> +++ b/tools/libxl/xl_cmdimpl.c        Mon Jul 18 14:52:29 2011 +0100
> @@ -1083,7 +1083,7 @@ skip_vfb:
>      }
>  
>      dm_info->type = c_info->hvm ?
> -        LIBXL_DOMAIN_TYPE_FV :
> +        LIBXL_DOMAIN_TYPE_HVM :
>          LIBXL_DOMAIN_TYPE_PV;
>  
>      xlu_cfg_destroy(config);
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread] Current Thread [Next in Thread>