summaryrefslogtreecommitdiffstats
path: root/hw/i386
diff options
context:
space:
mode:
authorGabriel L. Somlo2014-05-19 16:09:55 +0200
committerMichael S. Tsirkin2014-05-21 14:47:50 +0200
commit0d73394ad93aa12755316b3a90b3193aeeb95f90 (patch)
tree49638615f5506b68dad6b1a923bf13e2a5d2f257 /hw/i386
parentSMBIOS: Update Type 0 struct generator for machines >= 2.1 (diff)
downloadqemu-0d73394ad93aa12755316b3a90b3193aeeb95f90.tar.gz
qemu-0d73394ad93aa12755316b3a90b3193aeeb95f90.tar.xz
qemu-0d73394ad93aa12755316b3a90b3193aeeb95f90.zip
SMBIOS: Fix type 17 field sizes
Fields for configured_clock_speed and various voltage values introduced in spec v2.7+ should be "word", i.e. 16 bits. Reported-by: Laszlo Ersek <lersek@redhat.com> Signed-off-by: Gabriel Somlo <somlo@cmu.edu> Reviewed-by: Laszlo Ersek <lersek@redhat.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'hw/i386')
-rw-r--r--hw/i386/smbios.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/hw/i386/smbios.c b/hw/i386/smbios.c
index 17938215f6..b3bedde8b9 100644
--- a/hw/i386/smbios.c
+++ b/hw/i386/smbios.c
@@ -683,10 +683,10 @@ static void smbios_build_type_17_table(unsigned instance, ram_addr_t size)
SMBIOS_TABLE_SET_STR(17, asset_tag_number_str, type17.asset);
SMBIOS_TABLE_SET_STR(17, part_number_str, type17.part);
t->attributes = 0; /* Unknown */
- t->configured_clock_speed = cpu_to_le32(0); /* Unknown */
- t->minimum_voltage = cpu_to_le32(0); /* Unknown */
- t->maximum_voltage = cpu_to_le32(0); /* Unknown */
- t->configured_voltage = cpu_to_le32(0); /* Unknown */
+ t->configured_clock_speed = cpu_to_le16(0); /* Unknown */
+ t->minimum_voltage = cpu_to_le16(0); /* Unknown */
+ t->maximum_voltage = cpu_to_le16(0); /* Unknown */
+ t->configured_voltage = cpu_to_le16(0); /* Unknown */
SMBIOS_BUILD_TABLE_POST;
}