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

Re: [Xen-devel] {PATCH] Add p2m_lock in set_shared_p2m_entry


  • To: Tim Deegan <Tim.Deegan@xxxxxxxxxx>
  • From: Jui-Hao Chiang <juihaochiang@xxxxxxxxx>
  • Date: Thu, 6 Jan 2011 22:52:30 +0800
  • Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Thu, 06 Jan 2011 06:53:19 -0800
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=kg4oY/qKMNuQO6mfdiy3bmQPPG0URsJktc+v0rBNCHgY1Em7uyM69mAnw5YjkOGQdU 8NAd+DpFyZx+FvLUQMFiNyNfsTi+NnJm9Wb7qvhEU9M2vT9CunSXf9bQHUHUs9YR3vX4 OE4DLLB91ziE+o7lfTqhNfc81+TXagBNBHGfc=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

Hi, Tim

Thanks.  I'll have to make a slightly more complex fix, as
set_shared_p2m_entry() can be called (via p2m_teardown() and
mem_sharing_unshare_page()) with the p2m lock already held.  :(


Actually our original problem is the mem_sharing_share_pages() doesn't hold the p2m lock when calling set_shared_p2m_entry(), which causes the share operation to die.

 
That's a sign of a deeper illness which will probably lead to deadlocks
elsewhere.  I intend to give the MM locking (p2m, shadow, hap, log-dirty
and page-sharing locks) a good kicking some time soon, maybe after 4.1
is released.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

 


Rackspace

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