xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Ian Campbell <ian.campbell@citrix.com>
To: Julien Grall <julien.grall@citrix.com>,
	David Vrabel <david.vrabel@citrix.com>,
	Shannon Zhao <zhaoshenglong@huawei.com>,
	xen-devel <xen-devel@lists.xen.org>,
	Jan Beulich <jbeulich@suse.com>,
	Stefano Stabellini <stefano.stabellini@citrix.com>,
	Parth Dixit <parth.dixit@linaro.org>,
	Christoffer Dall <christoffer.dall@linaro.org>,
	Shannon Zhao <shannon.zhao@linaro.org>
Cc: Hangaohuai <hangaohuai@huawei.com>,
	"Huangpeng (Peter)" <peter.huangpeng@huawei.com>
Subject: Re: Design doc of adding ACPI support for arm64 on Xen - version 2
Date: Tue, 11 Aug 2015 16:19:19 +0100	[thread overview]
Message-ID: <1439306359.9747.280.camel@citrix.com> (raw)
In-Reply-To: <55CA1085.6050506@citrix.com>

On Tue, 2015-08-11 at 16:11 +0100, Julien Grall wrote:
> 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).
> 
> IIRC we talked about it few months ago and you said that using balloon
> page will split in 4K the 1G/2M mapping done in the stage-2 p2m.

Did I? Odd because I'm also of the opinion that alloc_ballooned_pages
should operate in chunks of 2M at the hypercall layer and keep any
resulting spare 4K pages on a free list to use for future such allocations.

IOW it should avoid such shattering where it can.

> Note that for DOM0 we never give back the ballooned page to Xen. This is
> because we have to respect the 1:1 mapping when DOM0 want to get back
> the page.
> 
> Regards,
> 

  reply	other threads:[~2015-08-11 15:19 UTC|newest]

Thread overview: 82+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-07  2:11 Design doc of adding ACPI support for arm64 on Xen - version 2 Shannon Zhao
2015-08-07  9:45 ` Stefano Stabellini
2015-08-07 10:33 ` Julien Grall
2015-08-07 10:37   ` Christoffer Dall
2015-08-07 10:42     ` Stefano Stabellini
2015-08-07 10:44     ` Julien Grall
2015-08-11  2:09   ` Shannon Zhao
2015-08-11  9:46     ` Julien Grall
2015-08-11 10:27       ` Shannon Zhao
2015-08-11 14:12 ` Ian Campbell
2015-08-11 14:51   ` David Vrabel
2015-08-11 14:59     ` Ian Campbell
2015-08-11 15:02       ` David Vrabel
2015-08-11 15:11       ` Julien Grall
2015-08-11 15:19         ` Ian Campbell [this message]
2015-08-11 15:25           ` David Vrabel
2015-08-11 16:01           ` Julien Grall
2015-08-12  2:42             ` Shannon Zhao
2015-08-12  8:46               ` Ian Campbell
2015-08-12  8:46             ` Ian Campbell
2015-08-12  9:02               ` Julien Grall
2015-08-12 15:48   ` Jan Beulich
2015-08-11 14:19 ` Ian Campbell
2015-08-11 14:21   ` Ian Campbell
2015-08-11 15:29     ` Boris Ostrovsky
2015-08-11 15:35       ` Ian Campbell
2015-08-11 15:52         ` Boris Ostrovsky
2015-08-12  2:47           ` Shannon Zhao
2015-08-12  8:47             ` Ian Campbell
2015-08-12  9:00               ` Shannon Zhao
2015-08-17 10:36   ` Roger Pau Monné
2015-08-18  1:44     ` Shannon Zhao
2015-08-11 16:19 ` Julien Grall
2015-08-12  3:04   ` Shannon Zhao
2015-08-12  8:52     ` Ian Campbell
2015-08-12  9:21       ` Julien Grall
2015-08-12 10:36         ` Andrew Turner
2015-08-12 10:48           ` Stefano Stabellini
2015-08-12 11:23             ` Ian Campbell
2015-08-12 12:11               ` Julien Grall
2015-09-02 11:27                 ` Ian Campbell
2015-08-12 12:20           ` Julien Grall
2015-09-02 11:30             ` Ian Campbell
2015-09-02 11:39               ` Julien Grall
2015-09-02 12:05                 ` Ian Campbell
2015-08-12 10:17       ` Stefano Stabellini
2015-08-12  7:22 ` Shannon Zhao
2015-08-12  9:11   ` Julien Grall
2015-08-14 14:05     ` Shannon Zhao
2015-08-14 14:17       ` xen/arm: Crash when allocating memory for ACPI table (Was Re: Design doc of adding ACPI support for arm64 on Xen - version 2) Julien Grall
2015-08-14 14:35         ` Shannon Zhao
2015-08-14 14:41           ` Julien Grall
2015-08-14 14:49             ` Shannon Zhao
2015-08-14 14:53               ` Julien Grall
2015-08-14 14:55                 ` Shannon Zhao
2015-08-12 15:45 ` Design doc of adding ACPI support for arm64 on Xen - version 2 Jan Beulich
2015-08-12 15:51   ` Christoffer Dall
2015-08-12 15:58     ` Jan Beulich
2015-08-12 16:18       ` Julien Grall
2015-08-13  6:41         ` Jan Beulich
2015-08-13  8:01           ` Christoffer Dall
2015-08-13  8:11             ` Jan Beulich
2015-08-13  8:18             ` Jan Beulich
2015-08-13  9:05           ` Ian Campbell
2015-08-13  9:20             ` Jan Beulich
2015-08-13 10:40               ` Julien Grall
2015-08-13  9:43           ` Stefano Stabellini
2015-08-13 10:03             ` Jan Beulich
2015-08-13 10:13               ` Stefano Stabellini
2015-08-13 10:22                 ` Ian Campbell
2015-08-13 10:29                   ` Christoffer Dall
2015-08-13 10:32                     ` Stefano Stabellini
2015-08-13 10:34                     ` Ian Campbell
2015-08-13 10:48                     ` Shannon Zhao
2015-08-13 10:54                       ` Jan Beulich
2015-08-13 11:00                         ` Julien Grall
2015-08-13 11:07                           ` Stefano Stabellini
2015-08-13 11:55                           ` Jan Beulich
2015-08-13 12:08                             ` Julien Grall
2015-08-13 14:49                               ` Ian Campbell
2015-08-14  5:34                     ` Shannon Zhao
2015-08-14  9:54                       ` Stefano Stabellini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1439306359.9747.280.camel@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=christoffer.dall@linaro.org \
    --cc=david.vrabel@citrix.com \
    --cc=hangaohuai@huawei.com \
    --cc=jbeulich@suse.com \
    --cc=julien.grall@citrix.com \
    --cc=parth.dixit@linaro.org \
    --cc=peter.huangpeng@huawei.com \
    --cc=shannon.zhao@linaro.org \
    --cc=stefano.stabellini@citrix.com \
    --cc=xen-devel@lists.xen.org \
    --cc=zhaoshenglong@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).