Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/sys/arch/xen/xen Do not escalate the local spl while taking ...



details:   https://anonhg.NetBSD.org/src/rev/30bc3844c5e5
branches:  trunk
changeset: 782898:30bc3844c5e5
user:      cherry <cherry%NetBSD.org@localhost>
date:      Sun Nov 25 20:56:33 2012 +0000

description:
Do not escalate the local spl while taking the interrupt handler list
lock. Re-entrance is protected against within the events handler
framework by toggling interrupts/events, so we don't need (and cannot
use) the spl framework in that path. The other consumers (interrupt
registration/removal code) are not spl sensitive wrt this lock.

diffstat:

 sys/arch/xen/xen/evtchn.c |  6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diffs (27 lines):

diff -r 9e594b8fe993 -r 30bc3844c5e5 sys/arch/xen/xen/evtchn.c
--- a/sys/arch/xen/xen/evtchn.c Sun Nov 25 20:40:30 2012 +0000
+++ b/sys/arch/xen/xen/evtchn.c Sun Nov 25 20:56:33 2012 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: evtchn.c,v 1.64 2012/11/25 08:39:36 cherry Exp $       */
+/*     $NetBSD: evtchn.c,v 1.65 2012/11/25 20:56:33 cherry Exp $       */
 
 /*
  * Copyright (c) 2006 Manuel Bouyer.
@@ -54,7 +54,7 @@
 
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: evtchn.c,v 1.64 2012/11/25 08:39:36 cherry Exp $");
+__KERNEL_RCSID(0, "$NetBSD: evtchn.c,v 1.65 2012/11/25 20:56:33 cherry Exp $");
 
 #include "opt_xen.h"
 #include "isa.h"
@@ -694,7 +694,7 @@
                 * is more explicitly implemented.
                 */
                evts->ev_cpu = ci;
-               mutex_init(&evtlock[evtch], MUTEX_DEFAULT, IPL_HIGH);
+               mutex_init(&evtlock[evtch], MUTEX_DEFAULT, IPL_NONE);
                evtsource[evtch] = evts;
                if (evname)
                        strncpy(evts->ev_evname, evname,



Home | Main Index | Thread Index | Old Index