Server::handle_client_request() ignores unknown client operation
by returning -ENOTSUPP, however, Server::perf_gather_op_latency()
aborts on unknown client op, thereby causing -ENOTSUPP to never
reach the client. ceph_abort() seems unnecessary here.
Note, we could have invoked Server::perf_gather_op_latency()
when the return value to client is not -ENOTSUPP, however,
a valid client operation *might* just return -ENOTSUPP in
some cases.
@mchangir ran into this with his getvxattr op changes (PR #42001).
Signed-off-by: Venky Shankar <vshankar@redhat.com>
(cherry picked from commit
2f4060b8c41004d10d9a64676ccd847f6e1304dd)
case CEPH_MDS_OP_RENAMESNAP:
code = l_mdss_req_renamesnap_latency;
break;
- default: ceph_abort();
+ default:
+ dout(1) << ": unknown client op" << dendl;
+ return;
}
logger->tinc(code, lat);
}