diff options
author | J. Bruce Fields <bfields@citi.umich.edu> | 2007-10-26 18:05:40 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@citi.umich.edu> | 2008-02-03 17:51:36 -0500 |
commit | b533184fc353d4a2d07929b4ac424a6f1bf5a3b9 (patch) | |
tree | 502634d5810735bcaea8666bdadf9bc0b6abc216 /include | |
parent | 9135f1901ee6449dfe338adf6e40e9c2025b8150 (diff) |
locks: clarify posix_locks_deadlock
For such a short function (with such a long comment),
posix_locks_deadlock() seems to cause a lot of confusion. Attempt to
make it a bit clearer:
- Remove the initial posix_same_owner() check, which can never
pass (since this is only called in the case that block_fl and
caller_fl conflict)
- Use an explicit loop (and a helper function) instead of a goto.
- Rewrite the comment, attempting a clearer explanation, and
removing some uninteresting historical detail.
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions