From: Jan Beulich Date: Wed, 20 Dec 2017 09:05:16 +0000 (+0100) Subject: x86/shadow: ignore sh_pin() failure in one more case X-Git-Tag: archive/raspbian/4.11.1-1+rpi1~1^2~66^2~888 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=51fc3633dfe50d0e04f438dbcd19e55beab9b9a7;p=xen.git x86/shadow: ignore sh_pin() failure in one more case Following what we've already done in the XSA-250 fix, convert another sh_pin() caller to no longer fail the higher level operation if pinning fails, as pinning is a performance optimization only in those places. Suggested-by: Tim Deegan Signed-off-by: Jan Beulich Reviewed-by: Tim Deegan --- diff --git a/xen/arch/x86/mm/shadow/multi.c b/xen/arch/x86/mm/shadow/multi.c index f7182e466a..c4e954efb9 100644 --- a/xen/arch/x86/mm/shadow/multi.c +++ b/xen/arch/x86/mm/shadow/multi.c @@ -3952,17 +3952,15 @@ sh_set_toplevel_shadow(struct vcpu *v, } ASSERT(mfn_valid(smfn)); - /* Pin the shadow and put it (back) on the list of pinned shadows */ - if ( sh_pin(d, smfn) == 0 ) - { - SHADOW_ERROR("can't pin %#lx as toplevel shadow\n", mfn_x(smfn)); - domain_crash(d); - } - /* Take a ref to this page: it will be released in sh_detach_old_tables() * or the next call to set_toplevel_shadow() */ if ( sh_get_ref(d, smfn, 0) ) + { + /* Pin the shadow and put it (back) on the list of pinned shadows */ + sh_pin(d, smfn); + new_entry = pagetable_from_mfn(smfn); + } else { SHADOW_ERROR("can't install %#lx as toplevel shadow\n", mfn_x(smfn));