diff options
author | Conrad Meyer <cem@FreeBSD.org> | 2016-11-30 21:59:52 +0000 |
---|---|---|
committer | Conrad Meyer <cem@FreeBSD.org> | 2016-11-30 21:59:52 +0000 |
commit | 58a639b77cdd79858e92f719fe11e3fa005eee9f (patch) | |
tree | f780ea0dd6e9e991fe1ea2790eccbbe3ec6ba9da /sys/dev/ioat/ioat_internal.h | |
parent | 26e287836b13fdb777857a47fe48f504d77d9696 (diff) | |
download | src-58a639b77cdd79858e92f719fe11e3fa005eee9f.tar.gz src-58a639b77cdd79858e92f719fe11e3fa005eee9f.zip |
ioat(4): Fix 'bogus completion_pending' KASSERT
Fix ioat_release to only set is_completion_pending if DMAs were actually
queued. Otherwise, the spurious flag could trigger an assert in the
reset path on INVARIANTS kernels.
Reviewed by: bdrewery, Suraj Raju @ Isilon
Sponsored by: Dell EMC Isilon
Notes
Notes:
svn path=/head/; revision=309338
Diffstat (limited to 'sys/dev/ioat/ioat_internal.h')
-rw-r--r-- | sys/dev/ioat/ioat_internal.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/sys/dev/ioat/ioat_internal.h b/sys/dev/ioat/ioat_internal.h index 0ed05fc677f4..5d107b0420cf 100644 --- a/sys/dev/ioat/ioat_internal.h +++ b/sys/dev/ioat/ioat_internal.h @@ -483,6 +483,7 @@ struct ioat_softc { boolean_t resetting_cleanup; /* cleanup_lock */ uint32_t head; + uint32_t acq_head; uint32_t tail; uint32_t hw_head; uint32_t ring_size_order; |