From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: Keir Fraser <keir@xen.org>,
Ian Campbell <Ian.Campbell@citrix.com>,
Matthew Rushton <mvrushton@gmail.com>,
AndrewCooper <andrew.cooper3@citrix.com>,
Tim Deegan <tim@xen.org>, Matt Wilson <msw@amazon.com>,
Matt Wilson <msw@linux.com>,
xen-devel@lists.xenproject.org
Subject: Re: [RFC PATCH] page_alloc: use first half of higher order chunks when halving
Date: Mon, 14 Apr 2014 10:40:17 -0400 [thread overview]
Message-ID: <20140414144017.GB23371@phenom.dumpdata.com> (raw)
In-Reply-To: <534BBDB602000078000084CA@nat28.tlf.novell.com>
On Mon, Apr 14, 2014 at 09:51:34AM +0100, Jan Beulich wrote:
> >>> On 13.04.14 at 23:32, <tim@xen.org> wrote:
> > (c) If for some reason the current hypercall API is not sufficient
> > for dom0 to get what it wants, we should consider adding some new
> > operation/flag/mode somewhere. But since AFAIK there's already
> > another path in linux that allocates contiguous DMA buffers
> > for device drivers, presumably this isn't the case.
>
> And it should be kept in mind that requesting contiguous memory
> shouldn't be done at will, as it may end up exhausting the portion
> of memory intended for DMA-style allocations (SWIOTLB / DMA-
> coherent allocations in Linux). I.e. neither Dom0 nor DomU should
> be trying to populate large parts of their memory with contiguous
> allocation requests to the hypervisor. They may, if they so desire,
> go and re-arrange their P2M mapping (solely based on what they
> got handed by doing order-0 allocations).
I did try that at some point - and it did not work.
The reason for trying this was that during the E820 parsing we would
find the MMIO holes/gaps and instead of doing the
'XENMEM_decrease_reservation'/ 'XENMEM_populate_physmap' dance I
thought I could just swap the P2M entries.
That was OK, but the M2P lookup table was not too thrilled with this.
Perhaps I should have used another hypercall to re-arrange the M2P?
I think I did try 'XENMEM_exchange' but that is not the right call either.
Perhaps I should use XENMEM_remove_from_physmap/XENMEM_add_to_physmap
combo ?
>
> Jan
>
next prev parent reply other threads:[~2014-04-14 14:41 UTC|newest]
Thread overview: 55+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-25 11:22 [RFC PATCH] page_alloc: use first half of higher order chunks when halving Matt Wilson
2014-03-25 11:44 ` Andrew Cooper
2014-03-25 13:20 ` Matt Wilson
2014-03-25 20:18 ` Matthew Rushton
2014-03-25 12:19 ` Tim Deegan
2014-03-25 13:27 ` Matt Wilson
2014-03-25 20:09 ` Matthew Rushton
2014-03-26 9:55 ` Tim Deegan
2014-03-26 10:17 ` Matt Wilson
2014-03-26 10:44 ` David Vrabel
2014-03-26 10:48 ` Matt Wilson
2014-03-26 11:13 ` Ian Campbell
2014-03-26 11:41 ` Matt Wilson
2014-03-26 11:45 ` Andrew Cooper
2014-03-26 11:50 ` Matt Wilson
2014-03-26 12:43 ` David Vrabel
2014-03-26 12:48 ` Matt Wilson
2014-03-26 15:08 ` Konrad Rzeszutek Wilk
2014-03-26 15:15 ` Matt Wilson
2014-03-26 15:59 ` Matthew Rushton
2014-03-26 16:36 ` Konrad Rzeszutek Wilk
2014-03-26 17:47 ` Matthew Rushton
2014-03-26 17:56 ` Konrad Rzeszutek Wilk
2014-03-26 22:15 ` Matthew Rushton
2014-03-28 17:02 ` Konrad Rzeszutek Wilk
2014-03-28 22:06 ` Matthew Rushton
2014-03-31 14:15 ` Konrad Rzeszutek Wilk
2014-04-01 3:25 ` Matthew Rushton
2014-04-01 10:48 ` Konrad Rzeszutek Wilk
2014-04-01 12:22 ` Tim Deegan
2014-04-02 0:17 ` Matthew Rushton
2014-04-02 7:52 ` Jan Beulich
2014-04-02 10:06 ` Ian Campbell
2014-04-02 10:15 ` Jan Beulich
2014-04-02 10:20 ` Ian Campbell
2014-04-09 22:21 ` Matthew Rushton
2014-04-10 6:14 ` Jan Beulich
2014-04-11 20:20 ` Matthew Rushton
2014-04-11 17:05 ` Konrad Rzeszutek Wilk
2014-04-11 20:28 ` Matthew Rushton
2014-04-12 1:34 ` Konrad Rzeszutek Wilk
2014-04-13 21:32 ` Tim Deegan
2014-04-14 8:51 ` Jan Beulich
2014-04-14 14:40 ` Konrad Rzeszutek Wilk [this message]
2014-04-14 15:34 ` Jan Beulich
2014-04-16 14:15 ` Konrad Rzeszutek Wilk
2014-04-17 1:34 ` Matthew Rushton
2014-05-07 23:16 ` Matthew Rushton
2014-05-08 18:05 ` Konrad Rzeszutek Wilk
2014-05-14 15:06 ` Konrad Rzeszutek Wilk
2014-05-20 19:26 ` Matthew Rushton
2014-05-23 19:00 ` Konrad Rzeszutek Wilk
2014-06-04 22:25 ` Matthew Rushton
2014-06-05 9:32 ` David Vrabel
2014-03-26 16:34 ` Konrad Rzeszutek Wilk
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=20140414144017.GB23371@phenom.dumpdata.com \
--to=konrad.wilk@oracle.com \
--cc=Ian.Campbell@citrix.com \
--cc=JBeulich@suse.com \
--cc=andrew.cooper3@citrix.com \
--cc=keir@xen.org \
--cc=msw@amazon.com \
--cc=msw@linux.com \
--cc=mvrushton@gmail.com \
--cc=tim@xen.org \
--cc=xen-devel@lists.xenproject.org \
/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.