From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:53212) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TNjFv-0005Lt-PI for qemu-devel@nongnu.org; Mon, 15 Oct 2012 07:54:48 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TNjFp-0006mL-Vu for qemu-devel@nongnu.org; Mon, 15 Oct 2012 07:54:47 -0400 Received: from mx1.redhat.com ([209.132.183.28]:14500) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TNjFp-0006mD-Nv for qemu-devel@nongnu.org; Mon, 15 Oct 2012 07:54:41 -0400 Date: Mon, 15 Oct 2012 13:54:34 +0200 From: Igor Mammedov Message-ID: <20121015135434.62d4965d@nial.usersys.redhat.com> In-Reply-To: <507A3B14.3050502@suse.de> References: <1350160539-22293-1-git-send-email-imammedo@redhat.com> <507A3B14.3050502@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v5] target-i386: initialize APIC at CPU level List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Andreas =?ISO-8859-1?B?RuRyYmVy?= Cc: jan.kiszka@siemens.com, aliguori@us.ibm.com, qemu-devel@nongnu.org, ehabkost@redhat.com On Sun, 14 Oct 2012 06:09:56 +0200 Andreas F=E4rber wrote: > Am 13.10.2012 22:35, schrieb Igor Mammedov: > > (L)APIC is a part of cpu [1] so move APIC initialization inside of > > x86_cpu object. Since cpu_model and override flags currently specify > > whether APIC should be created or not, APIC creation&initialization is > > moved into x86_cpu_apic_init() which is called from x86_cpu_realize(). > >=20 > > [1] - all x86 cpus have integrated APIC if we overlook existence of i48= 6, > > and it's more convenient to model after majority of them. > >=20 > > Signed-off-by: Igor Mammedov > > --- > > v5: fix *-user target build, smp_cpus is defined for softmmu only >=20 > I do not run into any build issue with v4. Is it a runtime issue? >=20 > Andreas >=20 I've run into this trying to build your latest CPUstate series with following configure options: './configure' '--enable-debug' '--target-list=3Dx86_64-softmmu,x86_64-li= nux-user' Anyway it's better not to build in any APIC checks in user target since it doesn't need it at all. I'm sorry for not noticing error earlier at v4 build time. Could you re-apply it, please?