From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Juergen Gross <jgross@suse.com>
Cc: linux-kernel@vger.kernel.org, xen-devel@lists.xensource.com,
david.vrabel@citrix.com, boris.ostrovsky@oracle.com
Subject: Re: [PATCH] xen: switch to post-init routines in xen mmu.c earlier
Date: Wed, 10 Dec 2014 11:13:53 -0500 [thread overview]
Message-ID: <20141210161353.GD4268@laptop.dumpdata.com> (raw)
In-Reply-To: <1418226963-24873-1-git-send-email-jgross@suse.com>
On Wed, Dec 10, 2014 at 04:56:03PM +0100, Juergen Gross wrote:
> With the virtual mapped linear p2m list the post-init mmu operations
> must be used for setting up the p2m mappings, as in case of
> CONFIG_FLATMEM the init routines may trigger BUGs.
Um, could you explain a bit more of why the CONFIG_FLATMEM
is such unique? What about the other CONFIG_*MEM cases?
>
> Reported-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
> Signed-off-by: Juergen Gross <jgross@suse.com>
> ---
> arch/x86/xen/mmu.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/x86/xen/mmu.c b/arch/x86/xen/mmu.c
> index 6ab6150..a1a429a 100644
> --- a/arch/x86/xen/mmu.c
> +++ b/arch/x86/xen/mmu.c
> @@ -1225,6 +1225,7 @@ static void __init xen_pagetable_p2m_setup(void)
> static void __init xen_pagetable_init(void)
> {
> paging_init();
> + xen_post_allocator_init();
>
> xen_pagetable_p2m_setup();
>
> @@ -1236,7 +1237,6 @@ static void __init xen_pagetable_init(void)
> xen_remap_memory();
>
> xen_setup_shared_info();
> - xen_post_allocator_init();
> }
> static void xen_write_cr2(unsigned long cr2)
> {
> --
> 2.1.2
>
next prev parent reply other threads:[~2014-12-10 16:14 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-10 15:56 [PATCH] xen: switch to post-init routines in xen mmu.c earlier Juergen Gross
2014-12-10 16:13 ` Konrad Rzeszutek Wilk [this message]
2014-12-11 5:17 ` Juergen Gross
2014-12-10 18:07 ` [Xen-devel] " David Vrabel
2014-12-10 18:07 ` David Vrabel
2014-12-11 5:32 ` Juergen Gross
2014-12-11 12:15 ` David Vrabel
2014-12-11 12:15 ` David Vrabel
2014-12-11 12:15 ` David Vrabel
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=20141210161353.GD4268@laptop.dumpdata.com \
--to=konrad.wilk@oracle.com \
--cc=boris.ostrovsky@oracle.com \
--cc=david.vrabel@citrix.com \
--cc=jgross@suse.com \
--cc=linux-kernel@vger.kernel.org \
--cc=xen-devel@lists.xensource.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.