All of lore.kernel.org
 help / color / mirror / Atom feed
From: Julien Grall <julien.grall@citrix.com>
To: Ian Campbell <ian.campbell@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: Wed, 12 Aug 2015 10:02:06 +0100	[thread overview]
Message-ID: <55CB0B8E.8010904@citrix.com> (raw)
In-Reply-To: <1439369174.9747.304.camel@citrix.com>



On 12/08/2015 09:46, Ian Campbell wrote:
> On Tue, 2015-08-11 at 17:01 +0100, Julien Grall wrote:
>> On 11/08/15 16:19, Ian Campbell wrote:
>>>> 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.
>>
>> That from what I recall from an IRL talk.
>>
>> Anyway, I've looked in my archive to see why we decided to keep the
>> grant table parameters (in Xen ACPI table at this point). We were not
>> sure that the domain as all the key in hand in order to find memory hole.
>>
>> I think it's quite important to not think only about Linux but all other
>> Operating Systems. If we ever require a parameters later, it would mean
>> that the OS won't be able to run as DOM0 on older Xen.
>>
>> Linux is using ballooned page, which means loosing ~128KB
>
> It's not "loosing" anything. It is _using_ 128KB for a legitimate data
> structure needed to support the platform (the fact that it happens to not
> really be using the underlying RAM is irrelevant here).
>
> It's not really inherently any different to "loosing" memory in order to
> allocate e.g. page tables.

If you read my sentence until the end, you would have noticed why I say 
"loosing". It's because we never give back this memory to Xen due to the 
1:1 mapping...

The grant table are already allocated in Xen separately which means 
doubling the size of memory used for DOM0 grant table.

Anyway, that's small amount of data and can optimized later using the 
hotplug.

Cheers,

-- 
Julien Grall

  reply	other threads:[~2015-08-12  9:02 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
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 [this message]
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=55CB0B8E.8010904@citrix.com \
    --to=julien.grall@citrix.com \
    --cc=christoffer.dall@linaro.org \
    --cc=david.vrabel@citrix.com \
    --cc=hangaohuai@huawei.com \
    --cc=ian.campbell@citrix.com \
    --cc=jbeulich@suse.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.