]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
msg/async/ProtocolV2: allow rxbuf/txbuf get bigger in testing 37081/head
authorIlya Dryomov <idryomov@gmail.com>
Sat, 29 Aug 2020 10:02:30 +0000 (12:02 +0200)
committerIlya Dryomov <idryomov@gmail.com>
Thu, 10 Sep 2020 09:43:46 +0000 (11:43 +0200)
We have a kernel client test case that constructs huge auth tickets
to exercise the three related code paths in the kernel.  One of the
tickets is bigger than 1000000 bytes, as required for triggering the
third code path.

We haven't bumped into this assert earlier because the kernel client
is still on msgr v1.  However, "rbd map" and "rbd unmap" commands
started connecting to the cluster in commit 96f05a7956b3 ("rbd: delay
determination of default pool name") and that happens via msgr v2.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit 94953dd9398a937d43026f73efaf437597071ca7)

src/msg/async/ProtocolV2.cc

index 0431414608500fa03567e1521b5d11e6890ad01f..8921693df79f98d5b809a55b8cec14c1e0661b1c 100644 (file)
@@ -740,7 +740,7 @@ CtPtr ProtocolV2::read(CONTINUATION_RXBPTR_TYPE<ProtocolV2> &next,
       if (unlikely(pre_auth.enabled) && r >= 0) {
         pre_auth.rxbuf.append(*next.node);
        ceph_assert(!cct->_conf->ms_die_on_bug ||
-                   pre_auth.rxbuf.length() < 1000000);
+                   pre_auth.rxbuf.length() < 10000000);
       }
       next.r = r;
       run_continuation(next);
@@ -750,7 +750,7 @@ CtPtr ProtocolV2::read(CONTINUATION_RXBPTR_TYPE<ProtocolV2> &next,
     if (unlikely(pre_auth.enabled) && r >= 0) {
       pre_auth.rxbuf.append(*next.node);
       ceph_assert(!cct->_conf->ms_die_on_bug ||
-                 pre_auth.rxbuf.length() < 1000000);
+                 pre_auth.rxbuf.length() < 10000000);
     }
     next.r = r;
     return &next;
@@ -782,7 +782,7 @@ CtPtr ProtocolV2::write(const std::string &desc,
   if (unlikely(pre_auth.enabled)) {
     pre_auth.txbuf.append(buffer);
     ceph_assert(!cct->_conf->ms_die_on_bug ||
-               pre_auth.txbuf.length() < 1000000);
+               pre_auth.txbuf.length() < 10000000);
   }
 
   ssize_t r =