From: Andres Lagar-Cavilla <andres@lagarcavilla.org>
To: xen-devel@lists.xensource.com
Cc: ian.campbell@citrix.com, andres@gridcentric.ca, tim@xen.org,
keir.xen@gmail.com, JBeulich@suse.com, ian.jackson@citrix.com,
adin@gridcentric.ca
Subject: [PATCH 00 of 13] Sharing overhaul V4
Date: Wed, 25 Jan 2012 22:32:24 -0500 [thread overview]
Message-ID: <patchbomb.1327548744@xdev.gridcentric.ca> (raw)
Changes from sharing overhaul v3 posted Jan 16th
- Patches 1,2: Improved description in header
- Patch 1:
+ Changed mem_sharing audit from int to void
+ Fixed format of audit debugtrace printk's
Patch 2: added comment to code explaining grant sharing raison-d'etre
Patches 4 (mainly), but 6 and 8 as well:
+ Using a per-cpu static struct instead of stack var for lock
order and deadlock detection book-keeping.
Patches 7, 9, 11: Added Acked-by's
(all the major tools patches acked-by tools maintainers now)
Patch 8:
+ Fixed off-by-one audit type accounting caused by PGT_locked bumping
the page type count
New patch 13: rename shared_info to sharing so that grep/cscope/etc
don't alias it with the unrelated shared info page.
Regular description follows...
--------------------
This patch series brings about a set of changes to the sharing support
in the hypervisor and tools. Specifically:
- Granular scalable locking: lock mfn's individually when sharing/unsharing
them. Do not rely on a global lock. Use RCU to maintain a list of all
shared frames for audit purposes.
- Refreshed stats: they now work (tm), and are accesible via libxc
and console.
- libxl/xl support to get sharing stats.
- New sharing command "add to physmap", to directly populate a new domain
with shared frames.
- Toolstack and memshr kept in sync.
A consequence of our modifications is a change to the sharing
API. While we refresh the only user of the sharing API in the
tree, we want to make sure we are not affecting any other
users of the sharing API.
Patches 9, 10 11 are tools patches. Patches 1 to 8 are x86/mm.
Patch 12 is a simple testing tool. Patch 13 is a style fix, no
functional changes.
Signed-off-by: Andres Lagar-Cavilla <andres@lagarcavilla.org>
Signed-off-by: Adin Scannell <adin@scannell.ca>
xen/arch/x86/mm/mem_sharing.c | 539 +++++++++++++++++-----------------
xen/include/asm-x86/mem_sharing.h | 19 +-
xen/include/asm-x86/mm.h | 11 +-
xen/include/public/domctl.h | 3 +
xen/arch/x86/mm/mem_sharing.c | 57 +++-
xen/include/public/domctl.h | 13 +
xen/arch/x86/mm.c | 74 +----
xen/arch/x86/mm/mem_sharing.c | 257 +++++++++++++++-
xen/arch/x86/mm/mm-locks.h | 15 +-
xen/include/asm-x86/mm.h | 27 +-
xen/arch/x86/mm/mem_sharing.c | 16 +
xen/arch/x86/mm/mm-locks.h | 18 +-
xen/include/asm-x86/mm.h | 3 +-
xen/arch/x86/mm.c | 6 -
xen/arch/x86/mm/mem_sharing.c | 25 +
xen/arch/x86/x86_64/compat/mm.c | 6 +
xen/arch/x86/x86_64/mm.c | 7 +
xen/include/asm-x86/mem_sharing.h | 1 +
xen/include/public/memory.h | 1 +
xen/arch/x86/mm/mem_sharing.c | 103 ++++++
xen/include/public/domctl.h | 3 +-
xen/arch/ia64/xen/mm.c | 5 +
xen/arch/x86/mm.c | 14 +
xen/common/keyhandler.c | 7 +-
xen/include/xen/mm.h | 3 +
xen/arch/x86/mm/mem_sharing.c | 93 ++---
xen/arch/x86/mm/mm-locks.h | 18 -
xen/include/asm-x86/mem_sharing.h | 3 +-
tools/blktap2/drivers/tapdisk-vbd.c | 6 +-
tools/blktap2/drivers/tapdisk.h | 2 +-
tools/libxc/xc_memshr.c | 63 +++-
tools/libxc/xenctrl.h | 19 +-
tools/memshr/bidir-daemon.c | 34 +-
tools/memshr/bidir-hash.h | 12 +-
tools/memshr/interface.c | 30 +-
tools/memshr/memshr.h | 11 +-
tools/libxc/xc_memshr.c | 10 +
tools/libxc/xenctrl.h | 17 +
docs/man/xl.pod.1 | 13 +
tools/libxl/libxl.c | 2 +
tools/libxl/libxl_types.idl | 2 +
tools/libxl/xl.h | 1 +
tools/libxl/xl_cmdimpl.c | 66 ++++
tools/libxl/xl_cmdtable.c | 5 +
tools/tests/mem-sharing/Makefile | 26 +
tools/tests/mem-sharing/memshrtool.c | 165 ++++++++++
xen/arch/x86/mm/mem_sharing.c | 64 ++--
xen/include/asm-x86/mm.h | 2 +-
48 files changed, 1360 insertions(+), 537 deletions(-)
next reply other threads:[~2012-01-26 3:32 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-26 3:32 Andres Lagar-Cavilla [this message]
2012-01-26 3:32 ` [PATCH 01 of 13] x86/mm: Eliminate hash table in sharing code as index of shared mfns Andres Lagar-Cavilla
2012-01-26 3:32 ` [PATCH 02 of 13] x86/mm: Update mem sharing interface to (re)allow sharing of grants Andres Lagar-Cavilla
2012-01-26 3:32 ` [PATCH 03 of 13] x86/mm: Add per-page locking for memory sharing, when audits are disabled Andres Lagar-Cavilla
2012-01-26 3:32 ` [PATCH 04 of 13] x86/mm: Enforce lock ordering for sharing page locks Andres Lagar-Cavilla
2012-01-26 3:32 ` [PATCH 05 of 13] x86/mm: Check how many mfns are shared, in addition to how many are saved Andres Lagar-Cavilla
2012-01-26 3:32 ` [PATCH 06 of 13] x86/mm: New domctl: add a shared page to the physmap Andres Lagar-Cavilla
2012-01-26 3:32 ` [PATCH 07 of 13] Add the ability to poll stats about shared memory via the console Andres Lagar-Cavilla
2012-01-26 3:32 ` [PATCH 08 of 13] x86/mm: use RCU in mem sharing audit list, eliminate global lock completely Andres Lagar-Cavilla
2012-01-26 3:32 ` [PATCH 09 of 13] Update memshr API and tools Andres Lagar-Cavilla
2012-01-26 3:32 ` [PATCH 10 of 13] Tools: Expose to libxc the total number of shared frames and space saved Andres Lagar-Cavilla
2012-01-26 3:32 ` [PATCH 11 of 13] Tools: Add a sharing command to xl for information about shared pages Andres Lagar-Cavilla
2012-01-26 3:32 ` [PATCH 12 of 13] Memshrtool: tool to test and exercise the sharing subsystem Andres Lagar-Cavilla
2012-01-27 18:21 ` Ian Jackson
2012-01-27 18:27 ` Andres Lagar-Cavilla
2012-01-26 3:32 ` [PATCH 13 of 13] x86/mm: Sharing overhaul style improvements Andres Lagar-Cavilla
2012-01-26 12:54 ` [PATCH 00 of 13] Sharing overhaul V4 Tim Deegan
2012-01-26 13:08 ` Andres Lagar-Cavilla
2012-01-26 13:16 ` Olaf Hering
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=patchbomb.1327548744@xdev.gridcentric.ca \
--to=andres@lagarcavilla.org \
--cc=JBeulich@suse.com \
--cc=adin@gridcentric.ca \
--cc=andres@gridcentric.ca \
--cc=ian.campbell@citrix.com \
--cc=ian.jackson@citrix.com \
--cc=keir.xen@gmail.com \
--cc=tim@xen.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).