From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH v2 4/4] KVM: Allow host IRQ sharing for passed-through PCI 2.3 devices Date: Mon, 13 Dec 2010 12:19:35 +0200 Message-ID: <4D05F337.3000701@redhat.com> References: <5a8f3388ba70d018510fa75f42ef677ad0765e69.1292152963.git.jan.kiszka@web.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Thomas Gleixner , Marcelo Tosatti , linux-kernel@vger.kernel.org, kvm , Tom Lyon , Alex Williamson , "Michael S. Tsirkin" , Jan Kiszka To: Jan Kiszka Return-path: In-Reply-To: <5a8f3388ba70d018510fa75f42ef677ad0765e69.1292152963.git.jan.kiszka@web.de> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On 12/12/2010 01:22 PM, Jan Kiszka wrote: > From: Jan Kiszka > > PCI 2.3 allows to generically disable IRQ sources at device level. This > enables us to share IRQs of such devices on the host side when passing > them to a guest. > > However, IRQ disabling via the PCI config space is more costly than > masking the line via disable_irq. Therefore we register the IRQ in adaptive > mode and switch between line and device level disabling on demand. > > This feature is optional, user space has to request it explicitly as it > also has to inform us about its view of PCI_COMMAND_INTX_DISABLE. That > way, we can avoid unmasking the interrupt and signaling it if the guest > masked it via the PCI config space. > Looks fine. > + ret =IRQ_NONE; > + Danger, whitespace error detected. Initiating self-destruct sequence. -- error compiling committee.c: too many arguments to function