[prev in list] [next in list] [prev in thread] [next in thread]
List: xen-devel
Subject: RE: [Xen-devel] [PATCH][RESEND] add "acpi" option
From: "Yu, Ke" <ke.yu () intel ! com>
Date: 2005-12-31 15:26:53
Message-ID: 8126E4F969BA254AB43EA03C59F44E8404583957 () pdsmsx404
[Download RAW message or body]
Forget to reset sum in func checksum_valid(). Update the patch again. sorry for the inconvenient.
> -----Original Message-----
> From: xen-devel-bounces@lists.xensource.com
> [mailto:xen-devel-bounces@lists.xensource.com] On Behalf Of Yu, Ke
> Sent: 2005年12月31日 22:26 To: Keir Fraser
> Cc: xen-devel@lists.xensource.com
> Subject: RE: [Xen-devel] [PATCH][RESEND] add "acpi" option
> inxmexample.vmxconfig file
>
> Hi Keir,
>
> This is the updated patch according your suggestion. Would you please
> review that? Thanks
>
> Best Regards
> Ke
["acpi_option_4.patch" (application/octet-stream)]
# HG changeset patch
# User Ke Yu <ke.yu@intel.com>
# Node ID e0e2a542a009a2ccc5bc313fe03b0431feab3d84
# Parent b54e981957ebd69c9b413df668d3a49908d13f23
This patch adds option "acpi" in xmexmaple.vmx. Vmxloader uses this option to decide loading ACPI or not.
Signed-off-by: Yu Ke <ke.yu@intel.com>
diff -r b54e981957eb -r e0e2a542a009 tools/examples/xmexample.vmx
--- a/tools/examples/xmexample.vmx Thu Dec 29 16:59:12 2005 +0100
+++ b/tools/examples/xmexample.vmx Sat Dec 31 22:14:04 2005 +0800
@@ -29,6 +29,9 @@
#-----------------------------------------------------------------------------
# the number of cpus guest platform has, default=1
vcpus=1
+
+# enable/disalbe vmx guest ACPI, default=0 (disabled)
+#acpi=0
# List of which CPUS this domain is allowed to use, default Xen picks
#cpus = "" # leave to Xen to pick
diff -r b54e981957eb -r e0e2a542a009 tools/firmware/vmxassist/Makefile
--- a/tools/firmware/vmxassist/Makefile Thu Dec 29 16:59:12 2005 +0100
+++ b/tools/firmware/vmxassist/Makefile Sat Dec 31 22:14:04 2005 +0800
@@ -24,7 +24,7 @@
# The emulator code lives in ROM space
TEXTADDR=0x000D0000
-DEFINES=-DDEBUG -D_ACPI_ -DTEXTADDR=$(TEXTADDR)
+DEFINES=-DDEBUG -DTEXTADDR=$(TEXTADDR)
XENINC=-I$(XEN_ROOT)/tools/libxc
LD = ld
diff -r b54e981957eb -r e0e2a542a009 tools/firmware/vmxassist/acpi_madt.c
--- a/tools/firmware/vmxassist/acpi_madt.c Thu Dec 29 16:59:12 2005 +0100
+++ b/tools/firmware/vmxassist/acpi_madt.c Sat Dec 31 22:14:04 2005 +0800
@@ -24,23 +24,78 @@
extern int puts(const char *s);
-#define VCPU_NR_PAGE 0x0009F000
-#define VCPU_NR_OFFSET 0x00000800
-#define VCPU_MAGIC 0x76637075 /* "vcpu" */
-
-/* xc_vmx_builder wrote vcpu block at 0x9F800. Return it. */
+#define HVM_INFO_PAGE 0x0009F000
+#define HVM_INFO_OFFSET 0x00000800
+#define HVM_INFO_SIGNATURE 0x48564d20494e464fULL /* "HVM INFO" */
+
+struct hvm_info_table {
+ uint64_t signature; /* "HVM INFO" */
+ uint32_t length;
+ uint8_t checksum;
+ uint8_t acpi_enabled;
+ uint8_t pad[2];
+ uint32_t nr_vcpus;
+};
+
+static struct hvm_info_table* table = NULL;
+
static int
+checksum_valid(uint8_t* ptr, int len)
+{
+ uint8_t sum=0;
+ int i;
+
+ for (i=0; i<len; i++){
+ sum += ptr[i];
+ }
+
+ return sum==0;
+}
+
+/* xc_vmx_builder wrote hvm info at 0x9F800. Return it. */
+static struct hvm_info_table*
+get_hvm_info_table(void)
+{
+ struct hvm_info_table* t;
+
+ if (table != NULL)
+ return table;
+
+ t = (struct hvm_info_table*) (HVM_INFO_PAGE + HVM_INFO_OFFSET);
+
+ if (t->signature != HVM_INFO_SIGNATURE) {
+ puts("Bad hvm info signature¥n");
+ return NULL;
+ }
+ if ( !checksum_valid((uint8_t*)t, t->length) ) {
+ puts("Bad hvm info checksum¥n");
+ return NULL;
+ }
+
+ table = t;
+ return table;
+}
+
+int
get_vcpu_nr(void)
{
- unsigned int *vcpus;
-
- vcpus = (unsigned int *)(VCPU_NR_PAGE + VCPU_NR_OFFSET);
- if (vcpus[0] != VCPU_MAGIC) {
- puts("Bad vcpus magic, set vcpu number to 1 by default.¥n");
- return 1;
- }
-
- return vcpus[1];
+ struct hvm_info_table* t;
+ t = get_hvm_info_table();
+ if (t==NULL)
+ return 1; /* bad hvm info, so set 1 vcpu by dafault */
+ else
+ return t->nr_vcpus;
+}
+
+int
+get_acpi_enabled(void)
+{
+ struct hvm_info_table* t;
+ t = get_hvm_info_table();
+ if (t==NULL)
+ return 0; /* bad hvm info, so disable acpi by default */
+ else
+ return t->acpi_enabled;
}
static void *
diff -r b54e981957eb -r e0e2a542a009 tools/firmware/vmxassist/vmxloader.c
--- a/tools/firmware/vmxassist/vmxloader.c Thu Dec 29 16:59:12 2005 +0100
+++ b/tools/firmware/vmxassist/vmxloader.c Sat Dec 31 22:14:04 2005 +0800
@@ -24,12 +24,10 @@
#include "machine.h"
#include "roms.h"
-#ifdef _ACPI_
#include "acpi.h"
#include "../acpi/acpi2_0.h" // for ACPI_PHYSICAL_ADDRESS
int acpi_madt_update(unsigned char* acpi_start);
-#endif
-
+int get_acpi_enabled(void);
/*
* C runtime start off
@@ -120,18 +118,17 @@
memcpy((void *)0xC0000,
vgabios_stdvga, sizeof(vgabios_stdvga));
}
-#ifdef _ACPI_
- puts("Loading ACPI ...¥n");
- acpi_madt_update(acpi);
-
- if (ACPI_PHYSICAL_ADDRESS+sizeof(acpi) <= 0xF0000) {
- /* make sure acpi table does not overlap rombios
- * currently acpi less than 8K will be OK.
- */
- memcpy((void *)ACPI_PHYSICAL_ADDRESS, acpi, sizeof(acpi));
- }
-#endif
+ if (get_acpi_enabled() != 0) {
+ puts("Loading ACPI ...¥n");
+ acpi_madt_update((unsigned char*)acpi);
+ if (ACPI_PHYSICAL_ADDRESS+sizeof(acpi) <= 0xF0000) {
+ /* make sure acpi table does not overlap rombios
+ * currently acpi less than 8K will be OK.
+ */
+ memcpy((void *)ACPI_PHYSICAL_ADDRESS, acpi, sizeof(acpi));
+ }
+ }
puts("Loading VMXAssist ...¥n");
memcpy((void *)TEXTADDR, vmxassist, sizeof(vmxassist));
diff -r b54e981957eb -r e0e2a542a009 tools/libxc/xc_vmx_build.c
--- a/tools/libxc/xc_vmx_build.c Thu Dec 29 16:59:12 2005 +0100
+++ b/tools/libxc/xc_vmx_build.c Sat Dec 31 22:14:04 2005 +0800
@@ -33,8 +33,18 @@
#define E820_MAP_NR_OFFSET 0x000001E8
#define E820_MAP_OFFSET 0x000002D0
-#define VCPU_NR_PAGE 0x0009F000
-#define VCPU_NR_OFFSET 0x00000800
+#define HVM_INFO_PAGE 0x0009F000
+#define HVM_INFO_OFFSET 0x00000800
+#define HVM_INFO_SIGNATURE 0x48564d20494e464fULL /* "HVM INFO" */
+
+struct hvm_info_table {
+ uint64_t signature; /* "HVM INFO" */
+ uint32_t length;
+ uint8_t checksum;
+ uint8_t acpi_enabled;
+ uint8_t pad[2];
+ uint32_t nr_vcpus;
+};
struct e820entry {
uint64_t addr;
@@ -119,26 +129,47 @@
return (*(((unsigned char *)e820_page) + E820_MAP_NR_OFFSET) = nr_map);
}
+static void
+set_hvm_info_checksum(struct hvm_info_table* t)
+{
+ unsigned char sum = 0;
+ unsigned char *ptr;
+
+ ptr = (unsigned char*)t;
+ t->checksum = 0;
+ len = t->length;
+ while (len--)
+ sum += *ptr++;
+
+ t->checksum = -sum;
+}
+
/*
- * Use E820 reserved memory 0x9F800 to pass number of vcpus to vmxloader
- * vmxloader will use it to config ACPI MADT table
+ * Use E820 reserved memory 0x9F800 to pass HVM info to vmxloader
+ * vmxloader will use this info to set BIOS accordingly
*/
-#define VCPU_MAGIC 0x76637075 /* "vcpu" */
-static int set_vcpu_nr(int xc_handle, uint32_t dom,
- unsigned long *pfn_list, unsigned int vcpus)
-{
- char *va_map;
- unsigned int *va_vcpus;
+static int set_hvm_info(int xc_handle, uint32_t dom,
+ unsigned long *pfn_list, unsigned int vcpus,
+ unsigned int acpi)
+{
+ char *va_map;
+ struct hvm_info_table *va_hvm;
va_map = xc_map_foreign_range(xc_handle, dom,
PAGE_SIZE, PROT_READ|PROT_WRITE,
- pfn_list[VCPU_NR_PAGE >> PAGE_SHIFT]);
+ pfn_list[HVM_INFO_PAGE >> PAGE_SHIFT]);
if ( va_map == NULL )
return -1;
- va_vcpus = (unsigned int *)(va_map + VCPU_NR_OFFSET);
- va_vcpus[0] = VCPU_MAGIC;
- va_vcpus[1] = vcpus;
+ va_hvm = (struct hvm_info_table *)(va_map + HVM_INFO_OFFSET);
+ va_hvm->signature = HVM_INFO_SIGNATURE;
+ va_hvm->length = sizeof(struct hvm_info_table);
+ va_hvm->acpi_enabled = acpi;
+ va_hvm->pad[0] = 0;
+ va_hvm->pad[1] = 0;
+ va_hvm->nr_vcpus = vcpus;
+
+ set_hvm_info_checksum(va_hvm);
munmap(va_map, PAGE_SIZE);
@@ -281,6 +312,7 @@
unsigned int control_evtchn,
unsigned int lapic,
unsigned int vcpus,
+ unsigned int acpi,
unsigned int store_evtchn,
unsigned long *store_mfn)
{
@@ -490,8 +522,8 @@
goto error_out;
}
- if (set_vcpu_nr(xc_handle, dom, page_array, vcpus)) {
- fprintf(stderr, "Couldn't set vcpu number for VMX guest.¥n");
+ if (set_hvm_info(xc_handle, dom, page_array, vcpus, acpi)) {
+ fprintf(stderr, "Couldn't set hvm info for VMX guest.¥n");
goto error_out;
}
@@ -604,6 +636,7 @@
unsigned int control_evtchn,
unsigned int lapic,
unsigned int vcpus,
+ unsigned int acpi,
unsigned int store_evtchn,
unsigned long *store_mfn)
{
@@ -659,7 +692,7 @@
if ( setup_guest(xc_handle, domid, memsize, image, image_size, nr_pages,
ctxt, op.u.getdomaininfo.shared_info_frame, control_evtchn,
- lapic, vcpus, store_evtchn, store_mfn) < 0)
+ lapic, vcpus, acpi, store_evtchn, store_mfn) < 0)
{
ERROR("Error constructing guest OS");
goto error_out;
diff -r b54e981957eb -r e0e2a542a009 tools/libxc/xenguest.h
--- a/tools/libxc/xenguest.h Thu Dec 29 16:59:12 2005 +0100
+++ b/tools/libxc/xenguest.h Sat Dec 31 22:14:04 2005 +0800
@@ -58,6 +58,7 @@
unsigned int control_evtchn,
unsigned int lapic,
unsigned int vcpus,
+ unsigned int acpi,
unsigned int store_evtchn,
unsigned long *store_mfn);
diff -r b54e981957eb -r e0e2a542a009 tools/python/xen/lowlevel/xc/xc.c
--- a/tools/python/xen/lowlevel/xc/xc.c Thu Dec 29 16:59:12 2005 +0100
+++ b/tools/python/xen/lowlevel/xc/xc.c Sat Dec 31 22:14:04 2005 +0800
@@ -364,19 +364,20 @@
int control_evtchn, store_evtchn;
int vcpus = 1;
int lapic = 0;
+ int acpi = 0;
int memsize;
unsigned long store_mfn = 0;
static char *kwd_list[] = { "dom", "control_evtchn", "store_evtchn",
- "memsize", "image", "lapic", "vcpus", NULL };
-
- if ( !PyArg_ParseTupleAndKeywords(args, kwds, "iiiisii", kwd_list,
+ "memsize", "image", "lapic", "vcpus", "acpi",NULL };
+
+ if ( !PyArg_ParseTupleAndKeywords(args, kwds, "iiiisiii", kwd_list,
&dom, &control_evtchn, &store_evtchn,
- &memsize, &image, &lapic, &vcpus) )
+ &memsize, &image, &lapic, &vcpus,&acpi) )
return NULL;
if ( xc_vmx_build(self->xc_handle, dom, memsize, image, control_evtchn,
- lapic, vcpus, store_evtchn, &store_mfn) != 0 )
+ lapic, vcpus, acpi, store_evtchn, &store_mfn) != 0 )
return PyErr_SetFromErrno(xc_error);
return Py_BuildValue("{s:i}", "store_mfn", store_mfn);
diff -r b54e981957eb -r e0e2a542a009 tools/python/xen/xend/image.py
--- a/tools/python/xen/xend/image.py Thu Dec 29 16:59:12 2005 +0100
+++ b/tools/python/xen/xend/image.py Sat Dec 31 22:14:04 2005 +0800
@@ -210,6 +210,8 @@
if not lapic is None:
self.lapic = int(lapic)
+ self.acpi = int(sxp.child_value(imageConfig, 'acpi', 0))
+
def buildDomain(self):
# Create an event channel
self.device_channel = xc.evtchn_alloc_unbound(dom=self.vm.getDomid(),
@@ -225,6 +227,7 @@
log.debug("memsize = %d", self.vm.getMemoryTarget() / 1024)
log.debug("lapic = %d", self.lapic)
log.debug("vcpus = %d", self.vm.getVCpuCount())
+ log.debug("acpi = %d", self.acpi)
return xc.vmx_build(dom = self.vm.getDomid(),
image = self.kernel,
@@ -232,8 +235,8 @@
store_evtchn = store_evtchn,
memsize = self.vm.getMemoryTarget() / 1024,
lapic = self.lapic,
+ acpi = self.acpi,
vcpus = self.vm.getVCpuCount())
-
# Return a list of cmd line args to the device models based on the
# xm config file
diff -r b54e981957eb -r e0e2a542a009 tools/python/xen/xm/create.py
--- a/tools/python/xen/xm/create.py Thu Dec 29 16:59:12 2005 +0100
+++ b/tools/python/xen/xm/create.py Sat Dec 31 22:14:04 2005 +0800
@@ -163,6 +163,10 @@
gopts.var('lapic', val='LAPIC',
fn=set_int, default=0,
use="Disable or enable local APIC of VMX domain.")
+
+gopts.var('acpi', val='ACPI',
+ fn=set_int, default=0,
+ use="Disable or enable ACPI of VMX domain.")
gopts.var('vcpus', val='VCPUS',
fn=set_int, default=1,
@@ -531,7 +535,7 @@
args = [ 'device_model', 'vcpus', 'cdrom', 'boot', 'fda', 'fdb',
'localtime', 'serial', 'stdvga', 'isa', 'nographic', 'audio',
'vnc', 'vncviewer', 'sdl', 'display', 'ne2000', 'lapic',
- 'xauthority' ]
+ 'xauthority', 'acpi' ]
for a in args:
if (vals.__dict__[a]):
config_image.append([a, vals.__dict__[a]])
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic