Commit 864e9197 authored by Yan, Zheng's avatar Yan, Zheng Committed by Ilya Dryomov

libceph: add CREATE osd operation support

Add CEPH_OSD_OP_CREATE support.  Also change libceph to not treat
CEPH_OSD_OP_DELETE as an extent op and add an assert to that end.
Signed-off-by: default avatarYan, Zheng <zyan@redhat.com>
Reviewed-by: default avatarIlya Dryomov <idryomov@redhat.com>
parent d74b50be
...@@ -480,8 +480,7 @@ void osd_req_op_extent_init(struct ceph_osd_request *osd_req, ...@@ -480,8 +480,7 @@ void osd_req_op_extent_init(struct ceph_osd_request *osd_req,
size_t payload_len = 0; size_t payload_len = 0;
BUG_ON(opcode != CEPH_OSD_OP_READ && opcode != CEPH_OSD_OP_WRITE && BUG_ON(opcode != CEPH_OSD_OP_READ && opcode != CEPH_OSD_OP_WRITE &&
opcode != CEPH_OSD_OP_DELETE && opcode != CEPH_OSD_OP_ZERO && opcode != CEPH_OSD_OP_ZERO && opcode != CEPH_OSD_OP_TRUNCATE);
opcode != CEPH_OSD_OP_TRUNCATE);
op->extent.offset = offset; op->extent.offset = offset;
op->extent.length = length; op->extent.length = length;
...@@ -663,7 +662,6 @@ static u64 osd_req_encode_op(struct ceph_osd_request *req, ...@@ -663,7 +662,6 @@ static u64 osd_req_encode_op(struct ceph_osd_request *req,
case CEPH_OSD_OP_READ: case CEPH_OSD_OP_READ:
case CEPH_OSD_OP_WRITE: case CEPH_OSD_OP_WRITE:
case CEPH_OSD_OP_ZERO: case CEPH_OSD_OP_ZERO:
case CEPH_OSD_OP_DELETE:
case CEPH_OSD_OP_TRUNCATE: case CEPH_OSD_OP_TRUNCATE:
if (src->op == CEPH_OSD_OP_WRITE) if (src->op == CEPH_OSD_OP_WRITE)
request_data_len = src->extent.length; request_data_len = src->extent.length;
...@@ -723,6 +721,9 @@ static u64 osd_req_encode_op(struct ceph_osd_request *req, ...@@ -723,6 +721,9 @@ static u64 osd_req_encode_op(struct ceph_osd_request *req,
ceph_osdc_msg_data_add(req->r_request, osd_data); ceph_osdc_msg_data_add(req->r_request, osd_data);
request_data_len = osd_data->pagelist->length; request_data_len = osd_data->pagelist->length;
break; break;
case CEPH_OSD_OP_CREATE:
case CEPH_OSD_OP_DELETE:
break;
default: default:
pr_err("unsupported osd opcode %s\n", pr_err("unsupported osd opcode %s\n",
ceph_osd_op_name(src->op)); ceph_osd_op_name(src->op));
...@@ -763,13 +764,11 @@ struct ceph_osd_request *ceph_osdc_new_request(struct ceph_osd_client *osdc, ...@@ -763,13 +764,11 @@ struct ceph_osd_request *ceph_osdc_new_request(struct ceph_osd_client *osdc,
u64 objnum = 0; u64 objnum = 0;
u64 objoff = 0; u64 objoff = 0;
u64 objlen = 0; u64 objlen = 0;
u32 object_size;
u64 object_base;
int r; int r;
BUG_ON(opcode != CEPH_OSD_OP_READ && opcode != CEPH_OSD_OP_WRITE && BUG_ON(opcode != CEPH_OSD_OP_READ && opcode != CEPH_OSD_OP_WRITE &&
opcode != CEPH_OSD_OP_DELETE && opcode != CEPH_OSD_OP_ZERO && opcode != CEPH_OSD_OP_ZERO && opcode != CEPH_OSD_OP_TRUNCATE &&
opcode != CEPH_OSD_OP_TRUNCATE); opcode != CEPH_OSD_OP_CREATE && opcode != CEPH_OSD_OP_DELETE);
req = ceph_osdc_alloc_request(osdc, snapc, num_ops, use_mempool, req = ceph_osdc_alloc_request(osdc, snapc, num_ops, use_mempool,
GFP_NOFS); GFP_NOFS);
...@@ -785,21 +784,24 @@ struct ceph_osd_request *ceph_osdc_new_request(struct ceph_osd_client *osdc, ...@@ -785,21 +784,24 @@ struct ceph_osd_request *ceph_osdc_new_request(struct ceph_osd_client *osdc,
return ERR_PTR(r); return ERR_PTR(r);
} }
object_size = le32_to_cpu(layout->fl_object_size); if (opcode == CEPH_OSD_OP_CREATE || opcode == CEPH_OSD_OP_DELETE) {
object_base = off - objoff; osd_req_op_init(req, 0, opcode);
if (!(truncate_seq == 1 && truncate_size == -1ULL)) { } else {
if (truncate_size <= object_base) { u32 object_size = le32_to_cpu(layout->fl_object_size);
truncate_size = 0; u32 object_base = off - objoff;
} else { if (!(truncate_seq == 1 && truncate_size == -1ULL)) {
truncate_size -= object_base; if (truncate_size <= object_base) {
if (truncate_size > object_size) truncate_size = 0;
truncate_size = object_size; } else {
truncate_size -= object_base;
if (truncate_size > object_size)
truncate_size = object_size;
}
} }
}
osd_req_op_extent_init(req, 0, opcode, objoff, objlen,
truncate_size, truncate_seq);
osd_req_op_extent_init(req, 0, opcode, objoff, objlen,
truncate_size, truncate_seq);
}
/* /*
* A second op in the ops array means the caller wants to * A second op in the ops array means the caller wants to
* also issue a include a 'startsync' command so that the * also issue a include a 'startsync' command so that the
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment