From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH] KVM: Use IRQF_ONESHOT for assigned device MSI interrupts Date: Mon, 11 Jun 2012 13:01:41 +0300 Message-ID: <4FD5C205.5070901@redhat.com> References: <20120601161521.26935.25606.stgit@bling.home> <4FC8F042.5050600@siemens.com> <1338570192.23475.25.camel@bling.home> <4FC8F867.7080103@siemens.com> <1338573558.23475.41.camel@bling.home> <4FC90961.8030701@siemens.com> <4FCB2359.9020505@redhat.com> <4FCC9EAC.9090007@siemens.com> <20120608143927.GA4949@redhat.com> <4FD21151.9090608@siemens.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: "Michael S. Tsirkin" , Thomas Gleixner , Alex Williamson , "kvm@vger.kernel.org" , "mtosatti@redhat.com" , "linux-kernel@vger.kernel.org" , "yongjie.ren@intel.com" To: Jan Kiszka Return-path: In-Reply-To: <4FD21151.9090608@siemens.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On 06/08/2012 05:50 PM, Jan Kiszka wrote: > >> >> Pls correct me if I'm wrong. > > Well, IIRC, the "don't loop over all vcpus with IRQs or preemption > disabled" was one argument against direct legacy interrupt injection as > well. That's what I kept in mind from those discussions. Maybe Avi can > comment on the current position. It's still my position. IMO we need something like struct gfn_to_hva_cache for interrupts. If it's in the cache, we fast-path it from the interrupt handler. If not, fall back to a workqueue and let it refill the cache. -- error compiling committee.c: too many arguments to function