[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 1/2] target-i386: move cpu halted decision into

From: Igor Mammedov
Subject: Re: [Qemu-devel] [PATCH 1/2] target-i386: move cpu halted decision into x86_cpu_reset
Date: Thu, 12 Jul 2012 15:09:30 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120615 Thunderbird/13.0.1

On 07/12/2012 08:38 AM, Gleb Natapov wrote:
On Tue, Jul 10, 2012 at 03:15:51PM +0200, Igor Mammedov wrote:
  /* pc.c */
-int cpu_is_bsp(CPUX86State *env);
Why remove it instead of modifying it to check BSP bit in apic base?
I think it will make the patch smaller and open code the check does not
look nice.

It's smaller than if I keep cpu_is_bsp() because keeping it would require
as minimum adapting Andreas' patch [1] and without cpu_is_bsp() there won't
be need in it as well.

plain check doesn't look horrible though. It's documented in patch
description and Intel's SDM also mentions APIC base and BSP bit in it. So
may be it's better to use it this way.

I've made a version that keeps cpu_is_bsp() with a bits from [1].
I'll send it as followup to this email, please see if that way is any better.

1) [PATCH qom-next 06/59] pc: Pass X86CPU to cpu_is_bsp()

reply via email to

[Prev in Thread] Current Thread [Next in Thread]