RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2025-03-30 23:40:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

crimson/osd: check for primary out side of is_misdirected()

Browse Source 
so we can check can_serve_replica_read() without checking primary again.

Signed-off-by: Kefu Chai <kchai@redhat.com>
This commit is contained in:
Kefu Chai 2020-08-30 23:09:14 +08:00
parent c4992739f2
commit f77072b551
1 changed files with 3 additions and 5 deletions

8
src/crimson/osd/osd_operations/client_request.cc
View File

@ -137,7 +137,9 @@ seastar::future<> ClientRequest::process_op(
return with_blocking_future(handle.enter(pp(pg).get_obc));
}).then([this, &pg]() -> PG::load_obc_ertr::future<> {
op_info.set_from_op(&*m, *pg.get_osdmap());
if (is_misdirected(pg)) {
if (pg.is_primary()) {
// primary can handle both normal ops and balanced reads
} else if (is_misdirected(pg)) {
logger().trace("process_op: dropping misdirected op");
return seastar::now();
}
@ -163,10 +165,6 @@ seastar::future<> ClientRequest::process_op(
bool ClientRequest::is_misdirected(const PG& pg) const
{
// primary can handle both normal ops and balanced reads
if (pg.is_primary()) {
return false;
}
// otherwise take a closer look
if (const int flags = m->get_flags();
flags & CEPH_OSD_FLAG_BALANCE_READS ||