Index: sys/alpha/alpha/machdep.c =================================================================== RCS file: /home/ncvs/src/sys/alpha/alpha/machdep.c,v retrieving revision 1.106 diff -u -r1.106 machdep.c --- sys/alpha/alpha/machdep.c 2001/01/10 04:43:44 1.106 +++ sys/alpha/alpha/machdep.c 2001/01/18 05:36:14 @@ -390,7 +390,7 @@ TAILQ_INIT(&callwheel[i]); } - mtx_init(&callout_lock, "callout", MTX_SPIN); + mtx_init(&callout_lock, "callout", MTX_SPIN | MTX_RECURSE); #if defined(USERCONFIG) #if defined(USERCONFIG_BOOT) @@ -1003,8 +1003,8 @@ /* * Initialise mutexes. */ - mtx_init(&Giant, "Giant", MTX_DEF | MTX_COLD); - mtx_init(&sched_lock, "sched lock", MTX_SPIN | MTX_COLD); + mtx_init(&Giant, "Giant", MTX_DEF | MTX_COLD | MTX_RECURSE); + mtx_init(&sched_lock, "sched lock", MTX_SPIN | MTX_COLD | MTX_RECURSE); /* * Look at arguments passed to us and compute boothowto. Index: sys/dev/an/if_an.c =================================================================== RCS file: /home/ncvs/src/sys/dev/an/if_an.c,v retrieving revision 1.13 diff -u -r1.13 if_an.c --- sys/dev/an/if_an.c 2000/11/30 18:52:30 1.13 +++ sys/dev/an/if_an.c 2001/01/18 05:36:14 @@ -288,7 +288,8 @@ { struct ifnet *ifp = &sc->arpcom.ac_if; - mtx_init(&sc->an_mtx, device_get_nameunit(sc->an_dev), MTX_DEF); + mtx_init(&sc->an_mtx, device_get_nameunit(sc->an_dev), MTX_DEF | + MTX_RECURSE); AN_LOCK(sc); sc->an_gone = 0; Index: sys/dev/usb/if_aue.c =================================================================== RCS file: /home/ncvs/src/sys/dev/usb/if_aue.c,v retrieving revision 1.33 diff -u -r1.33 if_aue.c --- sys/dev/usb/if_aue.c 2001/01/04 11:55:24 1.33 +++ sys/dev/usb/if_aue.c 2001/01/18 05:36:14 @@ -681,7 +681,8 @@ } } - mtx_init(&sc->aue_mtx, device_get_nameunit(self), MTX_DEF); + mtx_init(&sc->aue_mtx, device_get_nameunit(self), MTX_DEF | + MTX_RECURSE); AUE_LOCK(sc); /* Reset the adapter. */ Index: sys/dev/usb/if_cue.c =================================================================== RCS file: /home/ncvs/src/sys/dev/usb/if_cue.c,v retrieving revision 1.14 diff -u -r1.14 if_cue.c --- sys/dev/usb/if_cue.c 2000/10/24 22:38:54 1.14 +++ sys/dev/usb/if_cue.c 2001/01/18 05:36:14 @@ -504,7 +504,8 @@ } } - mtx_init(&sc->cue_mtx, device_get_nameunit(self), MTX_DEF); + mtx_init(&sc->cue_mtx, device_get_nameunit(self), MTX_DEF | + MTX_RECURSE); CUE_LOCK(sc); #ifdef notdef Index: sys/dev/usb/if_kue.c =================================================================== RCS file: /home/ncvs/src/sys/dev/usb/if_kue.c,v retrieving revision 1.26 diff -u -r1.26 if_kue.c --- sys/dev/usb/if_kue.c 2000/11/23 05:58:48 1.26 +++ sys/dev/usb/if_kue.c 2001/01/18 05:36:14 @@ -450,7 +450,8 @@ } } - mtx_init(&sc->kue_mtx, device_get_nameunit(self), MTX_DEF); + mtx_init(&sc->kue_mtx, device_get_nameunit(self), MTX_DEF | + MTX_RECURSE); KUE_LOCK(sc); /* Load the firmware into the NIC. */ Index: sys/dev/wi/if_wi.c =================================================================== RCS file: /home/ncvs/src/sys/dev/wi/if_wi.c,v retrieving revision 1.33 diff -u -r1.33 if_wi.c --- sys/dev/wi/if_wi.c 2001/01/17 12:31:59 1.33 +++ sys/dev/wi/if_wi.c 2001/01/18 05:36:14 @@ -304,7 +304,7 @@ return (error); } - mtx_init(&sc->wi_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->wi_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); WI_LOCK(sc); /* Reset the NIC. */ Index: sys/i386/i386/machdep.c =================================================================== RCS file: /home/ncvs/src/sys/i386/i386/machdep.c,v retrieving revision 1.429 diff -u -r1.429 machdep.c --- sys/i386/i386/machdep.c 2001/01/16 09:10:33 1.429 +++ sys/i386/i386/machdep.c 2001/01/18 05:36:14 @@ -422,7 +422,7 @@ TAILQ_INIT(&callwheel[i]); } - mtx_init(&callout_lock, "callout", MTX_SPIN); + mtx_init(&callout_lock, "callout", MTX_SPIN | MTX_RECURSE); #if defined(USERCONFIG) userconfig(); @@ -441,7 +441,7 @@ SLIST_INIT(&cpuhead); SLIST_INSERT_HEAD(&cpuhead, GLOBALDATA, gd_allcpu); - mtx_init(&sched_lock, "sched lock", MTX_SPIN | MTX_COLD); + mtx_init(&sched_lock, "sched lock", MTX_SPIN | MTX_COLD | MTX_RECURSE); #ifdef SMP /* @@ -1939,7 +1939,7 @@ /* * We need this mutex before the console probe. */ - mtx_init(&clock_lock, "clk", MTX_SPIN | MTX_COLD); + mtx_init(&clock_lock, "clk", MTX_SPIN | MTX_COLD | MTX_RECURSE); /* * Initialize the console before we print anything out. @@ -1954,7 +1954,7 @@ /* * Giant is used early for at least debugger traps and unexpected traps. */ - mtx_init(&Giant, "Giant", MTX_DEF | MTX_COLD); + mtx_init(&Giant, "Giant", MTX_DEF | MTX_COLD | MTX_RECURSE); #ifdef DDB kdb_init(); Index: sys/i386/isa/if_el.c =================================================================== RCS file: /home/ncvs/src/sys/i386/isa/if_el.c,v retrieving revision 1.52 diff -u -r1.52 if_el.c --- sys/i386/isa/if_el.c 2000/12/15 20:09:09 1.52 +++ sys/i386/isa/if_el.c 2001/01/18 05:36:15 @@ -149,7 +149,7 @@ sc->el_btag = rman_get_bustag(sc->el_res); sc->el_bhandle = rman_get_bushandle(sc->el_res); mtx_init(&sc->el_mtx, - device_get_nameunit(dev), MTX_DEF); + device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); EL_LOCK(sc); /* Now attempt to grab the station address from the PROM Index: sys/ia64/ia64/machdep.c =================================================================== RCS file: /home/ncvs/src/sys/ia64/ia64/machdep.c,v retrieving revision 1.15 diff -u -r1.15 machdep.c --- sys/ia64/ia64/machdep.c 2000/12/05 00:35:58 1.15 +++ sys/ia64/ia64/machdep.c 2001/01/18 05:36:15 @@ -319,7 +319,7 @@ TAILQ_INIT(&callwheel[i]); } - mtx_init(&callout_lock, "callout", MTX_SPIN); + mtx_init(&callout_lock, "callout", MTX_SPIN | MTX_RECURSE); #if defined(USERCONFIG) #if defined(USERCONFIG_BOOT) @@ -590,8 +590,8 @@ /* * Initialise mutexes. */ - mtx_init(&Giant, "Giant", MTX_DEF | MTX_COLD); - mtx_init(&sched_lock, "sched lock", MTX_SPIN | MTX_COLD); + mtx_init(&Giant, "Giant", MTX_DEF | MTX_COLD | MTX_RECURSE); + mtx_init(&sched_lock, "sched lock", MTX_SPIN | MTX_COLD | MTX_RECURSE); #if 0 /* Index: sys/kern/kern_mutex.c =================================================================== RCS file: /home/ncvs/src/sys/kern/kern_mutex.c,v retrieving revision 1.31 diff -u -r1.31 kern_mutex.c --- sys/kern/kern_mutex.c 2001/01/10 04:43:49 1.31 +++ sys/kern/kern_mutex.c 2001/01/18 05:36:15 @@ -259,7 +259,7 @@ case MTX_DEF: if ((m->mtx_lock & MTX_FLAGMASK) == (uintptr_t)p) { m->mtx_recurse++; - atomic_set_ptr(&m->mtx_lock, MTX_RECURSE); + atomic_set_ptr(&m->mtx_lock, MTX_RECURSED); if ((type & MTX_QUIET) == 0) CTR1(KTR_LOCK, "mtx_enter: 0x%p recurse", m); return; @@ -434,9 +434,9 @@ switch (type) { case MTX_DEF: case MTX_DEF | MTX_NOSWITCH: - if (m->mtx_recurse != 0) { + if (mtx_recursed(m)) { if (--(m->mtx_recurse) == 0) - atomic_clear_ptr(&m->mtx_lock, MTX_RECURSE); + atomic_clear_ptr(&m->mtx_lock, MTX_RECURSED); if ((type & MTX_QUIET) == 0) CTR1(KTR_LOCK, "mtx_exit: 0x%p unrecurse", m); return; @@ -501,7 +501,7 @@ break; case MTX_SPIN: case MTX_SPIN | MTX_FIRST: - if (m->mtx_recurse != 0) { + if (mtx_recursed(m)) { m->mtx_recurse--; return; } @@ -515,7 +515,7 @@ } break; case MTX_SPIN | MTX_TOPHALF: - if (m->mtx_recurse != 0) { + if (mtx_recursed(m)) { m->mtx_recurse--; return; } @@ -655,7 +655,7 @@ if (!mtx_owned(m)) { MPASS(m->mtx_lock == MTX_UNOWNED); } else { - MPASS((m->mtx_lock & (MTX_RECURSE|MTX_CONTESTED)) == 0); + MPASS((m->mtx_lock & (MTX_RECURSED|MTX_CONTESTED)) == 0); } mtx_validate(m, MV_DESTROY); /* diagnostic */ #endif @@ -701,8 +701,9 @@ u_char w_Giant_squawked:1; u_char w_other_squawked:1; u_char w_same_squawked:1; - u_char w_sleep:1; - u_char w_spin:1; /* this is a spin mutex */ + u_char w_sleep:1; /* MTX_DEF type mutex. */ + u_char w_spin:1; /* MTX_SPIN type mutex. */ + u_char w_recurse:1; /* MTX_RECURSE mutex option. */ u_int w_level; struct witness *w_children[WITNESS_NCHILDREN]; }; @@ -838,11 +839,16 @@ p = CURPROC; if (flags & MTX_SPIN) { - if (!w->w_spin) + if (!(w->w_spin)) panic("mutex_enter: MTX_SPIN on MTX_DEF mutex %s @" " %s:%d", m->mtx_description, file, line); - if (m->mtx_recurse != 0) + if (mtx_recursed(m)) { + if (!(w->w_recurse)) + panic("mutex_enter: recursion on non-recursive" + " mutex %s @ %s:%d", m->mtx_description, + file, line); return; + } mtx_enter(&w_mtx, MTX_SPIN | MTX_QUIET); i = PCPU_GET(witness_spin_check); if (i != 0 && w->w_level < i) { @@ -864,8 +870,13 @@ panic("mutex_enter: MTX_DEF on MTX_SPIN mutex %s @ %s:%d", m->mtx_description, file, line); - if (m->mtx_recurse != 0) + if (mtx_recursed(m)) { + if (!(w->w_recurse)) + panic("mutex_enter: recursion on non-recursive" + " mutex %s @ %s:%d", m->mtx_description, + file, line); return; + } if (witness_dead) goto out; if (cold) @@ -971,11 +982,16 @@ w = m->mtx_witness; if (flags & MTX_SPIN) { - if (!w->w_spin) + if (!(w->w_spin)) panic("mutex_exit: MTX_SPIN on MTX_DEF mutex %s @" " %s:%d", m->mtx_description, file, line); - if (m->mtx_recurse != 0) + if (mtx_recursed(m)) { + if (!(w->w_recurse)) + panic("mutex_exit: recursion on non-recursive" + " mutex %s @ %s:%d", m->mtx_description, + file, line); return; + } mtx_enter(&w_mtx, MTX_SPIN | MTX_QUIET); PCPU_SET(witness_spin_check, PCPU_GET(witness_spin_check) & ~w->w_level); @@ -986,8 +1002,13 @@ panic("mutex_exit: MTX_DEF on MTX_SPIN mutex %s @ %s:%d", m->mtx_description, file, line); - if (m->mtx_recurse != 0) + if (mtx_recursed(m)) { + if (!(w->w_recurse)) + panic("mutex_exit: recursion on non-recursive" + " mutex %s @ %s:%d", m->mtx_description, + file, line); return; + } if ((flags & MTX_NOSWITCH) == 0 && !mtx_legal2block() && !cold) panic("switchable mtx_exit() of %s when not legal @ %s:%d", @@ -1005,12 +1026,17 @@ if (panicstr) return; if (flags & MTX_SPIN) { - if (!w->w_spin) + if (!(w->w_spin)) panic("mutex_try_enter: " "MTX_SPIN on MTX_DEF mutex %s @ %s:%d", m->mtx_description, file, line); - if (m->mtx_recurse != 0) + if (mtx_recursed(m)) { + if (!(w->w_recurse)) + panic("mutex_try_enter: recursion on" + " non-recursive mutex %s @ %s:%d", + m->mtx_description, file, line); return; + } mtx_enter(&w_mtx, MTX_SPIN | MTX_QUIET); PCPU_SET(witness_spin_check, PCPU_GET(witness_spin_check) | w->w_level); @@ -1026,9 +1052,13 @@ panic("mutex_try_enter: MTX_DEF on MTX_SPIN mutex %s @ %s:%d", m->mtx_description, file, line); - if (m->mtx_recurse != 0) + if (mtx_recursed(m)) { + if (!(w->w_recurse)) + panic("mutex_try_enter: recursion on non-recursive" + " mutex %s @ %s:%d", m->mtx_description, file, + line); return; - + } w->w_file = file; w->w_line = line; m->mtx_line = line; @@ -1156,7 +1186,12 @@ if (*order == NULL) panic("spin lock %s not in order list", description); w->w_level = i; - } + } else + w->w_sleep = 1; + + if (flag & MTX_RECURSE) + w->w_recurse = 1; + return (w); } @@ -1270,7 +1305,7 @@ struct witness *w, *w1; for (w = w_all; w; w = w->w_next) - if (!w->w_spin) + if (!(w->w_spin)) w->w_level = 0; for (w = w_all; w; w = w->w_next) { if (w->w_spin) Index: sys/kern/subr_eventhandler.c =================================================================== RCS file: /home/ncvs/src/sys/kern/subr_eventhandler.c,v retrieving revision 1.9 diff -u -r1.9 subr_eventhandler.c --- sys/kern/subr_eventhandler.c 2000/12/12 04:01:35 1.9 +++ sys/kern/subr_eventhandler.c 2001/01/18 05:36:15 @@ -53,7 +53,7 @@ eventhandler_init(void *dummy __unused) { TAILQ_INIT(&eventhandler_lists); - mtx_init(&eventhandler_mutex, "eventhandler", MTX_DEF); + mtx_init(&eventhandler_mutex, "eventhandler", MTX_DEF | MTX_RECURSE); eventhandler_lists_initted = 1; } SYSINIT(eventhandlers, SI_SUB_EVENTHANDLER, SI_ORDER_FIRST, eventhandler_init, Index: sys/pc98/i386/machdep.c =================================================================== RCS file: /home/ncvs/src/sys/pc98/i386/machdep.c,v retrieving revision 1.196 diff -u -r1.196 machdep.c --- sys/pc98/i386/machdep.c 2001/01/16 09:10:34 1.196 +++ sys/pc98/i386/machdep.c 2001/01/18 05:36:15 @@ -436,7 +436,7 @@ TAILQ_INIT(&callwheel[i]); } - mtx_init(&callout_lock, "callout", MTX_SPIN); + mtx_init(&callout_lock, "callout", MTX_SPIN | MTX_RECURSE); #if defined(USERCONFIG) userconfig(); @@ -455,7 +455,7 @@ SLIST_INIT(&cpuhead); SLIST_INSERT_HEAD(&cpuhead, GLOBALDATA, gd_allcpu); - mtx_init(&sched_lock, "sched lock", MTX_SPIN | MTX_COLD); + mtx_init(&sched_lock, "sched lock", MTX_SPIN | MTX_COLD | MTX_RECURSE); #ifdef SMP /* @@ -2249,7 +2249,7 @@ /* * We need this mutex before the console probe. */ - mtx_init(&clock_lock, "clk", MTX_SPIN | MTX_COLD); + mtx_init(&clock_lock, "clk", MTX_SPIN | MTX_COLD | MTX_RECURSE); /* * Initialize the console before we print anything out. @@ -2264,7 +2264,7 @@ /* * Giant is used early for at least debugger traps and unexpected traps. */ - mtx_init(&Giant, "Giant", MTX_DEF | MTX_COLD); + mtx_init(&Giant, "Giant", MTX_DEF | MTX_COLD | MTX_RECURSE); #ifdef DDB kdb_init(); Index: sys/pci/if_dc.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_dc.c,v retrieving revision 1.41 diff -u -r1.41 if_dc.c --- sys/pci/if_dc.c 2000/12/18 21:53:05 1.41 +++ sys/pci/if_dc.c 2001/01/18 05:36:15 @@ -1762,7 +1762,7 @@ unit = device_get_unit(dev); bzero(sc, sizeof(struct dc_softc)); - mtx_init(&sc->dc_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->dc_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); DC_LOCK(sc); /* Index: sys/pci/if_fxp.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_fxp.c,v retrieving revision 1.101 diff -u -r1.101 if_fxp.c --- sys/pci/if_fxp.c 2000/12/18 22:06:12 1.101 +++ sys/pci/if_fxp.c 2001/01/18 05:36:15 @@ -316,7 +316,7 @@ u_long val; int rid; - mtx_init(&sc->sc_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->sc_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); callout_handle_init(&sc->stat_ch); FXP_LOCK(sc); Index: sys/pci/if_pcn.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_pcn.c,v retrieving revision 1.14 diff -u -r1.14 if_pcn.c --- sys/pci/if_pcn.c 2000/12/04 22:46:50 1.14 +++ sys/pci/if_pcn.c 2001/01/18 05:36:15 @@ -497,7 +497,7 @@ unit = device_get_unit(dev); /* Initialize our mutex. */ - mtx_init(&sc->pcn_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->pcn_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); PCN_LOCK(sc); /* Index: sys/pci/if_rl.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_rl.c,v retrieving revision 1.53 diff -u -r1.53 if_rl.c --- sys/pci/if_rl.c 2000/12/18 21:53:05 1.53 +++ sys/pci/if_rl.c 2001/01/18 05:36:15 @@ -802,7 +802,7 @@ unit = device_get_unit(dev); bzero(sc, sizeof(struct rl_softc)); - mtx_init(&sc->rl_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->rl_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); RL_LOCK(sc); /* Index: sys/pci/if_sf.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_sf.c,v retrieving revision 1.27 diff -u -r1.27 if_sf.c --- sys/pci/if_sf.c 2000/12/04 22:46:50 1.27 +++ sys/pci/if_sf.c 2001/01/18 05:36:15 @@ -679,7 +679,7 @@ unit = device_get_unit(dev); bzero(sc, sizeof(struct sf_softc)); - mtx_init(&sc->sf_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->sf_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); SF_LOCK(sc); /* * Handle power management nonsense. Index: sys/pci/if_sis.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_sis.c,v retrieving revision 1.26 diff -u -r1.26 if_sis.c --- sys/pci/if_sis.c 2000/12/04 22:46:50 1.26 +++ sys/pci/if_sis.c 2001/01/18 05:36:15 @@ -638,7 +638,7 @@ unit = device_get_unit(dev); bzero(sc, sizeof(struct sis_softc)); - mtx_init(&sc->sis_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->sis_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); SIS_LOCK(sc); if (pci_get_device(dev) == SIS_DEVICEID_900) Index: sys/pci/if_sk.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_sk.c,v retrieving revision 1.36 diff -u -r1.36 if_sk.c --- sys/pci/if_sk.c 2000/12/04 22:46:50 1.36 +++ sys/pci/if_sk.c 2001/01/18 05:36:15 @@ -1194,7 +1194,7 @@ unit = device_get_unit(dev); bzero(sc, sizeof(struct sk_softc)); - mtx_init(&sc->sk_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->sk_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); SK_LOCK(sc); /* Index: sys/pci/if_ste.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_ste.c,v retrieving revision 1.21 diff -u -r1.21 if_ste.c --- sys/pci/if_ste.c 2000/12/04 22:46:50 1.21 +++ sys/pci/if_ste.c 2001/01/18 05:36:15 @@ -917,7 +917,7 @@ unit = device_get_unit(dev); bzero(sc, sizeof(struct ste_softc)); - mtx_init(&sc->ste_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->ste_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); STE_LOCK(sc); /* Index: sys/pci/if_ti.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_ti.c,v retrieving revision 1.41 diff -u -r1.41 if_ti.c --- sys/pci/if_ti.c 2000/12/04 22:46:50 1.41 +++ sys/pci/if_ti.c 2001/01/18 05:36:15 @@ -1476,7 +1476,7 @@ unit = device_get_unit(dev); bzero(sc, sizeof(struct ti_softc)); - mtx_init(&sc->ti_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->ti_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); TI_LOCK(sc); /* Index: sys/pci/if_tl.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_tl.c,v retrieving revision 1.60 diff -u -r1.60 if_tl.c --- sys/pci/if_tl.c 2000/12/04 22:46:50 1.60 +++ sys/pci/if_tl.c 2001/01/18 05:36:15 @@ -1132,7 +1132,7 @@ goto fail; } - mtx_init(&sc->tl_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->tl_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); TL_LOCK(sc); /* Index: sys/pci/if_vr.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_vr.c,v retrieving revision 1.35 diff -u -r1.35 if_vr.c --- sys/pci/if_vr.c 2000/12/04 22:46:50 1.35 +++ sys/pci/if_vr.c 2001/01/18 05:36:15 @@ -643,7 +643,7 @@ unit = device_get_unit(dev); bzero(sc, sizeof(struct vr_softc *)); - mtx_init(&sc->vr_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->vr_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); VR_LOCK(sc); /* Index: sys/pci/if_wb.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_wb.c,v retrieving revision 1.36 diff -u -r1.36 if_wb.c --- sys/pci/if_wb.c 2000/12/04 22:46:50 1.36 +++ sys/pci/if_wb.c 2001/01/18 05:36:15 @@ -817,7 +817,7 @@ sc = device_get_softc(dev); unit = device_get_unit(dev); - mtx_init(&sc->wb_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->wb_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); WB_LOCK(sc); /* Index: sys/pci/if_wx.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_wx.c,v retrieving revision 1.13 diff -u -r1.13 if_wx.c --- sys/pci/if_wx.c 2000/12/06 00:52:28 1.13 +++ sys/pci/if_wx.c 2001/01/18 05:36:15 @@ -597,7 +597,7 @@ } #ifdef SMPNG - mtx_init(&sc->wx_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->wx_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); #endif WX_LOCK(sc); Index: sys/pci/if_xl.c =================================================================== RCS file: /home/ncvs/src/sys/pci/if_xl.c,v retrieving revision 1.88 diff -u -r1.88 if_xl.c --- sys/pci/if_xl.c 2000/12/18 21:53:05 1.88 +++ sys/pci/if_xl.c 2001/01/18 05:36:15 @@ -1239,7 +1239,7 @@ sc = device_get_softc(dev); unit = device_get_unit(dev); - mtx_init(&sc->xl_mtx, device_get_nameunit(dev), MTX_DEF); + mtx_init(&sc->xl_mtx, device_get_nameunit(dev), MTX_DEF | MTX_RECURSE); XL_LOCK(sc); sc->xl_flags = 0; Index: sys/sys/mutex.h =================================================================== RCS file: /home/ncvs/src/sys/sys/mutex.h,v retrieving revision 1.16 diff -u -r1.16 mutex.h --- sys/sys/mutex.h 2000/12/13 21:53:42 1.16 +++ sys/sys/mutex.h 2001/01/18 05:36:15 @@ -67,7 +67,8 @@ #define MTX_SPIN 0x1 /* Spin only lock */ /* Options */ -#define MTX_RLIKELY 0x4 /* (opt) Recursion likely */ +#define MTX_RECURSE 0x2 /* Recursive lock (for mtx_init) */ +#define MTX_RLIKELY 0x4 /* Recursion likely */ #define MTX_NORECURSE 0x8 /* No recursion possible */ #define MTX_NOSPIN 0x10 /* Don't spin before sleeping */ #define MTX_NOSWITCH 0x20 /* Do not switch on release */ @@ -80,9 +81,9 @@ #define MTX_HARDOPTS (MTX_SPIN | MTX_FIRST | MTX_TOPHALF | MTX_NOSWITCH) /* Flags/value used in mtx_lock */ -#define MTX_RECURSE 0x01 /* (non-spin) lock held recursively */ +#define MTX_RECURSED 0x01 /* (non-spin) lock held recursively */ #define MTX_CONTESTED 0x02 /* (non-spin) lock contested */ -#define MTX_FLAGMASK ~(MTX_RECURSE | MTX_CONTESTED) +#define MTX_FLAGMASK ~(MTX_RECURSED | MTX_CONTESTED) #define MTX_UNOWNED 0x8 /* Cookie for free mutex */ #endif /* _KERNEL */ @@ -360,7 +361,7 @@ if (((mp)->mtx_lock & MTX_FLAGMASK) != ((uintptr_t)(tid)))\ mtx_enter_hard(mp, (type) & MTX_HARDOPTS, 0); \ else { \ - atomic_set_ptr(&(mp)->mtx_lock, MTX_RECURSE); \ + atomic_set_ptr(&(mp)->mtx_lock, MTX_RECURSED); \ (mp)->mtx_recurse++; \ } \ } \ @@ -408,10 +409,10 @@ */ #define _exitlock(mp, tid, type) do { \ if (!_release_lock(mp, tid)) { \ - if ((mp)->mtx_lock & MTX_RECURSE) { \ + if ((mp)->mtx_lock & MTX_RECURSED) { \ if (--((mp)->mtx_recurse) == 0) \ atomic_clear_ptr(&(mp)->mtx_lock, \ - MTX_RECURSE); \ + MTX_RECURSED); \ } else { \ mtx_exit_hard((mp), (type) & MTX_HARDOPTS); \ } \ @@ -422,7 +423,7 @@ #ifndef _exitlock_spin /* Release a spin lock (with possible recursion). */ #define _exitlock_spin(mp) do { \ - if ((mp)->mtx_recurse == 0) { \ + if (!mtx_recursed((mp))) { \ int _mtx_intr = (mp)->mtx_saveintr; \ \ _release_lock_quick(mp); \