Re: [PATCH 0/4] xen/arm: Unbreak ACPI

[Julien Grall <julien@xxxxxxx>]

On 28/09/2020 07:47, Masami Hiramatsu wrote:
> Hello,
Hi Masami,

> This made progress with my Xen boot on DeveloperBox (
> https://www.96boards.org/product/developerbox/ ) with ACPI.
I have reviewed the patch attached and I have a couple of remarks about it.

The STAO table was originally created to allow an hypervisor to hide 
devices from a controller domain (such as Dom0). If this table is not 
present, then it means the OS/hypervisor can use any device listed in 
the ACPI table.
Additionally, the STAO table should never be present in the host ACPI table.

Therefore, I think the code should not try to find the STAO. Instead, it 
should check whether the SPCR table is present.
Cheers,

--
Julien Grall