From mboxrd@z Thu Jan 1 00:00:00 1970 From: Juergen Gross Subject: Re: [PATCHv1] xen/balloon: disable memory hotplug in PV guests Date: Thu, 19 Mar 2015 14:38:25 +0100 Message-ID: <550AD151.20302@suse.com> References: <5500544B.4020302@citrix.com> <550061F0.9030206@oracle.com> <55066B88.9040300@suse.com> <20150316100344.GV27971@olila.local.net-space.pl> <5506B115.8090804@suse.com> <5509553A.7030707@citrix.com> <5509843B.2080205@suse.com> <550984DE.4010800@citrix.com> <20150318151424.GB27971@olila.local.net-space.pl> <550A9D19.9040800@suse.com> <20150319113453.GC27971@olila.local.net-space.pl> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="------------050401030607080108000905" Return-path: Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1YYaeZ-0002Wr-Uc for xen-devel@lists.xenproject.org; Thu, 19 Mar 2015 13:38:28 +0000 In-Reply-To: <20150319113453.GC27971@olila.local.net-space.pl> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: David Vrabel Cc: xen-devel@lists.xenproject.org, Daniel Kiper , Boris Ostrovsky List-Id: xen-devel@lists.xenproject.org This is a multi-part message in MIME format. --------------050401030607080108000905 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit On 03/19/2015 12:34 PM, Daniel Kiper wrote: > On Thu, Mar 19, 2015 at 10:55:37AM +0100, Juergen Gross wrote: >> On 03/18/2015 04:14 PM, Daniel Kiper wrote: > > [...] > >>> I have checked new p2m code and I think that this is reasonable solution too. >> >> Do I need any patches for xl to be able to test this? I did: >> >> xl mem-max 2 4096 >> xl mem-set 2 4096 > > Yep. I started work on it but I was not able to finish it. > Here you can find some old patches for this issue: > > http://lists.xen.org/archives/html/xen-devel/2013-04/msg03072.html > > Bob and I will work on it sooner or later (probably a bit later). > > Hmmm... Boris, David, how did you manage to get this memory hotplug > stuff working. Did you use xm instead of xl? Whoever can test this - what about the attached patch? David, you said: This one can be trivially fixed by setting the new sections in the p2m to INVALID_P2M_ENTRY before calling add_memory(). Are you preparing a patch for this, or do you want me to do it? Juergen --------------050401030607080108000905 Content-Type: text/x-patch; name="0001-xen-prepare-p2m-list-for-memory-hotplug.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename*0="0001-xen-prepare-p2m-list-for-memory-hotplug.patch" >>From a700c3439edb9774a069302d4393e235edc8c264 Mon Sep 17 00:00:00 2001 From: Juergen Gross Date: Thu, 19 Mar 2015 07:05:39 +0100 Subject: [PATCH] xen: prepare p2m list for memory hotplug Commit 054954eb051f35e74b75a566a96fe756015352c8 ("xen: switch to linear virtual mapped sparse p2m list") introduced a regression regarding to memory hotplug for a pv-domain: as the virtual space for the p2m list is allocated for the to be expected memory size of the domain only, hotplugged memory above that size will not be usable by the domain. Correct this by using a configurable size for the p2m list in case of memory hotplug enabled (default supported memory size is 512 GB for 64 bit domains and 4 GB for 32 bit domains). Signed-off-by: Juergen Gross --- arch/x86/xen/p2m.c | 13 ++++++++++++- drivers/xen/Kconfig | 13 +++++++++++++ 2 files changed, 25 insertions(+), 1 deletion(-) diff --git a/arch/x86/xen/p2m.c b/arch/x86/xen/p2m.c index 9f93af5..30e84ae 100644 --- a/arch/x86/xen/p2m.c +++ b/arch/x86/xen/p2m.c @@ -91,6 +91,17 @@ EXPORT_SYMBOL_GPL(xen_p2m_size); unsigned long xen_max_p2m_pfn __read_mostly; EXPORT_SYMBOL_GPL(xen_max_p2m_pfn); +#ifdef CONFIG_XEN_BALLOON_MEMORY_HOTPLUG_LIMIT +#ifdef CONFIG_X86_32 +BUILD_BUG_ON_MSG(CONFIG_XEN_BALLOON_MEMORY_HOTPLUG_LIMIT > 64) +#endif +#define P2M_LIMIT max(xen_max_p2m_pfn, \ + ((unsigned long)((u64)CONFIG_XEN_BALLOON_MEMORY_HOTPLUG_LIMIT * \ + 1024 * 1024 * 1024 / PAGE_SIZE))) +#else +#define P2M_LIMIT xen_max_p2m_pfn +#endif + static DEFINE_SPINLOCK(p2m_update_lock); static unsigned long *p2m_mid_missing_mfn; @@ -387,7 +398,7 @@ void __init xen_vmalloc_p2m_tree(void) static struct vm_struct vm; vm.flags = VM_ALLOC; - vm.size = ALIGN(sizeof(unsigned long) * xen_max_p2m_pfn, + vm.size = ALIGN(sizeof(unsigned long) * P2M_LIMIT, PMD_SIZE * PMDS_PER_MID_PAGE); vm_area_register_early(&vm, PMD_SIZE * PMDS_PER_MID_PAGE); pr_notice("p2m virtual area at %p, size is %lx\n", vm.addr, vm.size); diff --git a/drivers/xen/Kconfig b/drivers/xen/Kconfig index b812462..0a61ddf 100644 --- a/drivers/xen/Kconfig +++ b/drivers/xen/Kconfig @@ -55,6 +55,19 @@ config XEN_BALLOON_MEMORY_HOTPLUG In that case step 3 should be omitted. +config XEN_BALLOON_MEMORY_HOTPLUG_LIMIT + int + default 512 if X86_64 + default 4 if X86_32 + depends on XEN_HAVE_PVMMU + depends on XEN_BALLOON_MEMORY_HOTPLUG + help + Upper limit in GBs a pv domain can be expanded to using memory + hotplug. + + This value is used to allocate enough space in internal tables needed + for physical memory administration. + config XEN_SCRUB_PAGES bool "Scrub pages before returning them to system" depends on XEN_BALLOON -- 2.1.4 --------------050401030607080108000905 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel --------------050401030607080108000905--