Part of the testing code in network_interface_asio_test_utils.h is a countdown latch. The latch relies on releasing writes alongside a condvar by writing to an atomic and calling notify (rather than sync'ing on the mutex the condvar is waiting on). This isn't the right way to use condvars and can be re-ordered to hang forever.

We should drop the atomic and always take the mutex when writing to a non-atomic