diff options
-rw-r--r-- | src/msg/Pipe.cc | 24 | ||||
-rw-r--r-- | src/msg/Pipe.h | 1 |
2 files changed, 23 insertions, 2 deletions
diff --git a/src/msg/Pipe.cc b/src/msg/Pipe.cc index 2bbaf0bc17f..1ebf2854473 100644 --- a/src/msg/Pipe.cc +++ b/src/msg/Pipe.cc @@ -160,6 +160,7 @@ void Pipe::join_reader() void Pipe::DelayedDelivery::discard() { + lgeneric_subdout(pipe->msgr->cct, ms, 20) << pipe->_pipe_prefix(_dout) << "DelayedDelivery::discard" << dendl; Mutex::Locker l(delay_lock); while (!delay_queue.empty()) { Message *m = delay_queue.front().second; @@ -169,6 +170,17 @@ void Pipe::DelayedDelivery::discard() } } +void Pipe::DelayedDelivery::flush() +{ + lgeneric_subdout(pipe->msgr->cct, ms, 20) << pipe->_pipe_prefix(_dout) << "DelayedDelivery::flush" << dendl; + Mutex::Locker l(delay_lock); + while (!delay_queue.empty()) { + Message *m = delay_queue.front().second; + delay_queue.pop_front(); + pipe->in_q->enqueue(m, m->get_priority(), pipe->conn_id); + } +} + void *Pipe::DelayedDelivery::entry() { Mutex::Locker locker(delay_lock); @@ -543,7 +555,11 @@ int Pipe::accept() // make existing Connection reference us existing->connection_state->reset_pipe(this); - + + // flush/queue any existing delayed messages + if (existing->delay_thread) + existing->delay_thread->flush(); + // steal incoming queue uint64_t replaced_conn_id = conn_id; conn_id = existing->conn_id; @@ -1113,6 +1129,10 @@ void Pipe::fault(bool onread) return; } + // queue delayed items immediately + if (delay_thread) + delay_thread->flush(); + // requeue sent items requeue_sent(); @@ -1120,7 +1140,7 @@ void Pipe::fault(bool onread) ldout(msgr->cct,0) << "fault with nothing to send, going to standby" << dendl; state = STATE_STANDBY; return; - } + } if (state != STATE_CONNECTING) { if (policy.server) { diff --git a/src/msg/Pipe.h b/src/msg/Pipe.h index 30fb7eede11..1bcc8263f4a 100644 --- a/src/msg/Pipe.h +++ b/src/msg/Pipe.h @@ -90,6 +90,7 @@ class DispatchQueue; delay_cond.Signal(); } void discard(); + void flush(); void stop() { delay_lock.Lock(); stop_delayed_delivery = true; |