From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36345) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VHw7i-0003YF-Cr for qemu-devel@nongnu.org; Fri, 06 Sep 2013 09:31:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VHw7Z-0002tZ-8o for qemu-devel@nongnu.org; Fri, 06 Sep 2013 09:30:54 -0400 Received: from e06smtp17.uk.ibm.com ([195.75.94.113]:44913) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VHw7Y-0002tV-WD for qemu-devel@nongnu.org; Fri, 06 Sep 2013 09:30:45 -0400 Received: from /spool/local by e06smtp17.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Fri, 6 Sep 2013 14:25:27 +0100 Received: from b06cxnps4074.portsmouth.uk.ibm.com (d06relay11.portsmouth.uk.ibm.com [9.149.109.196]) by d06dlp02.portsmouth.uk.ibm.com (Postfix) with ESMTP id 48228219005C for ; Fri, 6 Sep 2013 14:30:40 +0100 (BST) Received: from d06av09.portsmouth.uk.ibm.com (d06av09.portsmouth.uk.ibm.com [9.149.37.250]) by b06cxnps4074.portsmouth.uk.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id r86DUSD230343288 for ; Fri, 6 Sep 2013 13:30:28 GMT Received: from d06av09.portsmouth.uk.ibm.com (localhost [127.0.0.1]) by d06av09.portsmouth.uk.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id r86DUdXm029455 for ; Fri, 6 Sep 2013 07:30:39 -0600 Message-ID: <5229D8FE.90807@de.ibm.com> Date: Fri, 06 Sep 2013 15:30:38 +0200 From: Christian Borntraeger MIME-Version: 1.0 References: <1378469957-62290-1-git-send-email-jfrei@linux.vnet.ibm.com> In-Reply-To: <1378469957-62290-1-git-send-email-jfrei@linux.vnet.ibm.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/2] KVM: s390: add floating irq controller List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gleb Natapov Cc: Peter Maydell , Thomas Huth , kvm@vger.kernel.org, Alexander Graf , qemu-devel@nongnu.org, Jens Freimann , Cornelia Huck , Andreas Faerber , Christoffer Dall On 06/09/13 14:19, Jens Freimann wrote:> This series adds a kvm_device that acts as a irq controller for floating > interrupts. As a first step it implements functionality to retrieve and inject > interrupts for the purpose of migration and for hardening the reset code by > allowing user space to explicitly remove all pending floating interrupts. > > PFAULT patches will also use this device for enabling/disabling pfault, therefore > the pfault patch series will be reworked to use this device. > > * Patch 1/2 adds a new data structure to hold interrupt information. The current > one (struct kvm_s390_interrupt) does not allow to inject every kind of interrupt, > e.g. some data for program interrupts and machine check interruptions were > missing. > > * Patch 2/2 adds a kvm_device which supports getting/setting currently pending > floating interrupts as well as deleting all currently pending interrupts > > > Jens Freimann (2): > KVM: s390: add and extend interrupt information data structs > KVM: s390: add floating irq controller > > Documentation/virtual/kvm/devices/s390_flic.txt | 36 +++ > arch/s390/include/asm/kvm_host.h | 35 +-- > arch/s390/include/uapi/asm/kvm.h | 5 + > arch/s390/kvm/interrupt.c | 304 ++++++++++++++++++++---- > arch/s390/kvm/kvm-s390.c | 1 + > include/linux/kvm_host.h | 1 + > include/uapi/linux/kvm.h | 65 +++++ > virt/kvm/kvm_main.c | 5 + > 8 files changed, 368 insertions(+), 84 deletions(-) > create mode 100644 Documentation/virtual/kvm/devices/s390_flic.txt > Gleb, Paolo, since the qemu part relies on a kernel header file, it makes sense to not only let the kernel part go via the kvm tree, but also the qemu part. I want Alex to Ack the interface, and if he agrees then I am fine with applying the whole series. If nothing else comes up, feel free to apply the small change request from Peter yourself or ask Jens for a resend. ------snip---- --- a/include/uapi/linux/kvm.h +++ b/include/uapi/linux/kvm.h @@ -908,7 +908,7 @@ struct kvm_device_attr { #define KVM_DEV_TYPE_FSL_MPIC_20 1 #define KVM_DEV_TYPE_FSL_MPIC_42 2 #define KVM_DEV_TYPE_XICS 3 -#define KVM_DEV_TYPE_FLIC 4 +#define KVM_DEV_TYPE_FLIC 5 /* * ioctls for VM fds ------snip----