[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] [PATCH 0 of 2] x86/mm: Unsharing ENOMEM handling


  • To: xen-devel@xxxxxxxxxxxxx
  • From: Andres Lagar-Cavilla <andres@xxxxxxxxxxxxxxxx>
  • Date: Mon, 12 Mar 2012 11:29:36 -0400
  • Cc: andres@xxxxxxxxxxxxxx, tim@xxxxxxx, adin@xxxxxxxxxxxxxx
  • Delivery-date: Mon, 12 Mar 2012 15:29:41 +0000
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=lagarcavilla.org; h=content-type :mime-version:content-transfer-encoding:subject:message-id:date :from:to:cc; q=dns; s=lagarcavilla.org; b=pkC0Uro4nbb/PM6pCEWh0B ZklbsyD9NFygQt4jX6oloThHiFRtmQogFpLPCsZBJVy6Go0N8MrvLlBazH63D4GD 2U6h8/MtbeWM7Nhcjb0vVEO7vDnatzlks5f50aTaHrzdlJ3zh3kAE/VG/pc4CEn4 cmpNVcM8o1ocavhsj4ugM=
  • List-id: Xen developer discussion <xen-devel.lists.xen.org>

These two patches were originally posted on Feb 15th as part of a larger
series.

They were left to simmer as a discussion on wait queues took precedence.

Regardless of the ultimate fate of wait queues, these two patches are necessary
as they solve some bugs on the memory sharing side. When unsharing fails,
domains would spin forever, hosts would crash, etc.

The patches also clarify the semantics of unsharing, and comment how it's
handled.

Two comments against the Feb 15th series taken care of here: 
 - We assert that the unsharing code can only return success or ENOMEN.
 - Acked-by Tim Deegan added to patch #1

Signed-off-by: Andres Lagar-Cavilla <andres@xxxxxxxxxxxxxxxx>
Signed-off-by: Adin Scannell <adin@xxxxxxxxxxx>
Acked-by: Tim Deegan <tim@xxxxxxx>

 xen/arch/x86/mm/mem_event.c       |   5 ++-
 xen/include/asm-x86/mem_event.h   |  30 ++++++++++++++++++---
 xen/arch/x86/hvm/hvm.c            |  23 +++++++++++++++-
 xen/arch/x86/mm.c                 |   8 +++--
 xen/arch/x86/mm/mem_sharing.c     |  54 +++++++++++++++++++++++---------------
 xen/arch/x86/mm/p2m.c             |  18 ++++++++++++-
 xen/common/grant_table.c          |  11 ++++---
 xen/common/memory.c               |   1 +
 xen/include/asm-x86/mem_sharing.h |  27 ++++++++++++++++++-
 9 files changed, 138 insertions(+), 39 deletions(-)

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.