Re: [PATCH v1 5/5] hv_balloon: try to merge system ram resources
From: Wei Liu <wei.liu@kernel.org>
Date: 2020-09-04 18:10:30
Also in:
linux-mm, lkml, xen-devel
From: Wei Liu <wei.liu@kernel.org>
Date: 2020-09-04 18:10:30
Also in:
linux-mm, lkml, xen-devel
On Fri, Aug 21, 2020 at 12:34:31PM +0200, David Hildenbrand wrote:
Let's use the new mechanism to merge system ram resources below the root. We are the only one hotplugging system ram, e.g., DIMMs don't apply, so this is safe to be used. Cc: Andrew Morton <akpm@linux-foundation.org> Cc: Michal Hocko <mhocko@suse.com> Cc: "K. Y. Srinivasan" <kys@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Cc: Stephen Hemminger <redacted> Cc: Wei Liu <wei.liu@kernel.org> Cc: Pankaj Gupta <pankaj.gupta.linux@gmail.com> Cc: Baoquan He <bhe@redhat.com> Cc: Wei Yang <redacted> Signed-off-by: David Hildenbrand <redacted> --- drivers/hv/hv_balloon.c | 3 +++ 1 file changed, 3 insertions(+)diff --git a/drivers/hv/hv_balloon.c b/drivers/hv/hv_balloon.c index 32e3bc0aa665a..49a6305f0fb73 100644 --- a/drivers/hv/hv_balloon.c +++ b/drivers/hv/hv_balloon.c@@ -745,6 +745,9 @@ static void hv_mem_hot_add(unsigned long start, unsigned long size, has->covered_end_pfn -= processed_pfn; spin_unlock_irqrestore(&dm_device.ha_lock, flags); break; + } else { + /* Try to reduce the number of system ram resources. */ + merge_system_ram_resources(&iomem_resource); }
You don't need to put the call under the "else" branch. It will have broken out of the loop if ret is not zero. Wei.
/* -- 2.26.2