[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [PATCH v9 03/13] tools: Add vmware_hwver support
This is used to set HVM_PARAM_VMWARE_HWVER. It is set to the emulated VMware virtual hardware version. Currently 0, 3-4, 6-11 are good values. However the code only checks for == 0, != 0, or < 7. If non-zero then default VGA to VMware's VGA. Also now allows vga=vmware Signed-off-by: Don Slutz <dslutz@xxxxxxxxxxx> Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> --- v9: I assumed that s/vmware_hw/vmware_hwver/ is not a big enough change to drop the Reviewed-by. Did a minor edit to the commit message to add 7 to the list of values checked. v7: Default handling of hvm.vga.kind bad. Fixed. Default of vmware_port should be based on vmware_hw. Done. v5: Anything looking for Xen according to the Xen cpuid instructions... Adjusted doc to new wording. docs/man/xl.cfg.pod.5 | 25 +++++++++++++++++++++++-- docs/misc/hypervisor-cpuid.markdown | 33 +++++++++++++++++++++++++++++++++ tools/libxc/xc_domain_restore.c | 15 +++++++++++++++ tools/libxc/xc_domain_save.c | 11 +++++++++++ tools/libxc/xg_save_restore.h | 2 ++ tools/libxl/libxl.h | 10 ++++++++++ tools/libxl/libxl_create.c | 12 +++++++++--- tools/libxl/libxl_dm.c | 8 ++++++++ tools/libxl/libxl_dom.c | 2 ++ tools/libxl/libxl_types.idl | 2 ++ tools/libxl/xl_cmdimpl.c | 4 ++++ 11 files changed, 119 insertions(+), 5 deletions(-) create mode 100644 docs/misc/hypervisor-cpuid.markdown diff --git a/docs/man/xl.cfg.pod.5 b/docs/man/xl.cfg.pod.5 index 408653f..905ca8f 100644 --- a/docs/man/xl.cfg.pod.5 +++ b/docs/man/xl.cfg.pod.5 @@ -1271,6 +1271,23 @@ The viridian option can be specified as a boolean. A value of true (1) is equivalent to the list [ "defaults" ], and a value of false (0) is equivalent to an empty list. +=item B<vmware_hwver=NUMBER> + +Turns on or off the exposure of VMware cpuid. The number is +VMware's hardware version number, where 0 is off. If not zero it +changes the default VGA to VMware's VGA. + +The hardware version number (vmware_hwver) come from VMware config files. + +=over 4 + +In a .vmx it is virtualHW.version + +In a .ovf it is part of the value of vssd:VirtualSystemType. +For vssd:VirtualSystemType == vmx-07, vmware_hwver = 7. + +=back + =back =head3 Emulated VGA Graphics Device @@ -1307,10 +1324,14 @@ later (e.g. Windows XP onwards) then you should enable this. stdvga supports more video ram and bigger resolutions than Cirrus. This option is deprecated, use vga="stdvga" instead. +The deprecated B<stdvga=0> prevents the usage of vmware by default +if B<vmware_hwver> is non-zero. + =item B<vga="STRING"> -Selects the emulated video card (none|stdvga|cirrus). -The default is cirrus. +Selects the emulated video card (none|stdvga|cirrus|vmware). +The default is cirrus unless B<vmware_hwver> is non-zero in which case it +is vmware. =item B<vnc=BOOLEAN> diff --git a/docs/misc/hypervisor-cpuid.markdown b/docs/misc/hypervisor-cpuid.markdown new file mode 100644 index 0000000..473f6a6 --- /dev/null +++ b/docs/misc/hypervisor-cpuid.markdown @@ -0,0 +1,33 @@ +Hypervisor Cpuid +================ + +There is no agreed standard for the use of hypervisor cpuid leaves. + +AMD (Vol3, Appendix E.3.9) reserves 0x40000000 to 0x400000ff for +hypervisor use, while Intel (Vol 2a, 3.2 CPUID) guarantees that no +existing or future CPUs will use the range 0x40000000 to 0x4fffffff. + +Different hypervisors use the space as follows: + +MicroSoft Hyper-V (AKA viridian) leaves currently must be at +0x40000000. + +VMware leaves currently must be at 0x40000000. + +KVM leaves currently must be at 0x40000000 (from Seabios). + +Xen leaves can be found at the first otherwise unused 0x100 aligned +offset between 0x40000000 and 0x40010000. + +http://download.microsoft.com/download/F/B/0/FB0D01A3-8E3A-4F5F-AA59-08C8026D3B8A/requirements-for-implementing-microsoft-hypervisor-interface.docx + +http://kb.vmware.com/selfservice/microsites/search.do?language=en_US&cmd=displayKC&externalId=1009458 + +http://lwn.net/Articles/301888/ + Attempted to get this cleaned up. + +So if viridian or vmware_hwver is selected, return their format for +the range 0x40000000 to 0x400000ff. And return Xen format for the +range 0x40000100 to 0x400001ff. + +Otherwise return Xen format for the range 0x40000000 to 0x400000ff. diff --git a/tools/libxc/xc_domain_restore.c b/tools/libxc/xc_domain_restore.c index a382701..386ba3c 100644 --- a/tools/libxc/xc_domain_restore.c +++ b/tools/libxc/xc_domain_restore.c @@ -743,6 +743,7 @@ typedef struct { uint64_t vm_generationid_addr; uint64_t ioreq_server_pfn; uint64_t nr_ioreq_server_pages; + uint64_t vmware_hwver; struct toolstack_data_t tdata; } pagebuf_t; @@ -927,6 +928,16 @@ static int pagebuf_get_one(xc_interface *xch, struct restore_ctx *ctx, } return pagebuf_get_one(xch, ctx, buf, fd, dom); + case XC_SAVE_ID_HVM_VMWARE_HWVER: + /* Skip padding 4 bytes then read the vmware hw version. */ + if ( RDEXACT(fd, &buf->vmware_hwver, sizeof(uint32_t)) || + RDEXACT(fd, &buf->vmware_hwver, sizeof(uint64_t)) ) + { + PERROR("error read the vmware_hwver value"); + return -1; + } + return pagebuf_get_one(xch, ctx, buf, fd, dom); + case XC_SAVE_ID_TOOLSTACK: { if ( RDEXACT(fd, &buf->tdata.len, sizeof(buf->tdata.len)) ) @@ -1774,6 +1785,10 @@ int xc_domain_restore(xc_interface *xch, int io_fd, uint32_t dom, } } + if (pagebuf.vmware_hwver != 0) + xc_set_hvm_param(xch, dom, HVM_PARAM_VMWARE_HWVER, + pagebuf.vmware_hwver); + if (pagebuf.acpi_ioport_location == 1) { DBGPRINTF("Use new firmware ioport from the checkpoint\n"); xc_hvm_param_set(xch, dom, HVM_PARAM_ACPI_IOPORTS_LOCATION, 1); diff --git a/tools/libxc/xc_domain_save.c b/tools/libxc/xc_domain_save.c index 254fdb3..40d413f 100644 --- a/tools/libxc/xc_domain_save.c +++ b/tools/libxc/xc_domain_save.c @@ -1750,6 +1750,17 @@ int xc_domain_save(xc_interface *xch, int io_fd, uint32_t dom, uint32_t max_iter PERROR("Error when writing the ioreq server gmfn count"); goto out; } + + chunk.id = XC_SAVE_ID_HVM_VMWARE_HWVER; + chunk.data = 0; + xc_hvm_param_get(xch, dom, HVM_PARAM_VMWARE_HWVER, &chunk.data); + + if ( (chunk.data != 0) && + wrexact(io_fd, &chunk, sizeof(chunk)) ) + { + PERROR("Error when writing the vmware_hwver value"); + goto out; + } } if ( callbacks != NULL && callbacks->toolstack_save != NULL ) diff --git a/tools/libxc/xg_save_restore.h b/tools/libxc/xg_save_restore.h index bdd9009..d73d152 100644 --- a/tools/libxc/xg_save_restore.h +++ b/tools/libxc/xg_save_restore.h @@ -262,6 +262,8 @@ /* These are a pair; it is an error for one to exist without the other */ #define XC_SAVE_ID_HVM_IOREQ_SERVER_PFN -19 #define XC_SAVE_ID_HVM_NR_IOREQ_SERVER_PAGES -20 +/* VMware data */ +#define XC_SAVE_ID_HVM_VMWARE_HWVER -21 /* ** We process save/restore/migrate in batches of pages; the below diff --git a/tools/libxl/libxl.h b/tools/libxl/libxl.h index c219f59..0c27e5c 100644 --- a/tools/libxl/libxl.h +++ b/tools/libxl/libxl.h @@ -163,6 +163,16 @@ #define LIBXL_HAVE_BUILDINFO_HVM_MMIO_HOLE_MEMKB 1 /* + * The libxl_vga_interface_type has the type for vmware. + */ +#define LIBXL_HAVE_LIBXL_VGA_INTERFACE_TYPE_VMWARE 1 + +/* + * libxl_domain_build_info has the u.hvm.vmware_hwver field. + */ +#define LIBXL_HAVE_BUILDINFO_HVM_VMWARE_HWVER 1 + +/* * libxl ABI compatibility * * The only guarantee which libxl makes regarding ABI compatibility diff --git a/tools/libxl/libxl_create.c b/tools/libxl/libxl_create.c index 98687bd..8c910c4 100644 --- a/tools/libxl/libxl_create.c +++ b/tools/libxl/libxl_create.c @@ -238,8 +238,12 @@ int libxl__domain_build_info_setdefault(libxl__gc *gc, if (b_info->u.hvm.mmio_hole_memkb == LIBXL_MEMKB_DEFAULT) b_info->u.hvm.mmio_hole_memkb = 0; - if (!b_info->u.hvm.vga.kind) - b_info->u.hvm.vga.kind = LIBXL_VGA_INTERFACE_TYPE_CIRRUS; + if (!b_info->u.hvm.vga.kind) { + if (b_info->u.hvm.vmware_hwver) + b_info->u.hvm.vga.kind = LIBXL_VGA_INTERFACE_TYPE_VMWARE; + else + b_info->u.hvm.vga.kind = LIBXL_VGA_INTERFACE_TYPE_CIRRUS; + } switch (b_info->device_model_version) { case LIBXL_DEVICE_MODEL_VERSION_QEMU_XEN_TRADITIONAL: @@ -442,7 +446,7 @@ int libxl__domain_build(libxl__gc *gc, vments[4] = "start_time"; vments[5] = libxl__sprintf(gc, "%lu.%02d", start_time.tv_sec,(int)start_time.tv_usec/10000); - localents = libxl__calloc(gc, 9, sizeof(char *)); + localents = libxl__calloc(gc, 11, sizeof(char *)); i = 0; localents[i++] = "platform/acpi"; localents[i++] = libxl_defbool_val(info->u.hvm.acpi) ? "1" : "0"; @@ -450,6 +454,8 @@ int libxl__domain_build(libxl__gc *gc, localents[i++] = libxl_defbool_val(info->u.hvm.acpi_s3) ? "1" : "0"; localents[i++] = "platform/acpi_s4"; localents[i++] = libxl_defbool_val(info->u.hvm.acpi_s4) ? "1" : "0"; + localents[i++] = "platform/vmware_hwver"; + localents[i++] = libxl__sprintf(gc, "%"PRId64, info->u.hvm.vmware_hwver); if (info->u.hvm.mmio_hole_memkb) { uint64_t max_ram_below_4g = (1ULL << 32) - (info->u.hvm.mmio_hole_memkb << 10); diff --git a/tools/libxl/libxl_dm.c b/tools/libxl/libxl_dm.c index 8599a6a..b05fa73 100644 --- a/tools/libxl/libxl_dm.c +++ b/tools/libxl/libxl_dm.c @@ -244,6 +244,9 @@ static char ** libxl__build_device_model_args_old(libxl__gc *gc, case LIBXL_VGA_INTERFACE_TYPE_NONE: flexarray_append_pair(dm_args, "-vga", "none"); break; + case LIBXL_VGA_INTERFACE_TYPE_VMWARE: + flexarray_append_pair(dm_args, "-vga", "vmware"); + break; } if (b_info->u.hvm.boot) { @@ -599,6 +602,11 @@ static char ** libxl__build_device_model_args_new(libxl__gc *gc, break; case LIBXL_VGA_INTERFACE_TYPE_NONE: break; + case LIBXL_VGA_INTERFACE_TYPE_VMWARE: + flexarray_append_pair(dm_args, "-device", + GCSPRINTF("vmware-svga,vgamem_mb=%d", + libxl__sizekb_to_mb(b_info->video_memkb))); + break; } if (b_info->u.hvm.boot) { diff --git a/tools/libxl/libxl_dom.c b/tools/libxl/libxl_dom.c index 48d661a..34a36d3 100644 --- a/tools/libxl/libxl_dom.c +++ b/tools/libxl/libxl_dom.c @@ -294,6 +294,8 @@ static void hvm_set_conf_params(xc_interface *handle, uint32_t domid, #if defined(__i386__) || defined(__x86_64__) xc_hvm_param_set(handle, domid, HVM_PARAM_HPET_ENABLED, libxl_defbool_val(info->u.hvm.hpet)); + xc_set_hvm_param(handle, domid, HVM_PARAM_VMWARE_HWVER, + info->u.hvm.vmware_hwver); #endif xc_hvm_param_set(handle, domid, HVM_PARAM_TIMER_MODE, timer_mode(info)); xc_hvm_param_set(handle, domid, HVM_PARAM_VPT_ALIGN, diff --git a/tools/libxl/libxl_types.idl b/tools/libxl/libxl_types.idl index 02be466..ad596de 100644 --- a/tools/libxl/libxl_types.idl +++ b/tools/libxl/libxl_types.idl @@ -181,6 +181,7 @@ libxl_vga_interface_type = Enumeration("vga_interface_type", [ (1, "CIRRUS"), (2, "STD"), (3, "NONE"), + (4, "VMWARE"), ], init_val = "LIBXL_VGA_INTERFACE_TYPE_CIRRUS") libxl_vendor_device = Enumeration("vendor_device", [ @@ -417,6 +418,7 @@ libxl_domain_build_info = Struct("domain_build_info",[ ("hpet", libxl_defbool), ("vpt_align", libxl_defbool), ("mmio_hole_memkb", MemKB), + ("vmware_hwver", uint64), ("timer_mode", libxl_timer_mode), ("nested_hvm", libxl_defbool), ("smbios_firmware", string), diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c index 440db78..ca6d0ec 100644 --- a/tools/libxl/xl_cmdimpl.c +++ b/tools/libxl/xl_cmdimpl.c @@ -1280,6 +1280,8 @@ static void parse_config_data(const char *config_source, exit (1); } } + if (!xlu_cfg_get_long(config, "vmware_hwver", &l, 1)) + b_info->u.hvm.vmware_hwver = l; if (!xlu_cfg_get_long(config, "timer_mode", &l, 1)) { const char *s = libxl_timer_mode_to_string(l); fprintf(stderr, "WARNING: specifying \"timer_mode\" as an integer is deprecated. " @@ -1920,6 +1922,8 @@ skip_vfb: b_info->u.hvm.vga.kind = LIBXL_VGA_INTERFACE_TYPE_CIRRUS; } else if (!strcmp(buf, "none")) { b_info->u.hvm.vga.kind = LIBXL_VGA_INTERFACE_TYPE_NONE; + } else if (!strcmp(buf, "vmware")) { + b_info->u.hvm.vga.kind = LIBXL_VGA_INTERFACE_TYPE_VMWARE; } else { fprintf(stderr, "Unknown vga \"%s\" specified\n", buf); exit(1); -- 1.8.4 _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |