Trolle Selander writes ("Re: [Xen-devel] Re: [PATCH] ioemu-remote: ACPI S3
state wake up"):
> Don't leave the main rombios unprotected. I also added a comment to explain
> the "magic numbers", since their meaning is perhaps not as obvious as I
> thought, in particular now that we're no longer blanket-protecting the
> entire firmware area anymore.
Thanks ...
> - if (xc_hvm_set_mem_type(xc_handle, domid, mem_type, 0xc0, 0x40))
> + /* [These numbers] are pfns and page counts [for] the memory ranges:
> + 0xc0000 - 0xdffff - VGABIOS and option ROMS (extboot, ethboot, etc)
> + 0xf0000 - 0xfffff - the main ROMBIOS */
> + if (xc_hvm_set_mem_type(xc_handle, domid, mem_type, 0xc0, 0x20)
> + || xc_hvm_set_mem_type(xc_handle, domid, mem_type, 0xf0, 0x10))
> fprintf(logfile,"xen_platform: unable to change ro/rw "
> "state of ROM memory area!\n");
> else
... so 0xe0000..0xeffff are the ACPI tables that Jean was wanting the
guest to be able to modify. So I should say
+ 0xe0000 - 0xeffff - ACPI tables (guest writes on S3 wakeup)
or some such ?
Forgive my ignorance ...
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|