Am 30.09.2019 um 15:19 hat Christian Borntraeger geschrieben: > From: Paolo Bonzini <pbonzini@xxxxxxxxxx> > > Currently MemoryRegionSection has 1:1 mapping to KVMSlot. > However next patch will allow splitting MemoryRegionSection into > several KVMSlot-s, make sure that kvm_physical_log_slot_clear() > is able to handle such 1:N mapping. > > Signed-off-by: Paolo Bonzini <pbonzini@xxxxxxxxxx> > Signed-off-by: Igor Mammedov <imammedo@xxxxxxxxxx> > Reviewed-by: Peter Xu <peterx@xxxxxxxxxx> > Message-Id: <20190924144751.24149-3-imammedo@xxxxxxxxxx> > Acked-by: Paolo Bonzini <pbonzini@xxxxxxxxxx> > Signed-off-by: Christian Borntraeger <borntraeger@xxxxxxxxxx> This broke the build for me on F29: CC x86_64-softmmu/accel/kvm/kvm-all.o /tmp/qemu/accel/kvm/kvm-all.c: In function 'kvm_log_clear': /tmp/qemu/accel/kvm/kvm-all.c:1086:8: error: 'ret' may be used uninitialized in this function [-Werror=maybe-uninitialized] if (r < 0) { ^ cc1: all warnings being treated as errors Kevin