[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH qemu v5 04/18] memory: Move AddressSpaceDispatch
From: |
Alexey Kardashevskiy |
Subject: |
Re: [Qemu-devel] [PATCH qemu v5 04/18] memory: Move AddressSpaceDispatch from AddressSpace to FlatView |
Date: |
Thu, 21 Sep 2017 23:44:59 +1000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
On 21/09/17 21:51, Paolo Bonzini wrote:
> On 21/09/2017 10:50, Alexey Kardashevskiy wrote:
>> * since FlatView::rcu is used now to dispose FV, call_rcu() in
>> address_space_update_topology() is replaced with direct call to
>> flatview_unref()
>
> Hmm, this is not correct, as you could have
>
>
> thread 1 thread 2 RCU thread
> -------------------------------------------------------------
> rcu_read_lock
> read as->current_map
> set as->current_map
> flatview_unref
> '--> call_rcu
> flatview_ref
> rcu_read_unlock
> flatview_destroy
>
> I need to think a bit more about this (and possibly ask Paul...).
>
> Paolo
>
Nah, you're right, it should be like this:
diff --git a/memory.c b/memory.c
index 35b2fc5f7f..689bf53866 100644
--- a/memory.c
+++ b/memory.c
@@ -317,7 +317,7 @@ static void flatview_ref(FlatView *view)
static void flatview_unref(FlatView *view)
{
if (atomic_fetch_dec(&view->ref) == 1) {
- call_rcu(view, flatview_destroy, rcu);
+ flatview_destroy(view);
}
}
@@ -768,7 +768,7 @@ static FlatView *generate_memory_topology(MemoryRegion *mr)
flatview_simplify(view);
if (!view->nr) {
- flatview_destroy(view);
+ flatview_unref(view);
use_empty = true;
}
}
@@ -1026,7 +1026,7 @@ static void address_space_set_flatview(AddressSpace *as)
/* Writes are protected by the BQL. */
atomic_rcu_set(&as->current_map, new_view);
if (old_view) {
- flatview_unref(old_view);
+ call_rcu(view, flatview_unref, rcu);
}
--
Alexey
- [Qemu-devel] [PATCH qemu v5 00/18] memory: Store physical root MR in FlatView, Alexey Kardashevskiy, 2017/09/21
- [Qemu-devel] [PATCH qemu v5 07/18] memory: Cleanup after switching to FlatView, Alexey Kardashevskiy, 2017/09/21
- [Qemu-devel] [PATCH qemu v5 09/18] memory: Store physical root MR in FlatView, Alexey Kardashevskiy, 2017/09/21
- [Qemu-devel] [PATCH qemu v5 06/18] memory: Switch memory from using AddressSpace to FlatView, Alexey Kardashevskiy, 2017/09/21
- [Qemu-devel] [PATCH qemu v5 12/18] memory: Share FlatView's and dispatch trees between address spaces, Alexey Kardashevskiy, 2017/09/21
- [Qemu-devel] [PATCH qemu v5 08/18] memory: Rename mem_begin/mem_commit/mem_add helpers, Alexey Kardashevskiy, 2017/09/21
- [Qemu-devel] [PATCH qemu v5 11/18] memory: Move address_space_update_ioeventfds, Alexey Kardashevskiy, 2017/09/21
- [Qemu-devel] [PATCH qemu v5 13/18] memory: Do not allocate FlatView in address_space_init, Alexey Kardashevskiy, 2017/09/21
- [Qemu-devel] [PATCH qemu v5 10/18] memory: Alloc dispatch tree where topology is generared, Alexey Kardashevskiy, 2017/09/21
- [Qemu-devel] [PATCH qemu v5 18/18] memory: Avoid temporary FlatView allocation in a single child case, Alexey Kardashevskiy, 2017/09/21