diff mbox

libceph: allow STAT osd operations

Message ID 511525B3.30108@inktank.com (mailing list archive)
State New, archived
Headers show

Commit Message

Alex Elder Feb. 8, 2013, 4:20 p.m. UTC
Add support for CEPH_OSD_OP_STAT operations in the osd client
and in rbd.

This operation sends no data to the osd; everything required is
encoded in identity of the target object.

The result will be ENOENT if the object doesn't exist.  If it does
exist and no other error occurs the server returns the size and last
modification time of the target object as output data (in little
endian format).  The size is a 64 bit unsigned and the time is
ceph_timespec structure (two unsigned 32-bit integers, representing
a seconds and nanoseconds value).

This resolves:
    http://tracker.ceph.com/issues/4007

Signed-off-by: Alex Elder <elder@inktank.com>
---
 drivers/block/rbd.c   |   15 +++++++++++++++
 net/ceph/osd_client.c |    3 ++-
 2 files changed, 17 insertions(+), 1 deletion(-)

 	case CEPH_OSD_OP_CMPXATTR:

Comments

Josh Durgin Feb. 19, 2013, 7:16 p.m. UTC | #1
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>

On 02/08/2013 08:20 AM, Alex Elder wrote:
> Add support for CEPH_OSD_OP_STAT operations in the osd client
> and in rbd.
>
> This operation sends no data to the osd; everything required is
> encoded in identity of the target object.
>
> The result will be ENOENT if the object doesn't exist.  If it does
> exist and no other error occurs the server returns the size and last
> modification time of the target object as output data (in little
> endian format).  The size is a 64 bit unsigned and the time is
> ceph_timespec structure (two unsigned 32-bit integers, representing
> a seconds and nanoseconds value).
>
> This resolves:
>      http://tracker.ceph.com/issues/4007
>
> Signed-off-by: Alex Elder <elder@inktank.com>
> ---
>   drivers/block/rbd.c   |   15 +++++++++++++++
>   net/ceph/osd_client.c |    3 ++-
>   2 files changed, 17 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c
> index 6e9e2c2..37361bd 100644
> --- a/drivers/block/rbd.c
> +++ b/drivers/block/rbd.c
> @@ -1148,6 +1148,8 @@ struct ceph_osd_req_op *rbd_osd_req_op_create(u16
> opcode, ...)
>   		if (opcode == CEPH_OSD_OP_WRITE)
>   			op->payload_len = op->extent.length;
>   		break;
> +	case CEPH_OSD_OP_STAT:
> +		break;
>   	case CEPH_OSD_OP_CALL:
>   		/* rbd_osd_req_op_create(CALL, class, method, data, datalen) */
>   		op->cls.class_name = va_arg(args, char *);
> @@ -1277,6 +1279,16 @@ static void rbd_osd_write_callback(struct
> rbd_obj_request *obj_request,
>   	obj_request_done_set(obj_request);
>   }
>
> +/*
> + * For a simple stat call there's nothing to do.  We'll do more if
> + * this is part of a write sequence for a layered image.
> + */
> +static void rbd_osd_stat_callback(struct rbd_obj_request *obj_request,
> +				struct ceph_osd_op *op)
> +{
> +	obj_request_done_set(obj_request);
> +}
> +
>   static void rbd_osd_req_callback(struct ceph_osd_request *osd_req,
>   				struct ceph_msg *msg)
>   {
> @@ -1307,6 +1319,9 @@ static void rbd_osd_req_callback(struct
> ceph_osd_request *osd_req,
>   	case CEPH_OSD_OP_WRITE:
>   		rbd_osd_write_callback(obj_request, op);
>   		break;
> +	case CEPH_OSD_OP_STAT:
> +		rbd_osd_stat_callback(obj_request, op);
> +		break;
>   	case CEPH_OSD_OP_CALL:
>   	case CEPH_OSD_OP_NOTIFY_ACK:
>   	case CEPH_OSD_OP_WATCH:
> diff --git a/net/ceph/osd_client.c b/net/ceph/osd_client.c
> index d9d58bb..3e4d8c4 100644
> --- a/net/ceph/osd_client.c
> +++ b/net/ceph/osd_client.c
> @@ -244,7 +244,8 @@ static void osd_req_encode_op(struct
> ceph_osd_request *req,
>   		dst->extent.truncate_seq =
>   			cpu_to_le32(src->extent.truncate_seq);
>   		break;
> -
> +	case CEPH_OSD_OP_STAT:
> +		break;
>   	case CEPH_OSD_OP_GETXATTR:
>   	case CEPH_OSD_OP_SETXATTR:
>   	case CEPH_OSD_OP_CMPXATTR:
>

--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c
index 6e9e2c2..37361bd 100644
--- a/drivers/block/rbd.c
+++ b/drivers/block/rbd.c
@@ -1148,6 +1148,8 @@  struct ceph_osd_req_op *rbd_osd_req_op_create(u16
opcode, ...)
 		if (opcode == CEPH_OSD_OP_WRITE)
 			op->payload_len = op->extent.length;
 		break;
+	case CEPH_OSD_OP_STAT:
+		break;
 	case CEPH_OSD_OP_CALL:
 		/* rbd_osd_req_op_create(CALL, class, method, data, datalen) */
 		op->cls.class_name = va_arg(args, char *);
@@ -1277,6 +1279,16 @@  static void rbd_osd_write_callback(struct
rbd_obj_request *obj_request,
 	obj_request_done_set(obj_request);
 }

+/*
+ * For a simple stat call there's nothing to do.  We'll do more if
+ * this is part of a write sequence for a layered image.
+ */
+static void rbd_osd_stat_callback(struct rbd_obj_request *obj_request,
+				struct ceph_osd_op *op)
+{
+	obj_request_done_set(obj_request);
+}
+
 static void rbd_osd_req_callback(struct ceph_osd_request *osd_req,
 				struct ceph_msg *msg)
 {
@@ -1307,6 +1319,9 @@  static void rbd_osd_req_callback(struct
ceph_osd_request *osd_req,
 	case CEPH_OSD_OP_WRITE:
 		rbd_osd_write_callback(obj_request, op);
 		break;
+	case CEPH_OSD_OP_STAT:
+		rbd_osd_stat_callback(obj_request, op);
+		break;
 	case CEPH_OSD_OP_CALL:
 	case CEPH_OSD_OP_NOTIFY_ACK:
 	case CEPH_OSD_OP_WATCH:
diff --git a/net/ceph/osd_client.c b/net/ceph/osd_client.c
index d9d58bb..3e4d8c4 100644
--- a/net/ceph/osd_client.c
+++ b/net/ceph/osd_client.c
@@ -244,7 +244,8 @@  static void osd_req_encode_op(struct
ceph_osd_request *req,
 		dst->extent.truncate_seq =
 			cpu_to_le32(src->extent.truncate_seq);
 		break;
-
+	case CEPH_OSD_OP_STAT:
+		break;
 	case CEPH_OSD_OP_GETXATTR:
 	case CEPH_OSD_OP_SETXATTR: