Add a special assertion that would trigger if a thread
is in epoch. With EPOCH_TRACE present, it will also list the
epochs entered and entry points.
Details
Details
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 27020 Build 25309: arc lint + arc unit
Event Timeline
sys/net/if.c | ||
---|---|---|
3585 | This won't be part of commit. Added just to check functionality. |
Comment Actions
Re-do the patch a bit. Instead of new macro, just add epoch_trace_list()
to the three critical places where we must not be in an epoch.
sys/kern/kern_synch.c | ||
---|---|---|
152 ↗ | (On Diff #63297) | != 0 instead of > 0 matches the KASSERT() below. |
sys/kern/kern_synch.c | ||
---|---|---|
152 ↗ | (On Diff #63297) | I don't agree. The value isn't boolean, it is an unsigned counter. Using > 0 is more correct. |
sys/kern/kern_synch.c | ||
---|---|---|
152 ↗ | (On Diff #63297) | I thought it was signed integer. If unsigned, then your check is OK. |