From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Vrabel Subject: Re: Design doc of adding ACPI support for arm64 on Xen - version 2 Date: Tue, 11 Aug 2015 16:02:31 +0100 Message-ID: <55CA0E87.7000405@citrix.com> References: <55C413D5.7000709@huawei.com> <1439302364.9747.259.camel@citrix.com> <55CA0C0C.70403@citrix.com> <1439305140.9747.278.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1439305140.9747.278.camel@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell , Shannon Zhao , xen-devel , Jan Beulich , Stefano Stabellini , Julien Grall , Parth Dixit , Christoffer Dall , Shannon Zhao Cc: Hangaohuai , "Huangpeng (Peter)" List-Id: xen-devel@lists.xenproject.org On 11/08/15 15:59, Ian Campbell wrote: > On Tue, 2015-08-11 at 15:51 +0100, David Vrabel wrote: >> On 11/08/15 15:12, Ian Campbell wrote: >>> On Fri, 2015-08-07 at 10:11 +0800, Shannon Zhao wrote: >>>> >>> [...] >>>> 3. Dom0 gets grant table and event channel irq information >>>> ----------------------------------------------------------- >>>> As said above, we assign the hypervisor_id be "XenVMM" to tell Dom0 >>>> that >>>> it runs on Xen hypervisor. >>>> >>>> For grant table, add two new HVM_PARAMs: >>>> HVM_PARAM_GNTTAB_START_ADDRESS >>>> and HVM_PARAM_GNTTAB_SIZE. >>> >>> The reason we expose this range is essentially to allow OS authors to >>> take >>> a short cut by telling them about an IPA range which is unused, so it >>> is >>> available for remapping the grant table into. On x86 there is a BAR on >>> the >>> Xen platform PCI device which serves a similar purpose. >>> >>> IIRC somebody (perhaps David V, CCd) had proposed at some point to make >>> it >>> so that Linux was able to pick such an IPA itself by examining the >>> memory >>> map or by some other scheme. >> >> PVH in Linux uses ballooned pages which are vmap()'d into a virtually >> contiguous region. >> >> See xlated_setup_gnttab_pages(). > > So somewhat more concrete than a proposal then ;-) > > I don't see anything there which would be a problem on ARM, so we should > probably go that route there too (at least for ACPI, if not globally for > all ARM guests). If someone does this please move xlated_setup_gnttab_pages() into drivers/xen/xlate_mmu.c, and not copy it into an arm specific file. David