All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthew Rushton <mvrushton@gmail.com>
To: David Vrabel <david.vrabel@citrix.com>,
	konrad.wilk@oracle.com, boris.ostrovsky@oracle.com
Cc: msw@amazon.com, linux-kernel@vger.kernel.org,
	xen-devel@lists.xensource.com, Matt Rushton <mrushton@amazon.com>
Subject: Re: [PATCH v2 2/2] xen/setup: Remap Xen Identity Mapped RAM
Date: Tue, 26 Aug 2014 13:40:09 -0700	[thread overview]
Message-ID: <53FCF0A9.60400@gmail.com> (raw)
In-Reply-To: <53DBA9AC.8000302@citrix.com>

On 08/01/14 07:52, David Vrabel wrote:
> On 31/07/14 18:43, David Vrabel wrote:
>> On 20/07/14 01:01, Matt Rushton wrote:
>>> Instead of ballooning up and down dom0 memory this remaps the existing mfns
>>> that were replaced by the identity map. The reason for this is that the
>>> existing implementation ballooned memory up and and down which caused dom0
>>> to have discontiguous pages. In some cases this resulted in the use of bounce
>>> buffers which reduced network I/O performance significantly. This change will
>>> honor the existing order of the pages with the exception of some boundary
>>> conditions.
>>>
>>> To do this we need to update both the Linux p2m table and the Xen m2p table.
>>> Particular care must be taken when updating the p2m table since it's important
>>> to limit table memory consumption and reuse the existing leaf pages which get
>>> freed when an entire leaf page is set to the identity map. To implement this,
>>> mapping updates are grouped into blocks with table entries getting cached
>>> temporarily and then released.
>>>
>>> On my test system before:
>>> Total pages: 2105014
>>> Total contiguous: 1640635
>>>
>>> After:
>>> Total pages: 2105014
>>> Total contiguous: 2098904
>> Applied to devel/for-linus-3.17
> Unfortunately, this produces too many WARNINGs on some boxes or
> with certain configurations.
>
> I will remove this patch from the queue for 3.17.
>

Have you had a chance to try the v3 patch set?



  parent reply	other threads:[~2014-08-26 20:40 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-20  0:01 [PATCH v2 1/2] xen/setup: Remove Identity Map Debug Message Matt Rushton
2014-07-20  0:01 ` [PATCH v2 2/2] xen/setup: Remap Xen Identity Mapped RAM Matt Rushton
2014-07-28 13:18   ` David Vrabel
2014-07-28 13:18     ` David Vrabel
2014-07-31 17:43   ` David Vrabel
2014-07-31 17:43     ` David Vrabel
2014-08-01 14:52     ` David Vrabel
2014-08-01 14:52       ` David Vrabel
2014-08-01 15:52       ` Matt Wilson
2014-08-01 15:56         ` Boris Ostrovsky
2014-08-02  0:52           ` Matthew Rushton
2014-08-11 17:40           ` Matthew Rushton
2014-08-26 20:40       ` Matthew Rushton [this message]
2014-07-28 13:14 ` [PATCH v2 1/2] xen/setup: Remove Identity Map Debug Message David Vrabel
2014-07-28 13:14   ` 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=53FCF0A9.60400@gmail.com \
    --to=mvrushton@gmail.com \
    --cc=boris.ostrovsky@oracle.com \
    --cc=david.vrabel@citrix.com \
    --cc=konrad.wilk@oracle.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mrushton@amazon.com \
    --cc=msw@amazon.com \
    --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.