[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 16/20] KVM: Simplify dirty log sync in kvm_set_phys_mem
From: |
Paolo Bonzini |
Subject: |
[PULL 16/20] KVM: Simplify dirty log sync in kvm_set_phys_mem |
Date: |
Mon, 17 May 2021 07:19:57 -0400 |
From: Peter Xu <peterx@redhat.com>
kvm_physical_sync_dirty_bitmap() on the whole section is inaccurate, because
the section can be a superset of the memslot that we're working on. The result
is that if the section covers multiple kvm memslots, we could be doing the
synchronization for multiple times for each kvmslot in the section.
With the two helpers that we just introduced, it's very easy to do it right now
by calling the helpers.
Signed-off-by: Peter Xu <peterx@redhat.com>
Message-Id: <20210506160549.130416-7-peterx@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
accel/kvm/kvm-all.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/accel/kvm/kvm-all.c b/accel/kvm/kvm-all.c
index 7031dd4250..0ba6d48120 100644
--- a/accel/kvm/kvm-all.c
+++ b/accel/kvm/kvm-all.c
@@ -1161,7 +1161,8 @@ static void kvm_set_phys_mem(KVMMemoryListener *kml,
goto out;
}
if (mem->flags & KVM_MEM_LOG_DIRTY_PAGES) {
- kvm_physical_sync_dirty_bitmap(kml, section);
+ kvm_slot_get_dirty_log(kvm_state, mem);
+ kvm_slot_sync_dirty_pages(mem);
}
/* unregister the slot */
--
2.27.0
- [PULL 01/20] configure: Only clone softfloat-3 repositories if TCG is enabled, (continued)
- [PULL 01/20] configure: Only clone softfloat-3 repositories if TCG is enabled, Paolo Bonzini, 2021/05/17
- [PULL 07/20] meson: bump submodule to 0.57.2, Paolo Bonzini, 2021/05/17
- [PULL 10/20] KVM: do not allow setting properties at runtime, Paolo Bonzini, 2021/05/17
- [PULL 09/20] qtest: add a QOM object for qtest, Paolo Bonzini, 2021/05/17
- [PULL 06/20] i386/cpu: Expose AVX_VNNI instruction to guest, Paolo Bonzini, 2021/05/17
- [PULL 11/20] memory: Introduce log_sync_global() to memory listener, Paolo Bonzini, 2021/05/17
- [PULL 12/20] KVM: Use a big lock to replace per-kml slots_lock, Paolo Bonzini, 2021/05/17
- [PULL 13/20] KVM: Create the KVMSlot dirty bitmap on flag changes, Paolo Bonzini, 2021/05/17
- [PULL 14/20] KVM: Provide helper to get kvm dirty log, Paolo Bonzini, 2021/05/17
- [PULL 15/20] KVM: Provide helper to sync dirty bitmap from slot to ramblock, Paolo Bonzini, 2021/05/17
- [PULL 16/20] KVM: Simplify dirty log sync in kvm_set_phys_mem,
Paolo Bonzini <=
- [PULL 18/20] KVM: Add dirty-ring-size property, Paolo Bonzini, 2021/05/17
- [PULL 17/20] KVM: Cache kvm slot dirty bitmap size, Paolo Bonzini, 2021/05/17
- [PULL 19/20] KVM: Disable manual dirty log when dirty ring enabled, Paolo Bonzini, 2021/05/17
- [PULL 20/20] KVM: Dirty ring support, Paolo Bonzini, 2021/05/17
- Re: [PULL 00/20] Misc patches for 2020-05-17, no-reply, 2021/05/17
- Re: [PULL 00/20] Misc patches for 2020-05-17, Peter Maydell, 2021/05/18