From: Wei Liu <email@example.com>
To: Juergen Gross <firstname.lastname@example.org>
Cc: Wei Liu <email@example.com>,
Ian Campbell <firstname.lastname@example.org>,
Subject: Re: [PATCH v4 1/9] libxc: reorganize domain builder guest memory allocator
Date: Thu, 12 Nov 2015 11:32:46 +0000 [thread overview]
Message-ID: <20151112113246.GA28451@zion.uk.xensource.com> (raw)
On Thu, Nov 12, 2015 at 12:29:06PM +0100, Juergen Gross wrote:
> On 12/11/15 12:20, Ian Campbell wrote:
> > On Thu, 2015-11-12 at 11:14 +0000, Wei Liu wrote:
> >> On Thu, Nov 05, 2015 at 03:36:27PM +0100, Juergen Gross wrote:
> >>> Guest memory allocation in the domain builder of libxc is done via
> >>> virtual addresses only. In order to be able to support preallocated
> >>> areas not virtually mapped reorganize the memory allocator to keep
> >>> track of allocated pages globally and in allocated segments.
> >>> This requires an interface change of the allocate callback of the
> >>> domain builder which currently is using the last mapped virtual
> >>> address as a parameter. This is no problem as the only user of this
> >>> callback is stubdom/grub/kexec.c using this virtual address to
> >>> calculate the last used pfn.
> >>> Signed-off-by: Juergen Gross <email@example.com>
> >> One question: did you test building with pygrub after this change? I
> >> think the code is correct but we'd better be sure it doesn't break
> >> pygrub.
> > Did you mean pvgrub? (with a Vee)
> > pygrub (with a Why) runs way before the libxc domain builder gets in on the
> > act, doesn't it.
> To be clear: we have:
> - pygrub: running on dom0, mounts the root disk and analyzes the grub
> configuration found, selects kernel and initrd and loads those via
> - pvgrub: based on grub2, is started via libxl as paravirtualized kernel
> in the new domain. pvgrub is reading the grub configuration and starts
> the appropriate kernel via it's own mechanisms (patches for support of
> huge domains are sent, no comments up to now).
> - stubdom based grub: subject to the patch we are discussing here. It is
> being built in my environment, but not tested up to now.
I think we use pvgrub to denote the third case (according to our wiki).
The second case is just ... upstream grub2 with pv capability ?
(So many grubs...)
next prev parent reply other threads:[~2015-11-12 11:32 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-05 14:36 [PATCH v4 0/9] libxc: support building large pv-domains Juergen Gross
2015-11-05 14:36 ` [PATCH v4 1/9] libxc: reorganize domain builder guest memory allocator Juergen Gross
2015-11-12 11:14 ` Wei Liu
2015-11-12 11:20 ` Ian Campbell
2015-11-12 11:22 ` Wei Liu
2015-11-12 11:29 ` Juergen Gross
2015-11-12 11:32 ` Wei Liu [this message]
2015-11-12 12:28 ` Juergen Gross
2015-11-12 11:21 ` Juergen Gross
2015-11-12 11:24 ` Wei Liu
2015-11-05 14:36 ` [PATCH v4 2/9] xen: add generic flag to elf_dom_parms indicating support of unmapped initrd Juergen Gross
2015-11-05 17:38 ` Andrew Cooper
2015-11-05 14:36 ` [PATCH v4 3/9] libxc: rename domain builder count_pgtables to alloc_pgtables Juergen Gross
2015-11-05 14:36 ` [PATCH v4 4/9] libxc: introduce domain builder architecture specific data Juergen Gross
2015-11-05 14:36 ` [PATCH v4 5/9] libxc: use domain builder architecture private data for x86 pv domains Juergen Gross
2015-11-05 14:36 ` [PATCH v4 6/9] libxc: create unmapped initrd in domain builder if supported Juergen Gross
2015-11-05 14:36 ` [PATCH v4 7/9] libxc: split p2m allocation in domain builder from other magic pages Juergen Gross
2015-11-05 14:36 ` [PATCH v4 8/9] libxc: rework of domain builder's page table handler Juergen Gross
2015-11-12 12:39 ` Wei Liu
2015-11-12 12:45 ` Juergen Gross
2015-11-05 14:36 ` [PATCH v4 9/9] libxc: create p2m list outside of kernel mapping if supported Juergen Gross
2015-11-12 12:42 ` Wei Liu
2015-11-12 5:09 ` [PATCH v4 0/9] libxc: support building large pv-domains Juergen Gross
2015-11-12 9:39 ` Wei Liu
2015-11-12 9:41 ` Juergen Gross
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:
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
* 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.