Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Tue, 22 Oct 2002 17:30:30 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Tue, 22 Oct 2002 17:30:29 -0400 Received: from 12-237-170-171.client.attbi.com ([12.237.170.171]:53776 "EHLO wf-rch.cirr.com") by vger.kernel.org with ESMTP id ; Tue, 22 Oct 2002 17:30:21 -0400 Message-ID: <3DB5C4F3.5030102@mvista.com> Date: Tue, 22 Oct 2002 16:36:51 -0500 From: Corey Minyard User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0rc3) Gecko/20020523 X-Accept-Language: en-us, en MIME-Version: 1.0 To: linux-kernel@vger.kernel.org CC: John Levon , dipankar@gamebox.net Subject: [PATCH] NMI request/release, version 3 References: <3DB4AABF.9020400@mvista.com> <20021022021005.GA39792@compsoc.man.ac.uk> <3DB4B8A7.5060807@mvista.com> <20021022025346.GC41678@compsoc.man.ac.uk> <3DB54C53.9010603@mvista.com> <20021022232345.A25716@dikhow> <3DB59385.6050003@mvista.com> <20021022233853.B25716@dikhow> <3DB59923.9050002@mvista.com> <20021022190818.GA84745@compsoc.man.ac.uk> Content-Type: multipart/mixed; boundary="------------030904010801020302000906" Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 11151 Lines: 392 This is a multi-part message in MIME format. --------------030904010801020302000906 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Add a request/release mechanism to the kernel (x86 only for now) for NMIs. This version uses the rcupdate mechanism to free the link items, instead of using the previous algorithm for interacting with the NMI handler code to free the item. It should be more scalable. Thanks to everyone who helped me with this. -Corey --------------030904010801020302000906 Content-Type: text/plain; name="linux-nmi-v3.diff" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="linux-nmi-v3.diff" diff -ur linux.orig/arch/i386/kernel/i386_ksyms.c linux/arch/i386/kernel/i386_ksyms.c --- linux.orig/arch/i386/kernel/i386_ksyms.c Mon Oct 21 13:25:58 2002 +++ linux/arch/i386/kernel/i386_ksyms.c Tue Oct 22 12:19:59 2002 @@ -90,6 +90,9 @@ EXPORT_SYMBOL(cpu_khz); EXPORT_SYMBOL(apm_info); +EXPORT_SYMBOL(request_nmi); +EXPORT_SYMBOL(release_nmi); + #ifdef CONFIG_DEBUG_IOVIRT EXPORT_SYMBOL(__io_virt_debug); #endif diff -ur linux.orig/arch/i386/kernel/irq.c linux/arch/i386/kernel/irq.c --- linux.orig/arch/i386/kernel/irq.c Mon Oct 21 13:25:58 2002 +++ linux/arch/i386/kernel/irq.c Tue Oct 22 12:08:20 2002 @@ -131,6 +131,8 @@ * Generic, controller-independent functions: */ +extern void nmi_append_user_names(struct seq_file *p); + int show_interrupts(struct seq_file *p, void *v) { int i, j; @@ -166,6 +168,8 @@ for (j = 0; j < NR_CPUS; j++) if (cpu_online(j)) p += seq_printf(p, "%10u ", nmi_count(j)); + seq_printf(p, " "); + nmi_append_user_names(p); seq_putc(p, '\n'); #if CONFIG_X86_LOCAL_APIC seq_printf(p, "LOC: "); diff -ur linux.orig/arch/i386/kernel/nmi.c linux/arch/i386/kernel/nmi.c --- linux.orig/arch/i386/kernel/nmi.c Mon Oct 21 13:25:45 2002 +++ linux/arch/i386/kernel/nmi.c Tue Oct 22 16:18:00 2002 @@ -20,6 +20,7 @@ #include #include #include +#include #include #include @@ -102,6 +103,17 @@ return 0; } +static int nmi_watchdog_tick (void * dev_id, struct pt_regs * regs); + +static struct nmi_handler nmi_watchdog_handler = +{ + .dev_name = "nmi_watchdog", + .dev_id = NULL, + .handler = nmi_watchdog_tick, + .priority = 255, /* We want to be relatively high priority. */ + .freed = NULL +}; + static int __init setup_nmi_watchdog(char *str) { int nmi; @@ -110,6 +122,12 @@ if (nmi >= NMI_INVALID) return 0; + + if (request_nmi(&nmi_watchdog_handler) != 0) { + /* Couldn't add a watchdog handler, give up. */ + return 0; + } + if (nmi == NMI_NONE) nmi_watchdog = nmi; /* @@ -350,7 +368,7 @@ alert_counter[i] = 0; } -void nmi_watchdog_tick (struct pt_regs * regs) +static int nmi_watchdog_tick (void * dev_id, struct pt_regs * regs) { /* @@ -401,4 +419,6 @@ } wrmsr(nmi_perfctr_msr, -(cpu_khz/nmi_hz*1000), -1); } + + return NOTIFY_OK; } diff -ur linux.orig/arch/i386/kernel/traps.c linux/arch/i386/kernel/traps.c --- linux.orig/arch/i386/kernel/traps.c Mon Oct 21 13:25:45 2002 +++ linux/arch/i386/kernel/traps.c Tue Oct 22 16:01:33 2002 @@ -23,6 +23,10 @@ #include #include #include +#include +#include +#include +#include #ifdef CONFIG_EISA #include @@ -485,21 +489,191 @@ printk("Do you have a strange power saving mode enabled?\n"); } +/* + * A list of handlers for NMIs. This list will be called in order + * when an NMI from an otherwise unidentifiable source somes in. If + * one of these handles the NMI, it should return NOTIFY_OK, otherwise + * it should return NOTIFY_DONE. NMI handlers cannot claim spinlocks, + * so we have to handle freeing these in a different manner. A + * spinlock protects the list from multiple writers. When something + * is removed from the list, it is thrown into another list (with + * another link, so the "next" element stays valid) and scheduled to + * run as an rcu. When the rcu runs, it is guaranteed that nothing in + * the NMI code will be using it. + */ +static struct nmi_handler *nmi_handler_list = NULL; +static spinlock_t nmi_handler_lock = SPIN_LOCK_UNLOCKED; +static struct nmi_handler *nmi_to_free_list = NULL; +static spinlock_t nmi_to_free_lock = SPIN_LOCK_UNLOCKED; + +struct rcu_head nmi_rcu; + +/* + * To free the list item, we use an rcu. The rcu-function will not + * run until all processors have done a context switch, gone idle, or + * gone to a user process, so it's guaranteed that when this runs, any + * NMI handler running at release time has completed and the list item + * can be safely freed. + */ +static void really_free_nmi_list(void *unused) +{ + unsigned long flags; + struct nmi_handler *item; + + spin_lock_irqsave(&nmi_to_free_lock, flags); + while (nmi_to_free_list) { + item = nmi_to_free_list; + nmi_to_free_list = item->link2; + item->freed(item); + } + spin_unlock_irqrestore(&nmi_to_free_lock, flags); +} +static inline void free_nmi_handler(struct nmi_handler *item) +{ + unsigned long flags; + + if (!item->freed) + return; + + spin_lock_irqsave(&nmi_to_free_lock, flags); + /* We only have one copy of nmi_rcu, so we only want to add it + once. If there are items in the list, then it has already + been added. */ + if (nmi_to_free_list == NULL) + call_rcu(&nmi_rcu, really_free_nmi_list, NULL); + item->link2 = nmi_to_free_list; + nmi_to_free_list = item; + spin_unlock_irqrestore(&nmi_to_free_lock, flags); +} + +static inline struct nmi_handler *find_nmi_handler( + struct nmi_handler *handler, + struct nmi_handler **rprev) +{ + struct nmi_handler *curr, *prev; + + curr = nmi_handler_list; + prev = NULL; + while (curr) { + if (curr == handler) { + break; + } + prev = curr; + curr = curr->next; + } + + if (rprev) + *rprev = prev; + + return curr; +} + +int request_nmi(struct nmi_handler *handler) +{ + struct nmi_handler *curr; + + spin_lock(&nmi_handler_lock); + + /* Make sure the thing is not already in the list. */ + if (find_nmi_handler(handler, NULL)) + return EBUSY; + + /* Add it into the list in priority order. */ + curr = nmi_handler_list; + if ((!curr) || (curr->priority < handler->priority)) { + handler->next = nmi_handler_list; + smp_mb(); + xchg(&nmi_handler_list, handler); + } else { + while (curr->next && + (curr->next->priority > handler->priority)) + { + curr = curr->next; + } + handler->next = curr->next; + smp_mb(); + xchg(&(curr->next), handler); + } + + spin_unlock(&nmi_handler_lock); + return 0; +} + +void release_nmi(struct nmi_handler *handler) +{ + struct nmi_handler *curr, *prev; + + spin_lock(&nmi_handler_lock); + + /* Find it in the list. */ + curr = find_nmi_handler(handler, &prev); + + if (curr) { + /* If it was found, remove it from the list. We + assume the write operation here is atomic. */ + smp_mb(); + if (prev) + xchg(&(prev->next), curr->next); + else + xchg(&nmi_handler_list, curr->next); + + free_nmi_handler(curr); + } + spin_unlock(&nmi_handler_lock); +} + +static int call_nmi_handlers(struct pt_regs * regs) +{ + struct nmi_handler *curr, *next; + int handled = 0; + int val; + + curr = nmi_handler_list; + while (curr) { + next = curr->next; + val = curr->handler(curr->dev_id, regs); + switch (val & ~NOTIFY_STOP_MASK) { + case NOTIFY_OK: + handled = 1; + break; + + case NOTIFY_DONE: + default: + } + if (val & NOTIFY_STOP_MASK) + break; + curr = next; + } + + return handled; +} + +void nmi_append_user_names(struct seq_file *p) +{ + struct nmi_handler *curr; + + spin_lock(&nmi_handler_lock); + curr = nmi_handler_list; + while (curr) { + if (curr->dev_name) + p += seq_printf(p, " %s", curr->dev_name); + curr = curr->next; + } + spin_unlock(&nmi_handler_lock); +} + static void default_do_nmi(struct pt_regs * regs) { unsigned char reason = inb(0x61); if (!(reason & 0xc0)) { -#if CONFIG_X86_LOCAL_APIC /* - * Ok, so this is none of the documented NMI sources, - * so it must be the NMI watchdog. + * Check our handler list to see if anyone can handle this + * nmi. */ - if (nmi_watchdog) { - nmi_watchdog_tick(regs); + if (call_nmi_handlers(regs)) return; - } -#endif + unknown_nmi_error(reason, regs); return; } diff -ur linux.orig/include/asm-i386/apic.h linux/include/asm-i386/apic.h --- linux.orig/include/asm-i386/apic.h Mon Oct 21 13:26:04 2002 +++ linux/include/asm-i386/apic.h Tue Oct 22 12:40:16 2002 @@ -79,7 +79,6 @@ extern void setup_boot_APIC_clock (void); extern void setup_secondary_APIC_clock (void); extern void setup_apic_nmi_watchdog (void); -extern inline void nmi_watchdog_tick (struct pt_regs * regs); extern int APIC_init_uniprocessor (void); extern void disable_APIC_timer(void); extern void enable_APIC_timer(void); diff -ur linux.orig/include/asm-i386/irq.h linux/include/asm-i386/irq.h --- linux.orig/include/asm-i386/irq.h Mon Oct 21 13:24:41 2002 +++ linux/include/asm-i386/irq.h Tue Oct 22 16:13:30 2002 @@ -28,4 +28,43 @@ #define ARCH_HAS_NMI_WATCHDOG /* See include/linux/nmi.h */ #endif + +/** + * Register a handler to get called when an NMI occurs. If the + * handler actually handles the NMI, it should return NOTIFY_OK. If + * it did not handle the NMI, it should return NOTIFY_DONE. It may "or" + * on NOTIFY_STOP_MASK to the return value if it does not want other + * handlers after it to be notified. Note that this is a slow-path + * handler, if you have a fast-path handler there's another tie-in + * for you. See the oprofile code. + * + * Note that when you release the handler, you may NOT immediately + * free or reuse the handler item, and you may not unload the module + * of any handler, because it still may be referenced in an NMI call. + * Instead, you should wait until the supplied "freed" callback is + * called. + */ +#define HAVE_NMI_HANDLER 1 +struct nmi_handler +{ + char *dev_name; + void *dev_id; + int (*handler)(void *dev_id, struct pt_regs *regs); + int priority; /* Handlers called in priority order. */ + + /* If "freed" is not NULL, this will be called when the item + is no longer in use. */ + void (*freed)(const void *arg); + + /* Don't mess with anything below here. */ + + struct nmi_handler *next; + /* This is for linking into the list of things release in the + rcu callback. We can't use next because we can't touch it + until the rcu callback runs. */ + struct nmi_handler *link2; +}; +int request_nmi(struct nmi_handler *handler); +void release_nmi(struct nmi_handler *handler); + #endif /* _ASM_IRQ_H */ --------------030904010801020302000906-- - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/