From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marc Zyngier Subject: Re: [PATCH] KVM: EVENTFD: remove inclusion of irq.h Date: Thu, 11 Sep 2014 11:24:13 +0100 Message-ID: <5411784D.3010405@arm.com> References: <1409560568-6911-1-git-send-email-eric.auger@linaro.org> <54044647.4030605@redhat.com> <20140911030932.GA2784@lvm> <54114B21.1080102@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=WINDOWS-1252 Content-Transfer-Encoding: 8BIT Cc: Christoffer Dall , Eric Auger , "eric.auger@st.com" , "linux-arm-kernel@lists.infradead.org" , "kvmarm@lists.cs.columbia.edu" , "kvm@vger.kernel.org" , "gleb@kernel.org" , "borntraeger@de.ibm.com" , "cornelia.huck@de.ibm.com" , "xiantao.zhang@intel.com" , "agraf@suse.de" , "linux-kernel@vger.kernel.org" , "patches@linaro.org" To: Paolo Bonzini Return-path: In-Reply-To: <54114B21.1080102@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On 11/09/14 08:11, Paolo Bonzini wrote: > Il 11/09/2014 05:09, Christoffer Dall ha scritto: >> On Mon, Sep 01, 2014 at 12:11:19PM +0200, Paolo Bonzini wrote: >>> Il 01/09/2014 10:36, Eric Auger ha scritto: >>>> No more needed. irq.h would be void on ARM. >>>> >>>> Signed-off-by: Eric Auger >>>> >>>> --- >>>> >>>> I don't think irq.h is needed anymore since Paul Mackerras' work. However >>>> I did not compile for all architectures. >>>> --- >>>> virt/kvm/eventfd.c | 1 - >>>> 1 file changed, 1 deletion(-) >>>> >>>> diff --git a/virt/kvm/eventfd.c b/virt/kvm/eventfd.c >>>> index 3c5981c..0c712a7 100644 >>>> --- a/virt/kvm/eventfd.c >>>> +++ b/virt/kvm/eventfd.c >>>> @@ -36,7 +36,6 @@ >>>> #include >>>> #include >>>> >>>> -#include "irq.h" >>>> #include "iodev.h" >>>> >>>> #ifdef CONFIG_HAVE_KVM_IRQFD >>>> >>> >>> Acked-by: Paolo Bonzini >>> >>> Christoffer, please include this via the ARM tree, together with ARM >>> irqfd support. Thanks, >>> >> Marc is dealing with the tree this week and the next so he can apply it >> to kvmarm/queue. Do you want it to wait and go with the irqfd patch >> (which has dependencies not yet resolved) or should we just queue it? > > As you prefer. Added to kvmarm/queue. Thanks, M. -- Jazz is not dead. It just smells funny...