Re: [Xen-devel] Re: [PATCH] ioemu-remote: ACPI S3 state wake up

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

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