Yu, Ke
2005-Dec-31 14:26 UTC
RE: [Xen-devel] [PATCH][RESEND] add "acpi" option in xmexample.vmxconfig file
Hi Keir, This is the updated patch according your suggestion. Would you please review that? Thanks Best Regards Ke>-----Original Message----- >From: Keir Fraser [mailto:Keir.Fraser@cl.cam.ac.uk] >Sent: 2005年12月30日 18:33 >To: Yu, Ke >Cc: xen-devel@lists.xensource.com >Subject: Re: [Xen-devel] [PATCH][RESEND] add "acpi" option in >xmexample.vmxconfig file > > >On 30 Dec 2005, at 01:52, Yu, Ke wrote: > >> Rebase the patch due to its conflict with new changeset 8445. >> Would somebody apply it or give some comments? Thanks > >The info block written by vmx builder is no longer merely vcpu >information. Hence its name and the magic string at the start of the >block are badly named. > >I would like to see a more ACPI-styled table. At the moment, can you >please define a struct something like the following (where the >definition is available both to vmx builder and vmxassist -- I guess >the struct may need to be copied in two different header files): > >struct hvm_info_table { > char signature[8]; /* "HVM INFO" */ > u32 length; > u8 checksum; > u8 acpi_enabled; > u8 pad[2]; > u32 nr_vcpus; >}; > >Where the checksum is a simple ACPI-style twos-complement byte >summation and test for zero. > >This has the advantages of a meaningful name and gets rid of magic >numbers (indexes into array of unsigned ints). > > -- Keir >_______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel