nvme: Always lock and only avoid processing for recovery state

[imp]

Summary

When we lose a race with the timeout code, shift towards waiting for
that timeout code to complete so we can acquire the lock. This way we
can make sure we're in 'normal' mode before processing I/O
completions. If we're not in 'normal' mode, then we're resetting and we
should avoid completions.

Sponsored by: Netflix

[gallatin]

sys/dev/nvme/nvme_qpair.c

701

Speaking of name changes, with the direction you're heading in, I'd call this "event_lock" or something, rather than "recovery"

[imp]

Misc updates with testing