diff options
| author | Adrian Chadd <adrian@FreeBSD.org> | 2015-01-26 04:04:57 +0000 |
|---|---|---|
| committer | Adrian Chadd <adrian@FreeBSD.org> | 2015-01-26 04:04:57 +0000 |
| commit | 9500dd9f0bbfddbda46306f53d1d63a4984cb119 (patch) | |
| tree | 9e33408b7bb54b09cfe177c9157fbda5e9cc2536 /sys | |
| parent | e505aef3ed06c161f4637495cc3355588d85b556 (diff) | |
| download | src-9500dd9f0bbfddbda46306f53d1d63a4984cb119.tar.gz src-9500dd9f0bbfddbda46306f53d1d63a4984cb119.zip | |
Call WITNESS_WARN() in callout_drain() to check whether any locks are
being held before sleeping.
This has bitten me (in ath(4)) once before and I'd like to see this
not bite anyone else.
Differential Revision: D1638
Reviewed by: jhb, hselasky
MFC after: 1 week
Notes
Notes:
svn path=/head/; revision=277723
Diffstat (limited to 'sys')
| -rw-r--r-- | sys/kern/kern_timeout.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/sys/kern/kern_timeout.c b/sys/kern/kern_timeout.c index 13822fd99ac6..1d5d24faa40b 100644 --- a/sys/kern/kern_timeout.c +++ b/sys/kern/kern_timeout.c @@ -1096,6 +1096,10 @@ _callout_stop_safe(struct callout *c, int safe) struct lock_class *class; int direct, sq_locked, use_lock; + if (safe) + WITNESS_WARN(WARN_GIANTOK | WARN_SLEEPOK, c->c_lock, + "calling %s", __func__); + /* * Some old subsystems don't hold Giant while running a callout_stop(), * so just discard this check for the moment. |