From 93ae366bffcbd4d2d98d22c0c1274f6d58de5493 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Fri, 1 Sep 2017 12:11:32 -0400 Subject: [PATCH] osd/PrimaryLogPG: do_op: drop snapset_obc lookup on snapdir We'll never find it. This also means we can drop the goofy hack to clear it for cache tiers. Signed-off-by: Sage Weil --- src/osd/PrimaryLogPG.cc | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/src/osd/PrimaryLogPG.cc b/src/osd/PrimaryLogPG.cc index 1dae1b3f4454..9a8523f1a609 100644 --- a/src/osd/PrimaryLogPG.cc +++ b/src/osd/PrimaryLogPG.cc @@ -2232,17 +2232,6 @@ void PrimaryLogPG::do_op(OpRequestRef& op) OpContext *ctx = new OpContext(op, m->get_reqid(), &m->ops, obc, this); - if (!obc->obs.exists) - ctx->snapset_obc = get_object_context(obc->obs.oi.soid.get_snapdir(), false); - - /* Due to obc caching, we might have a cached non-existent snapset_obc - * for the snapdir. If so, we can ignore it. Subsequent parts of the - * do_op pipeline make decisions based on whether snapset_obc is - * populated. - */ - if (ctx->snapset_obc && !ctx->snapset_obc->obs.exists) - ctx->snapset_obc = ObjectContextRef(); - if (m->has_flag(CEPH_OSD_FLAG_SKIPRWLOCKS)) { dout(20) << __func__ << ": skipping rw locks" << dendl; } else if (m->get_flags() & CEPH_OSD_FLAG_FLUSH) { -- 2.47.3