All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Vrabel <david.vrabel@citrix.com>
To: Juergen Gross <jgross@suse.com>, <linux-kernel@vger.kernel.org>,
	<xen-devel@lists.xensource.com>, <konrad.wilk@oracle.com>,
	<boris.ostrovsky@oracle.com>, <jbeulich@suse.com>
Subject: Re: [PATCH V3] xen: eliminate scalability issues from initial mapping setup
Date: Tue, 23 Sep 2014 14:10:29 +0100	[thread overview]
Message-ID: <54217145.7090206@citrix.com> (raw)
In-Reply-To: <5420EFD2.4070801@suse.com>

On 23/09/14 04:58, Juergen Gross wrote:
> On 09/17/2014 04:59 PM, Juergen Gross wrote:
>> Direct Xen to place the initial P->M table outside of the initial
>> mapping, as otherwise the 1G (implementation) / 2G (theoretical)
>> restriction on the size of the initial mapping limits the amount
>> of memory a domain can be handed initially.
>>
>> As the initial P->M table is copied rather early during boot to
>> domain private memory and it's initial virtual mapping is dropped,
>> the easiest way to avoid virtual address conflicts with other
>> addresses in the kernel is to use a user address area for the
>> virtual address of the initial P->M table. This allows us to just
>> throw away the page tables of the initial mapping after the copy
>> without having to care about address invalidation.
>>
>> It should be noted that this patch won't enable a pv-domain to USE
>> more than 512 GB of RAM. It just enables it to be started with a
>> P->M table covering more memory. This is especially important for
>> being able to boot a Dom0 on a system with more than 512 GB memory.
>>
>> Signed-off-by: Juergen Gross <jgross@suse.com>
>> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> 
> Any Acks/Naks?

Sorry for the delay.  We only have one machine with 1 TB of RAM and it's
very busy.

>> +static void xen_cleanmfnmap_free_pgtbl(void *pgtbl)

This needs to be __init.  But I'll fix this up.  In future, can you make
sure you build with the option that finds these
(CONFIG_DEBUG_SECTION_MISMATCH).

David

WARNING: multiple messages have this Message-ID (diff)
From: David Vrabel <david.vrabel@citrix.com>
To: Juergen Gross <jgross@suse.com>,
	linux-kernel@vger.kernel.org, xen-devel@lists.xensource.com,
	konrad.wilk@oracle.com, boris.ostrovsky@oracle.com,
	jbeulich@suse.com
Subject: Re: [PATCH V3] xen: eliminate scalability issues from initial mapping setup
Date: Tue, 23 Sep 2014 14:10:29 +0100	[thread overview]
Message-ID: <54217145.7090206@citrix.com> (raw)
In-Reply-To: <5420EFD2.4070801@suse.com>

On 23/09/14 04:58, Juergen Gross wrote:
> On 09/17/2014 04:59 PM, Juergen Gross wrote:
>> Direct Xen to place the initial P->M table outside of the initial
>> mapping, as otherwise the 1G (implementation) / 2G (theoretical)
>> restriction on the size of the initial mapping limits the amount
>> of memory a domain can be handed initially.
>>
>> As the initial P->M table is copied rather early during boot to
>> domain private memory and it's initial virtual mapping is dropped,
>> the easiest way to avoid virtual address conflicts with other
>> addresses in the kernel is to use a user address area for the
>> virtual address of the initial P->M table. This allows us to just
>> throw away the page tables of the initial mapping after the copy
>> without having to care about address invalidation.
>>
>> It should be noted that this patch won't enable a pv-domain to USE
>> more than 512 GB of RAM. It just enables it to be started with a
>> P->M table covering more memory. This is especially important for
>> being able to boot a Dom0 on a system with more than 512 GB memory.
>>
>> Signed-off-by: Juergen Gross <jgross@suse.com>
>> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> 
> Any Acks/Naks?

Sorry for the delay.  We only have one machine with 1 TB of RAM and it's
very busy.

>> +static void xen_cleanmfnmap_free_pgtbl(void *pgtbl)

This needs to be __init.  But I'll fix this up.  In future, can you make
sure you build with the option that finds these
(CONFIG_DEBUG_SECTION_MISMATCH).

David

  reply	other threads:[~2014-09-23 13:10 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-17 14:59 [PATCH V3] xen: remove some memory limits from pv-domains Juergen Gross
2014-09-17 14:59 ` [PATCH V3] xen: eliminate scalability issues from initial mapping setup Juergen Gross
2014-09-23  3:58   ` Juergen Gross
2014-09-23 13:10     ` David Vrabel [this message]
2014-09-23 13:10       ` David Vrabel
2014-09-24 13:20   ` David Vrabel
2014-09-24 13:20     ` David Vrabel
2014-09-24 14:03     ` Juergen Gross
2014-09-26  7:54     ` Juergen Gross

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=54217145.7090206@citrix.com \
    --to=david.vrabel@citrix.com \
    --cc=boris.ostrovsky@oracle.com \
    --cc=jbeulich@suse.com \
    --cc=jgross@suse.com \
    --cc=konrad.wilk@oracle.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.