|
 |
9bac43 |
From 5643e64ad775e2a14c76faa867ad698c5e1fe9c7 Mon Sep 17 00:00:00 2001
|
|
|
9bac43 |
From: David Gibson <dgibson@redhat.com>
|
|
|
9bac43 |
Date: Thu, 16 Nov 2017 03:07:32 +0100
|
|
|
9bac43 |
Subject: [PATCH 28/30] memory: Share special empty FlatView
|
|
|
9bac43 |
|
|
|
9bac43 |
RH-Author: David Gibson <dgibson@redhat.com>
|
|
|
9bac43 |
Message-id: <20171116030732.8560-23-dgibson@redhat.com>
|
|
|
9bac43 |
Patchwork-id: 77711
|
|
|
9bac43 |
O-Subject: [PATCH 22/22] memory: Share special empty FlatView
|
|
|
9bac43 |
Bugzilla: 1481593
|
|
|
9bac43 |
RH-Acked-by: Thomas Huth <thuth@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: Laurent Vivier <lvivier@redhat.com>
|
|
|
9bac43 |
|
|
|
9bac43 |
From: Alexey Kardashevskiy <aik@ozlabs.ru>
|
|
|
9bac43 |
|
|
|
9bac43 |
This shares an cached empty FlatView among address spaces. The empty
|
|
|
9bac43 |
FV is used every time when a root MR renders into a FV without memory
|
|
|
9bac43 |
sections which happens when MR or its children are not enabled or
|
|
|
9bac43 |
zero-sized. The empty_view is not NULL to keep the rest of memory
|
|
|
9bac43 |
API intact; it also has a dispatch tree for the same reason.
|
|
|
9bac43 |
|
|
|
9bac43 |
On POWER8 with 255 CPUs, 255 virtio-net, 40 PCI bridges guest this halves
|
|
|
9bac43 |
the amount of FlatView's in use (557 -> 260) and dispatch tables
|
|
|
9bac43 |
(~800000 -> ~370000). In an unrelated experiment with 112 non-virtio
|
|
|
9bac43 |
devices on x86 ("-M pc"), only 4 FlatViews are alive, and about ~2000
|
|
|
9bac43 |
are created at startup.
|
|
|
9bac43 |
|
|
|
9bac43 |
Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
|
|
|
9bac43 |
Message-Id: <20170921085110.25598-16-aik@ozlabs.ru>
|
|
|
9bac43 |
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
9bac43 |
(cherry picked from commit 092aa2fc65b7a35121616aad8f39d47b8f921618)
|
|
|
9bac43 |
|
|
|
9bac43 |
Signed-off-by: David Gibson <dgibson@redhat.com>
|
|
|
9bac43 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9bac43 |
---
|
|
|
9bac43 |
memory.c | 18 ++++++++++++++++--
|
|
|
9bac43 |
1 file changed, 16 insertions(+), 2 deletions(-)
|
|
|
9bac43 |
|
|
|
9bac43 |
diff --git a/memory.c b/memory.c
|
|
|
9bac43 |
index 38eabe7..a1f1e68 100644
|
|
|
9bac43 |
--- a/memory.c
|
|
|
9bac43 |
+++ b/memory.c
|
|
|
9bac43 |
@@ -317,6 +317,7 @@ static void flatview_unref(FlatView *view)
|
|
|
9bac43 |
{
|
|
|
9bac43 |
if (atomic_fetch_dec(&view->ref) == 1) {
|
|
|
9bac43 |
trace_flatview_destroy_rcu(view, view->root);
|
|
|
9bac43 |
+ assert(view->root);
|
|
|
9bac43 |
call_rcu(view, flatview_destroy, rcu);
|
|
|
9bac43 |
}
|
|
|
9bac43 |
}
|
|
|
9bac43 |
@@ -760,16 +761,19 @@ static MemoryRegion *memory_region_get_flatview_root(MemoryRegion *mr)
|
|
|
9bac43 |
}
|
|
|
9bac43 |
}
|
|
|
9bac43 |
}
|
|
|
9bac43 |
+ if (found == 0) {
|
|
|
9bac43 |
+ return NULL;
|
|
|
9bac43 |
+ }
|
|
|
9bac43 |
if (next) {
|
|
|
9bac43 |
mr = next;
|
|
|
9bac43 |
continue;
|
|
|
9bac43 |
}
|
|
|
9bac43 |
}
|
|
|
9bac43 |
|
|
|
9bac43 |
- break;
|
|
|
9bac43 |
+ return mr;
|
|
|
9bac43 |
}
|
|
|
9bac43 |
|
|
|
9bac43 |
- return mr;
|
|
|
9bac43 |
+ return NULL;
|
|
|
9bac43 |
}
|
|
|
9bac43 |
|
|
|
9bac43 |
/* Render a memory topology into a list of disjoint absolute ranges. */
|
|
|
9bac43 |
@@ -965,12 +969,22 @@ static void address_space_update_topology_pass(AddressSpace *as,
|
|
|
9bac43 |
|
|
|
9bac43 |
static void flatviews_init(void)
|
|
|
9bac43 |
{
|
|
|
9bac43 |
+ static FlatView *empty_view;
|
|
|
9bac43 |
+
|
|
|
9bac43 |
if (flat_views) {
|
|
|
9bac43 |
return;
|
|
|
9bac43 |
}
|
|
|
9bac43 |
|
|
|
9bac43 |
flat_views = g_hash_table_new_full(g_direct_hash, g_direct_equal, NULL,
|
|
|
9bac43 |
(GDestroyNotify) flatview_unref);
|
|
|
9bac43 |
+ if (!empty_view) {
|
|
|
9bac43 |
+ empty_view = generate_memory_topology(NULL);
|
|
|
9bac43 |
+ /* We keep it alive forever in the global variable. */
|
|
|
9bac43 |
+ flatview_ref(empty_view);
|
|
|
9bac43 |
+ } else {
|
|
|
9bac43 |
+ g_hash_table_replace(flat_views, NULL, empty_view);
|
|
|
9bac43 |
+ flatview_ref(empty_view);
|
|
|
9bac43 |
+ }
|
|
|
9bac43 |
}
|
|
|
9bac43 |
|
|
|
9bac43 |
static void flatviews_reset(void)
|
|
|
9bac43 |
--
|
|
|
9bac43 |
1.8.3.1
|
|
|
9bac43 |
|