diff options
| author | Eduardo Habkost <ehabkost@redhat.com> | 2019-06-15 17:05:05 -0300 |
|---|---|---|
| committer | Eduardo Habkost <ehabkost@redhat.com> | 2019-07-05 17:08:03 -0300 |
| commit | 4f2beda4539a0e6665fca52c2bc37cca7fc792a3 (patch) | |
| tree | b6e31a13cf738b96c77e32d48e6da924e146d0a0 | |
| parent | fea306520ea4b2f189dd23c70a6afd2fc4ffafdc (diff) | |
| download | qemu-4f2beda4539a0e6665fca52c2bc37cca7fc792a3.zip qemu-4f2beda4539a0e6665fca52c2bc37cca7fc792a3.tar.gz qemu-4f2beda4539a0e6665fca52c2bc37cca7fc792a3.tar.bz2 | |
i386: Fix signedness of hyperv_spinlock_attempts
The current default value for hv-spinlocks is 0xFFFFFFFF (meaning
"never retry"). However, the value is stored as a signed
integer, making the getter of the hv-spinlocks QOM property
return -1 instead of 0xFFFFFFFF.
Fix this by changing the type of X86CPU::hyperv_spinlock_attempts
to uint32_t. This has no visible effect to guest operating
systems, affecting just the behavior of the QOM getter.
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Message-Id: <20190615200505.31348-1-ehabkost@redhat.com>
Reviewed-by: Vitaly Kuznetsov <vkuznets@redhat.com>
Reviewed-by: Roman Kagan <rkagan@virtuozzo.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
| -rw-r--r-- | target/i386/cpu.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/target/i386/cpu.h b/target/i386/cpu.h index 4d2ae23..ff26351 100644 --- a/target/i386/cpu.h +++ b/target/i386/cpu.h @@ -1408,7 +1408,7 @@ struct X86CPU { CPUNegativeOffsetState neg; CPUX86State env; - int hyperv_spinlock_attempts; + uint32_t hyperv_spinlock_attempts; char *hyperv_vendor_id; bool hyperv_synic_kvm_only; uint64_t hyperv_features; |