From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH for-4.6 v2] x86/mm: Make {hap, shadow}_teardown() preemptible Date: Thu, 6 Aug 2015 10:28:26 +0100 Message-ID: <1438853306.9747.125.camel@citrix.com> References: <1438789679-8439-1-git-send-email-andrew.cooper3@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: George Dunlap , Andrew Cooper Cc: Anshul Makkar , Wei Liu , Jan Beulich , Xen-devel List-Id: xen-devel@lists.xenproject.org On Wed, 2015-08-05 at 18:00 +0100, George Dunlap wrote: > On Wed, Aug 5, 2015 at 4:47 PM, Andrew Cooper > wrote: > > From: Anshul Makkar > > > > A domain with sufficient shadow allocation can cause a watchdog timeout > > during domain destruction. Expand the existing -ERESTART logic in > > paging_teardown() to allow {hap/sh}_set_allocation() to become > > restartable during the DOMCTL_destroydomain hypercall. > > > > Signed-off-by: Anshul Makkar > > Signed-off-by: Andrew Cooper > > Reviewed-by: Tim Deegan > > Looks good, thanks: > > Reviewed-by: George Dunlap Applied, thanks.