Subject: Re: [PATCH v10 00/17] Add ACPI support for Xen Dom0 on ARM64
To: Shannon Zhao <shannon.zhao@linaro.org>,
        linux-arm-kernel@lists.infradead.org, stefano.stabellini@citrix.com
References: <1459525755-36968-1-git-send-email-shannon.zhao@linaro.org>
Cc: david.vrabel@citrix.com, devicetree@vger.kernel.org,
        linux-efi@vger.kernel.org, linux-kernel@vger.kernel.org,
        catalin.marinas@arm.com, will.deacon@arm.com,
        peter.huangpeng@huawei.com, xen-devel@lists.xen.org,
        zhaoshenglong@huawei.com
From: Julien Grall <julien.grall@arm.com>
Message-ID: <5704F3E1.7020508@arm.com>
Date: Wed, 6 Apr 2016 12:32:49 +0100
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:38.0)
 Gecko/20100101 Thunderbird/38.6.0
MIME-Version: 1.0
In-Reply-To: <1459525755-36968-1-git-send-email-shannon.zhao@linaro.org>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1555
Lines: 38

Hi Shannon,

On 01/04/2016 16:48, Shannon Zhao wrote:
> This patch set adds ACPI support for Xen Dom0 on ARM64. The relevant Xen
> ACPI on ARM64 design document could be found from [1].
>
> This patch set adds a new FDT node "uefi" under /hypervisor to pass UEFI
> information. Introduce a bus notifier of AMBA and Platform bus to map
> the new added device's MMIO space. Make Xen domain use
> xlated_setup_gnttab_pages to setup grant table and a new hypercall to
> get event-channel irq.
>
> Regarding the initialization flow of Linux kernel, it needs to move
> xen_early_init() before efi_init(). Then xen_early_init() will check
> whether it runs on Xen through the /hypervisor node and efi_init() will
> call a new function fdt_find_xen_uefi_params(), to parse those
> xen,uefi-* parameters just like the existing efi_get_fdt_params().
>
> And in arm64_enable_runtime_services() it will check whether it runs on
> Xen and call another new function xen_efi_runtime_setup() to setup
> runtime service instead of efi_native_runtime_setup(). The
> xen_efi_runtime_setup() will assign the runtime function pointers with
> the functions of driver/xen/efi.c.
>
> And since we pass a /hypervisor node and a /chosen node to Dom0, it
> needs to check whether the DTS only contains a /hypervisor node and a
> /chosen node in acpi_boot_table_init().
>
> Patches are tested on FVP base model. They can be fetched from[2].

I have tested this series and Linux is booting up to the prompt:

Tested-by: Julien Grall <julien.grall@arm.com>

Regards,

-- 
Julien Grall