From: Samuel Just <sam.just@inktank.com>
Date: Wed, 27 Aug 2014 23:21:41 +0000 (-0700)
Subject: PG::can_discard_op: do discard old subopreplies
X-Git-Tag: v0.80.6~15
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=d73eb895cdc792ddef2626643e61c1521cf53434;p=ceph.git

PG::can_discard_op: do discard old subopreplies

Otherwise, a sub_op_reply from a previous interval can stick around
until we either one day go active again and get rid of it or delete the
pg which is holding it on its waiting_for_active list.  While it sticks
around futily waiting for the pg to once more go active, it will cause
harmless slow request warnings.

Fixes: #9259
Backport: firefly
Signed-off-by: Samuel Just <sam.just@inktank.com>
(cherry picked from commit ae3d87348ca4e2dde809c9593b0d54ce0469f7a0)
---

diff --git a/src/osd/PG.cc b/src/osd/PG.cc
index 6394e1bd4be1f..8f4ef2c8bf77f 100644
--- a/src/osd/PG.cc
+++ b/src/osd/PG.cc
@@ -5138,7 +5138,7 @@ bool PG::can_discard_request(OpRequestRef op)
   case MSG_OSD_PG_PUSH_REPLY:
     return can_discard_replica_op<MOSDPGPushReply, MSG_OSD_PG_PUSH_REPLY>(op);
   case MSG_OSD_SUBOPREPLY:
-    return false;
+    return can_discard_replica_op<MOSDSubOpReply, MSG_OSD_SUBOPREPLY>(op);
 
   case MSG_OSD_EC_WRITE:
     return can_discard_replica_op<MOSDECSubOpWrite, MSG_OSD_EC_WRITE>(op);