Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752457AbbFZI4r (ORCPT ); Fri, 26 Jun 2015 04:56:47 -0400 Received: from smtp.citrix.com ([66.165.176.89]:12854 "EHLO SMTP.CITRIX.COM" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751546AbbFZI4f (ORCPT ); Fri, 26 Jun 2015 04:56:35 -0400 X-IronPort-AV: E=Sophos;i="5.13,683,1427760000"; d="scan'208";a="275857563" Message-ID: <558D13BF.9030907@citrix.com> Date: Fri, 26 Jun 2015 09:56:31 +0100 From: David Vrabel User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.4.0 MIME-Version: 1.0 To: Daniel Kiper , David Vrabel CC: , , Boris Ostrovsky , Subject: Re: [Xen-devel] [PATCHv1 6/8] xen/balloon: only hotplug additional memory if required References: <1435252263-31952-1-git-send-email-david.vrabel@citrix.com> <1435252263-31952-7-git-send-email-david.vrabel@citrix.com> <20150625211834.GO14050@olila.local.net-space.pl> In-Reply-To: <20150625211834.GO14050@olila.local.net-space.pl> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-DLP: MIA2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2279 Lines: 59 On 25/06/15 22:18, Daniel Kiper wrote: > On Thu, Jun 25, 2015 at 06:11:01PM +0100, David Vrabel wrote: >> Now that we track the total number of pages (included hotplugged >> regions), it is easy to determine if more memory needs to be >> hotplugged. >> >> Signed-off-by: David Vrabel >> --- >> drivers/xen/balloon.c | 16 +++++++++++++--- >> 1 file changed, 13 insertions(+), 3 deletions(-) >> >> diff --git a/drivers/xen/balloon.c b/drivers/xen/balloon.c >> index 960ac79..dd41da8 100644 >> --- a/drivers/xen/balloon.c >> +++ b/drivers/xen/balloon.c >> @@ -241,12 +241,22 @@ static void release_memory_resource(struct resource *resource) >> * bit set). Real size of added memory is established at page onlining stage. >> */ >> >> -static enum bp_state reserve_additional_memory(long credit) >> +static enum bp_state reserve_additional_memory(void) >> { >> + long credit; >> struct resource *resource; >> int nid, rc; >> unsigned long balloon_hotplug; >> >> + credit = balloon_stats.target_pages - balloon_stats.total_pages; >> + >> + /* >> + * Already hotplugged enough pages? Wait for them to be >> + * onlined. >> + */ > > Comment is wrong or at least misleading. Both values does not depend on onlining. If we get here and credit <=0 then the balloon is empty and we have already hotplugged enough sections to reach target. We need to wait for userspace to online the sections that already exist. >> + if (credit <= 0) >> + return BP_EAGAIN; > > Not BP_EAGAIN for sure. It should be BP_DONE but then balloon_process() will go > into loop until memory is onlined at least up to balloon_stats.target_pages. > BP_ECANCELED does work but it is misleading because it is not an error. So, maybe > we should introduce BP_STOP (or something like that) which works like BP_ECANCELED > and is not BP_ECANCELED. We don't want to spin while waiting for userspace to online a new section so BP_EAGAIN is correct here as it causes the balloon process to be rescheduled at a later time. David -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/