From: "Yan, Zheng" <zheng.z.yan@xxxxxxxxx>
Forwarding client request that was from MDS will trigger assertion
in MDS::forward_message_mds(). MDS only send client requests for
stray migration/reintegration, so it's safe to drop them.
Signed-off-by: Yan, Zheng <zheng.z.yan@xxxxxxxxx>
---
src/mds/MDCache.cc | 16 ++++++++++------
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/src/mds/MDCache.cc b/src/mds/MDCache.cc
index 5027469..88a248b 100644
--- a/src/mds/MDCache.cc
+++ b/src/mds/MDCache.cc
@@ -7475,13 +7475,17 @@ void MDCache::request_finish(MDRequest *mdr)
void MDCache::request_forward(MDRequest *mdr, int who, int port)
{
- dout(7) << "request_forward " << *mdr << " to mds." << who << " req " << *mdr << dendl;
-
- mds->forward_message_mds(mdr->client_request, who);
- mdr->client_request = 0;
+ if (mdr->client_request->get_source().is_client()) {
+ dout(7) << "request_forward " << *mdr << " to mds." << who << " req "
+ << *mdr->client_request << dendl;
+ mds->forward_message_mds(mdr->client_request, who);
+ mdr->client_request = 0;
+ if (mds->logger) mds->logger->inc(l_mds_fw);
+ } else {
+ dout(7) << "request_forward drop " << *mdr << " req " << *mdr->client_request
+ << " was from mds" << dendl;
+ }
request_cleanup(mdr);
-
- if (mds->logger) mds->logger->inc(l_mds_fw);
}
--
1.7.11.7
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
