* GPL HEADER END
*/
/*
- * Copyright (c) 2013, Intel Corporation.
+ * Copyright (c) 2013, 2017, Intel Corporation.
*
- * lustre/mdt/out_handler.c
+ * lustre/target/out_handler.c
*
* Object update handler between targets.
*
#define DEBUG_SUBSYSTEM S_CLASS
-#include <obd_class.h>
+#include <llog_swab.h>
+#include <lustre_obdo.h>
+#include <lustre_swab.h>
+#include <lustre_update.h>
#include <md_object.h>
+#include <obd_class.h>
#include "tgt_internal.h"
-#include <lustre_update.h>
-struct tx_arg *tx_add_exec(struct thandle_exec_args *ta, tx_exec_func_t func,
- tx_exec_func_t undo, char *file, int line)
+static inline void orr_cpu_to_le(struct out_read_reply *orr_dst,
+ const struct out_read_reply *orr_src)
{
- int i;
-
- LASSERT(ta);
- LASSERT(func);
-
- i = ta->ta_argno;
- LASSERT(i < UPDATE_MAX_OPS);
-
- ta->ta_argno++;
-
- ta->ta_args[i].exec_fn = func;
- ta->ta_args[i].undo_fn = undo;
- ta->ta_args[i].file = file;
- ta->ta_args[i].line = line;
-
- return &ta->ta_args[i];
-}
-
-static int out_tx_start(const struct lu_env *env, struct dt_device *dt,
- struct thandle_exec_args *ta)
-{
- memset(ta, 0, sizeof(*ta));
- ta->ta_handle = dt_trans_create(env, dt);
- if (IS_ERR(ta->ta_handle)) {
- CERROR("%s: start handle error: rc = %ld\n",
- dt_obd_name(dt), PTR_ERR(ta->ta_handle));
- return PTR_ERR(ta->ta_handle);
- }
- ta->ta_dev = dt;
- /*For phase I, sync for cross-ref operation*/
- ta->ta_handle->th_sync = 1;
- return 0;
-}
-
-static int out_trans_start(const struct lu_env *env,
- struct thandle_exec_args *ta)
-{
- /* Always do sync commit for Phase I */
- LASSERT(ta->ta_handle->th_sync != 0);
- return dt_trans_start(env, ta->ta_dev, ta->ta_handle);
-}
-
-static int out_trans_stop(const struct lu_env *env,
- struct thandle_exec_args *ta, int err)
-{
- int i;
- int rc;
-
- ta->ta_handle->th_result = err;
- LASSERT(ta->ta_handle->th_sync != 0);
- rc = dt_trans_stop(env, ta->ta_dev, ta->ta_handle);
- for (i = 0; i < ta->ta_argno; i++) {
- if (ta->ta_args[i].object != NULL) {
- lu_object_put(env, &ta->ta_args[i].object->do_lu);
- ta->ta_args[i].object = NULL;
- }
- }
-
- return rc;
-}
-
-int out_tx_end(const struct lu_env *env, struct thandle_exec_args *ta)
-{
- struct tgt_session_info *tsi = tgt_ses_info(env);
- int i = 0, rc;
-
- LASSERT(ta->ta_dev);
- LASSERT(ta->ta_handle);
-
- if (ta->ta_err != 0 || ta->ta_argno == 0)
- GOTO(stop, rc = ta->ta_err);
-
- rc = out_trans_start(env, ta);
- if (unlikely(rc))
- GOTO(stop, rc);
-
- for (i = 0; i < ta->ta_argno; i++) {
- rc = ta->ta_args[i].exec_fn(env, ta->ta_handle,
- &ta->ta_args[i]);
- if (unlikely(rc)) {
- CDEBUG(D_INFO, "error during execution of #%u from"
- " %s:%d: rc = %d\n", i, ta->ta_args[i].file,
- ta->ta_args[i].line, rc);
- while (--i >= 0) {
- LASSERTF(ta->ta_args[i].undo_fn != NULL,
- "can't undo changes, hope for failover!\n");
- ta->ta_args[i].undo_fn(env, ta->ta_handle,
- &ta->ta_args[i]);
- }
- break;
- }
- }
-
- /* Only fail for real update */
- tsi->tsi_reply_fail_id = OBD_FAIL_UPDATE_OBJ_NET_REP;
-stop:
- CDEBUG(D_INFO, "%s: executed %u/%u: rc = %d\n",
- dt_obd_name(ta->ta_dev), i, ta->ta_argno, rc);
- out_trans_stop(env, ta, rc);
- ta->ta_handle = NULL;
- ta->ta_argno = 0;
- ta->ta_err = 0;
-
- RETURN(rc);
+ orr_dst->orr_size = cpu_to_le32(orr_src->orr_size);
+ orr_dst->orr_padding = cpu_to_le32(orr_src->orr_padding);
+ orr_dst->orr_offset = cpu_to_le64(orr_dst->orr_offset);
}
static void out_reconstruct(const struct lu_env *env, struct dt_device *dt,
- struct dt_object *obj, struct update_reply *reply,
+ struct dt_object *obj,
+ struct object_update_reply *reply,
int index)
{
CDEBUG(D_INFO, "%s: fork reply reply %p index %d: rc = %d\n",
dt_obd_name(dt), reply, index, 0);
- update_insert_reply(reply, NULL, 0, index, 0);
+ object_update_result_insert(reply, NULL, 0, index, 0);
return;
}
typedef void (*out_reconstruct_t)(const struct lu_env *env,
struct dt_device *dt,
struct dt_object *obj,
- struct update_reply *reply,
+ struct object_update_reply *reply,
int index);
static inline int out_check_resent(const struct lu_env *env,
struct dt_object *obj,
struct ptlrpc_request *req,
out_reconstruct_t reconstruct,
- struct update_reply *reply,
+ struct object_update_reply *reply,
int index)
{
if (likely(!(lustre_msg_get_flags(req->rq_reqmsg) & MSG_RESENT)))
return 0;
if (req_xid_is_last(req)) {
- reconstruct(env, dt, obj, reply, index);
- return 1;
- }
- DEBUG_REQ(D_HA, req, "no reply for RESENT req (have "LPD64")",
- req->rq_export->exp_target_data.ted_lcd->lcd_last_xid);
- return 0;
-}
-
-static int out_obj_destroy(const struct lu_env *env, struct dt_object *dt_obj,
- struct thandle *th)
-{
- int rc;
-
- CDEBUG(D_INFO, "%s: destroy "DFID"\n", dt_obd_name(th->th_dev),
- PFID(lu_object_fid(&dt_obj->do_lu)));
+ struct lsd_client_data *lcd;
- dt_write_lock(env, dt_obj, MOR_TGT_CHILD);
- rc = dt_destroy(env, dt_obj, th);
- dt_write_unlock(env, dt_obj);
+ /* XXX this does not support mulitple transactions yet, i.e.
+ * only 1 update RPC each time betwee MDTs */
+ lcd = req->rq_export->exp_target_data.ted_lcd;
- return rc;
-}
+ req->rq_transno = lcd->lcd_last_transno;
+ req->rq_status = lcd->lcd_last_result;
+ if (req->rq_status != 0)
+ req->rq_transno = 0;
+ lustre_msg_set_transno(req->rq_repmsg, req->rq_transno);
+ lustre_msg_set_status(req->rq_repmsg, req->rq_status);
-/**
- * All of the xxx_undo will be used once execution failed,
- * But because all of the required resource has been reserved in
- * declare phase, i.e. if declare succeed, it should make sure
- * the following executing phase succeed in anyway, so these undo
- * should be useless for most of the time in Phase I
- */
-int out_tx_create_undo(const struct lu_env *env, struct thandle *th,
- struct tx_arg *arg)
-{
- int rc;
-
- rc = out_obj_destroy(env, arg->object, th);
- if (rc != 0)
- CERROR("%s: undo failure, we are doomed!: rc = %d\n",
- dt_obd_name(th->th_dev), rc);
- return rc;
-}
-
-int out_tx_create_exec(const struct lu_env *env, struct thandle *th,
- struct tx_arg *arg)
-{
- struct dt_object *dt_obj = arg->object;
- int rc;
-
- CDEBUG(D_OTHER, "%s: create "DFID": dof %u, mode %o\n",
- dt_obd_name(th->th_dev),
- PFID(lu_object_fid(&arg->object->do_lu)),
- arg->u.create.dof.dof_type,
- arg->u.create.attr.la_mode & S_IFMT);
-
- dt_write_lock(env, dt_obj, MOR_TGT_CHILD);
- rc = dt_create(env, dt_obj, &arg->u.create.attr,
- &arg->u.create.hint, &arg->u.create.dof, th);
-
- dt_write_unlock(env, dt_obj);
-
- CDEBUG(D_INFO, "%s: insert create reply %p index %d: rc = %d\n",
- dt_obd_name(th->th_dev), arg->reply, arg->index, rc);
-
- update_insert_reply(arg->reply, NULL, 0, arg->index, rc);
-
- return rc;
-}
-
-static int __out_tx_create(const struct lu_env *env, struct dt_object *obj,
- struct lu_attr *attr, struct lu_fid *parent_fid,
- struct dt_object_format *dof,
- struct thandle_exec_args *ta,
- struct update_reply *reply,
- int index, char *file, int line)
-{
- struct tx_arg *arg;
-
- LASSERT(ta->ta_handle != NULL);
- ta->ta_err = dt_declare_create(env, obj, attr, NULL, dof,
- ta->ta_handle);
- if (ta->ta_err != 0)
- return ta->ta_err;
-
- arg = tx_add_exec(ta, out_tx_create_exec, out_tx_create_undo, file,
- line);
- LASSERT(arg);
-
- /* release the object in out_trans_stop */
- lu_object_get(&obj->do_lu);
- arg->object = obj;
- arg->u.create.attr = *attr;
- if (parent_fid)
- arg->u.create.fid = *parent_fid;
- memset(&arg->u.create.hint, 0, sizeof(arg->u.create.hint));
- arg->u.create.dof = *dof;
- arg->reply = reply;
- arg->index = index;
+ DEBUG_REQ(D_RPCTRACE, req, "restoring resent RPC");
+ reconstruct(env, dt, obj, reply, index);
+ return 1;
+ }
+ DEBUG_REQ(D_HA, req, "no reply for RESENT req (have %lld)",
+ req->rq_export->exp_target_data.ted_lcd->lcd_last_xid);
return 0;
}
static int out_create(struct tgt_session_info *tsi)
{
struct tgt_thread_info *tti = tgt_th_info(tsi->tsi_env);
- struct update *update = tti->tti_u.update.tti_update;
+ struct object_update *update = tti->tti_u.update.tti_update;
struct dt_object *obj = tti->tti_u.update.tti_dt_object;
struct dt_object_format *dof = &tti->tti_u.update.tti_update_dof;
struct obdo *lobdo = &tti->tti_u.update.tti_obdo;
struct lu_attr *attr = &tti->tti_attr;
struct lu_fid *fid = NULL;
struct obdo *wobdo;
- int size;
+ size_t size;
int rc;
ENTRY;
- wobdo = update_param_buf(update, 0, &size);
- if (wobdo == NULL || size != sizeof(*wobdo)) {
- CERROR("%s: obdo is NULL, invalid RPC: rc = %d\n",
- tgt_name(tsi->tsi_tgt), -EPROTO);
- RETURN(err_serious(-EPROTO));
+ wobdo = object_update_param_get(update, 0, &size);
+ if (IS_ERR(wobdo) || size != sizeof(*wobdo)) {
+ CERROR("%s: obdo is NULL, invalid RPC: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(wobdo));
+ RETURN(PTR_ERR(wobdo));
}
- obdo_le_to_cpu(wobdo, wobdo);
+ if (ptlrpc_req_need_swab(tsi->tsi_pill->rc_req))
+ lustre_swab_obdo(wobdo);
lustre_get_wire_obdo(NULL, lobdo, wobdo);
la_from_obdo(attr, lobdo, lobdo->o_valid);
dof->dof_type = dt_mode_to_dft(attr->la_mode);
- if (S_ISDIR(attr->la_mode)) {
- int size;
-
- fid = update_param_buf(update, 1, &size);
- if (fid == NULL || size != sizeof(*fid)) {
- CERROR("%s: invalid fid: rc = %d\n",
- tgt_name(tsi->tsi_tgt), -EPROTO);
- RETURN(err_serious(-EPROTO));
+ if (update->ou_params_count > 1) {
+ fid = object_update_param_get(update, 1, &size);
+ if (IS_ERR(fid) || size != sizeof(*fid)) {
+ CERROR("%s: invalid fid: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(fid));
+ RETURN(PTR_ERR(fid));
}
- fid_le_to_cpu(fid, fid);
+ if (ptlrpc_req_need_swab(tsi->tsi_pill->rc_req))
+ lustre_swab_lu_fid(fid);
if (!fid_is_sane(fid)) {
CERROR("%s: invalid fid "DFID": rc = %d\n",
tgt_name(tsi->tsi_tgt), PFID(fid), -EPROTO);
- RETURN(err_serious(-EPROTO));
+ RETURN(-EPROTO);
}
}
RETURN(-EEXIST);
rc = out_tx_create(tsi->tsi_env, obj, attr, fid, dof,
- &tti->tti_tea,
+ &tti->tti_tea, tti->tti_tea.ta_handle,
tti->tti_u.update.tti_update_reply,
tti->tti_u.update.tti_update_reply_index);
RETURN(rc);
}
-static int out_tx_attr_set_undo(const struct lu_env *env,
- struct thandle *th, struct tx_arg *arg)
-{
- CERROR("%s: attr set undo "DFID" unimplemented yet!: rc = %d\n",
- dt_obd_name(th->th_dev),
- PFID(lu_object_fid(&arg->object->do_lu)), -ENOTSUPP);
-
- return -ENOTSUPP;
-}
-
-static int out_tx_attr_set_exec(const struct lu_env *env, struct thandle *th,
- struct tx_arg *arg)
-{
- struct dt_object *dt_obj = arg->object;
- int rc;
-
- CDEBUG(D_OTHER, "%s: attr set "DFID"\n", dt_obd_name(th->th_dev),
- PFID(lu_object_fid(&dt_obj->do_lu)));
-
- dt_write_lock(env, dt_obj, MOR_TGT_CHILD);
- rc = dt_attr_set(env, dt_obj, &arg->u.attr_set.attr, th, NULL);
- dt_write_unlock(env, dt_obj);
-
- CDEBUG(D_INFO, "%s: insert attr_set reply %p index %d: rc = %d\n",
- dt_obd_name(th->th_dev), arg->reply, arg->index, rc);
-
- update_insert_reply(arg->reply, NULL, 0, arg->index, rc);
-
- return rc;
-}
-
-static int __out_tx_attr_set(const struct lu_env *env,
- struct dt_object *dt_obj,
- const struct lu_attr *attr,
- struct thandle_exec_args *th,
- struct update_reply *reply, int index,
- char *file, int line)
-{
- struct tx_arg *arg;
-
- LASSERT(th->ta_handle != NULL);
- th->ta_err = dt_declare_attr_set(env, dt_obj, attr, th->ta_handle);
- if (th->ta_err != 0)
- return th->ta_err;
-
- arg = tx_add_exec(th, out_tx_attr_set_exec, out_tx_attr_set_undo,
- file, line);
- LASSERT(arg);
- lu_object_get(&dt_obj->do_lu);
- arg->object = dt_obj;
- arg->u.attr_set.attr = *attr;
- arg->reply = reply;
- arg->index = index;
- return 0;
-}
-
static int out_attr_set(struct tgt_session_info *tsi)
{
struct tgt_thread_info *tti = tgt_th_info(tsi->tsi_env);
- struct update *update = tti->tti_u.update.tti_update;
+ struct object_update *update = tti->tti_u.update.tti_update;
struct lu_attr *attr = &tti->tti_attr;
struct dt_object *obj = tti->tti_u.update.tti_dt_object;
struct obdo *lobdo = &tti->tti_u.update.tti_obdo;
struct obdo *wobdo;
- int size;
+ size_t size;
int rc;
ENTRY;
- wobdo = update_param_buf(update, 0, &size);
- if (wobdo == NULL || size != sizeof(*wobdo)) {
- CERROR("%s: empty obdo in the update: rc = %d\n",
- tgt_name(tsi->tsi_tgt), -EPROTO);
- RETURN(err_serious(-EPROTO));
+ wobdo = object_update_param_get(update, 0, &size);
+ if (IS_ERR(wobdo) || size != sizeof(*wobdo)) {
+ CERROR("%s: empty obdo in the update: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(wobdo));
+ RETURN(PTR_ERR(wobdo));
}
attr->la_valid = 0;
attr->la_valid = 0;
- obdo_le_to_cpu(wobdo, wobdo);
+
+ if (ptlrpc_req_need_swab(tsi->tsi_pill->rc_req))
+ lustre_swab_obdo(wobdo);
lustre_get_wire_obdo(NULL, lobdo, wobdo);
la_from_obdo(attr, lobdo, lobdo->o_valid);
rc = out_tx_attr_set(tsi->tsi_env, obj, attr, &tti->tti_tea,
+ tti->tti_tea.ta_handle,
tti->tti_u.update.tti_update_reply,
tti->tti_u.update.tti_update_reply_index);
{
const struct lu_env *env = tsi->tsi_env;
struct tgt_thread_info *tti = tgt_th_info(env);
+ struct object_update *update = tti->tti_u.update.tti_update;
struct obdo *obdo = &tti->tti_u.update.tti_obdo;
struct lu_attr *la = &tti->tti_attr;
struct dt_object *obj = tti->tti_u.update.tti_dt_object;
+ int idx = tti->tti_u.update.tti_update_reply_index;
int rc;
ENTRY;
- if (!lu_object_exists(&obj->do_lu))
+ if (unlikely(update->ou_result_size < sizeof(*obdo)))
+ return -EPROTO;
+
+ if (!lu_object_exists(&obj->do_lu)) {
+ /* Usually, this will be called when the master MDT try
+ * to init a remote object(see osp_object_init), so if
+ * the object does not exist on slave, we need set BANSHEE flag,
+ * so the object can be removed from the cache immediately */
+ set_bit(LU_OBJECT_HEARD_BANSHEE,
+ &obj->do_lu.lo_header->loh_flags);
RETURN(-ENOENT);
+ }
- dt_read_lock(env, obj, MOR_TGT_CHILD);
- rc = dt_attr_get(env, obj, la, NULL);
+ dt_read_lock(env, obj, DT_TGT_CHILD);
+ rc = dt_attr_get(env, obj, la);
if (rc)
GOTO(out_unlock, rc);
- /*
- * If it is a directory, we will also check whether the
- * directory is empty.
- * la_flags = 0 : Empty.
- * = 1 : Not empty.
- */
- la->la_flags = 0;
- if (S_ISDIR(la->la_mode)) {
- struct dt_it *it;
- const struct dt_it_ops *iops;
-
- if (!dt_try_as_dir(env, obj))
- GOTO(out_unlock, rc = -ENOTDIR);
-
- iops = &obj->do_index_ops->dio_it;
- it = iops->init(env, obj, LUDA_64BITHASH, BYPASS_CAPA);
- if (!IS_ERR(it)) {
- int result;
- result = iops->get(env, it, (const void *)"");
- if (result > 0) {
- int i;
- for (result = 0, i = 0; result == 0 && i < 3;
- ++i)
- result = iops->next(env, it);
- if (result == 0)
- la->la_flags = 1;
- } else if (result == 0)
- /*
- * Huh? Index contains no zero key?
- */
- rc = -EIO;
-
- iops->put(env, it);
- iops->fini(env, it);
- }
- }
obdo->o_valid = 0;
obdo_from_la(obdo, la, la->la_valid);
- obdo_cpu_to_le(obdo, obdo);
- lustre_set_wire_obdo(NULL, obdo, obdo);
out_unlock:
dt_read_unlock(env, obj);
tgt_name(tsi->tsi_tgt), tti->tti_u.update.tti_update_reply,
0, rc);
- update_insert_reply(tti->tti_u.update.tti_update_reply, obdo,
- sizeof(*obdo), 0, rc);
+ object_update_result_insert(tti->tti_u.update.tti_update_reply, obdo,
+ sizeof(*obdo), idx, rc);
+
RETURN(rc);
}
static int out_xattr_get(struct tgt_session_info *tsi)
{
- const struct lu_env *env = tsi->tsi_env;
- struct tgt_thread_info *tti = tgt_th_info(env);
- struct update *update = tti->tti_u.update.tti_update;
- struct lu_buf *lbuf = &tti->tti_buf;
- struct update_reply *reply = tti->tti_u.update.tti_update_reply;
- struct dt_object *obj = tti->tti_u.update.tti_dt_object;
- char *name;
- void *ptr;
- int rc;
+ const struct lu_env *env = tsi->tsi_env;
+ struct tgt_thread_info *tti = tgt_th_info(env);
+ struct object_update *update = tti->tti_u.update.tti_update;
+ struct lu_buf *lbuf = &tti->tti_buf;
+ struct object_update_reply *reply = tti->tti_u.update.tti_update_reply;
+ struct dt_object *obj = tti->tti_u.update.tti_dt_object;
+ char *name;
+ struct object_update_result *update_result;
+ int idx = tti->tti_u.update.tti_update_reply_index;
+ int rc;
ENTRY;
- name = (char *)update_param_buf(update, 0, NULL);
- if (name == NULL) {
+ if (!lu_object_exists(&obj->do_lu)) {
+ set_bit(LU_OBJECT_HEARD_BANSHEE,
+ &obj->do_lu.lo_header->loh_flags);
+ RETURN(-ENOENT);
+ }
+
+ name = object_update_param_get(update, 0, NULL);
+ if (IS_ERR(name)) {
+ CERROR("%s: empty name for xattr get: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(name));
+ RETURN(PTR_ERR(name));
+ }
+
+ update_result = object_update_result_get(reply, idx, NULL);
+ if (update_result == NULL) {
CERROR("%s: empty name for xattr get: rc = %d\n",
tgt_name(tsi->tsi_tgt), -EPROTO);
- RETURN(err_serious(-EPROTO));
+ RETURN(-EPROTO);
}
- ptr = update_get_buf_internal(reply, 0, NULL);
- LASSERT(ptr != NULL);
+ lbuf->lb_len = (int)tti->tti_u.update.tti_update->ou_result_size;
+ if (lbuf->lb_len == 0)
+ lbuf->lb_buf = NULL;
+ else
+ lbuf->lb_buf = update_result->our_data;
- /* The first 4 bytes(int) are used to store the result */
- lbuf->lb_buf = (char *)ptr + sizeof(int);
- lbuf->lb_len = UPDATE_BUFFER_SIZE - sizeof(struct update_reply);
- dt_read_lock(env, obj, MOR_TGT_CHILD);
- rc = dt_xattr_get(env, obj, lbuf, name, NULL);
+ dt_read_lock(env, obj, DT_TGT_CHILD);
+ rc = dt_xattr_get(env, obj, lbuf, name);
dt_read_unlock(env, obj);
- if (rc < 0) {
+ if (rc <= 0) {
lbuf->lb_len = 0;
- GOTO(out, rc);
+ if (unlikely(!rc))
+ rc = -ENODATA;
+ } else if (lbuf->lb_buf) {
+ lbuf->lb_len = rc;
}
- if (rc == 0) {
- lbuf->lb_len = 0;
- GOTO(out, rc = -ENOENT);
- }
- lbuf->lb_len = rc;
- rc = 0;
CDEBUG(D_INFO, "%s: "DFID" get xattr %s len %d\n",
tgt_name(tsi->tsi_tgt), PFID(lu_object_fid(&obj->do_lu)),
- name, (int)lbuf->lb_len);
+ name, rc);
+
+ GOTO(out, rc);
+
out:
- *(int *)ptr = rc;
- reply->ur_lens[0] = lbuf->lb_len + sizeof(int);
- RETURN(rc);
+ object_update_result_insert(reply, lbuf->lb_buf, lbuf->lb_len, idx, rc);
+ RETURN(0);
+}
+
+static int out_xattr_list(struct tgt_session_info *tsi)
+{
+ const struct lu_env *env = tsi->tsi_env;
+ struct tgt_thread_info *tti = tgt_th_info(env);
+ struct lu_buf *lbuf = &tti->tti_buf;
+ struct object_update_reply *reply = tti->tti_u.update.tti_update_reply;
+ struct dt_object *obj = tti->tti_u.update.tti_dt_object;
+ struct object_update_result *update_result;
+ int idx = tti->tti_u.update.tti_update_reply_index;
+ int rc;
+
+ ENTRY;
+
+ if (!lu_object_exists(&obj->do_lu)) {
+ set_bit(LU_OBJECT_HEARD_BANSHEE,
+ &obj->do_lu.lo_header->loh_flags);
+ RETURN(-ENOENT);
+ }
+
+ update_result = object_update_result_get(reply, 0, NULL);
+ if (!update_result) {
+ rc = -EPROTO;
+ CERROR("%s: empty buf for xattr list: rc = %d\n",
+ tgt_name(tsi->tsi_tgt), rc);
+ RETURN(rc);
+ }
+
+ lbuf->lb_len = (int)tti->tti_u.update.tti_update->ou_result_size;
+ lbuf->lb_buf = update_result->our_data;
+ if (lbuf->lb_len == 0)
+ lbuf->lb_buf = 0;
+
+ dt_read_lock(env, obj, DT_TGT_CHILD);
+ rc = dt_xattr_list(env, obj, lbuf);
+ dt_read_unlock(env, obj);
+ if (rc <= 0) {
+ lbuf->lb_len = 0;
+ if (unlikely(!rc))
+ rc = -ENODATA;
+ } else if (lbuf->lb_buf) {
+ lbuf->lb_len = rc;
+ }
+
+ CDEBUG(D_INFO, "%s: "DFID" list xattr len %d\n",
+ tgt_name(tsi->tsi_tgt), PFID(lu_object_fid(&obj->do_lu)), rc);
+
+ /* Since we directly use update_result->our_data as the lbuf->lb_buf,
+ * then use NULL for result_insert to avoid unnecessary memory copy. */
+ object_update_result_insert(reply, NULL, lbuf->lb_len, idx, rc);
+
+ RETURN(0);
}
static int out_index_lookup(struct tgt_session_info *tsi)
{
const struct lu_env *env = tsi->tsi_env;
struct tgt_thread_info *tti = tgt_th_info(env);
- struct update *update = tti->tti_u.update.tti_update;
+ struct object_update *update = tti->tti_u.update.tti_update;
struct dt_object *obj = tti->tti_u.update.tti_dt_object;
char *name;
int rc;
ENTRY;
+ if (unlikely(update->ou_result_size < sizeof(tti->tti_fid1)))
+ return -EPROTO;
+
if (!lu_object_exists(&obj->do_lu))
RETURN(-ENOENT);
- name = (char *)update_param_buf(update, 0, NULL);
- if (name == NULL) {
- CERROR("%s: empty name for lookup: rc = %d\n",
- tgt_name(tsi->tsi_tgt), -EPROTO);
- RETURN(err_serious(-EPROTO));
+ name = object_update_param_get(update, 0, NULL);
+ if (IS_ERR(name)) {
+ CERROR("%s: empty name for lookup: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(name));
+ RETURN(PTR_ERR(name));
}
- dt_read_lock(env, obj, MOR_TGT_CHILD);
+ dt_read_lock(env, obj, DT_TGT_CHILD);
if (!dt_try_as_dir(env, obj))
GOTO(out_unlock, rc = -ENOTDIR);
rc = dt_lookup(env, obj, (struct dt_rec *)&tti->tti_fid1,
- (struct dt_key *)name, NULL);
+ (struct dt_key *)name);
if (rc < 0)
GOTO(out_unlock, rc);
if (rc == 0)
rc += 1;
+out_unlock:
+ dt_read_unlock(env, obj);
+
CDEBUG(D_INFO, "lookup "DFID" %s get "DFID" rc %d\n",
PFID(lu_object_fid(&obj->do_lu)), name,
PFID(&tti->tti_fid1), rc);
- fid_cpu_to_le(&tti->tti_fid1, &tti->tti_fid1);
-
-out_unlock:
- dt_read_unlock(env, obj);
CDEBUG(D_INFO, "%s: insert lookup reply %p index %d: rc = %d\n",
tgt_name(tsi->tsi_tgt), tti->tti_u.update.tti_update_reply,
0, rc);
- update_insert_reply(tti->tti_u.update.tti_update_reply,
- &tti->tti_fid1, sizeof(tti->tti_fid1), 0, rc);
+ object_update_result_insert(tti->tti_u.update.tti_update_reply,
+ &tti->tti_fid1, sizeof(tti->tti_fid1),
+ tti->tti_u.update.tti_update_reply_index, rc);
RETURN(rc);
}
-static int out_tx_xattr_set_exec(const struct lu_env *env,
- struct thandle *th,
- struct tx_arg *arg)
-{
- struct dt_object *dt_obj = arg->object;
- int rc;
-
- CDEBUG(D_INFO, "%s: set xattr buf %p name %s flag %d\n",
- dt_obd_name(th->th_dev), arg->u.xattr_set.buf.lb_buf,
- arg->u.xattr_set.name, arg->u.xattr_set.flags);
-
- dt_write_lock(env, dt_obj, MOR_TGT_CHILD);
- rc = dt_xattr_set(env, dt_obj, &arg->u.xattr_set.buf,
- arg->u.xattr_set.name, arg->u.xattr_set.flags,
- th, NULL);
- dt_write_unlock(env, dt_obj);
- /**
- * Ignore errors if this is LINK EA
- **/
- if (unlikely(rc && !strncmp(arg->u.xattr_set.name, XATTR_NAME_LINK,
- strlen(XATTR_NAME_LINK))))
- rc = 0;
-
- CDEBUG(D_INFO, "%s: insert xattr set reply %p index %d: rc = %d\n",
- dt_obd_name(th->th_dev), arg->reply, arg->index, rc);
-
- update_insert_reply(arg->reply, NULL, 0, arg->index, rc);
-
- return rc;
-}
-
-static int __out_tx_xattr_set(const struct lu_env *env,
- struct dt_object *dt_obj,
- const struct lu_buf *buf,
- const char *name, int flags,
- struct thandle_exec_args *ta,
- struct update_reply *reply, int index,
- char *file, int line)
-{
- struct tx_arg *arg;
-
- LASSERT(ta->ta_handle != NULL);
- ta->ta_err = dt_declare_xattr_set(env, dt_obj, buf, name,
- flags, ta->ta_handle);
- if (ta->ta_err != 0)
- return ta->ta_err;
-
- arg = tx_add_exec(ta, out_tx_xattr_set_exec, NULL, file, line);
- LASSERT(arg);
- lu_object_get(&dt_obj->do_lu);
- arg->object = dt_obj;
- arg->u.xattr_set.name = name;
- arg->u.xattr_set.flags = flags;
- arg->u.xattr_set.buf = *buf;
- arg->reply = reply;
- arg->index = index;
- arg->u.xattr_set.csum = 0;
- return 0;
-}
-
static int out_xattr_set(struct tgt_session_info *tsi)
{
struct tgt_thread_info *tti = tgt_th_info(tsi->tsi_env);
- struct update *update = tti->tti_u.update.tti_update;
+ struct object_update *update = tti->tti_u.update.tti_update;
struct dt_object *obj = tti->tti_u.update.tti_dt_object;
struct lu_buf *lbuf = &tti->tti_buf;
char *name;
char *buf;
- char *tmp;
- int buf_len = 0;
+ __u32 *tmp;
+ size_t buf_len = 0;
int flag;
+ size_t size = 0;
int rc;
ENTRY;
- name = update_param_buf(update, 0, NULL);
- if (name == NULL) {
- CERROR("%s: empty name for xattr set: rc = %d\n",
- tgt_name(tsi->tsi_tgt), -EPROTO);
- RETURN(err_serious(-EPROTO));
+ name = object_update_param_get(update, 0, NULL);
+ if (IS_ERR(name)) {
+ CERROR("%s: empty name for xattr set: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(name));
+ RETURN(PTR_ERR(name));
}
- buf = (char *)update_param_buf(update, 1, &buf_len);
- if (buf == NULL || buf_len == 0) {
- CERROR("%s: empty buf for xattr set: rc = %d\n",
- tgt_name(tsi->tsi_tgt), -EPROTO);
- RETURN(err_serious(-EPROTO));
- }
+ /* If buffer == NULL (-ENODATA), then it might mean delete xattr */
+ buf = object_update_param_get(update, 1, &buf_len);
+ if (IS_ERR(buf) && PTR_ERR(buf) != -ENODATA)
+ RETURN(PTR_ERR(buf));
lbuf->lb_buf = buf;
lbuf->lb_len = buf_len;
- tmp = (char *)update_param_buf(update, 2, NULL);
- if (tmp == NULL) {
- CERROR("%s: empty flag for xattr set: rc = %d\n",
- tgt_name(tsi->tsi_tgt), -EPROTO);
- RETURN(err_serious(-EPROTO));
+ tmp = object_update_param_get(update, 2, &size);
+ if (IS_ERR(tmp) || size != sizeof(*tmp)) {
+ CERROR("%s: emptry or wrong size %zu flag: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), size, PTR_ERR(tmp));
+ RETURN(PTR_ERR(tmp));
}
- flag = le32_to_cpu(*(int *)tmp);
+ if (ptlrpc_req_need_swab(tsi->tsi_pill->rc_req))
+ __swab32s(tmp);
+ flag = *tmp;
rc = out_tx_xattr_set(tsi->tsi_env, obj, lbuf, name, flag,
- &tti->tti_tea,
+ &tti->tti_tea, tti->tti_tea.ta_handle,
tti->tti_u.update.tti_update_reply,
tti->tti_u.update.tti_update_reply_index);
RETURN(rc);
}
-static int out_obj_ref_add(const struct lu_env *env,
- struct dt_object *dt_obj,
- struct thandle *th)
+static int out_xattr_del(struct tgt_session_info *tsi)
{
- int rc;
-
- dt_write_lock(env, dt_obj, MOR_TGT_CHILD);
- rc = dt_ref_add(env, dt_obj, th);
- dt_write_unlock(env, dt_obj);
-
- return rc;
-}
-
-static int out_obj_ref_del(const struct lu_env *env,
- struct dt_object *dt_obj,
- struct thandle *th)
-{
- int rc;
-
- dt_write_lock(env, dt_obj, MOR_TGT_CHILD);
- rc = dt_ref_del(env, dt_obj, th);
- dt_write_unlock(env, dt_obj);
-
- return rc;
-}
-
-static int out_tx_ref_add_exec(const struct lu_env *env, struct thandle *th,
- struct tx_arg *arg)
-{
- struct dt_object *dt_obj = arg->object;
- int rc;
-
- rc = out_obj_ref_add(env, dt_obj, th);
-
- CDEBUG(D_INFO, "%s: insert ref_add reply %p index %d: rc = %d\n",
- dt_obd_name(th->th_dev), arg->reply, arg->index, rc);
-
- update_insert_reply(arg->reply, NULL, 0, arg->index, rc);
- return rc;
-}
+ struct tgt_thread_info *tti = tgt_th_info(tsi->tsi_env);
+ struct object_update *update = tti->tti_u.update.tti_update;
+ struct dt_object *obj = tti->tti_u.update.tti_dt_object;
+ char *name;
+ int rc;
+ ENTRY;
-static int out_tx_ref_add_undo(const struct lu_env *env, struct thandle *th,
- struct tx_arg *arg)
-{
- return out_obj_ref_del(env, arg->object, th);
-}
+ name = object_update_param_get(update, 0, NULL);
+ if (IS_ERR(name)) {
+ CERROR("%s: empty name for xattr set: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(name));
+ RETURN(PTR_ERR(name));
+ }
-static int __out_tx_ref_add(const struct lu_env *env,
- struct dt_object *dt_obj,
- struct thandle_exec_args *ta,
- struct update_reply *reply,
- int index, char *file, int line)
-{
- struct tx_arg *arg;
-
- LASSERT(ta->ta_handle != NULL);
- ta->ta_err = dt_declare_ref_add(env, dt_obj, ta->ta_handle);
- if (ta->ta_err != 0)
- return ta->ta_err;
-
- arg = tx_add_exec(ta, out_tx_ref_add_exec, out_tx_ref_add_undo, file,
- line);
- LASSERT(arg);
- lu_object_get(&dt_obj->do_lu);
- arg->object = dt_obj;
- arg->reply = reply;
- arg->index = index;
- return 0;
+ rc = out_tx_xattr_del(tsi->tsi_env, obj, name, &tti->tti_tea,
+ tti->tti_tea.ta_handle,
+ tti->tti_u.update.tti_update_reply,
+ tti->tti_u.update.tti_update_reply_index);
+ RETURN(rc);
}
/**
ENTRY;
rc = out_tx_ref_add(tsi->tsi_env, obj, &tti->tti_tea,
+ tti->tti_tea.ta_handle,
tti->tti_u.update.tti_update_reply,
tti->tti_u.update.tti_update_reply_index);
RETURN(rc);
}
-static int out_tx_ref_del_exec(const struct lu_env *env, struct thandle *th,
- struct tx_arg *arg)
-{
- struct dt_object *dt_obj = arg->object;
- int rc;
-
- rc = out_obj_ref_del(env, dt_obj, th);
-
- CDEBUG(D_INFO, "%s: insert ref_del reply %p index %d: rc = %d\n",
- dt_obd_name(th->th_dev), arg->reply, arg->index, 0);
-
- update_insert_reply(arg->reply, NULL, 0, arg->index, rc);
-
- return rc;
-}
-
-static int out_tx_ref_del_undo(const struct lu_env *env, struct thandle *th,
- struct tx_arg *arg)
-{
- return out_obj_ref_add(env, arg->object, th);
-}
-
-static int __out_tx_ref_del(const struct lu_env *env,
- struct dt_object *dt_obj,
- struct thandle_exec_args *ta,
- struct update_reply *reply,
- int index, char *file, int line)
-{
- struct tx_arg *arg;
-
- LASSERT(ta->ta_handle != NULL);
- ta->ta_err = dt_declare_ref_del(env, dt_obj, ta->ta_handle);
- if (ta->ta_err != 0)
- return ta->ta_err;
-
- arg = tx_add_exec(ta, out_tx_ref_del_exec, out_tx_ref_del_undo, file,
- line);
- LASSERT(arg);
- lu_object_get(&dt_obj->do_lu);
- arg->object = dt_obj;
- arg->reply = reply;
- arg->index = index;
- return 0;
-}
-
static int out_ref_del(struct tgt_session_info *tsi)
{
struct tgt_thread_info *tti = tgt_th_info(tsi->tsi_env);
RETURN(-ENOENT);
rc = out_tx_ref_del(tsi->tsi_env, obj, &tti->tti_tea,
+ tti->tti_tea.ta_handle,
tti->tti_u.update.tti_update_reply,
tti->tti_u.update.tti_update_reply_index);
RETURN(rc);
}
-static int out_obj_index_insert(const struct lu_env *env,
- struct dt_object *dt_obj,
- const struct dt_rec *rec,
- const struct dt_key *key,
- struct thandle *th)
-{
- int rc;
-
- CDEBUG(D_INFO, "%s: index insert "DFID" name: %s fid "DFID"\n",
- dt_obd_name(th->th_dev), PFID(lu_object_fid(&dt_obj->do_lu)),
- (char *)key, PFID((struct lu_fid *)rec));
-
- if (dt_try_as_dir(env, dt_obj) == 0)
- return -ENOTDIR;
-
- dt_write_lock(env, dt_obj, MOR_TGT_CHILD);
- rc = dt_insert(env, dt_obj, rec, key, th, NULL, 0);
- dt_write_unlock(env, dt_obj);
-
- return rc;
-}
-
-static int out_obj_index_delete(const struct lu_env *env,
- struct dt_object *dt_obj,
- const struct dt_key *key,
- struct thandle *th)
-{
- int rc;
-
- CDEBUG(D_INFO, "%s: index delete "DFID" name: %s\n",
- dt_obd_name(th->th_dev), PFID(lu_object_fid(&dt_obj->do_lu)),
- (char *)key);
-
- if (dt_try_as_dir(env, dt_obj) == 0)
- return -ENOTDIR;
-
- dt_write_lock(env, dt_obj, MOR_TGT_CHILD);
- rc = dt_delete(env, dt_obj, key, th, NULL);
- dt_write_unlock(env, dt_obj);
-
- return rc;
-}
-
-static int out_tx_index_insert_exec(const struct lu_env *env,
- struct thandle *th, struct tx_arg *arg)
-{
- struct dt_object *dt_obj = arg->object;
- int rc;
-
- rc = out_obj_index_insert(env, dt_obj, arg->u.insert.rec,
- arg->u.insert.key, th);
-
- CDEBUG(D_INFO, "%s: insert idx insert reply %p index %d: rc = %d\n",
- dt_obd_name(th->th_dev), arg->reply, arg->index, rc);
-
- update_insert_reply(arg->reply, NULL, 0, arg->index, rc);
-
- return rc;
-}
-
-static int out_tx_index_insert_undo(const struct lu_env *env,
- struct thandle *th, struct tx_arg *arg)
-{
- return out_obj_index_delete(env, arg->object, arg->u.insert.key, th);
-}
-
-static int __out_tx_index_insert(const struct lu_env *env,
- struct dt_object *dt_obj,
- char *name, struct lu_fid *fid,
- struct thandle_exec_args *ta,
- struct update_reply *reply,
- int index, char *file, int line)
-{
- struct tx_arg *arg;
-
- LASSERT(ta->ta_handle != NULL);
-
- if (lu_object_exists(&dt_obj->do_lu)) {
- if (dt_try_as_dir(env, dt_obj) == 0) {
- ta->ta_err = -ENOTDIR;
- return ta->ta_err;
- }
- ta->ta_err = dt_declare_insert(env, dt_obj,
- (struct dt_rec *)fid,
- (struct dt_key *)name,
- ta->ta_handle);
- }
-
- if (ta->ta_err != 0)
- return ta->ta_err;
-
- arg = tx_add_exec(ta, out_tx_index_insert_exec,
- out_tx_index_insert_undo, file,
- line);
- LASSERT(arg);
- lu_object_get(&dt_obj->do_lu);
- arg->object = dt_obj;
- arg->reply = reply;
- arg->index = index;
- arg->u.insert.rec = (struct dt_rec *)fid;
- arg->u.insert.key = (struct dt_key *)name;
-
- return 0;
-}
-
static int out_index_insert(struct tgt_session_info *tsi)
{
- struct tgt_thread_info *tti = tgt_th_info(tsi->tsi_env);
- struct update *update = tti->tti_u.update.tti_update;
- struct dt_object *obj = tti->tti_u.update.tti_dt_object;
- struct lu_fid *fid;
- char *name;
- int rc = 0;
- int size;
-
+ struct tgt_thread_info *tti = tgt_th_info(tsi->tsi_env);
+ struct object_update *update = tti->tti_u.update.tti_update;
+ struct dt_object *obj = tti->tti_u.update.tti_dt_object;
+ struct dt_insert_rec *rec = &tti->tti_rec;
+ struct lu_fid *fid;
+ char *name;
+ __u32 *ptype;
+ int rc = 0;
+ size_t size;
ENTRY;
- name = (char *)update_param_buf(update, 0, NULL);
- if (name == NULL) {
- CERROR("%s: empty name for index insert: rc = %d\n",
- tgt_name(tsi->tsi_tgt), -EPROTO);
- RETURN(err_serious(-EPROTO));
+ name = object_update_param_get(update, 0, NULL);
+ if (IS_ERR(name)) {
+ CERROR("%s: empty name for index insert: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(name));
+ RETURN(PTR_ERR(name));
}
- fid = (struct lu_fid *)update_param_buf(update, 1, &size);
- if (fid == NULL || size != sizeof(*fid)) {
- CERROR("%s: invalid fid: rc = %d\n",
- tgt_name(tsi->tsi_tgt), -EPROTO);
- RETURN(err_serious(-EPROTO));
+ fid = object_update_param_get(update, 1, &size);
+ if (IS_ERR(fid) || size != sizeof(*fid)) {
+ CERROR("%s: invalid fid: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(fid));
+ RETURN(PTR_ERR(fid));
}
- fid_le_to_cpu(fid, fid);
+ if (ptlrpc_req_need_swab(tsi->tsi_pill->rc_req))
+ lustre_swab_lu_fid(fid);
+
if (!fid_is_sane(fid)) {
CERROR("%s: invalid FID "DFID": rc = %d\n",
tgt_name(tsi->tsi_tgt), PFID(fid), -EPROTO);
- RETURN(err_serious(-EPROTO));
+ RETURN(-EPROTO);
}
- rc = out_tx_index_insert(tsi->tsi_env, obj, name, fid,
- &tti->tti_tea,
- tti->tti_u.update.tti_update_reply,
- tti->tti_u.update.tti_update_reply_index);
- RETURN(rc);
-}
-
-static int out_tx_index_delete_exec(const struct lu_env *env,
- struct thandle *th,
- struct tx_arg *arg)
-{
- int rc;
-
- rc = out_obj_index_delete(env, arg->object, arg->u.insert.key, th);
-
- CDEBUG(D_INFO, "%s: insert idx insert reply %p index %d: rc = %d\n",
- dt_obd_name(th->th_dev), arg->reply, arg->index, rc);
-
- update_insert_reply(arg->reply, NULL, 0, arg->index, rc);
+ ptype = object_update_param_get(update, 2, &size);
+ if (IS_ERR(ptype) || size != sizeof(*ptype)) {
+ CERROR("%s: invalid type for index insert: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(ptype));
+ RETURN(PTR_ERR(ptype));
+ }
- return rc;
-}
+ if (ptlrpc_req_need_swab(tsi->tsi_pill->rc_req))
+ __swab32s(ptype);
-static int out_tx_index_delete_undo(const struct lu_env *env,
- struct thandle *th,
- struct tx_arg *arg)
-{
- CERROR("%s: Oops, can not rollback index_delete yet: rc = %d\n",
- dt_obd_name(th->th_dev), -ENOTSUPP);
- return -ENOTSUPP;
-}
+ rec->rec_fid = fid;
+ rec->rec_type = *ptype;
-static int __out_tx_index_delete(const struct lu_env *env,
- struct dt_object *dt_obj, char *name,
- struct thandle_exec_args *ta,
- struct update_reply *reply,
- int index, char *file, int line)
-{
- struct tx_arg *arg;
+ rc = out_tx_index_insert(tsi->tsi_env, obj, (const struct dt_rec *)rec,
+ (const struct dt_key *)name, &tti->tti_tea,
+ tti->tti_tea.ta_handle,
+ tti->tti_u.update.tti_update_reply,
+ tti->tti_u.update.tti_update_reply_index);
- if (dt_try_as_dir(env, dt_obj) == 0) {
- ta->ta_err = -ENOTDIR;
- return ta->ta_err;
- }
+ CDEBUG(D_INFO, "%s: "DFID" index insert %s: rc = %d\n",
+ tgt_name(tsi->tsi_tgt), PFID(lu_object_fid(&obj->do_lu)),
+ name, rc);
- LASSERT(ta->ta_handle != NULL);
- ta->ta_err = dt_declare_delete(env, dt_obj,
- (struct dt_key *)name,
- ta->ta_handle);
- if (ta->ta_err != 0)
- return ta->ta_err;
-
- arg = tx_add_exec(ta, out_tx_index_delete_exec,
- out_tx_index_delete_undo, file,
- line);
- LASSERT(arg);
- lu_object_get(&dt_obj->do_lu);
- arg->object = dt_obj;
- arg->reply = reply;
- arg->index = index;
- arg->u.insert.key = (struct dt_key *)name;
- return 0;
+ RETURN(rc);
}
static int out_index_delete(struct tgt_session_info *tsi)
{
struct tgt_thread_info *tti = tgt_th_info(tsi->tsi_env);
- struct update *update = tti->tti_u.update.tti_update;
+ struct object_update *update = tti->tti_u.update.tti_update;
struct dt_object *obj = tti->tti_u.update.tti_dt_object;
char *name;
int rc = 0;
if (!lu_object_exists(&obj->do_lu))
RETURN(-ENOENT);
- name = (char *)update_param_buf(update, 0, NULL);
- if (name == NULL) {
- CERROR("%s: empty name for index delete: rc = %d\n",
- tgt_name(tsi->tsi_tgt), -EPROTO);
- RETURN(err_serious(-EPROTO));
+ name = object_update_param_get(update, 0, NULL);
+ if (IS_ERR(name)) {
+ CERROR("%s: empty name for index delete: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(name));
+ RETURN(PTR_ERR(name));
}
- rc = out_tx_index_delete(tsi->tsi_env, obj, name, &tti->tti_tea,
+ rc = out_tx_index_delete(tsi->tsi_env, obj, (const struct dt_key *)name,
+ &tti->tti_tea, tti->tti_tea.ta_handle,
tti->tti_u.update.tti_update_reply,
tti->tti_u.update.tti_update_reply_index);
RETURN(rc);
}
-static int out_tx_destroy_exec(const struct lu_env *env, struct thandle *th,
- struct tx_arg *arg)
-{
- struct dt_object *dt_obj = arg->object;
- int rc;
-
- rc = out_obj_destroy(env, dt_obj, th);
-
- CDEBUG(D_INFO, "%s: insert destroy reply %p index %d: rc = %d\n",
- dt_obd_name(th->th_dev), arg->reply, arg->index, rc);
-
- update_insert_reply(arg->reply, NULL, 0, arg->index, rc);
-
- RETURN(rc);
-}
-
-static int out_tx_destroy_undo(const struct lu_env *env, struct thandle *th,
- struct tx_arg *arg)
-{
- CERROR("%s: not support destroy undo yet!: rc = %d\n",
- dt_obd_name(th->th_dev), -ENOTSUPP);
- return -ENOTSUPP;
-}
-
-static int __out_tx_destroy(const struct lu_env *env, struct dt_object *dt_obj,
- struct thandle_exec_args *ta,
- struct update_reply *reply,
- int index, char *file, int line)
-{
- struct tx_arg *arg;
-
- LASSERT(ta->ta_handle != NULL);
- ta->ta_err = dt_declare_destroy(env, dt_obj, ta->ta_handle);
- if (ta->ta_err)
- return ta->ta_err;
-
- arg = tx_add_exec(ta, out_tx_destroy_exec, out_tx_destroy_undo,
- file, line);
- LASSERT(arg);
- lu_object_get(&dt_obj->do_lu);
- arg->object = dt_obj;
- arg->reply = reply;
- arg->index = index;
- return 0;
-}
-
static int out_destroy(struct tgt_session_info *tsi)
{
struct tgt_thread_info *tti = tgt_th_info(tsi->tsi_env);
- struct update *update = tti->tti_u.update.tti_update;
+ struct object_update *update = tti->tti_u.update.tti_update;
struct dt_object *obj = tti->tti_u.update.tti_dt_object;
struct lu_fid *fid;
int rc;
ENTRY;
- fid = &update->u_fid;
- fid_le_to_cpu(fid, fid);
+ fid = &update->ou_fid;
if (!fid_is_sane(fid)) {
CERROR("%s: invalid FID "DFID": rc = %d\n",
tgt_name(tsi->tsi_tgt), PFID(fid), -EPROTO);
- RETURN(err_serious(-EPROTO));
+ RETURN(-EPROTO);
}
if (!lu_object_exists(&obj->do_lu))
RETURN(-ENOENT);
rc = out_tx_destroy(tsi->tsi_env, obj, &tti->tti_tea,
+ tti->tti_tea.ta_handle,
tti->tti_u.update.tti_update_reply,
tti->tti_u.update.tti_update_reply_index);
RETURN(rc);
}
+static int out_write(struct tgt_session_info *tsi)
+{
+ struct tgt_thread_info *tti = tgt_th_info(tsi->tsi_env);
+ struct object_update *update = tti->tti_u.update.tti_update;
+ struct dt_object *obj = tti->tti_u.update.tti_dt_object;
+ struct lu_buf *lbuf = &tti->tti_buf;
+ char *buf;
+ __u64 *tmp;
+ size_t size = 0;
+ size_t buf_len = 0;
+ loff_t pos;
+ int rc;
+ ENTRY;
+
+ buf = object_update_param_get(update, 0, &buf_len);
+ if (IS_ERR(buf) || buf_len == 0) {
+ CERROR("%s: empty buf for xattr set: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(buf));
+ RETURN(PTR_ERR(buf));
+ }
+ lbuf->lb_buf = buf;
+ lbuf->lb_len = buf_len;
+
+ tmp = object_update_param_get(update, 1, &size);
+ if (IS_ERR(tmp) || size != sizeof(*tmp)) {
+ CERROR("%s: empty or wrong size %zu pos: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), size, PTR_ERR(tmp));
+ RETURN(PTR_ERR(tmp));
+ }
+
+ if (ptlrpc_req_need_swab(tsi->tsi_pill->rc_req))
+ __swab64s(tmp);
+ pos = *tmp;
+
+ rc = out_tx_write(tsi->tsi_env, obj, lbuf, pos,
+ &tti->tti_tea, tti->tti_tea.ta_handle,
+ tti->tti_u.update.tti_update_reply,
+ tti->tti_u.update.tti_update_reply_index);
+ RETURN(rc);
+}
+
+static int out_read(struct tgt_session_info *tsi)
+{
+ const struct lu_env *env = tsi->tsi_env;
+ struct tgt_thread_info *tti = tgt_th_info(env);
+ struct object_update *update = tti->tti_u.update.tti_update;
+ struct dt_object *obj = tti->tti_u.update.tti_dt_object;
+ struct object_update_reply *reply = tti->tti_u.update.tti_update_reply;
+ int index = tti->tti_u.update.tti_update_reply_index;
+ struct lu_rdbuf *rdbuf;
+ struct object_update_result *update_result;
+ struct out_read_reply *orr;
+ void *tmp;
+ size_t size;
+ size_t total_size = 0;
+ __u64 pos;
+ unsigned int i;
+ unsigned int nbufs;
+ int rc = 0;
+ ENTRY;
+
+ update_result = object_update_result_get(reply, index, NULL);
+ LASSERT(update_result != NULL);
+ update_result->our_datalen = sizeof(*orr);
+
+ if (!lu_object_exists(&obj->do_lu))
+ GOTO(out, rc = -ENOENT);
+
+ tmp = object_update_param_get(update, 0, NULL);
+ if (IS_ERR(tmp)) {
+ CERROR("%s: empty size for read: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(tmp));
+ GOTO(out, rc = PTR_ERR(tmp));
+ }
+ size = le64_to_cpu(*(size_t *)(tmp));
+
+ tmp = object_update_param_get(update, 1, NULL);
+ if (IS_ERR(tmp)) {
+ CERROR("%s: empty pos for read: rc = %ld\n",
+ tgt_name(tsi->tsi_tgt), PTR_ERR(tmp));
+ GOTO(out, rc = PTR_ERR(tmp));
+ }
+ pos = le64_to_cpu(*(__u64 *)(tmp));
+
+ /* Put the offset into the begining of the buffer in reply */
+ orr = (struct out_read_reply *)update_result->our_data;
+
+ nbufs = (size + OUT_BULK_BUFFER_SIZE - 1) / OUT_BULK_BUFFER_SIZE;
+ OBD_ALLOC(rdbuf, sizeof(struct lu_rdbuf) +
+ nbufs * sizeof(rdbuf->rb_bufs[0]));
+ if (rdbuf == NULL)
+ GOTO(out, rc = -ENOMEM);
+
+ rdbuf->rb_nbufs = 0;
+ total_size = 0;
+ for (i = 0; i < nbufs; i++) {
+ __u32 read_size;
+
+ read_size = size > OUT_BULK_BUFFER_SIZE ?
+ OUT_BULK_BUFFER_SIZE : size;
+ OBD_ALLOC(rdbuf->rb_bufs[i].lb_buf, read_size);
+ if (rdbuf->rb_bufs[i].lb_buf == NULL)
+ GOTO(out_free, rc = -ENOMEM);
+
+ rdbuf->rb_bufs[i].lb_len = read_size;
+ dt_read_lock(env, obj, DT_TGT_CHILD);
+ rc = dt_read(env, obj, &rdbuf->rb_bufs[i], &pos);
+ dt_read_unlock(env, obj);
+
+ total_size += rc < 0 ? 0 : rc;
+ if (rc <= 0)
+ break;
+
+ rdbuf->rb_nbufs++;
+ size -= read_size;
+ }
+
+ /* send pages to client */
+ rc = tgt_send_buffer(tsi, rdbuf);
+ if (rc < 0)
+ GOTO(out_free, rc);
+
+ orr->orr_size = total_size;
+ orr->orr_offset = pos;
+
+ orr_cpu_to_le(orr, orr);
+ update_result->our_datalen += orr->orr_size;
+out_free:
+ for (i = 0; i < nbufs; i++) {
+ if (rdbuf->rb_bufs[i].lb_buf != NULL) {
+ OBD_FREE(rdbuf->rb_bufs[i].lb_buf,
+ rdbuf->rb_bufs[i].lb_len);
+ }
+ }
+ OBD_FREE(rdbuf, sizeof(struct lu_rdbuf) +
+ nbufs * sizeof(rdbuf->rb_bufs[0]));
+out:
+ /* Insert read buffer */
+ update_result->our_rc = ptlrpc_status_hton(rc);
+ reply->ourp_lens[index] = cfs_size_round(update_result->our_datalen +
+ sizeof(*update_result));
+ RETURN(rc);
+}
+
+static int out_noop(struct tgt_session_info *tsi)
+{
+ return 0;
+}
+
#define DEF_OUT_HNDL(opc, name, flags, fn) \
-[opc - OBJ_CREATE] = { \
+[opc - OUT_CREATE] = { \
.th_name = name, \
.th_fail_id = 0, \
.th_opc = opc, \
.th_version = 0, \
}
-#define out_handler mdt_handler
static struct tgt_handler out_update_ops[] = {
- DEF_OUT_HNDL(OBJ_CREATE, "obj_create", MUTABOR | HABEO_REFERO,
+ DEF_OUT_HNDL(OUT_CREATE, "out_create", IS_MUTABLE | HAS_REPLY,
out_create),
- DEF_OUT_HNDL(OBJ_DESTROY, "obj_create", MUTABOR | HABEO_REFERO,
+ DEF_OUT_HNDL(OUT_DESTROY, "out_create", IS_MUTABLE | HAS_REPLY,
out_destroy),
- DEF_OUT_HNDL(OBJ_REF_ADD, "obj_ref_add", MUTABOR | HABEO_REFERO,
+ DEF_OUT_HNDL(OUT_REF_ADD, "out_ref_add", IS_MUTABLE | HAS_REPLY,
out_ref_add),
- DEF_OUT_HNDL(OBJ_REF_DEL, "obj_ref_del", MUTABOR | HABEO_REFERO,
+ DEF_OUT_HNDL(OUT_REF_DEL, "out_ref_del", IS_MUTABLE | HAS_REPLY,
out_ref_del),
- DEF_OUT_HNDL(OBJ_ATTR_SET, "obj_attr_set", MUTABOR | HABEO_REFERO,
+ DEF_OUT_HNDL(OUT_ATTR_SET, "out_attr_set", IS_MUTABLE | HAS_REPLY,
out_attr_set),
- DEF_OUT_HNDL(OBJ_ATTR_GET, "obj_attr_get", HABEO_REFERO,
+ DEF_OUT_HNDL(OUT_ATTR_GET, "out_attr_get", HAS_REPLY,
out_attr_get),
- DEF_OUT_HNDL(OBJ_XATTR_SET, "obj_xattr_set", MUTABOR | HABEO_REFERO,
+ DEF_OUT_HNDL(OUT_XATTR_SET, "out_xattr_set", IS_MUTABLE | HAS_REPLY,
out_xattr_set),
- DEF_OUT_HNDL(OBJ_XATTR_GET, "obj_xattr_get", HABEO_REFERO,
+ DEF_OUT_HNDL(OUT_XATTR_DEL, "out_xattr_del", IS_MUTABLE | HAS_REPLY,
+ out_xattr_del),
+ DEF_OUT_HNDL(OUT_XATTR_GET, "out_xattr_get", HAS_REPLY,
out_xattr_get),
- DEF_OUT_HNDL(OBJ_INDEX_LOOKUP, "obj_index_lookup", HABEO_REFERO,
+ DEF_OUT_HNDL(OUT_INDEX_LOOKUP, "out_index_lookup", HAS_REPLY,
out_index_lookup),
- DEF_OUT_HNDL(OBJ_INDEX_INSERT, "obj_index_insert",
- MUTABOR | HABEO_REFERO, out_index_insert),
- DEF_OUT_HNDL(OBJ_INDEX_DELETE, "obj_index_delete",
- MUTABOR | HABEO_REFERO, out_index_delete),
+ DEF_OUT_HNDL(OUT_INDEX_INSERT, "out_index_insert",
+ IS_MUTABLE | HAS_REPLY, out_index_insert),
+ DEF_OUT_HNDL(OUT_INDEX_DELETE, "out_index_delete",
+ IS_MUTABLE | HAS_REPLY, out_index_delete),
+ DEF_OUT_HNDL(OUT_WRITE, "out_write", IS_MUTABLE | HAS_REPLY, out_write),
+ DEF_OUT_HNDL(OUT_READ, "out_read", HAS_REPLY, out_read),
+ DEF_OUT_HNDL(OUT_NOOP, "out_noop", HAS_REPLY, out_noop),
+ DEF_OUT_HNDL(OUT_XATTR_LIST, "out_xattr_list", HAS_REPLY,
+ out_xattr_list),
};
-struct tgt_handler *out_handler_find(__u32 opc)
+static struct tgt_handler *out_handler_find(__u32 opc)
{
struct tgt_handler *h;
h = NULL;
- if (OBJ_CREATE <= opc && opc < OBJ_LAST) {
- h = &out_update_ops[opc - OBJ_CREATE];
+ if (OUT_CREATE <= opc && opc < OUT_LAST) {
+ h = &out_update_ops[opc - OUT_CREATE];
LASSERTF(h->th_opc == opc, "opcode mismatch %d != %d\n",
h->th_opc, opc);
} else {
return h;
}
+static int out_tx_start(const struct lu_env *env, struct dt_device *dt,
+ struct thandle_exec_args *ta, struct obd_export *exp)
+{
+ ta->ta_argno = 0;
+ ta->ta_handle = dt_trans_create(env, dt);
+ if (IS_ERR(ta->ta_handle)) {
+ int rc;
+
+ rc = PTR_ERR(ta->ta_handle);
+ ta->ta_handle = NULL;
+ CERROR("%s: start handle error: rc = %d\n", dt_obd_name(dt),
+ rc);
+ return rc;
+ }
+ if (exp->exp_need_sync)
+ ta->ta_handle->th_sync = 1;
+
+ return 0;
+}
+
+static int out_trans_start(const struct lu_env *env,
+ struct thandle_exec_args *ta)
+{
+ return dt_trans_start(env, ta->ta_handle->th_dev, ta->ta_handle);
+}
+
+static int out_trans_stop(const struct lu_env *env,
+ struct thandle_exec_args *ta, int err)
+{
+ int i;
+ int rc;
+
+ ta->ta_handle->th_result = err;
+ rc = dt_trans_stop(env, ta->ta_handle->th_dev, ta->ta_handle);
+ for (i = 0; i < ta->ta_argno; i++) {
+ if (ta->ta_args[i]->object != NULL) {
+ dt_object_put(env, ta->ta_args[i]->object);
+ ta->ta_args[i]->object = NULL;
+ }
+ }
+ ta->ta_handle = NULL;
+ ta->ta_argno = 0;
+
+ return rc;
+}
+
+static int out_tx_end(const struct lu_env *env, struct thandle_exec_args *ta,
+ int declare_ret)
+{
+ struct tgt_session_info *tsi = tgt_ses_info(env);
+ int i;
+ int rc;
+ int rc1;
+ ENTRY;
+
+ if (ta->ta_handle == NULL)
+ RETURN(0);
+
+ if (declare_ret != 0 || ta->ta_argno == 0)
+ GOTO(stop, rc = declare_ret);
+
+ LASSERT(ta->ta_handle->th_dev != NULL);
+ rc = out_trans_start(env, ta);
+ if (unlikely(rc != 0))
+ GOTO(stop, rc);
+
+ for (i = 0; i < ta->ta_argno; i++) {
+ rc = ta->ta_args[i]->exec_fn(env, ta->ta_handle,
+ ta->ta_args[i]);
+ if (unlikely(rc != 0)) {
+ CDEBUG(D_INFO, "error during execution of #%u from"
+ " %s:%d: rc = %d\n", i, ta->ta_args[i]->file,
+ ta->ta_args[i]->line, rc);
+ while (--i >= 0) {
+ if (ta->ta_args[i]->undo_fn != NULL)
+ ta->ta_args[i]->undo_fn(env,
+ ta->ta_handle,
+ ta->ta_args[i]);
+ else
+ CERROR("%s: undo for %s:%d: rc = %d\n",
+ dt_obd_name(ta->ta_handle->th_dev),
+ ta->ta_args[i]->file,
+ ta->ta_args[i]->line, -ENOTSUPP);
+ }
+ break;
+ }
+ CDEBUG(D_INFO, "%s: executed %u/%u: rc = %d\n",
+ dt_obd_name(ta->ta_handle->th_dev), i, ta->ta_argno, rc);
+ }
+
+ /* Only fail for real updates, XXX right now llog updates will be
+ * ignore, whose updates count is usually 1, so failover test
+ * case will spot this FAIL_UPDATE_NET_REP precisely, and it will
+ * be removed after async update patch is landed. */
+ if (ta->ta_argno > 1)
+ tsi->tsi_reply_fail_id = OBD_FAIL_OUT_UPDATE_NET_REP;
+
+stop:
+ rc1 = out_trans_stop(env, ta, rc);
+ if (rc == 0)
+ rc = rc1;
+
+ ta->ta_handle = NULL;
+ ta->ta_argno = 0;
+
+ RETURN(rc);
+}
+
/**
* Object updates between Targets. Because all the updates has been
* dis-assemblied into object updates at sender side, so OUT will
struct thandle_exec_args *ta = &tti->tti_tea;
struct req_capsule *pill = tsi->tsi_pill;
struct dt_device *dt = tsi->tsi_tgt->lut_bottom;
- struct update_buf *ubuf;
- struct update *update;
- struct update_reply *update_reply;
- int bufsize;
- int count;
- int old_batchid = -1;
- unsigned off;
- int i;
- int rc = 0;
- int rc1 = 0;
-
+ struct out_update_header *ouh;
+ struct out_update_buffer *oub = NULL;
+ struct object_update *update;
+ struct object_update_reply *reply;
+ struct ptlrpc_bulk_desc *desc = NULL;
+ struct l_wait_info lwi;
+ void **update_bufs;
+ int current_batchid = -1;
+ __u32 update_buf_count;
+ unsigned int i;
+ unsigned int reply_index = 0;
+ int rc = 0;
+ int rc1 = 0;
+ int ouh_size, reply_size;
+ int updates;
ENTRY;
- req_capsule_set(pill, &RQF_UPDATE_OBJ);
- bufsize = req_capsule_get_size(pill, &RMF_UPDATE, RCL_CLIENT);
- if (bufsize != UPDATE_BUFFER_SIZE) {
- CERROR("%s: invalid bufsize %d: rc = %d\n",
- tgt_name(tsi->tsi_tgt), bufsize, -EPROTO);
+ req_capsule_set(pill, &RQF_OUT_UPDATE);
+ ouh_size = req_capsule_get_size(pill, &RMF_OUT_UPDATE_HEADER,
+ RCL_CLIENT);
+ if (ouh_size <= 0)
RETURN(err_serious(-EPROTO));
- }
- ubuf = req_capsule_client_get(pill, &RMF_UPDATE);
- if (ubuf == NULL) {
- CERROR("%s: No buf!: rc = %d\n", tgt_name(tsi->tsi_tgt),
- -EPROTO);
+ ouh = req_capsule_client_get(pill, &RMF_OUT_UPDATE_HEADER);
+ if (ouh == NULL)
RETURN(err_serious(-EPROTO));
- }
- if (le32_to_cpu(ubuf->ub_magic) != UPDATE_BUFFER_MAGIC) {
- CERROR("%s: invalid magic %x expect %x: rc = %d\n",
- tgt_name(tsi->tsi_tgt), le32_to_cpu(ubuf->ub_magic),
- UPDATE_BUFFER_MAGIC, -EPROTO);
+ if (ouh->ouh_magic != OUT_UPDATE_HEADER_MAGIC) {
+ CERROR("%s: invalid update buffer magic %x expect %x: "
+ "rc = %d\n", tgt_name(tsi->tsi_tgt), ouh->ouh_magic,
+ UPDATE_REQUEST_MAGIC, -EPROTO);
RETURN(err_serious(-EPROTO));
}
- count = le32_to_cpu(ubuf->ub_count);
- if (count <= 0) {
- CERROR("%s: No update!: rc = %d\n",
- tgt_name(tsi->tsi_tgt), -EPROTO);
+ update_buf_count = ouh->ouh_count;
+ if (update_buf_count == 0)
RETURN(err_serious(-EPROTO));
+
+ OBD_ALLOC(update_bufs, sizeof(*update_bufs) * update_buf_count);
+ if (update_bufs == NULL)
+ RETURN(err_serious(-ENOMEM));
+
+ if (ouh->ouh_inline_length > 0) {
+ update_bufs[0] = ouh->ouh_inline_data;
+ } else {
+ struct out_update_buffer *tmp;
+
+ oub = req_capsule_client_get(pill, &RMF_OUT_UPDATE_BUF);
+ if (oub == NULL)
+ GOTO(out_free, rc = err_serious(-EPROTO));
+
+ desc = ptlrpc_prep_bulk_exp(pill->rc_req, update_buf_count,
+ PTLRPC_BULK_OPS_COUNT,
+ PTLRPC_BULK_GET_SINK |
+ PTLRPC_BULK_BUF_KVEC,
+ MDS_BULK_PORTAL,
+ &ptlrpc_bulk_kvec_ops);
+ if (desc == NULL)
+ GOTO(out_free, rc = err_serious(-ENOMEM));
+
+ tmp = oub;
+ for (i = 0; i < update_buf_count; i++, tmp++) {
+ if (tmp->oub_size >= OUT_MAXREQSIZE)
+ GOTO(out_free, rc = err_serious(-EPROTO));
+
+ OBD_ALLOC_LARGE(update_bufs[i], tmp->oub_size);
+ if (update_bufs[i] == NULL)
+ GOTO(out_free, rc = err_serious(-ENOMEM));
+
+ desc->bd_frag_ops->add_iov_frag(desc, update_bufs[i],
+ tmp->oub_size);
+ }
+
+ pill->rc_req->rq_bulk_write = 1;
+ rc = sptlrpc_svc_prep_bulk(pill->rc_req, desc);
+ if (rc != 0)
+ GOTO(out_free, rc = err_serious(rc));
+
+ rc = target_bulk_io(pill->rc_req->rq_export, desc, &lwi);
+ if (rc < 0)
+ GOTO(out_free, rc = err_serious(rc));
}
+ /* validate the request and calculate the total update count and
+ * set it to reply */
+ reply_size = 0;
+ updates = 0;
+ for (i = 0; i < update_buf_count; i++) {
+ struct object_update_request *our;
+ int j;
+
+ our = update_bufs[i];
+ if (ptlrpc_req_need_swab(pill->rc_req))
+ lustre_swab_object_update_request(our);
+
+ if (our->ourq_magic != UPDATE_REQUEST_MAGIC) {
+ CERROR("%s: invalid update buffer magic %x"
+ " expect %x: rc = %d\n",
+ tgt_name(tsi->tsi_tgt), our->ourq_magic,
+ UPDATE_REQUEST_MAGIC, -EPROTO);
+ GOTO(out_free, rc = err_serious(-EPROTO));
+ }
+ updates += our->ourq_count;
+
+ /* need to calculate reply size */
+ for (j = 0; j < our->ourq_count; j++) {
+ update = object_update_request_get(our, j, NULL);
+ if (update == NULL)
+ GOTO(out, rc = err_serious(-EPROTO));
+ if (ptlrpc_req_need_swab(pill->rc_req))
+ lustre_swab_object_update(update);
+
+ if (!fid_is_sane(&update->ou_fid)) {
+ CERROR("%s: invalid FID "DFID": rc = %d\n",
+ tgt_name(tsi->tsi_tgt),
+ PFID(&update->ou_fid), -EPROTO);
+ GOTO(out, rc = err_serious(-EPROTO));
+ }
- req_capsule_set_size(pill, &RMF_UPDATE_REPLY, RCL_SERVER,
- UPDATE_BUFFER_SIZE);
+ /* XXX: what ou_result_size can be considered safe? */
+
+ reply_size += sizeof(reply->ourp_lens[0]);
+ reply_size += sizeof(struct object_update_result);
+ reply_size += update->ou_result_size;
+ }
+ }
+ reply_size += sizeof(*reply);
+
+ if (unlikely(reply_size > ouh->ouh_reply_size)) {
+ CERROR("%s: too small reply buf %u for %u, need %u at least\n",
+ tgt_name(tsi->tsi_tgt), ouh->ouh_reply_size,
+ updates, reply_size);
+ GOTO(out_free, rc = err_serious(-EPROTO));
+ }
+
+ req_capsule_set_size(pill, &RMF_OUT_UPDATE_REPLY, RCL_SERVER,
+ ouh->ouh_reply_size);
rc = req_capsule_server_pack(pill);
if (rc != 0) {
CERROR("%s: Can't pack response: rc = %d\n",
tgt_name(tsi->tsi_tgt), rc);
- RETURN(rc);
+ GOTO(out_free, rc = err_serious(-EPROTO));
}
/* Prepare the update reply buffer */
- update_reply = req_capsule_server_get(pill, &RMF_UPDATE_REPLY);
- update_init_reply_buf(update_reply, count);
- tti->tti_u.update.tti_update_reply = update_reply;
-
- rc = out_tx_start(env, dt, ta);
- if (rc != 0)
- RETURN(rc);
-
- /* Walk through updates in the request to execute them synchronously */
- off = cfs_size_round(offsetof(struct update_buf, ub_bufs[0]));
- for (i = 0; i < count; i++) {
+ reply = req_capsule_server_get(pill, &RMF_OUT_UPDATE_REPLY);
+ if (reply == NULL)
+ GOTO(out_free, rc = -EPROTO);
+ reply->ourp_magic = UPDATE_REPLY_MAGIC;
+ reply->ourp_count = updates;
+ tti->tti_u.update.tti_update_reply = reply;
+ tti->tti_mult_trans = !req_is_replay(tgt_ses_req(tsi));
+
+ /* Walk through updates in the request to execute them */
+ for (i = 0; i < update_buf_count; i++) {
struct tgt_handler *h;
struct dt_object *dt_obj;
+ int update_count;
+ struct object_update_request *our;
+ int j;
+
+ our = update_bufs[i];
+ update_count = our->ourq_count;
+ for (j = 0; j < update_count; j++) {
+ struct lu_object_conf conf;
+
+ update = object_update_request_get(our, j, NULL);
+ if (update->ou_type == OUT_CREATE)
+ conf.loc_flags = LOC_F_NEW;
+ else
+ conf.loc_flags = 0;
+
+ dt_obj = dt_locate_at(env, dt, &update->ou_fid,
+ dt->dd_lu_dev.ld_site->ls_top_dev, &conf);
+ if (IS_ERR(dt_obj))
+ GOTO(out, rc = PTR_ERR(dt_obj));
+
+ if (dt->dd_record_fid_accessed) {
+ struct lfsck_req_local *lrl = &tti->tti_lrl;
+
+ lfsck_pack_rfa(lrl,
+ lu_object_fid(&dt_obj->do_lu),
+ LEL_FID_ACCESSED,
+ LFSCK_TYPE_LAYOUT);
+ tgt_lfsck_in_notify_local(env, dt, lrl, NULL);
+ }
- update = (struct update *)((char *)ubuf + off);
- if (old_batchid == -1) {
- old_batchid = update->u_batchid;
- } else if (old_batchid != update->u_batchid) {
- /* Stop the current update transaction,
- * create a new one */
- rc = out_tx_end(env, ta);
- if (rc != 0)
- RETURN(rc);
-
- rc = out_tx_start(env, dt, ta);
- if (rc != 0)
- RETURN(rc);
- old_batchid = update->u_batchid;
- }
-
- fid_le_to_cpu(&update->u_fid, &update->u_fid);
- if (!fid_is_sane(&update->u_fid)) {
- CERROR("%s: invalid FID "DFID": rc = %d\n",
- tgt_name(tsi->tsi_tgt), PFID(&update->u_fid),
- -EPROTO);
- GOTO(out, rc = err_serious(-EPROTO));
- }
-
- dt_obj = dt_locate(env, dt, &update->u_fid);
- if (IS_ERR(dt_obj))
- GOTO(out, rc = PTR_ERR(dt_obj));
+ tti->tti_u.update.tti_dt_object = dt_obj;
+ tti->tti_u.update.tti_update = update;
+ tti->tti_u.update.tti_update_reply_index = reply_index;
- tti->tti_u.update.tti_dt_object = dt_obj;
- tti->tti_u.update.tti_update = update;
- tti->tti_u.update.tti_update_reply_index = i;
+ h = out_handler_find(update->ou_type);
+ if (unlikely(h == NULL)) {
+ CERROR("%s: unsupported opc: 0x%x\n",
+ tgt_name(tsi->tsi_tgt), update->ou_type);
+ GOTO(next, rc = -ENOTSUPP);
+ }
- h = out_handler_find(update->u_type);
- if (likely(h != NULL)) {
- /* For real modification RPC, check if the update
- * has been executed */
- if (h->th_flags & MUTABOR) {
+ /* Check resend case only for modifying RPC */
+ if (h->th_flags & IS_MUTABLE) {
struct ptlrpc_request *req = tgt_ses_req(tsi);
if (out_check_resent(env, dt, dt_obj, req,
- out_reconstruct,
- update_reply, i))
+ out_reconstruct, reply,
+ reply_index))
+ GOTO(next, rc = 0);
+
+ if (dt->dd_rdonly)
+ GOTO(next, rc = -EROFS);
+ }
+
+ /* start transaction for modification RPC only */
+ if (h->th_flags & IS_MUTABLE && current_batchid == -1) {
+ current_batchid = update->ou_batchid;
+ rc = out_tx_start(env, dt, ta, tsi->tsi_exp);
+ if (rc != 0)
+ GOTO(next, rc);
+
+ if (update->ou_flags & UPDATE_FL_SYNC)
+ ta->ta_handle->th_sync = 1;
+ }
+
+ /* Stop the current update transaction, if the update
+ * has different batchid, or read-only update */
+ if (((current_batchid != update->ou_batchid) ||
+ !(h->th_flags & IS_MUTABLE)) &&
+ ta->ta_handle != NULL) {
+ rc = out_tx_end(env, ta, rc);
+ current_batchid = -1;
+ if (rc != 0)
GOTO(next, rc);
+
+ /* start a new transaction if needed */
+ if (h->th_flags & IS_MUTABLE) {
+ rc = out_tx_start(env, dt, ta,
+ tsi->tsi_exp);
+ if (rc != 0)
+ GOTO(next, rc);
+ if (update->ou_flags & UPDATE_FL_SYNC)
+ ta->ta_handle->th_sync = 1;
+ current_batchid = update->ou_batchid;
+ }
}
rc = h->th_act(tsi);
- } else {
- CERROR("%s: The unsupported opc: 0x%x\n",
- tgt_name(tsi->tsi_tgt), update->u_type);
- lu_object_put(env, &dt_obj->do_lu);
- GOTO(out, rc = -ENOTSUPP);
- }
next:
- lu_object_put(env, &dt_obj->do_lu);
- if (rc < 0)
- GOTO(out, rc);
- off += cfs_size_round(update_size(update));
+ reply_index++;
+ dt_object_put(env, dt_obj);
+ if (rc < 0)
+ GOTO(out, rc);
+ }
}
out:
- rc1 = out_tx_end(env, ta);
- if (rc == 0)
- rc = rc1;
+ if (current_batchid != -1) {
+ rc1 = out_tx_end(env, ta, rc);
+ if (rc == 0)
+ rc = rc1;
+ }
+
+out_free:
+ if (update_bufs != NULL) {
+ if (oub != NULL) {
+ for (i = 0; i < update_buf_count; i++, oub++) {
+ if (update_bufs[i] != NULL)
+ OBD_FREE_LARGE(update_bufs[i],
+ oub->oub_size);
+ }
+ }
+
+ OBD_FREE(update_bufs, sizeof(*update_bufs) * update_buf_count);
+ }
+
+ if (desc != NULL)
+ ptlrpc_free_bulk(desc);
+
RETURN(rc);
}
struct tgt_handler tgt_out_handlers[] = {
-TGT_UPDATE_HDL(MUTABOR, UPDATE_OBJ, out_handle),
+TGT_UPDATE_HDL(IS_MUTABLE, OUT_UPDATE, out_handle),
};
EXPORT_SYMBOL(tgt_out_handlers);
git://git.whamcloud.com / fs / lustre-release.git / blobdiff