From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:42918) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ra96m-0002U5-RY for qemu-devel@nongnu.org; Mon, 12 Dec 2011 11:52:12 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Ra96g-0003d2-TB for qemu-devel@nongnu.org; Mon, 12 Dec 2011 11:52:08 -0500 Received: from goliath.siemens.de ([192.35.17.28]:27211) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ra96g-0003cp-JN for qemu-devel@nongnu.org; Mon, 12 Dec 2011 11:52:02 -0500 Message-ID: <4EE63123.101@siemens.com> Date: Mon, 12 Dec 2011 17:51:47 +0100 From: Jan Kiszka MIME-Version: 1.0 References: <20111212163725.GA32142@amt.cnet> In-Reply-To: <20111212163725.GA32142@amt.cnet> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v4 00/15] uq/master: Introduce basic irqchip support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Marcelo Tosatti Cc: Anthony Liguori , Lai Jiangshan , "kvm@vger.kernel.org" , "Michael S. Tsirkin" , qemu-devel , Blue Swirl , Avi Kivity On 2011-12-12 17:37, Marcelo Tosatti wrote: > On Thu, Dec 08, 2011 at 12:52:19PM +0100, Jan Kiszka wrote: >> Changes in v4: >> - rebased of current uq/master >> - fixed stupid bugs that broke bisectability and user space irqchip mode >> - integrated NMI-over-LINT1 injection logic >> >> CC: Lai Jiangshan >> >> Jan Kiszka (15): >> msi: Generalize msix_supported to msi_supported >> kvm: Move kvmclock into hw/kvm folder >> apic: Stop timer on reset >> apic: Inject external NMI events via LINT1 >> apic: Introduce backend/frontend infrastructure for KVM reuse >> apic: Open-code timer save/restore >> i8259: Introduce backend/frontend infrastructure for KVM reuse >> ioapic: Introduce backend/frontend infrastructure for KVM reuse >> memory: Introduce memory_region_init_reservation >> kvm: Introduce core services for in-kernel irqchip support >> kvm: x86: Establish IRQ0 override control >> kvm: x86: Add user space part for in-kernel APIC >> kvm: x86: Add user space part for in-kernel i8259 >> kvm: x86: Add user space part for in-kernel IOAPIC >> kvm: Arm in-kernel irqchip support > > Looks good to me. > > Any thoughts on the qemu-kvm merge plan? Sounds painful. Pain will be where the existing qemu-kvm extensions collide with these refactored upstream devices (backend/frontend split specifically). That's where we have to merge very carefully. Haven't tried this yet, will give it a spin tomorrow or so. >>From that point on, disabling the new stuff for now and at some point switching over from the old one should be simple again. BTW, PIT+HPET+speaker will cause similar issues for the same reasons. Jan -- Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux