From: Jan Beulich Date: Mon, 23 Sep 2019 12:41:00 +0000 (+0200) Subject: x86/shadow: don't enable shadow mode with too small a shadow allocation (part 2) X-Git-Tag: archive/raspbian/4.11.3+24-g14b62ab3e5-1+rpi1^2~55^2~104 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=8bbb3e900bb65dbb6bedb628ffc22af6bd7665e7;p=xen.git x86/shadow: don't enable shadow mode with too small a shadow allocation (part 2) Commit 2634b997af ("x86/shadow: don't enable shadow mode with too small a shadow allocation") was incomplete: The adjustment done there to shadow_enable() is also needed in shadow_one_bit_enable(). The (new) problem report was (apparently) a failed PV guest migration followed by another migration attempt for that same guest. Disabling log-dirty mode after the first one had left a couple of shadow pages allocated (perhaps something that also wants fixing), and hence the second enabling of log-dirty mode wouldn't have allocated anything further. Reported-by: James Wang Signed-off-by: Jan Beulich Acked-by: Tim Deegan master commit: 8b25551baa3307af0aa1ef8f7f43403f01c2c5d7 master date: 2019-09-05 09:56:42 +0200 --- diff --git a/xen/arch/x86/mm/shadow/common.c b/xen/arch/x86/mm/shadow/common.c index d2946affe2..34913c1ade 100644 --- a/xen/arch/x86/mm/shadow/common.c +++ b/xen/arch/x86/mm/shadow/common.c @@ -3506,7 +3506,8 @@ static int shadow_one_bit_enable(struct domain *d, u32 mode) mode |= PG_SH_enable; - if ( d->arch.paging.shadow.total_pages == 0 ) + if ( d->arch.paging.shadow.total_pages < + sh_min_allocation(d) + d->arch.paging.shadow.p2m_pages ) { /* Init the shadow memory allocation if the user hasn't done so */ if ( shadow_set_allocation(d, 1, NULL) != 0 )