xl: absense of vcpus parameter

xl: absense of vcpus parameter

[Sergey Tovpeko]

[-- Attachment #1: Type: text/plain, Size: 277 bytes --]

Hello, list!

Using xl, you should obligatory specify 'vcpus' parameter to domain's 
configuration file.
Without this parameter HVM Windows7 goes to BSOD : 
HAL_INITIALIZATION_FAILED,
and WinXP reboots constantly.

I prepared patch, that initializes vcpus parameter inside xl.

[-- Attachment #2: vcpus.patch --]
[-- Type: text/x-patch, Size: 514 bytes --]

diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c
index cb9dda7..b5947c1 100644
--- a/tools/libxl/xl_cmdimpl.c
+++ b/tools/libxl/xl_cmdimpl.c
@@ -638,6 +638,9 @@ static void parse_config_data(const char *configfile_filename_report,
     if (!xlu_cfg_get_long (config, "vcpus", &l)) {
         b_info->max_vcpus = l;
         b_info->cur_vcpus = (1 << l) - 1;
+    } else {
+        b_info->max_vcpus = 1;
+        b_info->cur_vcpus = 1;
     }
 
     if (!xlu_cfg_get_long (config, "maxvcpus", &l))

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

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

Re: xl: absense of vcpus parameter

[Ian Jackson]

Sergey Tovpeko writes ("[Xen-devel] xl: absense of vcpus parameter"):
> Using xl, you should obligatory specify 'vcpus' parameter to domain's 
> configuration file.
> Without this parameter HVM Windows7 goes to BSOD : 
> HAL_INITIALIZATION_FAILED,
> and WinXP reboots constantly.
> 
> I prepared patch, that initializes vcpus parameter inside xl.

I think this should be done in libxl, not xl.

Ian.