From mboxrd@z Thu Jan  1 00:00:00 1970
From: George Dunlap <george.dunlap@citrix.com>
Subject: Re: [PATCH 2 of 5] Clean up locking now that p2m
 lockups are fully synchronized
Date: Thu, 2 Feb 2012 14:31:22 +0000
Message-ID: <1328193082.21722.31.camel@elijah>
References: <patchbomb.1328126164@xdev.gridcentric.ca>
	<fb9c06df05f20461744a.1328126166@xdev.gridcentric.ca>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xensource.com>
In-Reply-To: <fb9c06df05f20461744a.1328126166@xdev.gridcentric.ca>
List-Unsubscribe: <http://lists.xensource.com/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xensource.com>
List-Help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-Subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
Sender: xen-devel-bounces@lists.xensource.com
Errors-To: xen-devel-bounces@lists.xensource.com
To: Andres Lagar-Cavilla <andres@lagarcavilla.org>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>, George Dunlap <George.Dunlap@eu.citrix.com>, "andres@gridcentric.ca" <andres@gridcentric.ca>, "Tim
	(Xen.org)" <tim@xen.org>, "keir.xen@gmail.com" <keir.xen@gmail.com>, "adin@gridcentric.ca" <adin@gridcentric.ca>
List-Id: xen-devel@lists.xenproject.org

On Wed, 2012-02-01 at 19:56 +0000, Andres Lagar-Cavilla wrote:
> @@ -1445,7 +1449,7 @@ p2m_get_nestedp2m(struct vcpu *v, uint64
>              nestedp2m_unlock(d);
>              return p2m;
>          }
> -        p2m_unlock(p2m);
> +        gfn_unlock(p2m, gfn, 0);
>      }

It looks like maybe you forgot to change the corresponding p2m_lock() to
gfn_lock(), here in p2m.c:p2m_get_nestedp2m()?

Other than that, I think the PoD stuff looks fine.  So regarding PoD
stuff:
Acked-by: George Dunlap <george.dunlap@eu.citrix.com>