#include "librbd/asio/Utils.h"
#include "librbd/io/AioCompletion.h"
#include "librbd/io/CopyupRequest.h"
-#include "librbd/io/ImageRequest.h"
-#include "librbd/io/ReadResult.h"
#include "librbd/io/Utils.h"
#include <boost/optional.hpp>
namespace io {
using librbd::util::data_object_name;
+using librbd::util::create_context_callback;
+using librbd::util::create_trace;
namespace {
const ZTracer::Trace &trace, Context *completion)
: m_ictx(ictx), m_object_no(objectno), m_object_off(off),
m_object_len(len), m_snap_id(snap_id), m_completion(completion),
- m_trace(librbd::util::create_trace(*ictx, "", trace)) {
+ m_trace(create_trace(*ictx, "", trace)) {
ceph_assert(m_ictx->data_ctx.is_valid());
if (m_trace.valid()) {
m_trace.copy_name(trace_name + std::string(" ") +
template <typename I>
void ObjectReadRequest<I>::read_parent() {
I *image_ctx = this->m_ictx;
-
- std::shared_lock image_locker{image_ctx->image_lock};
-
- // calculate reverse mapping onto the image
- Extents parent_extents;
- Striper::extent_to_file(image_ctx->cct, &image_ctx->layout,
- this->m_object_no, this->m_object_off,
- this->m_object_len, parent_extents);
-
- uint64_t parent_overlap = 0;
- uint64_t object_overlap = 0;
- int r = image_ctx->get_parent_overlap(this->m_snap_id, &parent_overlap);
- if (r == 0) {
- object_overlap = image_ctx->prune_parent_extents(parent_extents,
- parent_overlap);
- }
-
- if (object_overlap == 0) {
- image_locker.unlock();
-
- this->finish(-ENOENT);
- return;
- }
-
ldout(image_ctx->cct, 20) << dendl;
- auto parent_completion = AioCompletion::create_and_start<
- ObjectReadRequest<I>, &ObjectReadRequest<I>::handle_read_parent>(
- this, librbd::util::get_image_ctx(image_ctx->parent), AIO_TYPE_READ);
- ImageRequest<I>::aio_read(image_ctx->parent, parent_completion,
- std::move(parent_extents), ReadResult{m_read_data},
- 0, this->m_trace);
+ auto ctx = create_context_callback<
+ ObjectReadRequest<I>, &ObjectReadRequest<I>::handle_read_parent>(this);
+
+ io::util::read_parent<I>(image_ctx, this->m_object_no, this->m_object_off,
+ this->m_object_len, this->m_snap_id, this->m_trace,
+ m_read_data, ctx);
}
template <typename I>
// vim: ts=8 sw=2 smarttab
#include "librbd/io/Utils.h"
+#include "common/dout.h"
#include "include/buffer.h"
#include "include/rados/librados.hpp"
#include "include/neorados/RADOS.hpp"
+#include "librbd/io/AioCompletion.h"
+#include "librbd/io/ImageRequest.h"
#include "osd/osd_types.h"
+#include "osdc/Striper.h"
+
+#define dout_subsys ceph_subsys_rbd
+#undef dout_prefix
+#define dout_prefix *_dout << "librbd::io::util: " << __func__ << ": "
namespace librbd {
namespace io {
return false;
}
+template <typename I>
+void read_parent(I *image_ctx, uint64_t object_no, uint64_t off,
+ uint64_t len, librados::snap_t snap_id,
+ const ZTracer::Trace &trace, ceph::bufferlist* data,
+ Context* on_finish) {
+
+ auto cct = image_ctx->cct;
+
+ std::shared_lock image_locker{image_ctx->image_lock};
+
+ // calculate reverse mapping onto the image
+ Extents parent_extents;
+ Striper::extent_to_file(cct, &image_ctx->layout, object_no, off, len,
+ parent_extents);
+
+ uint64_t parent_overlap = 0;
+ uint64_t object_overlap = 0;
+ int r = image_ctx->get_parent_overlap(snap_id, &parent_overlap);
+ if (r == 0) {
+ object_overlap = image_ctx->prune_parent_extents(parent_extents,
+ parent_overlap);
+ }
+
+ if (object_overlap == 0) {
+ image_locker.unlock();
+
+ on_finish->complete(-ENOENT);
+ return;
+ }
+
+ ldout(cct, 20) << dendl;
+
+ auto comp = AioCompletion::create_and_start(on_finish, image_ctx->parent,
+ AIO_TYPE_READ);
+ ldout(cct, 20) << "completion " << comp << ", extents " << parent_extents
+ << dendl;
+
+ ImageRequest<I>::aio_read(image_ctx->parent, comp, std::move(parent_extents),
+ ReadResult{data}, 0, trace);
+}
+
} // namespace util
} // namespace io
} // namespace librbd
+template void librbd::io::util::read_parent(
+ librbd::ImageCtx *image_ctx, uint64_t object_no, uint64_t off, uint64_t len,
+ librados::snap_t snap_id, const ZTracer::Trace &trace,
+ ceph::bufferlist* data, Context* on_finish);
#include "test/librados_test_stub/MockTestMemRadosClient.h"
#include "include/rbd/librbd.hpp"
#include "librbd/io/CopyupRequest.h"
-#include "librbd/io/ImageRequest.h"
#include "librbd/io/ObjectRequest.h"
+#include "librbd/io/Utils.h"
namespace librbd {
namespace {
}
};
-template <>
-struct ImageRequest<librbd::MockTestImageCtx> {
- static ImageRequest *s_instance;
- static void aio_read(librbd::MockImageCtx *ictx, AioCompletion *c,
- Extents &&image_extents, ReadResult &&read_result,
- int op_flags, const ZTracer::Trace &parent_trace) {
- s_instance->aio_read(c, image_extents);
- }
+CopyupRequest<librbd::MockTestImageCtx>* CopyupRequest<librbd::MockTestImageCtx>::s_instance = nullptr;
- MOCK_METHOD2(aio_read, void(AioCompletion *, const Extents&));
+namespace util {
+
+namespace {
- ImageRequest() {
+struct Mock {
+ static Mock* s_instance;
+
+ Mock() {
s_instance = this;
}
+
+ MOCK_METHOD8(read_parent,
+ void(librbd::MockTestImageCtx *, uint64_t, uint64_t, uint64_t,
+ librados::snap_t, const ZTracer::Trace &, ceph::bufferlist*,
+ Context*));
};
-CopyupRequest<librbd::MockTestImageCtx>* CopyupRequest<librbd::MockTestImageCtx>::s_instance = nullptr;
-ImageRequest<librbd::MockTestImageCtx>* ImageRequest<librbd::MockTestImageCtx>::s_instance = nullptr;
+Mock *Mock::s_instance = nullptr;
+
+} // anonymous namespace
+
+template<> void read_parent(
+ librbd::MockTestImageCtx *image_ctx, uint64_t object_no, uint64_t off,
+ uint64_t len, librados::snap_t snap_id, const ZTracer::Trace &trace,
+ ceph::bufferlist* data, Context* on_finish) {
+ Mock::s_instance->read_parent(image_ctx, object_no, off, len, snap_id, trace,
+ data, on_finish);
+}
+
+} // namespace util
} // namespace io
} // namespace librbd
typedef ObjectCompareAndWriteRequest<librbd::MockTestImageCtx> MockObjectCompareAndWriteRequest;
typedef AbstractObjectWriteRequest<librbd::MockTestImageCtx> MockAbstractObjectWriteRequest;
typedef CopyupRequest<librbd::MockTestImageCtx> MockCopyupRequest;
- typedef ImageRequest<librbd::MockTestImageCtx> MockImageRequest;
+ typedef util::Mock MockUtils;
void expect_object_may_exist(MockTestImageCtx &mock_image_ctx,
uint64_t object_no, bool exists) {
}
}
- void expect_aio_read(MockTestImageCtx &mock_image_ctx,
- MockImageRequest& mock_image_request,
- Extents&& extents, int r) {
- EXPECT_CALL(mock_image_request, aio_read(_, extents))
- .WillOnce(WithArg<0>(Invoke([&mock_image_ctx, r](AioCompletion* aio_comp) {
- aio_comp->set_request_count(1);
- mock_image_ctx.image_ctx->op_work_queue->queue(new C_AioRequest(aio_comp), r);
- })));
+ void expect_read_parent(MockUtils &mock_utils, uint64_t object_no,
+ uint64_t off, uint64_t len, librados::snap_t snap_id,
+ int r) {
+ EXPECT_CALL(mock_utils,
+ read_parent(_, object_no, off, len, snap_id, _, _, _))
+ .WillOnce(WithArg<7>(CompleteContext(r, static_cast<asio::ContextWQ*>(nullptr))));
}
void expect_copyup(MockCopyupRequest& mock_copyup_request, int r) {
expect_get_read_flags(mock_image_ctx, CEPH_NOSNAP, 0);
expect_read(mock_image_ctx, ictx->get_object_name(0), 0, 4096, "", -ENOENT);
- MockImageRequest mock_image_request;
- expect_get_parent_overlap(mock_image_ctx, CEPH_NOSNAP, 4096, 0);
- expect_prune_parent_extents(mock_image_ctx, {{0, 4096}}, 4096, 4096);
- expect_aio_read(mock_image_ctx, mock_image_request, {{0, 4096}}, 0);
+ MockUtils mock_utils;
+ expect_read_parent(mock_utils, 0, 0, 4096, CEPH_NOSNAP, 0);
bufferlist bl;
Extents extent_map;
expect_get_read_flags(mock_image_ctx, CEPH_NOSNAP, 0);
expect_read(mock_image_ctx, ictx->get_object_name(0), 0, 4096, "", -ENOENT);
- MockImageRequest mock_image_request;
- expect_get_parent_overlap(mock_image_ctx, CEPH_NOSNAP, 4096, 0);
- expect_prune_parent_extents(mock_image_ctx, {{0, 4096}}, 4096, 4096);
- expect_aio_read(mock_image_ctx, mock_image_request, {{0, 4096}}, -EPERM);
+ MockUtils mock_utils;
+ expect_read_parent(mock_utils, 0, 0, 4096, CEPH_NOSNAP, -EPERM);
bufferlist bl;
Extents extent_map;
expect_get_read_flags(mock_image_ctx, CEPH_NOSNAP, 0);
expect_read(mock_image_ctx, ictx->get_object_name(0), 0, 4096, "", -ENOENT);
- MockImageRequest mock_image_request;
- expect_get_parent_overlap(mock_image_ctx, CEPH_NOSNAP, 4096, 0);
- expect_prune_parent_extents(mock_image_ctx, {{0, 4096}}, 4096, 4096);
- expect_aio_read(mock_image_ctx, mock_image_request, {{0, 4096}}, 0);
+ MockUtils mock_utils;
+ expect_read_parent(mock_utils, 0, 0, 4096, CEPH_NOSNAP, 0);
MockCopyupRequest mock_copyup_request;
expect_get_parent_overlap(mock_image_ctx, CEPH_NOSNAP, 4096, 0);
projects / ceph.git / commitdiff