- Change log_debug() for normal locking operations to log_debug1() where
DLM_DEBUG1 isn't set by default. This way abnormal or infrequent
conditions from log_debug() aren't quickly blown away.
- Don't call dlm_locks_dump() when DLM_ASSERT fails.
- In rcom_send_message(), only test/clear the READY flag when doing a
synchronous request. It's not used for async requests and because
these async requests can be called from different threads it could
lead to incorrect assertion failures.
- In rcom_send_message() get rid of the RECCOMM_WAIT flag which was
only used as a sanity check. Given that rcom_send_message can be
called from multiple threads and only synchronous calls are serialized,
an async caller could change the flag for the sync request and cause
the reply message to be discarded.