Revert "buffer: add a test for iterator::copy(huge_chunk, dest)"

This reverts commit 3caafc44765d331da9d8e1d52e03c6f2562848ae.

Fixes: http://tracker.ceph.com/issues/17809
Signed-off-by: Kefu Chai <kchai@redhat.com>

diff --git a/src/common/buffer.cc b/src/common/buffer.cc
index ddb8004a8bb5672c6989cbcde42a73a15f295050..ff687e2891897f74b6911b849e82ad21b22c1315 100644 (file)
--- a/src/common/buffer.cc
+++ b/src/common/buffer.cc
@@ -781,20 +781,6 @@ static simple_spinlock_t buffer_debug_lock = SIMPLE_SPINLOCK_INITIALIZER;
     return new raw_unshareable(len);
   }
 
-  class dummy_raw : public buffer::raw {
-  public:
-    dummy_raw()
-      : raw(UINT_MAX)
-    {}
-    virtual raw* clone_empty() override {
-      return new dummy_raw();
-    }
-  };
-
-  buffer::raw* buffer::create_dummy() {
-    return new dummy_raw();
-  }
-
   buffer::ptr::ptr(raw *r) : _raw(r), _off(0), _len(r->len)   // no lock needed; this is an unref raw.
   {
     r->nref.inc();

diff --git a/src/include/buffer.h b/src/include/buffer.h
index 4c05e628dbee1647c7f2e2d7f582d1c80d8a7432..7306b9906dd9dddc7c7af4d4d59e182718b19993 100644 (file)
--- a/src/include/buffer.h
+++ b/src/include/buffer.h
@@ -155,7 +155,6 @@ namespace buffer CEPH_BUFFER_API {
   raw* create_page_aligned(unsigned len);
   raw* create_zero_copy(unsigned len, int fd, int64_t *offset);
   raw* create_unshareable(unsigned len);
-  raw* create_dummy();
   raw* create_static(unsigned len, char *buf);
   raw* claim_buffer(unsigned len, char *buf, deleter del);
 

diff --git a/src/test/bufferlist.cc b/src/test/bufferlist.cc
index fc7627aeded43486530a476f90d5e587ca3ef83b..6042c515ff78fa95eece97ec0c72a3a840dded90 100644 (file)
--- a/src/test/bufferlist.cc
+++ b/src/test/bufferlist.cc
@@ -1297,21 +1297,6 @@ TEST(BufferListIterator, copy) {
   }
 }
 
-TEST(BufferListIterator, copy_huge) {
-  constexpr unsigned len = 2268888894U;
-  static_assert(int(len) < 0,
-               "should be a number underflows when being casted to int.");
-  bufferptr ptr(buffer::create_dummy());
-  ptr.set_length(len);
-
-  bufferlist src, dest;
-  src.append(ptr);
-  auto bp = src.begin();
-  bp.copy(len, dest);
-  // contents_equal() is not for this test
-  EXPECT_EQ(len, dest.length());
-}
-
 TEST(BufferListIterator, copy_in) {
   bufferlist bl;
   const char *existing = "XXX";