Whamcloud - gitweb
LU-6158 mdt: always shrink_capsule in getxattr_all
[fs/lustre-release.git] / lustre / osp / osp_md_object.c
index 46de68b..65879d5 100644 (file)
@@ -153,30 +153,6 @@ update_buffer_get_update(struct object_update_request *request,
        return ptr;
 }
 
-int osp_extend_update_buffer(const struct lu_env *env,
-                            struct osp_update_request *our)
-{
-       struct object_update_request *obj_update_req;
-       size_t new_size = our->our_req_size + OUT_UPDATE_BUFFER_SIZE_ADD;
-
-       /* enlarge object update request size */
-       if (new_size > OUT_UPDATE_BUFFER_SIZE_MAX)
-               return -E2BIG;
-
-       OBD_ALLOC_LARGE(obj_update_req, new_size);
-       if (obj_update_req == NULL)
-               return -ENOMEM;
-
-       memcpy(obj_update_req, our->our_req, our->our_req_size);
-
-       OBD_FREE_LARGE(our->our_req, our->our_req_size);
-
-       our->our_req = obj_update_req;
-       our->our_req_size = new_size;
-
-       return 0;
-}
-
 /**
  * Implementation of dt_object_operations::do_create
  *
@@ -922,6 +898,9 @@ static int osp_md_object_lock(const struct lu_env *env,
        if (mode > 0)
                return ELDLM_OK;
 
+       if (einfo->ei_nonblock)
+               flags |= LDLM_FL_BLOCK_NOWAIT;
+
        req = ldlm_enqueue_pack(osp->opd_exp, 0);
        if (IS_ERR(req))
                RETURN(PTR_ERR(req));
@@ -980,35 +959,6 @@ int osp_md_declare_object_destroy(const struct lu_env *env,
 }
 
 /**
- * Interpreter call for object destroy
- *
- * Object destroy interpreter, which will be called after destroying
- * the remote object to set flags and status.
- *
- * \param[in] env      execution environment
- * \param[in] reply    update reply
- * \param[in] req      ptlrpc update request for destroying object
- * \param[in] obj      object to be destroyed
- * \param[in] data     data used in this function.
- * \param[in] index    index(position) of destroy update in the whole
- *                      updates
- * \param[in] rc       update result on the remote MDT.
- *
- * \retval             only return 0 for now
- */
-static int osp_md_object_destroy_interpreter(const struct lu_env *env,
-                                            struct object_update_reply *reply,
-                                            struct ptlrpc_request *req,
-                                            struct osp_object *obj,
-                                            void *data, int index, int rc)
-{
-       /* not needed in cache any more */
-       set_bit(LU_OBJECT_HEARD_BANSHEE,
-               &obj->opo_obj.do_lu.lo_header->loh_flags);
-       return 0;
-}
-
-/**
  * Implement OSP layer dt_object_operations::do_destroy() interface.
  *
  * Pack the destroy update into the RPC buffer, which will be sent
@@ -1043,8 +993,10 @@ int osp_md_object_destroy(const struct lu_env *env, struct dt_object *dt,
        if (rc != 0)
                RETURN(rc);
 
+       set_bit(LU_OBJECT_HEARD_BANSHEE, &dt->do_lu.lo_header->loh_flags);
        rc = osp_insert_update_callback(env, update, dt2osp_obj(dt), NULL,
-                                       osp_md_object_destroy_interpreter);
+                                       NULL);
+
        RETURN(rc);
 }
 
@@ -1157,68 +1109,101 @@ static ssize_t osp_md_read(const struct lu_env *env, struct dt_object *dt,
        struct osp_device       *osp    = lu2osp_dev(dt->do_lu.lo_dev);
        struct dt_device        *dt_dev = &osp->opd_dt_dev;
        struct lu_buf           *lbuf   = &osp_env_info(env)->osi_lb2;
-       struct osp_update_request   *update;
+       struct osp_update_request   *update = NULL;
        struct object_update_reply *reply;
        struct out_read_reply      *orr;
+       char                       *ptr = rbuf->lb_buf;
        struct ptlrpc_request      *req = NULL;
+       size_t                     total_length = rbuf->lb_len;
+       size_t                     max_buf_size;
+       loff_t                     offset = *pos;
        int                        rc;
        ENTRY;
 
-       /* Because it needs send the update buffer right away,
-        * just create an update buffer, instead of attaching the
-        * update_remote list of the thandle.  */
-       update = osp_update_request_create(dt_dev);
-       if (IS_ERR(update))
-               RETURN(PTR_ERR(update));
+       /* Calculate the maxium buffer length for each read request */
+       max_buf_size = OUT_UPDATE_REPLY_SIZE - cfs_size_round(sizeof(*orr)) -
+                      cfs_size_round(sizeof(struct object_update_result)) -
+                      cfs_size_round(offsetof(struct object_update_reply,
+                                     ourp_lens[1]));
+       while (total_length > 0) {
+               size_t  read_length;
+
+               /* Because it needs send the update buffer right away,
+                * just create an update buffer, instead of attaching the
+                * update_remote list of the thandle.  */
+               update = osp_update_request_create(dt_dev);
+               if (IS_ERR(update))
+                       GOTO(out, rc = PTR_ERR(update));
+
+               read_length = total_length > max_buf_size ?
+                             max_buf_size : total_length;
+
+               rc = osp_update_rpc_pack(env, read, update, OUT_READ,
+                                        lu_object_fid(&dt->do_lu),
+                                        read_length, offset);
+               if (rc != 0) {
+                       CERROR("%s: cannot insert update: rc = %d\n",
+                              dt_dev->dd_lu_dev.ld_obd->obd_name, rc);
+                       GOTO(out, rc);
+               }
 
-       rc = osp_update_rpc_pack(env, read, update, OUT_READ,
-                                lu_object_fid(&dt->do_lu), rbuf->lb_len, *pos);
-       if (rc != 0) {
-               CERROR("%s: cannot insert update: rc = %d\n",
-                      dt_dev->dd_lu_dev.ld_obd->obd_name, rc);
-               GOTO(out, rc);
-       }
+               rc = osp_remote_sync(env, osp, update, &req);
+               if (rc < 0)
+                       GOTO(out, rc);
 
-       rc = osp_remote_sync(env, osp, update, &req);
-       if (rc < 0)
-               GOTO(out, rc);
+               reply = req_capsule_server_sized_get(&req->rq_pill,
+                                                    &RMF_OUT_UPDATE_REPLY,
+                                                    OUT_UPDATE_REPLY_SIZE);
 
-       reply = req_capsule_server_sized_get(&req->rq_pill,
-                                            &RMF_OUT_UPDATE_REPLY,
-                                            OUT_UPDATE_REPLY_SIZE);
-       if (reply->ourp_magic != UPDATE_REPLY_MAGIC) {
-               CERROR("%s: invalid update reply magic %x expected %x:"
-                      " rc = %d\n", dt_dev->dd_lu_dev.ld_obd->obd_name,
-                      reply->ourp_magic, UPDATE_REPLY_MAGIC, -EPROTO);
-               GOTO(out, rc = -EPROTO);
-       }
+               if (reply->ourp_magic != UPDATE_REPLY_MAGIC) {
+                       CERROR("%s: invalid update reply magic %x expected %x:"
+                              " rc = %d\n", dt_dev->dd_lu_dev.ld_obd->obd_name,
+                              reply->ourp_magic, UPDATE_REPLY_MAGIC, -EPROTO);
+                       GOTO(out, rc = -EPROTO);
+               }
 
-       rc = object_update_result_data_get(reply, lbuf, 0);
-       if (rc < 0)
-               GOTO(out, rc);
+               rc = object_update_result_data_get(reply, lbuf, 0);
+               if (rc < 0)
+                       GOTO(out, rc);
 
-       if (lbuf->lb_len < sizeof(*orr))
-               GOTO(out, rc = -EPROTO);
+               if (lbuf->lb_len < sizeof(*orr))
+                       GOTO(out, rc = -EPROTO);
 
-       orr = lbuf->lb_buf;
-       orr_le_to_cpu(orr, orr);
+               orr = lbuf->lb_buf;
+               orr_le_to_cpu(orr, orr);
+               offset = orr->orr_offset;
+               if (orr->orr_size > max_buf_size)
+                       GOTO(out, rc = -EPROTO);
 
-       *pos = orr->orr_offset;
+               memcpy(ptr, orr->orr_data, orr->orr_size);
+               ptr += orr->orr_size;
+               total_length -= orr->orr_size;
 
-       if (orr->orr_size > rbuf->lb_len)
-               GOTO(out, rc = -EPROTO);
+               CDEBUG(D_INFO, "%s: read "DFID" pos "LPU64" len %u left %zu\n",
+                      osp->opd_obd->obd_name, PFID(lu_object_fid(&dt->do_lu)),
+                      offset, orr->orr_size, total_length);
 
-       memcpy(rbuf->lb_buf, orr->orr_data, orr->orr_size);
+               if (orr->orr_size < read_length)
+                       break;
 
-       CDEBUG(D_INFO, "%s: read "DFID" pos "LPU64" len %u\n",
+               ptlrpc_req_finished(req);
+               osp_update_request_destroy(update);
+               req = NULL;
+               update = NULL;
+       }
+
+       total_length = rbuf->lb_len - total_length;
+       *pos = offset;
+       CDEBUG(D_INFO, "%s: total read "DFID" pos "LPU64" len %zu\n",
               osp->opd_obd->obd_name, PFID(lu_object_fid(&dt->do_lu)),
-              *pos, orr->orr_size);
-       GOTO(out, rc = (int)orr->orr_size);
+              *pos, total_length);
+       GOTO(out, rc = (int)total_length);
 out:
        if (req != NULL)
                ptlrpc_req_finished(req);
 
-       osp_update_request_destroy(update);
+       if (update != NULL)
+               osp_update_request_destroy(update);
 
        return rc;
 }