rbd: pull rbd_img_request_create() dout out into the callers
authorIlya Dryomov <idryomov@gmail.com>
Fri, 30 Aug 2019 15:31:06 +0000 (17:31 +0200)
committerIlya Dryomov <idryomov@gmail.com>
Mon, 16 Sep 2019 10:06:25 +0000 (12:06 +0200)
Make it more informative: log op_type, offset and length for block
layer requests and initiating obj_req for child requests.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
drivers/block/rbd.c

index 69db738..7c4350c 100644 (file)
@@ -1754,8 +1754,6 @@ static struct rbd_img_request *rbd_img_request_create(
        mutex_init(&img_request->state_mutex);
        kref_init(&img_request->kref);
 
-       dout("%s: rbd_dev %p %s -> img %p\n", __func__, rbd_dev,
-            obj_op_name(op_type), img_request);
        return img_request;
 }
 
@@ -2944,6 +2942,9 @@ static int rbd_obj_read_from_parent(struct rbd_obj_request *obj_req)
        __set_bit(IMG_REQ_CHILD, &child_img_req->flags);
        child_img_req->obj_request = obj_req;
 
+       dout("%s child_img_req %p for obj_req %p\n", __func__, child_img_req,
+            obj_req);
+
        if (!rbd_img_is_write(img_req)) {
                switch (img_req->data_type) {
                case OBJ_REQUEST_BIO:
@@ -4877,6 +4878,9 @@ static void rbd_queue_workfn(struct work_struct *work)
        img_request->rq = rq;
        snapc = NULL; /* img_request consumes a ref */
 
+       dout("%s rbd_dev %p img_req %p %s %llu~%llu\n", __func__, rbd_dev,
+            img_request, obj_op_name(op_type), offset, length);
+
        if (op_type == OBJ_OP_DISCARD || op_type == OBJ_OP_ZEROOUT)
                result = rbd_img_fill_nodata(img_request, offset, length);
        else