[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] smbios: Don't report unknown CPU speed (fix
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] smbios: Don't report unknown CPU speed (fix SVVP regression) |
Date: |
Wed, 07 Jan 2015 18:46:09 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 |
On 07/01/2015 18:36, Eduardo Habkost wrote:
> SVVP requires processor speed on Type 4 structures to not be unknown.
> This was fixed in SeaBIOS 0.5.0 (in 2009), but the bug was reintroduced
> in QEMU 2.1.
>
> Revert to old behavior and report CPU speed as 2000 MHz instead of
> unknown.
>
> Signed-off-by: Eduardo Habkost <address@hidden>
> ---
> hw/i386/smbios.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/hw/i386/smbios.c b/hw/i386/smbios.c
> index ae7032a..12d2137 100644
> --- a/hw/i386/smbios.c
> +++ b/hw/i386/smbios.c
> @@ -618,8 +618,9 @@ static void smbios_build_type_4_table(unsigned instance)
> SMBIOS_TABLE_SET_STR(4, processor_version_str, type4.version);
> t->voltage = 0;
> t->external_clock = cpu_to_le16(0); /* Unknown */
> - t->max_speed = cpu_to_le16(0); /* Unknown */
> - t->current_speed = cpu_to_le16(0); /* Unknown */
> + /* SVVP requires max_speed and current_speed to not be unknown. */
> + t->max_speed = cpu_to_le16(2000); /* 2000 MHz */
> + t->current_speed = cpu_to_le16(2000); /* 2000 MHz */
> t->status = 0x41; /* Socket populated, CPU enabled */
> t->processor_upgrade = 0x01; /* Other */
> t->l1_cache_handle = cpu_to_le16(0xFFFF); /* N/A */
>
Reviewed-by: Paolo Bonzini <address@hidden>