diff options
author | Thomas Huth | 2021-01-29 09:43:54 +0100 |
---|---|---|
committer | Paolo Bonzini | 2021-02-08 14:43:55 +0100 |
commit | 38e0b7904eca7cd32f8953c33701e1f226ecc3fe (patch) | |
tree | d5e411627ca6dbaa92e87f2cd7c3d0bcdc84e49f /accel/tcg | |
parent | qapi/meson: Restrict UI module to system emulation and tools (diff) | |
download | qemu-38e0b7904eca7cd32f8953c33701e1f226ecc3fe.tar.gz qemu-38e0b7904eca7cd32f8953c33701e1f226ecc3fe.tar.xz qemu-38e0b7904eca7cd32f8953c33701e1f226ecc3fe.zip |
accel/kvm/kvm-all: Fix wrong return code handling in dirty log code
The kvm_vm_ioctl() wrapper already returns -errno if the ioctl itself
returned -1, so the callers of kvm_vm_ioctl() should not check for -1
but for a value < 0 instead.
This problem has been fixed once already in commit b533f658a98325d0e4
but that commit missed that the ENOENT error code is not fatal for
this ioctl, so the commit has been reverted in commit 50212d6346f33d6e
since the problem occurred close to a pending release at that point
in time. The plan was to fix it properly after the release, but it
seems like this has been forgotten. So let's do it now finally instead.
Resolves: https://bugs.launchpad.net/qemu/+bug/1294227
Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20210129084354.42928-1-thuth@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'accel/tcg')
0 files changed, 0 insertions, 0 deletions