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 | focaccia-qemu-4f2beda4539a0e6665fca52c2bc37cca7fc792a3.tar.gz focaccia-qemu-4f2beda4539a0e6665fca52c2bc37cca7fc792a3.zip | |
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 4d2ae2384e..ff26351538 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; |