From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:57449) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZFfqO-0003ta-UO for qemu-devel@nongnu.org; Thu, 16 Jul 2015 05:52:45 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZFfqL-00035G-Nl for qemu-devel@nongnu.org; Thu, 16 Jul 2015 05:52:44 -0400 Received: from mx1.redhat.com ([209.132.183.28]:56888) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZFfqL-00035B-II for qemu-devel@nongnu.org; Thu, 16 Jul 2015 05:52:41 -0400 Date: Thu, 16 Jul 2015 11:52:37 +0200 From: Igor Mammedov Message-ID: <20150716115237.6415d01e@nial.brq.redhat.com> In-Reply-To: <55A71AD5.5030301@cn.fujitsu.com> References: <55A71AD5.5030301@cn.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v9 0/4] remove icc bus/bridge List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Zhu Guihua Cc: ehabkost@redhat.com, qemu-devel@nongnu.org, izumi.taku@jp.fujitsu.com, chen.fan.fnst@cn.fujitsu.com, pbonzini@redhat.com, afaerber@suse.de On Thu, 16 Jul 2015 10:45:41 +0800 Zhu Guihua wrote: > ping... I'll look at it once 2.4 is released. > > On 07/03/2015 05:38 PM, Zhu Guihua wrote: > > ICC Bus was used for providing a hotpluggable bus for APIC and CPU, > > but now we use HotplugHandler to make hotplug. So ICC Bus is > > unnecessary. > > > > This code has passed the new pc-cpu-test. > > And I have tested with kvm along with kernel_irqchip=on/off, > > it works fine. > > > > This patch series is based on the latest master. > > > > v9: > > -use a callback to correct reset sequence for x86 > > -update apic mmio mapping > > > > v8: > > -add a wrapper to specify reset order > > > > v7: > > -update to register reset handler for main_system_bus when created > > -register reset handler for apic after all devices are initialized > > > > Chen Fan (2): > > apic: map APIC's MMIO region at each CPU's address space > > cpu/apic: drop icc bus/bridge > > > > Zhu Guihua (2): > > x86: use new method to correct reset sequence > > icc_bus: drop the unused files > > > > default-configs/i386-softmmu.mak | 1 - > > default-configs/x86_64-softmmu.mak | 1 - > > hw/cpu/Makefile.objs | 1 - > > hw/cpu/icc_bus.c | 118 ------------------------------------- > > hw/i386/pc.c | 43 +++++++------- > > hw/i386/pc_piix.c | 9 +-- > > hw/i386/pc_q35.c | 9 +-- > > hw/intc/apic_common.c | 11 +--- > > include/hw/cpu/icc_bus.h | 82 -------------------------- > > include/hw/i386/apic_internal.h | 7 ++- > > include/hw/i386/pc.h | 2 +- > > target-i386/cpu.c | 30 +++++++--- > > 12 files changed, 52 insertions(+), 262 deletions(-) > > delete mode 100644 hw/cpu/icc_bus.c > > delete mode 100644 include/hw/cpu/icc_bus.h > > > >