All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Vrabel <david.vrabel@citrix.com>
To: xen-devel@lists.xenproject.org
Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>,
	David Vrabel <david.vrabel@citrix.com>
Subject: [PATCHv2 00/14] xen: fix many long-standing grant mapping bugs
Date: Mon, 12 Jan 2015 15:43:23 +0000	[thread overview]
Message-ID: <1421077417-7162-1-git-send-email-david.vrabel@citrix.com> (raw)

This series fixes a number of long-standing bugs in the handling of
grant maps.  Refer to the following for all the details.

  http://xenbits.xen.org/people/dvrabel/grant-improvements-C.pdf

In summary, the important uses that this enables are:

1. Block backends can use networked storage safely.

2. Block backends can use network storage provided by other guests on
   the same host.

3. User space block backends can use direct I/O or asynchronous I/O.

The first two patches are the core MM changes necessary.  I shall be
sending these to the MM maintainers seperately.

Patches #3 and #4 remove existing (broken) mechanisms.  This does
temporarily break some previously working use cases, but it does make
the subsequent additions much easier to review.

As a happy side effect, performance is also likely to be improved in
some areas (but I've not got any measurements yet).  User space
backends using grant mapping should see some good improvements from
reduced overheads and better unmap batching.  VIF to VIF network
traffic may also see a small improvement.

Finally, thanks to Jenny who did much of the implementation.

Changes in v2:
- Add find_page() VMA op instead of pages field (so struct
  vm_area_struct doesn't increase in size.
- Make foreign page tracking arch-independant and improve the API.
- Alloc extra memory (for 32-bit archs) for the (domain, gref) when
  allocating the page (instead of during the map).
- Convert gntdev's lock to a mutex.

David

             reply	other threads:[~2015-01-12 15:44 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-12 15:43 David Vrabel [this message]
2015-01-12 15:43 ` [PATCH 01/14] mm: provide a find_page vma operation David Vrabel
2015-01-12 15:43 ` [PATCH 02/14] mm: add 'foreign' alias for the 'pinned' page flag David Vrabel
2015-01-12 15:43 ` [PATCH 03/14] xen/grant-table: pre-populate kernel unmap ops for xen_gnttab_unmap_refs() David Vrabel
2015-01-12 15:43 ` [PATCH 04/14] xen: remove scratch frames for ballooned pages and m2p override David Vrabel
2015-01-12 15:43 ` [PATCH 05/14] x86/xen: require ballooned pages for grant maps David Vrabel
2015-01-12 15:43 ` [PATCH 06/14] xen/grant-table: add helpers for allocating pages David Vrabel
2015-01-12 15:43 ` [PATCH 07/14] xen: mark grant mapped pages as foreign David Vrabel
2015-01-12 16:54   ` Ian Campbell
2015-01-12 16:56     ` David Vrabel
2015-01-13 22:46   ` Boris Ostrovsky
2015-01-12 15:43 ` [PATCH 08/14] xen-netback: use foreign page information from the pages themselves David Vrabel
2015-01-12 16:56   ` Ian Campbell
2015-01-12 17:16     ` David Vrabel
2015-01-12 17:20       ` Ian Campbell
2015-01-13 14:43   ` David Vrabel
2015-01-13 14:43   ` [Xen-devel] " David Vrabel
2015-01-13 21:57     ` David Miller
2015-01-13 21:57     ` David Miller
2015-01-12 15:43 ` [PATCH 09/14] xen/grant-table: add a mechanism to safely unmap pages that are in use David Vrabel
2015-01-13 23:31   ` Boris Ostrovsky
2015-01-19 15:27     ` David Vrabel
2015-01-12 15:43 ` [PATCH 10/14] xen/gntdev: convert priv->lock to a mutex David Vrabel
2015-01-12 15:43 ` [PATCH 11/14] xen/gntdev: safely unmap grants in case they are still in use David Vrabel
2015-01-12 15:43 ` [PATCH 12/14] xen-blkback: " David Vrabel
2015-01-14 15:17   ` Boris Ostrovsky
2015-01-14 15:47   ` Boris Ostrovsky
2015-01-14 16:00     ` David Vrabel
2015-01-14 16:22       ` Boris Ostrovsky
2015-01-14 16:33         ` David Vrabel
2015-01-12 15:43 ` [PATCH 13/14] xen/gntdev: mark userspace PTEs as special on x86 PV guests David Vrabel
2015-01-12 15:43 ` [PATCH 14/14] xen/gntdev: provide find_page VMA operation 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=1421077417-7162-1-git-send-email-david.vrabel@citrix.com \
    --to=david.vrabel@citrix.com \
    --cc=boris.ostrovsky@oracle.com \
    --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.