]> xenbits.xensource.com Git - qemu-xen.git/commit
kvm: Take into account the unaligned section size when preparing bitmap
authorZenghui Yu <yuzenghui@huawei.com>
Tue, 8 Dec 2020 11:40:13 +0000 (19:40 +0800)
committerPaolo Bonzini <pbonzini@redhat.com>
Tue, 15 Dec 2020 17:52:05 +0000 (12:52 -0500)
commit4054adbdd2f66d78f73cdce0092eb5513b3b3521
treed37d9e0599b9990f6afaec4afc047464db4a0996
parentcfd4e36352d4426221aa94da44a172da1aaa741b
kvm: Take into account the unaligned section size when preparing bitmap

The kernel KVM_CLEAR_DIRTY_LOG interface has align requirement on both the
start and the size of the given range of pages. We have been careful to
handle the unaligned cases when performing CLEAR on one slot. But it seems
that we forget to take the unaligned *size* case into account when
preparing bitmap for the interface, and we may end up clearing dirty status
for pages outside of [start, start + size).

If the size is unaligned, let's go through the slow path to manipulate a
temp bitmap for the interface so that we won't bother with those unaligned
bits at the end of bitmap.

I don't think this can happen in practice since the upper layer would
provide us with the alignment guarantee. I'm not sure if kvm-all could rely
on it. And this patch is mainly intended to address correctness of the
specific algorithm used inside kvm_log_clear_one_slot().

Signed-off-by: Zenghui Yu <yuzenghui@huawei.com>
Message-Id: <20201208114013.875-1-yuzenghui@huawei.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
accel/kvm/kvm-all.c