}
static int mdd_declare_rename(const struct lu_env *env,
- struct mdd_device *mdd,
- struct mdd_object *mdd_spobj,
- struct mdd_object *mdd_tpobj,
- struct mdd_object *mdd_sobj,
- struct mdd_object *mdd_tobj,
- const struct lu_name *tname,
+ struct mdd_device *mdd,
+ struct mdd_object *mdd_spobj,
+ struct mdd_object *mdd_tpobj,
+ struct mdd_object *mdd_sobj,
+ struct mdd_object *mdd_tobj,
+ const struct lu_name *tname,
const struct lu_name *sname,
- struct md_attr *ma,
- struct thandle *handle)
+ struct md_attr *ma,
+ struct linkea_data *ldata,
+ struct thandle *handle)
{
- int rc;
+ struct lu_attr *la = &mdd_env_info(env)->mti_la_for_fix;
+ int rc;
LASSERT(mdd_spobj);
LASSERT(mdd_tpobj);
rc = mdo_declare_ref_del(env, mdd_spobj, handle);
if (rc)
return rc;
+ if (mdd_spobj != mdd_tpobj) {
+ rc = mdo_declare_index_delete(env, mdd_sobj, dotdot,
+ handle);
+ if (rc)
+ return rc;
- rc = mdo_declare_index_delete(env, mdd_sobj, dotdot, handle);
- if (rc)
- return rc;
-
- rc = mdo_declare_index_insert(env, mdd_sobj, mdo2fid(mdd_tpobj),
- dotdot, handle);
- if (rc)
- return rc;
-
+ rc = mdo_declare_index_insert(env, mdd_sobj,
+ mdo2fid(mdd_tpobj),
+ dotdot, handle);
+ if (rc)
+ return rc;
+ }
/* new target child can be directory,
* counted by target dir's nlink */
rc = mdo_declare_ref_add(env, mdd_tpobj, handle);
if (rc)
return rc;
- rc = mdo_declare_attr_set(env, mdd_sobj, NULL, handle);
- if (rc)
- return rc;
- mdd_declare_links_add(env, mdd_sobj, handle, NULL);
- if (rc)
- return rc;
+ LASSERT(ma->ma_attr.la_valid & LA_CTIME);
+ la->la_ctime = la->la_mtime = ma->ma_attr.la_ctime;
+ la->la_valid = LA_CTIME | LA_MTIME;
+ rc = mdo_declare_attr_set(env, mdd_sobj, la, handle);
+ if (rc)
+ return rc;
+
+ rc = mdd_declare_links_add(env, mdd_sobj, handle, ldata);
+ if (rc)
+ return rc;
rc = mdo_declare_attr_set(env, mdd_tpobj, NULL, handle);
if (rc)
struct md_object *tobj, const struct lu_name *ltname,
struct md_attr *ma)
{
- const char *sname = lsname->ln_name;
- const char *tname = ltname->ln_name;
- struct lu_attr *la = &mdd_env_info(env)->mti_la_for_fix;
+ const char *sname = lsname->ln_name;
+ const char *tname = ltname->ln_name;
+ struct lu_attr *la = &mdd_env_info(env)->mti_la_for_fix;
struct lu_attr *so_attr = &mdd_env_info(env)->mti_cattr;
struct lu_attr *tg_attr = &mdd_env_info(env)->mti_pattr;
- struct mdd_object *mdd_spobj = md2mdd_obj(src_pobj); /* source parent */
- struct mdd_object *mdd_tpobj = md2mdd_obj(tgt_pobj);
- struct mdd_device *mdd = mdo2mdd(src_pobj);
- struct mdd_object *mdd_sobj = NULL; /* source object */
- struct mdd_object *mdd_tobj = NULL;
- struct dynlock_handle *sdlh, *tdlh;
- struct thandle *handle;
- const struct lu_fid *tpobj_fid = mdo2fid(mdd_tpobj);
- const struct lu_fid *spobj_fid = mdo2fid(mdd_spobj);
- bool is_dir;
+ struct mdd_object *mdd_spobj = md2mdd_obj(src_pobj); /* source parent */
+ struct mdd_object *mdd_tpobj = md2mdd_obj(tgt_pobj);
+ struct mdd_device *mdd = mdo2mdd(src_pobj);
+ struct mdd_object *mdd_sobj = NULL; /* source object */
+ struct mdd_object *mdd_tobj = NULL;
+ struct dynlock_handle *sdlh = NULL, *tdlh = NULL;
+ struct thandle *handle;
+ struct linkea_data *ldata = &mdd_env_info(env)->mti_link_data;
+ const struct lu_fid *tpobj_fid = mdo2fid(mdd_tpobj);
+ const struct lu_fid *spobj_fid = mdo2fid(mdd_spobj);
+ bool is_dir;
bool tobj_ref = 0;
bool tobj_locked = 0;
unsigned cl_flags = 0;
- int rc, rc2;
- ENTRY;
+ int rc, rc2;
+ ENTRY;
if (tobj)
mdd_tobj = md2mdd_obj(tobj);
mdd_sobj = mdd_object_find(env, mdd, lf);
+ rc = mdd_la_get(env, mdd_sobj, so_attr,
+ mdd_object_capa(env, mdd_sobj));
+ if (rc)
+ GOTO(out_pending, rc);
+
+ if (mdd_tobj) {
+ rc = mdd_la_get(env, mdd_tobj, tg_attr,
+ mdd_object_capa(env, mdd_tobj));
+ if (rc)
+ GOTO(out_pending, rc);
+ }
+
+ rc = mdd_rename_sanity_check(env, mdd_spobj, mdd_tpobj, mdd_sobj,
+ mdd_tobj, so_attr, tg_attr);
+ if (rc)
+ GOTO(out_pending, rc);
+
handle = mdd_trans_create(env, mdd);
if (IS_ERR(handle))
GOTO(out_pending, rc = PTR_ERR(handle));
- rc = mdd_declare_rename(env, mdd, mdd_spobj, mdd_tpobj, mdd_sobj,
- mdd_tobj, lsname, ltname, ma, handle);
- if (rc)
- GOTO(stop, rc);
+ memset(ldata, 0, sizeof(*ldata));
+ mdd_linkea_prepare(env, mdd_sobj, NULL, NULL, mdd_object_fid(mdd_tpobj),
+ ltname, 1, 0, ldata);
+ rc = mdd_declare_rename(env, mdd, mdd_spobj, mdd_tpobj, mdd_sobj,
+ mdd_tobj, lsname, ltname, ma, ldata, handle);
+ if (rc)
+ GOTO(stop, rc);
rc = mdd_trans_start(env, mdd, handle);
if (rc)
if (sdlh == NULL || tdlh == NULL)
GOTO(cleanup, rc = -ENOMEM);
- rc = mdd_la_get(env, mdd_sobj, so_attr,
- mdd_object_capa(env, mdd_sobj));
- if (rc)
- GOTO(cleanup, rc);
-
- if (mdd_tobj) {
- rc = mdd_la_get(env, mdd_tobj, tg_attr,
- mdd_object_capa(env, mdd_tobj));
- if (rc)
- GOTO(cleanup, rc);
- }
-
- rc = mdd_rename_sanity_check(env, mdd_spobj, mdd_tpobj, mdd_sobj,
- mdd_tobj, so_attr, tg_attr);
- if (rc)
- GOTO(cleanup, rc);
-
is_dir = S_ISDIR(so_attr->la_mode);
/* Remove source name from source directory */
handle, 0);
}
- if (rc == 0 && mdd_sobj) {
- mdd_write_lock(env, mdd_sobj, MOR_SRC_CHILD);
+ if (rc == 0 && mdd_sobj) {
+ mdd_write_lock(env, mdd_sobj, MOR_SRC_CHILD);
rc = mdd_links_rename(env, mdd_sobj, mdo2fid(mdd_spobj), lsname,
mdo2fid(mdd_tpobj), ltname, handle, NULL,
0, 0);
}
fixup_spobj:
- if (rc && is_dir && mdd_sobj) {
- rc2 = __mdd_index_delete_only(env, mdd_sobj, dotdot, handle,
- BYPASS_CAPA);
+ if (rc && is_dir && mdd_sobj && mdd_spobj != mdd_tpobj) {
+ rc2 = __mdd_index_delete_only(env, mdd_sobj, dotdot, handle,
+ BYPASS_CAPA);
- if (rc2)
- CWARN("sp obj dotdot delete error %d\n",rc2);
+ if (rc2)
+ CWARN("%s: sp obj dotdot delete error: rc = %d\n",
+ mdd2obd_dev(mdd)->obd_name, rc2);
- rc2 = __mdd_index_insert_only(env, mdd_sobj, spobj_fid,
- dotdot, handle, BYPASS_CAPA);
- if (rc2)
- CWARN("sp obj dotdot insert error %d\n",rc2);
- }
+ rc2 = __mdd_index_insert_only(env, mdd_sobj, spobj_fid,
+ dotdot, handle, BYPASS_CAPA);
+ if (rc2)
+ CWARN("%s: sp obj dotdot insert error: rc = %d\n",
+ mdd2obd_dev(mdd)->obd_name, rc2);
+ }
fixup_spobj2:
if (rc) {
* 2 - src child; 3 - tgt child.
* Update on disk version of src child.
*/
+/**
+ * For DNE phase I, only these renames are allowed
+ * mv src_p/src_c tgt_p/tgt_c
+ * 1. src_p/src_c/tgt_p/tgt_c are in the same MDT.
+ * 2. src_p and tgt_p are same directory, and tgt_c does not
+ * exists. In this case, all of modification will happen
+ * in the MDT where ithesource parent is, only one remote
+ * update is needed, i.e. set c_time/m_time on the child.
+ * And tgt_c will be still in the same MDT as the original
+ * src_c.
+ */
static int mdt_reint_rename(struct mdt_thread_info *info,
struct mdt_lock_handle *lhc)
{
mold = mdt_object_find(info->mti_env, info->mti_mdt, old_fid);
if (IS_ERR(mold))
GOTO(out_unlock_target, rc = PTR_ERR(mold));
- if (mdt_object_remote(mold)) {
- mdt_object_put(info->mti_env, mold);
- CDEBUG(D_INFO, "Source child "DFID" is on another MDT\n",
- PFID(old_fid));
- GOTO(out_unlock_target, rc = -EXDEV);
- }
lh_oldp = &info->mti_lh[MDT_LH_OLD];
mdt_lock_reg_init(lh_oldp, LCK_EX);
if (fid_is_obf(new_fid) || fid_is_dot_lustre(new_fid))
GOTO(out_unlock_old, rc = -EPERM);
+ if (mdt_object_remote(mold)) {
+ CDEBUG(D_INFO, "Src child "DFID" is on another MDT\n",
+ PFID(old_fid));
+ GOTO(out_unlock_old, rc = -EXDEV);
+ }
+
mdt_lock_reg_init(lh_newp, LCK_EX);
mnew = mdt_object_find(info->mti_env, info->mti_mdt, new_fid);
if (IS_ERR(mnew))