From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: David Vrabel <david.vrabel@citrix.com>
Cc: xen-devel@lists.xenproject.org,
Boris Ostrovsky <boris.ostrovsky@oracle.com>,
Daniel Kiper <daniel.kiper@oracle.com>,
linux-kernel@vger.kernel.org, linux-mm@kvack.org
Subject: Re: [PATCHv2 05/10] xen/balloon: rationalize memory hotplug stats
Date: Fri, 24 Jul 2015 14:54:51 -0400 [thread overview]
Message-ID: <20150724185451.GC12824__38698.7709477127$1437764182$gmane$org@l.oracle.com> (raw)
In-Reply-To: <1437738468-24110-6-git-send-email-david.vrabel@citrix.com>
On Fri, Jul 24, 2015 at 12:47:43PM +0100, David Vrabel wrote:
> The stats used for memory hotplug make no sense and are fiddled with
> in odd ways. Remove them and introduce total_pages to track the total
> number of pages (both populated and unpopulated) including those within
> hotplugged regions (note that this includes not yet onlined pages).
>
> This will be used in the following commit when deciding whether
s/the following commit/"xen/balloon: only hotplug additional memory if required"
patch
> additional memory needs to be hotplugged.
>
> Signed-off-by: David Vrabel <david.vrabel@citrix.com>
> ---
> drivers/xen/balloon.c | 75 +++++++++------------------------------------------
> include/xen/balloon.h | 5 +---
> 2 files changed, 13 insertions(+), 67 deletions(-)
>
> diff --git a/drivers/xen/balloon.c b/drivers/xen/balloon.c
> index 29aeb8f..b5037b1 100644
> --- a/drivers/xen/balloon.c
> +++ b/drivers/xen/balloon.c
> @@ -194,21 +194,6 @@ static enum bp_state update_schedule(enum bp_state state)
> }
>
> #ifdef CONFIG_XEN_BALLOON_MEMORY_HOTPLUG
> -static long current_credit(void)
> -{
> - return balloon_stats.target_pages - balloon_stats.current_pages -
> - balloon_stats.hotplug_pages;
> -}
> -
> -static bool balloon_is_inflated(void)
> -{
> - if (balloon_stats.balloon_low || balloon_stats.balloon_high ||
> - balloon_stats.balloon_hotplug)
> - return true;
> - else
> - return false;
> -}
> -
> static struct resource *additional_memory_resource(phys_addr_t size)
> {
> struct resource *res;
> @@ -300,10 +285,7 @@ static enum bp_state reserve_additional_memory(long credit)
> goto err;
> }
>
> - balloon_hotplug -= credit;
> -
> - balloon_stats.hotplug_pages += credit;
> - balloon_stats.balloon_hotplug = balloon_hotplug;
> + balloon_stats.total_pages += balloon_hotplug;
>
> return BP_DONE;
> err:
> @@ -319,11 +301,6 @@ static void xen_online_page(struct page *page)
>
> __balloon_append(page);
>
> - if (balloon_stats.hotplug_pages)
> - --balloon_stats.hotplug_pages;
> - else
> - --balloon_stats.balloon_hotplug;
> -
> mutex_unlock(&balloon_mutex);
> }
>
> @@ -340,32 +317,22 @@ static struct notifier_block xen_memory_nb = {
> .priority = 0
> };
> #else
> -static long current_credit(void)
> +static enum bp_state reserve_additional_memory(long credit)
> {
> - unsigned long target = balloon_stats.target_pages;
> -
> - target = min(target,
> - balloon_stats.current_pages +
> - balloon_stats.balloon_low +
> - balloon_stats.balloon_high);
> -
> - return target - balloon_stats.current_pages;
> + balloon_stats.target_pages = balloon_stats.current_pages;
> + return BP_DONE;
> }
> +#endif /* CONFIG_XEN_BALLOON_MEMORY_HOTPLUG */
>
> -static bool balloon_is_inflated(void)
> +static long current_credit(void)
> {
> - if (balloon_stats.balloon_low || balloon_stats.balloon_high)
> - return true;
> - else
> - return false;
> + return balloon_stats.target_pages - balloon_stats.current_pages;
> }
>
> -static enum bp_state reserve_additional_memory(long credit)
> +static bool balloon_is_inflated(void)
> {
> - balloon_stats.target_pages = balloon_stats.current_pages;
> - return BP_DONE;
> + return balloon_stats.balloon_low || balloon_stats.balloon_high;
> }
> -#endif /* CONFIG_XEN_BALLOON_MEMORY_HOTPLUG */
>
> static enum bp_state increase_reservation(unsigned long nr_pages)
> {
> @@ -378,15 +345,6 @@ static enum bp_state increase_reservation(unsigned long nr_pages)
> .domid = DOMID_SELF
> };
>
> -#ifdef CONFIG_XEN_BALLOON_MEMORY_HOTPLUG
> - if (!balloon_stats.balloon_low && !balloon_stats.balloon_high) {
> - nr_pages = min(nr_pages, balloon_stats.balloon_hotplug);
> - balloon_stats.hotplug_pages += nr_pages;
> - balloon_stats.balloon_hotplug -= nr_pages;
> - return BP_DONE;
> - }
> -#endif
> -
> if (nr_pages > ARRAY_SIZE(frame_list))
> nr_pages = ARRAY_SIZE(frame_list);
>
> @@ -449,15 +407,6 @@ static enum bp_state decrease_reservation(unsigned long nr_pages, gfp_t gfp)
> .domid = DOMID_SELF
> };
>
> -#ifdef CONFIG_XEN_BALLOON_MEMORY_HOTPLUG
> - if (balloon_stats.hotplug_pages) {
> - nr_pages = min(nr_pages, balloon_stats.hotplug_pages);
> - balloon_stats.hotplug_pages -= nr_pages;
> - balloon_stats.balloon_hotplug += nr_pages;
> - return BP_DONE;
> - }
> -#endif
> -
> if (nr_pages > ARRAY_SIZE(frame_list))
> nr_pages = ARRAY_SIZE(frame_list);
>
> @@ -647,6 +596,8 @@ static void __init balloon_add_region(unsigned long start_pfn,
> don't subtract from it. */
> __balloon_append(page);
> }
> +
> + balloon_stats.total_pages += extra_pfn_end - start_pfn;
> }
>
> static int __init balloon_init(void)
> @@ -664,6 +615,7 @@ static int __init balloon_init(void)
> balloon_stats.target_pages = balloon_stats.current_pages;
> balloon_stats.balloon_low = 0;
> balloon_stats.balloon_high = 0;
> + balloon_stats.total_pages = balloon_stats.current_pages;
>
> balloon_stats.schedule_delay = 1;
> balloon_stats.max_schedule_delay = 32;
> @@ -671,9 +623,6 @@ static int __init balloon_init(void)
> balloon_stats.max_retry_count = RETRY_UNLIMITED;
>
> #ifdef CONFIG_XEN_BALLOON_MEMORY_HOTPLUG
> - balloon_stats.hotplug_pages = 0;
> - balloon_stats.balloon_hotplug = 0;
> -
> set_online_page_callback(&xen_online_page);
> register_memory_notifier(&xen_memory_nb);
> #endif
> diff --git a/include/xen/balloon.h b/include/xen/balloon.h
> index cc2e1a7..c8aee7a 100644
> --- a/include/xen/balloon.h
> +++ b/include/xen/balloon.h
> @@ -11,14 +11,11 @@ struct balloon_stats {
> /* Number of pages in high- and low-memory balloons. */
> unsigned long balloon_low;
> unsigned long balloon_high;
> + unsigned long total_pages;
> unsigned long schedule_delay;
> unsigned long max_schedule_delay;
> unsigned long retry_count;
> unsigned long max_retry_count;
> -#ifdef CONFIG_XEN_BALLOON_MEMORY_HOTPLUG
> - unsigned long hotplug_pages;
> - unsigned long balloon_hotplug;
> -#endif
> };
>
> extern struct balloon_stats balloon_stats;
> --
> 2.1.4
>
next prev parent reply other threads:[~2015-07-24 18:54 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1437738468-24110-1-git-send-email-david.vrabel@citrix.com>
2015-07-24 11:47 ` [PATCHv2 01/10] mm: memory hotplug with an existing resource David Vrabel
2015-07-24 11:47 ` [PATCHv2 02/10] xen/balloon: remove scratch page left overs David Vrabel
2015-07-24 11:47 ` [PATCHv2 03/10] x86/xen: discard RAM regions above the maximum reservation David Vrabel
2015-07-24 11:47 ` [PATCHv2 04/10] xen/balloon: find non-conflicting regions to place hotplugged memory David Vrabel
2015-07-24 11:47 ` [PATCHv2 05/10] xen/balloon: rationalize memory hotplug stats David Vrabel
2015-07-24 11:47 ` [PATCHv2 06/10] xen/balloon: only hotplug additional memory if required David Vrabel
2015-07-24 11:47 ` [PATCHv2 07/10] xen/balloon: make alloc_xenballoon_pages() always allocate low pages David Vrabel
2015-07-24 11:47 ` [PATCHv2 08/10] xen/balloon: use hotplugged pages for foreign mappings etc David Vrabel
2015-07-24 11:47 ` [PATCHv2 09/10] x86/xen: export xen_alloc_p2m_entry() David Vrabel
2015-07-24 11:47 ` [PATCHv2 10/10] xen/balloon: pre-allocate p2m entries for ballooned pages David Vrabel
[not found] ` <1437738468-24110-10-git-send-email-david.vrabel@citrix.com>
2015-07-24 18:51 ` [PATCHv2 09/10] x86/xen: export xen_alloc_p2m_entry() Konrad Rzeszutek Wilk
2015-07-29 16:10 ` Daniel Kiper
[not found] ` <1437738468-24110-6-git-send-email-david.vrabel@citrix.com>
2015-07-24 18:54 ` Konrad Rzeszutek Wilk [this message]
2015-07-28 15:22 ` [PATCHv2 05/10] xen/balloon: rationalize memory hotplug stats Daniel Kiper
[not found] ` <1437738468-24110-9-git-send-email-david.vrabel@citrix.com>
2015-07-24 18:55 ` [PATCHv2 08/10] xen/balloon: use hotplugged pages for foreign mappings etc Konrad Rzeszutek Wilk
[not found] ` <20150724185545.GD12824@l.oracle.com>
2015-07-27 10:18 ` David Vrabel
[not found] ` <1437738468-24110-11-git-send-email-david.vrabel@citrix.com>
2015-07-24 23:21 ` [PATCHv2 10/10] xen/balloon: pre-allocate p2m entries for ballooned pages Julien Grall
[not found] ` <55B2C882.8050903@citrix.com>
2015-07-27 9:30 ` David Vrabel
[not found] ` <55B5FA39.8000401@citrix.com>
2015-07-27 11:01 ` Julien Grall
[not found] ` <55B60F6E.3040901@citrix.com>
2015-07-27 12:40 ` David Vrabel
2015-07-27 19:37 ` [PATCHv2 00/10] mm, xen/balloon: memory hotplug improvements Daniel Kiper
[not found] ` <1437738468-24110-2-git-send-email-david.vrabel@citrix.com>
2015-07-28 14:54 ` [PATCHv2 01/10] mm: memory hotplug with an existing resource Daniel Kiper
[not found] ` <1437738468-24110-7-git-send-email-david.vrabel@citrix.com>
2015-07-29 15:55 ` [PATCHv2 06/10] xen/balloon: only hotplug additional memory if required Daniel Kiper
[not found] ` <20150729155535.GL3492@olila.local.net-space.pl>
2015-07-29 16:02 ` David Vrabel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='20150724185451.GC12824__38698.7709477127$1437764182$gmane$org@l.oracle.com' \
--to=konrad.wilk@oracle.com \
--cc=boris.ostrovsky@oracle.com \
--cc=daniel.kiper@oracle.com \
--cc=david.vrabel@citrix.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).