From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wright Subject: [PATCH] kvm: external module: fix request_irq for < 2.6.19 Date: Tue, 28 Apr 2009 08:56:51 -0700 Message-ID: <20090428155651.GF3036@sequoia.sous-sol.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: kvm@vger.kernel.org, ddutile@redhat.com To: Avi Kivity Return-path: Received: from sous-sol.org ([216.99.217.87]:60201 "EHLO sequoia.sous-sol.org" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1762518AbZD1P5G (ORCPT ); Tue, 28 Apr 2009 11:57:06 -0400 Content-Disposition: inline Sender: kvm-owner@vger.kernel.org List-ID: The irq handler changes (introduced in 2.6.19, not 2.6.20) dropped struct pt_regs from the handler prototype, they are found globally now. This introduces the back compat for older kernels. The handler is just a thin layer which calls the real registered handler (all this to work around a minor little compiler warning ;-) Needed for device assignment on older kernels. Signed-off-by: Chris Wright --- kernel/external-module-compat-comm.h | 30 ++++++++++++++++++++++++++---- kernel/ia64/hack-module.awk | 2 +- kernel/x86/hack-module.awk | 2 +- 3 files changed, 28 insertions(+), 6 deletions(-) diff --git a/kernel/external-module-compat-comm.h b/kernel/external-module-compat-comm.h index 5649d33..032a0f1 100644 --- a/kernel/external-module-compat-comm.h +++ b/kernel/external-module-compat-comm.h @@ -645,19 +645,41 @@ static inline int pci_reset_function(struct pci_dev *dev) #endif #include -#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,20) +#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,19) + +typedef irqreturn_t (*kvm_irq_handler_t)(int, void *); +static kvm_irq_handler_t kvm_irq_handlers[NR_IRQS]; + +static irqreturn_t kvm_irq_thunk(int irq, void *dev_id, struct pt_regs *regs) +{ + kvm_irq_handler_t handler = kvm_irq_handlers[irq]; + return handler(irq, dev_id); +} -typedef irqreturn_t (*kvm_irq_handler_t)(int, void *, struct pt_regs *); static inline int kvm_request_irq(unsigned int a, kvm_irq_handler_t handler, unsigned long c, const char *d, void *e) { - /* FIXME: allocate thunk, etc. */ - return -EINVAL; + int rc; + kvm_irq_handler_t old = kvm_irq_handlers[a]; + if (old) + return -EBUSY; + kvm_irq_handlers[a] = handler; + rc = request_irq(a, kvm_irq_thunk, c, d, e); + if (rc) + kvm_irq_handlers[a] = NULL; + return rc; +} + +static inline void kvm_free_irq(unsigned int irq, void *dev_id) +{ + free_irq(irq, dev_id); + kvm_irq_handlers[irq] = NULL; } #else #define kvm_request_irq request_irq +#define kvm_free_irq free_irq #endif diff --git a/kernel/ia64/hack-module.awk b/kernel/ia64/hack-module.awk index d0ef130..2eb557a 100644 --- a/kernel/ia64/hack-module.awk +++ b/kernel/ia64/hack-module.awk @@ -2,7 +2,7 @@ BEGIN { split("INIT_WORK on_each_cpu smp_call_function " \ "hrtimer_add_expires_ns hrtimer_get_expires " \ "hrtimer_get_expires_ns hrtimer_start_expires " \ "hrtimer_expires_remaining " \ - "request_irq", compat_apis); } + "request_irq free_irq", compat_apis); } /MODULE_AUTHOR/ { printf("MODULE_INFO(version, \"%s\");\n", version) diff --git a/kernel/x86/hack-module.awk b/kernel/x86/hack-module.awk index c04e073..325d399 100644 --- a/kernel/x86/hack-module.awk +++ b/kernel/x86/hack-module.awk @@ -2,7 +2,7 @@ BEGIN { split("INIT_WORK tsc_khz desc_struct ldttss_desc64 desc_ptr " \ "hrtimer_add_expires_ns hrtimer_get_expires " \ "hrtimer_get_expires_ns hrtimer_start_expires " \ "hrtimer_expires_remaining " \ - "on_each_cpu relay_open request_irq" , compat_apis); } + "on_each_cpu relay_open request_irq free_irq" , compat_apis); } /^int kvm_init\(/ { anon_inodes = 1 }