From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH v3 1/8] ARM: KVM: Initial skeleton to compile KVM support Date: Sun, 05 Jun 2011 18:10:24 +0300 Message-ID: <4DEB9C60.1030001@redhat.com> References: <20110603150318.17011.82777.stgit@ubuntu> <4DE8FE38.6030405@siemens.com> <4DE90397.5080801@siemens.com> <4DEB74DA.2040909@redhat.com> <4DEB8F22.1020802@web.de> <4DEB9033.9060109@redhat.com> <4DEB998E.1010805@web.de> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Christoffer Dall , catalin.marinas@arm.com, android-virt@lists.cs.columbia.edu, s.raho@virtualopensystems.com, a.motakis@virtualopensystems.com, c.dall@virtualopensystems.com, kvm@vger.kernel.org, a.costa@virtualopensystems.com To: Jan Kiszka Return-path: Received: from mx1.redhat.com ([209.132.183.28]:53950 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750957Ab1FEPKr (ORCPT ); Sun, 5 Jun 2011 11:10:47 -0400 In-Reply-To: <4DEB998E.1010805@web.de> Sender: kvm-owner@vger.kernel.org List-ID: On 06/05/2011 05:58 PM, Jan Kiszka wrote: > > > > Note that with KVM_SET_GSI_ROUTING (bad name for ARM...) we can even > > choose if an irq line is connected to a kernel-emulated interrupt > > controller or to the core's irq input. > > Makes some sense: Add KVM_IRQ_ROUTING_CPU, and kvm_irq_routing_entry's > union would require some struct kvm_irq_routing_cpu containing the > target identifier. Right. Note it would be the default, so no need to implement KVM_SET_GSI_ROUTING just yet. An additional advantage is that this is a vm ioctl, not a vcpu ioctl, so no need to interrupt the vcpu thread in userspace in order to queue an interrupt. Of course, it still happens in the kernel, but it's easier for userspace to implement its device model this way. > However, I would recommend to carefully check the generic irq routing > bits before use - if they still contain some x86/ia64 specifics or > unwanted irqchip_in_kernel(). Most likely yes. -- error compiling committee.c: too many arguments to function