From: Samuel Just <sjust@redhat.com>
Date: Sat, 3 Dec 2016 19:48:44 +0000 (-0800)
Subject: ReplicatedPG: add debugging to EINVAL returns
X-Git-Tag: v11.1.0~58^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=c0c7f926d81675756cf86e31867fa3155b239477;p=ceph.git

ReplicatedPG: add debugging to EINVAL returns

Signed-off-by: Samuel Just <sjust@redhat.com>
---

diff --git a/src/osd/ReplicatedPG.cc b/src/osd/ReplicatedPG.cc
index 689567e2da75..eb77dd73a4de 100644
--- a/src/osd/ReplicatedPG.cc
+++ b/src/osd/ReplicatedPG.cc
@@ -1741,8 +1741,11 @@ void ReplicatedPG::do_op(OpRequestRef& op)
   m->finish_decode();
   m->clear_payload();
 
+  dout(20) << __func__ << ": op " << *m << dendl;
+
   if (m->has_flag(CEPH_OSD_FLAG_PARALLELEXEC)) {
     // not implemented.
+    dout(20) << __func__ << ": PARALLELEXEC not implemented " << *m << dendl;
     osd->reply_op_error(op, -EINVAL);
     return;
   }
@@ -1862,6 +1865,7 @@ void ReplicatedPG::do_op(OpRequestRef& op)
 
     // invalid?
     if (m->get_snapid() != CEPH_NOSNAP) {
+      dout(20) << __func__ << ": write to clone not valid " << *m << dendl;
       osd->reply_op_error(op, -EINVAL);
       return;
     }
@@ -1945,6 +1949,7 @@ void ReplicatedPG::do_op(OpRequestRef& op)
  
   // asking for SNAPDIR is only ok for reads
   if (m->get_snapid() == CEPH_SNAPDIR && op->may_write()) {
+    dout(20) << __func__ << ": write to snapdir not valid " << *m << dendl;
     osd->reply_op_error(op, -EINVAL);
     return;
   }