From: Jan Beulich <jbeulich@suse.com>
To: George Dunlap <gwd@xenproject.org>
Cc: "Andrew Cooper" <andrew.cooper3@citrix.com>,
"Roger Pau Monné" <roger@xenproject.org>,
"Teddy Astie" <teddy.astie@vates.tech>,
"Anthony PERARD" <anthony.perard@vates.tech>,
"Michal Orzel" <michal.orzel@amd.com>,
"Julien Grall" <julien@xen.org>,
"Stefano Stabellini" <sstabellini@kernel.org>,
xen-devel@lists.xenproject.org
Subject: Re: [PATCH 0/7] x86: Address Space Isolation, part 1: per-domain area mapping rework
Date: Fri, 21 Aug 2026 10:45:28 +0200 [thread overview]
Message-ID: <ec040560-4377-474d-a929-3080b74bb0af@suse.com> (raw)
In-Reply-To: <20260820-asi-part1-0-f2dbd92b8459@xenproject.org>
On 20.08.2026 19:43, George Dunlap wrote:
> One point reviewers may want to look at specifically: patch 1 changes
> where the per-domain page-tables are allocated from, and its commit
> message discusses the (minor) NUMA-placement consequence.
While I don't recall which recent patch (series) it was, I can't very well
say "no new xenheap allocations please" there without also saying so here.
I've read over patch 1's description, and while it tries to justify this
accordingly, I still remain concerned. I think we simply have to accept
the mapping overhead, to avoid allocating from a pool which - over time -
is representing a decreasing portion of total memory systems have (on
average, and not even considering systems with extremely sparse memory
layouts, and with perhaps PDX compression not doing good enough to
compensate).
Jan
next prev parent reply other threads:[~2026-08-21 8:45 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-20 17:43 [PATCH 0/7] x86: Address Space Isolation, part 1: per-domain area mapping rework George Dunlap
2026-08-20 17:43 ` [PATCH 1/7] x86/mm: allocate the per-domain page-tables from the xenheap George Dunlap
2026-08-20 17:43 ` [PATCH 2/7] x86/mm: introduce populate_perdomain_mapping() George Dunlap
2026-08-20 17:43 ` [PATCH 3/7] x86/pv: use populate_perdomain_mapping() to map the Xen GDT George Dunlap
2026-08-21 20:59 ` Andrew Cooper
2026-08-20 17:43 ` [PATCH 4/7] x86/pv: set/clear guest GDT mappings using populate_perdomain_mapping() George Dunlap
2026-08-20 17:43 ` [PATCH 5/7] x86/pv: update guest LDT mappings using {populate,destroy}_perdomain_mapping() George Dunlap
2026-08-20 17:43 ` [PATCH 6/7] x86/pv: remove stashing of GDT/LDT L1 page-tables George Dunlap
2026-08-20 17:43 ` [PATCH 7/7] x86/mm: simplify create_perdomain_mapping() interface George Dunlap
2026-08-21 8:45 ` Jan Beulich [this message]
2026-08-21 15:17 ` [PATCH 0/7] x86: Address Space Isolation, part 1: per-domain area mapping rework George Dunlap
2026-08-21 15:36 ` George Dunlap
2026-08-24 9:02 ` Jan Beulich
2026-08-25 11:42 ` George Dunlap
2026-08-25 12:00 ` Juergen Gross
2026-08-25 13:28 ` Jan Beulich
2026-08-25 14:40 ` George Dunlap
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=ec040560-4377-474d-a929-3080b74bb0af@suse.com \
--to=jbeulich@suse.com \
--cc=andrew.cooper3@citrix.com \
--cc=anthony.perard@vates.tech \
--cc=gwd@xenproject.org \
--cc=julien@xen.org \
--cc=michal.orzel@amd.com \
--cc=roger@xenproject.org \
--cc=sstabellini@kernel.org \
--cc=teddy.astie@vates.tech \
--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.