summaryrefslogtreecommitdiffstats
path: root/drivers/xen/balloon.c
diff options
context:
space:
mode:
authorDan Magenheimer <dan.magenheimer@oracle.com>2009-01-22 14:36:08 -0800
committerIngo Molnar <mingo@elte.hu>2009-01-22 23:41:44 +0100
commit1058a75f07b9bb8323fb5197be5526220f8b75cf (patch)
tree3dcdc077c2c045043629b1999a40f8e1f13996d5 /drivers/xen/balloon.c
parent066941bd4eeb159307a5d7d795100d0887c00442 (diff)
downloadop-kernel-dev-1058a75f07b9bb8323fb5197be5526220f8b75cf.zip
op-kernel-dev-1058a75f07b9bb8323fb5197be5526220f8b75cf.tar.gz
xen: actually release memory when shrinking domain
Fix this: > It appears that in the upstream balloon driver, > the call to HYPERVISOR_update_va_mapping is missing > from decrease_reservation. I think as a result, > the balloon driver is eating memory but not > releasing it to Xen, thus rendering the balloon > driver essentially useless. (Can be observed via xentop.) Signed-off-by: Dan Magenheimer <dan.magenheimer@oracle.com> Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'drivers/xen/balloon.c')
-rw-r--r--drivers/xen/balloon.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/xen/balloon.c b/drivers/xen/balloon.c
index 8dc7109..8069d52 100644
--- a/drivers/xen/balloon.c
+++ b/drivers/xen/balloon.c
@@ -298,6 +298,11 @@ static int decrease_reservation(unsigned long nr_pages)
frame_list[i] = pfn_to_mfn(pfn);
scrub_page(page);
+
+ ret = HYPERVISOR_update_va_mapping(
+ (unsigned long)__va(pfn << PAGE_SHIFT),
+ __pte_ma(0), 0);
+ BUG_ON(ret);
}
/* Ensure that ballooned highmem pages don't have kmaps. */
OpenPOWER on IntegriCloud