From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:56796) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SpaOC-00058i-2D for qemu-devel@nongnu.org; Fri, 13 Jul 2012 03:34:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SpaO6-0006wU-83 for qemu-devel@nongnu.org; Fri, 13 Jul 2012 03:34:12 -0400 Received: from mx1.redhat.com ([209.132.183.28]:58094) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SpaO5-0006wO-VD for qemu-devel@nongnu.org; Fri, 13 Jul 2012 03:34:06 -0400 Message-ID: <4FFFCF66.3060205@redhat.com> Date: Fri, 13 Jul 2012 09:33:58 +0200 From: Igor Mammedov MIME-Version: 1.0 References: In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v2] MP initialization protocol differs between cpu families, and for P6 and onward models it is up to CPU to decide if it will be BSP using this protocol, so try to model this. However there is no point in implementing MP init List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: aliguori@us.ibm.com, ehabkost@redhat.com, gleb@redhat.com, jan.kiszka@siemens.com, mtosatti@redhat.com, qemu-devel@nongnu.org, mdroth@linux.vnet.ibm.com, blauwirbel@gmail.com, avi@redhat.com, pbonzini@redhat.com, afaerber@suse.de On 07/12/2012 03:39 PM, Peter Maydell wrote: > On 12 July 2012 14:22, Igor Mammedov wrote: >> This patch: >> - moves decision to designate BSP from board into cpu, making cpu >> self-sufficient in this regard. Later it will allow to cleanup hw/pc.c >> and remove cpu_reset and wrappers from there. >> - stores flag that CPU is BSP in IA32_APIC_BASE to model behavior >> described in Inted SDM vol 3a part 1 chapter 8.4.1 >> - uses MSR_IA32_APICBASE_BSP flag in apic_base for checking if cpu i= s BSP >> >> patch is based on Jan Kiszka's proposal: >> http://thread.gmane.org/gmane.comp.emulators.qemu/100806 > > This patch has a summary (subject) line that is way too long: > it should be short enough to fit reasonably on one line. > >> v2: >> - fix build for i386-linux-user >> spotted-by: Peter Maydell >> v3: >> - style change requested by Andreas F=C3=A4rber >> >> v4: >> - reuse cpu_is_bsp() rather than open code check if apicbase has BS= P bit set >> requested by Gleb Natapov >> - hijacked Andreas' patch [1] to use X86CPU instead of CPUX86State = in >> cpu_is_bsp() >> >> 1) http://lists.gnu.org/archive/html/qemu-devel/2012-05/msg03185.h= tml > > It's usual to put between-patchver changes below the '---' line. > Thanks! I'll fix it in next respin. > -- PMM > --=20 ----- Igor