Whamcloud - gitweb
LU-6210 mdd: Change positional struct initializers to C99
[fs/lustre-release.git] / lustre / mdd / mdd_dir.c
index 2cb3736..b37c795 100644 (file)
  *
  * You should have received a copy of the GNU General Public License
  * version 2 along with this program; If not, see
- * http://www.sun.com/software/products/lustre/docs/GPLv2.pdf
- *
- * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
- * CA 95054 USA or visit www.sun.com if you need additional information or
- * have any questions.
+ * http://www.gnu.org/licenses/gpl-2.0.html
  *
  * GPL HEADER END
  */
@@ -27,7 +23,7 @@
  * Copyright (c) 2007, 2010, Oracle and/or its affiliates. All rights reserved.
  * Use is subject to license terms.
  *
- * Copyright (c) 2011, 2015, Intel Corporation.
+ * Copyright (c) 2011, 2016, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
@@ -53,8 +49,8 @@ static const char dot[] = ".";
 static const char dotdot[] = "..";
 
 static struct lu_name lname_dotdot = {
-        (char *) dotdot,
-        sizeof(dotdot) - 1
+       .ln_name        = (char *) dotdot,
+       .ln_namelen     = sizeof(dotdot) - 1,
 };
 
 static inline int
@@ -124,6 +120,77 @@ int mdd_lookup(const struct lu_env *env,
         RETURN(rc);
 }
 
+/** Read the link EA into a temp buffer.
+ * Uses the mdd_thread_info::mti_big_buf since it is generally large.
+ * A pointer to the buffer is stored in \a ldata::ld_buf.
+ *
+ * \retval 0 or error
+ */
+static int __mdd_links_read(const struct lu_env *env,
+                           struct mdd_object *mdd_obj,
+                           struct linkea_data *ldata)
+{
+       int rc;
+
+       if (!mdd_object_exists(mdd_obj))
+               return -ENODATA;
+
+       /* First try a small buf */
+       LASSERT(env != NULL);
+       ldata->ld_buf = lu_buf_check_and_alloc(&mdd_env_info(env)->mti_link_buf,
+                                              PAGE_SIZE);
+       if (ldata->ld_buf->lb_buf == NULL)
+               return -ENOMEM;
+
+       rc = mdo_xattr_get(env, mdd_obj, ldata->ld_buf, XATTR_NAME_LINK);
+       if (rc == -ERANGE) {
+               /* Buf was too small, figure out what we need. */
+               lu_buf_free(ldata->ld_buf);
+               rc = mdo_xattr_get(env, mdd_obj, ldata->ld_buf,
+                                  XATTR_NAME_LINK);
+               if (rc < 0)
+                       return rc;
+               ldata->ld_buf = lu_buf_check_and_alloc(ldata->ld_buf, rc);
+               if (ldata->ld_buf->lb_buf == NULL)
+                       return -ENOMEM;
+               rc = mdo_xattr_get(env, mdd_obj, ldata->ld_buf,
+                                 XATTR_NAME_LINK);
+       }
+       if (rc < 0) {
+               lu_buf_free(ldata->ld_buf);
+               ldata->ld_buf = NULL;
+               return rc;
+       }
+
+       return linkea_init(ldata);
+}
+
+static int mdd_links_read(const struct lu_env *env,
+                         struct mdd_object *mdd_obj,
+                         struct linkea_data *ldata)
+{
+       int rc;
+
+       rc = __mdd_links_read(env, mdd_obj, ldata);
+       if (!rc)
+               rc = linkea_init(ldata);
+
+       return rc;
+}
+
+static int mdd_links_read_with_rec(const struct lu_env *env,
+                                  struct mdd_object *mdd_obj,
+                                  struct linkea_data *ldata)
+{
+       int rc;
+
+       rc = __mdd_links_read(env, mdd_obj, ldata);
+       if (!rc)
+               rc = linkea_init_with_rec(ldata);
+
+       return rc;
+}
+
 /**
  * Get parent FID of the directory
  *
@@ -158,7 +225,7 @@ static inline int mdd_parent_fid(const struct lu_env *env,
                GOTO(lookup, rc = 0);
 
        ldata.ld_buf = buf;
-       rc = mdd_links_read(env, obj, &ldata);
+       rc = mdd_links_read_with_rec(env, obj, &ldata);
        if (rc != 0)
                GOTO(lookup, rc);
 
@@ -966,41 +1033,32 @@ static int mdd_linkea_prepare(const struct lu_env *env,
                              struct linkea_data *ldata)
 {
        int rc = 0;
-       int rc2 = 0;
        ENTRY;
 
        if (OBD_FAIL_CHECK(OBD_FAIL_FID_IGIF))
-               return 0;
+               RETURN(0);
 
        LASSERT(oldpfid != NULL || newpfid != NULL);
 
-       if (mdd_obj->mod_flags & DEAD_OBJ) {
-               /* Prevent linkea to be updated which is NOT necessary. */
-               ldata->ld_reclen = 0;
-               /* No more links, don't bother */
+       if (mdd_obj->mod_flags & DEAD_OBJ)
+               /* Unnecessary to update linkEA for dead object.  */
                RETURN(0);
-       }
 
        if (oldpfid != NULL) {
                rc = __mdd_links_del(env, mdd_obj, ldata, oldlname, oldpfid);
                if (rc) {
-                       if ((check == 1) ||
-                           (rc != -ENODATA && rc != -ENOENT))
+                       if ((check == 1) || (rc != -ENODATA && rc != -ENOENT))
                                RETURN(rc);
+
                        /* No changes done. */
                        rc = 0;
                }
        }
 
        /* If renaming, add the new record */
-       if (newpfid != NULL) {
-               /* even if the add fails, we still delete the out-of-date
-                * old link */
-               rc2 = __mdd_links_add(env, mdd_obj, ldata, newlname, newpfid,
-                                     first, check);
-       }
-
-       rc = rc != 0 ? rc : rc2;
+       if (newpfid != NULL)
+               rc = __mdd_links_add(env, mdd_obj, ldata, newlname, newpfid,
+                                    first, check);
 
        RETURN(rc);
 }
@@ -1022,41 +1080,34 @@ int mdd_links_rename(const struct lu_env *env,
                ldata = &mdd_env_info(env)->mti_link_data;
                memset(ldata, 0, sizeof(*ldata));
                rc = mdd_linkea_prepare(env, mdd_obj, oldpfid, oldlname,
-                                       newpfid, newlname, first, check,
-                                       ldata);
-               if (rc != 0)
+                                       newpfid, newlname, first, check, ldata);
+               if (rc)
                        GOTO(out, rc);
        }
 
-       if (ldata->ld_reclen != 0)
+       if (!(mdd_obj->mod_flags & DEAD_OBJ))
                rc = mdd_links_write(env, mdd_obj, ldata, handle);
-       EXIT;
+
+       GOTO(out, rc);
+
 out:
        if (rc != 0) {
-               int error = 1;
-               if (rc == -EOVERFLOW || rc == -ENOSPC)
-                       error = 0;
                if (newlname == NULL)
-                       CDEBUG(error ? D_ERROR : D_OTHER,
-                              "link_ea add failed %d "DFID"\n",
+                       CERROR("link_ea add failed %d "DFID"\n",
                               rc, PFID(mdd_object_fid(mdd_obj)));
                else if (oldpfid == NULL)
-                       CDEBUG(error ? D_ERROR : D_OTHER,
-                              "link_ea add '%.*s' failed %d "DFID"\n",
-                              newlname->ln_namelen, newlname->ln_name,
-                              rc, PFID(mdd_object_fid(mdd_obj)));
+                       CERROR("link_ea add '%.*s' failed %d "DFID"\n",
+                              newlname->ln_namelen, newlname->ln_name, rc,
+                              PFID(mdd_object_fid(mdd_obj)));
                else if (newpfid == NULL)
-                       CDEBUG(error ? D_ERROR : D_OTHER,
-                              "link_ea del '%.*s' failed %d "DFID"\n",
-                              oldlname->ln_namelen, oldlname->ln_name,
-                              rc, PFID(mdd_object_fid(mdd_obj)));
+                       CERROR("link_ea del '%.*s' failed %d "DFID"\n",
+                              oldlname->ln_namelen, oldlname->ln_name, rc,
+                              PFID(mdd_object_fid(mdd_obj)));
                else
-                       CDEBUG(error ? D_ERROR : D_OTHER,
-                              "link_ea rename '%.*s'->'%.*s' failed %d "
-                              DFID"\n",
-                              oldlname->ln_namelen, oldlname->ln_name,
-                              newlname->ln_namelen, newlname->ln_name,
-                              rc, PFID(mdd_object_fid(mdd_obj)));
+                       CERROR("link_ea rename '%.*s'->'%.*s' failed %d "DFID
+                              "\n", oldlname->ln_namelen, oldlname->ln_name,
+                              newlname->ln_namelen, newlname->ln_name, rc,
+                              PFID(mdd_object_fid(mdd_obj)));
        }
 
        if (is_vmalloc_addr(ldata->ld_buf))
@@ -1088,50 +1139,6 @@ static inline int mdd_links_del(const struct lu_env *env,
 }
 
 /** Read the link EA into a temp buffer.
- * Uses the mdd_thread_info::mti_big_buf since it is generally large.
- * A pointer to the buffer is stored in \a ldata::ld_buf.
- *
- * \retval 0 or error
- */
-int mdd_links_read(const struct lu_env *env, struct mdd_object *mdd_obj,
-                  struct linkea_data *ldata)
-{
-       int rc;
-
-       if (!mdd_object_exists(mdd_obj))
-               return -ENODATA;
-
-       /* First try a small buf */
-       LASSERT(env != NULL);
-       ldata->ld_buf = lu_buf_check_and_alloc(&mdd_env_info(env)->mti_link_buf,
-                                              PAGE_CACHE_SIZE);
-       if (ldata->ld_buf->lb_buf == NULL)
-               return -ENOMEM;
-
-       rc = mdo_xattr_get(env, mdd_obj, ldata->ld_buf, XATTR_NAME_LINK);
-       if (rc == -ERANGE) {
-               /* Buf was too small, figure out what we need. */
-               lu_buf_free(ldata->ld_buf);
-               rc = mdo_xattr_get(env, mdd_obj, ldata->ld_buf,
-                                  XATTR_NAME_LINK);
-               if (rc < 0)
-                       return rc;
-               ldata->ld_buf = lu_buf_check_and_alloc(ldata->ld_buf, rc);
-               if (ldata->ld_buf->lb_buf == NULL)
-                       return -ENOMEM;
-               rc = mdo_xattr_get(env, mdd_obj, ldata->ld_buf,
-                                 XATTR_NAME_LINK);
-       }
-       if (rc < 0) {
-               lu_buf_free(ldata->ld_buf);
-               ldata->ld_buf = NULL;
-               return rc;
-       }
-
-       return linkea_init(ldata);
-}
-
-/** Read the link EA into a temp buffer.
  * Uses the name_buf since it is generally large.
  * \retval IS_ERR err
  * \retval ptr to \a lu_buf (always \a mti_big_buf)
@@ -1156,38 +1163,26 @@ int mdd_links_write(const struct lu_env *env, struct mdd_object *mdd_obj,
            ldata->ld_leh == NULL)
                return 0;
 
-       buf = mdd_buf_get_const(env, ldata->ld_buf->lb_buf,
-                               ldata->ld_leh->leh_len);
        if (OBD_FAIL_CHECK(OBD_FAIL_LFSCK_NO_LINKEA))
                return 0;
 
+again:
+       buf = mdd_buf_get_const(env, ldata->ld_buf->lb_buf,
+                               ldata->ld_leh->leh_len);
        rc = mdo_xattr_set(env, mdd_obj, buf, XATTR_NAME_LINK, 0, handle);
-       if (unlikely(rc == -ENOSPC) && S_ISREG(mdd_object_type(mdd_obj)) &&
-           mdd_object_remote(mdd_obj) == 0) {
-               struct lfsck_request *lr = &mdd_env_info(env)->mti_lr;
-               struct thandle  *sub_th;
-
-               /* XXX: If the linkEA is overflow, then we need to notify the
-                *      namespace LFSCK to skip "nlink" attribute verification
-                *      on this object to avoid the "nlink" to be shrinked by
-                *      wrong. It may be not good an interaction with LFSCK
-                *      like this. We will consider to replace it with other
-                *      mechanism in future. LU-5802. */
-               lfsck_pack_rfa(lr, mdo2fid(mdd_obj), LE_SKIP_NLINK,
-                              LFSCK_TYPE_NAMESPACE);
-
-               sub_th = thandle_get_sub_by_dt(env, handle,
-                               mdo2mdd(&mdd_obj->mod_obj)->mdd_bottom);
-               lfsck_in_notify(env, mdo2mdd(&mdd_obj->mod_obj)->mdd_bottom,
-                               lr, sub_th);
+       if (unlikely(rc == -ENOSPC)) {
+               rc = linkea_overflow_shrink(ldata);
+               if (likely(rc > 0))
+                       goto again;
        }
 
        return rc;
 }
 
-int mdd_declare_links_add(const struct lu_env *env, struct mdd_object *mdd_obj,
-                         struct thandle *handle, struct linkea_data *ldata,
-                         enum mdd_links_add_overflow overflow)
+static int mdd_declare_links_add(const struct lu_env *env,
+                                struct mdd_object *mdd_obj,
+                                struct thandle *handle,
+                                struct linkea_data *ldata)
 {
        int     rc;
        int     ea_len;
@@ -1197,36 +1192,13 @@ int mdd_declare_links_add(const struct lu_env *env, struct mdd_object *mdd_obj,
                ea_len = ldata->ld_leh->leh_len;
                linkea = ldata->ld_buf->lb_buf;
        } else {
-               ea_len = DEFAULT_LINKEA_SIZE;
+               ea_len = MAX_LINKEA_SIZE;
                linkea = NULL;
        }
 
-       /* XXX: max size? */
        rc = mdo_declare_xattr_set(env, mdd_obj,
                                   mdd_buf_get_const(env, linkea, ea_len),
                                   XATTR_NAME_LINK, 0, handle);
-       if (rc != 0)
-               return rc;
-
-       if (mdd_object_remote(mdd_obj) == 0 && overflow == MLAO_CHECK) {
-               struct lfsck_request *lr = &mdd_env_info(env)->mti_lr;
-               struct thandle  *sub_th;
-
-               /* XXX: If the linkEA is overflow, then we need to notify the
-                *      namespace LFSCK to skip "nlink" attribute verification
-                *      on this object to avoid the "nlink" to be shrinked by
-                *      wrong. It may be not good an interaction with LFSCK
-                *      like this. We will consider to replace it with other
-                *      mechanism in future. LU-5802. */
-               lfsck_pack_rfa(lr, mdo2fid(mdd_obj), LE_SKIP_NLINK_DECLARE,
-                              LFSCK_TYPE_NAMESPACE);
-
-               sub_th = thandle_get_sub_by_dt(env, handle,
-                               mdo2mdd(&mdd_obj->mod_obj)->mdd_bottom);
-               rc = lfsck_in_notify(env,
-                                    mdo2mdd(&mdd_obj->mod_obj)->mdd_bottom,
-                                    lr, sub_th);
-       }
 
        return rc;
 }
@@ -1240,7 +1212,7 @@ static inline int mdd_declare_links_del(const struct lu_env *env,
        /* For directory, the linkEA will be removed together
         * with the object. */
        if (!S_ISDIR(mdd_object_type(c)))
-               rc = mdd_declare_links_add(env, c, handle, NULL, MLAO_IGNORE);
+               rc = mdd_declare_links_add(env, c, handle, NULL);
 
        return rc;
 }
@@ -1254,9 +1226,13 @@ static int mdd_declare_link(const struct lu_env *env,
                            struct lu_attr *la,
                            struct linkea_data *data)
 {
+       struct lu_fid tfid = *mdo2fid(c);
        int rc;
 
-       rc = mdo_declare_index_insert(env, p, mdo2fid(c), mdd_object_type(c),
+       if (OBD_FAIL_CHECK(OBD_FAIL_LFSCK_DANGLING3))
+               tfid.f_oid = cfs_fail_val;
+
+       rc = mdo_declare_index_insert(env, p, &tfid, mdd_object_type(c),
                                      name->ln_name, handle);
        if (rc != 0)
                return rc;
@@ -1265,12 +1241,6 @@ static int mdd_declare_link(const struct lu_env *env,
        if (rc != 0)
                return rc;
 
-       if (OBD_FAIL_CHECK(OBD_FAIL_LFSCK_MORE_NLINK)) {
-               rc = mdo_declare_ref_add(env, c, handle);
-               if (rc != 0)
-                       return rc;
-       }
-
        la->la_valid = LA_CTIME | LA_MTIME;
        rc = mdo_declare_attr_set(env, p, la, handle);
        if (rc != 0)
@@ -1281,8 +1251,7 @@ static int mdd_declare_link(const struct lu_env *env,
        if (rc != 0)
                return rc;
 
-       rc = mdd_declare_links_add(env, c, handle, data,
-                       S_ISREG(mdd_object_type(c)) ? MLAO_CHECK : MLAO_IGNORE);
+       rc = mdd_declare_links_add(env, c, handle, data);
        if (rc != 0)
                return rc;
 
@@ -1301,8 +1270,9 @@ static int mdd_link(const struct lu_env *env, struct md_object *tgt_obj,
         struct mdd_object *mdd_sobj = md2mdd_obj(src_obj);
        struct lu_attr    *cattr = MDD_ENV_VAR(env, cattr);
        struct lu_attr    *tattr = MDD_ENV_VAR(env, tattr);
-        struct mdd_device *mdd = mdo2mdd(src_obj);
-        struct thandle *handle;
+       struct mdd_device *mdd = mdo2mdd(src_obj);
+       struct thandle *handle;
+       struct lu_fid *tfid = &mdd_env_info(env)->mti_fid2;
        struct linkea_data *ldata = &mdd_env_info(env)->mti_link_data;
        int rc;
        ENTRY;
@@ -1324,6 +1294,14 @@ static int mdd_link(const struct lu_env *env, struct md_object *tgt_obj,
        LASSERT(ma->ma_attr.la_valid & LA_CTIME);
        la->la_ctime = la->la_mtime = ma->ma_attr.la_ctime;
 
+       /* Note: even this function will change ldata, but it comes from
+        * thread_info, which is completely temporary and only seen in
+        * this function, so we do not need reset ldata once it fails.*/
+       rc = mdd_linkea_prepare(env, mdd_sobj, NULL, NULL, mdo2fid(mdd_tobj),
+                               lname, 0, 0, ldata);
+       if (rc != 0)
+               GOTO(stop, rc);
+
        rc = mdd_declare_link(env, mdd, mdd_tobj, mdd_sobj, lname, handle,
                              la, ldata);
         if (rc)
@@ -1345,25 +1323,12 @@ static int mdd_link(const struct lu_env *env, struct md_object *tgt_obj,
                        GOTO(out_unlock, rc);
        }
 
-       if (OBD_FAIL_CHECK(OBD_FAIL_LFSCK_MORE_NLINK)) {
-               rc = mdo_ref_add(env, mdd_sobj, handle);
-               if (rc != 0)
-                       GOTO(out_unlock, rc);
-       }
-
-       if (OBD_FAIL_CHECK(OBD_FAIL_LFSCK_DANGLING3)) {
-               struct lu_fid tfid = *mdo2fid(mdd_sobj);
-
-               tfid.f_oid++;
-               rc = __mdd_index_insert_only(env, mdd_tobj, &tfid,
-                                            mdd_object_type(mdd_sobj),
-                                            name, handle);
-       } else {
-               rc = __mdd_index_insert_only(env, mdd_tobj, mdo2fid(mdd_sobj),
-                                            mdd_object_type(mdd_sobj),
-                                            name, handle);
-       }
+       *tfid = *mdo2fid(mdd_sobj);
+       if (OBD_FAIL_CHECK(OBD_FAIL_LFSCK_DANGLING3))
+               tfid->f_oid = cfs_fail_val;
 
+       rc = __mdd_index_insert_only(env, mdd_tobj, tfid,
+                                    mdd_object_type(mdd_sobj), name, handle);
        if (rc != 0) {
                mdo_ref_del(env, mdd_sobj, handle);
                GOTO(out_unlock, rc);
@@ -1376,32 +1341,26 @@ static int mdd_link(const struct lu_env *env, struct md_object *tgt_obj,
 
        la->la_valid = LA_CTIME;
        rc = mdd_update_time(env, mdd_sobj, cattr, la, handle);
-       if (rc == 0) {
-               rc = mdd_linkea_prepare(env, mdd_sobj, NULL, NULL,
-                                       mdo2fid(mdd_tobj), lname, 0, 0,
-                                       ldata);
-               if (rc == 0)
-                       mdd_links_add(env, mdd_sobj, mdo2fid(mdd_tobj),
-                                     lname, handle, ldata, 0);
-               /* The failure of links_add should not cause the link
-                * failure, reset rc here */
-               rc = 0;
-       }
-        EXIT;
+       if (rc == 0)
+               /* Note: The failure of links_add should not cause the
+                * link failure, so do not check return value. */
+               mdd_links_add(env, mdd_sobj, mdo2fid(mdd_tobj),
+                             lname, handle, ldata, 0);
+
+       EXIT;
 out_unlock:
-        mdd_write_unlock(env, mdd_sobj);
-        if (rc == 0)
+       mdd_write_unlock(env, mdd_sobj);
+       if (rc == 0)
                rc = mdd_changelog_ns_store(env, mdd, CL_HARDLINK, 0, mdd_sobj,
                                            mdo2fid(mdd_tobj), NULL, NULL,
                                            lname, NULL, handle);
 stop:
-       mdd_trans_stop(env, mdd, rc, handle);
-
+       rc = mdd_trans_stop(env, mdd, rc, handle);
        if (is_vmalloc_addr(ldata->ld_buf))
                /* if we vmalloced a large buffer drop it */
                lu_buf_free(ldata->ld_buf);
 out_pending:
-        return rc;
+       return rc;
 }
 
 static int mdd_mark_orphan_object(const struct lu_env *env,
@@ -1467,10 +1426,6 @@ int mdd_finish_unlink(const struct lu_env *env,
                 * will be deleted during mdd_close() */
                obj->mod_flags |= DEAD_OBJ;
                if (obj->mod_count) {
-                       rc = mdd_mark_orphan_object(env, obj, th, false);
-                       if (rc != 0)
-                               RETURN(rc);
-
                        rc = __mdd_orphan_add(env, obj, th);
                        if (rc == 0)
                                CDEBUG(D_HA, "Object "DFID" is inserted into "
@@ -1483,6 +1438,12 @@ int mdd_finish_unlink(const struct lu_env *env,
                                       "open replay\n",
                                        PFID(mdd_object_fid(obj)),
                                        obj->mod_count);
+
+                       /* mark object as an orphan here, not
+                        * before __mdd_orphan_add() as racing
+                        * mdd_la_get() may propagate ORPHAN_OBJ
+                        * causing the asserition */
+                       rc = mdd_mark_orphan_object(env, obj, th, false);
                } else {
                        rc = mdo_destroy(env, obj, th);
                }
@@ -1591,7 +1552,7 @@ static bool mdd_hsm_archive_exists(const struct lu_env *env,
                if (rc < 0)
                        RETURN(false);
 
-               ma->ma_valid = MA_HSM;
+               ma->ma_valid |= MA_HSM;
        }
        if (ma->ma_hsm.mh_flags & HS_EXISTS)
                RETURN(true);
@@ -1619,7 +1580,7 @@ static int mdd_unlink(const struct lu_env *env, struct md_object *pobj,
        struct mdd_object *mdd_cobj = NULL;
        struct mdd_device *mdd = mdo2mdd(pobj);
        struct thandle    *handle;
-       int rc, is_dir = 0;
+       int rc, is_dir = 0, cl_flags = 0;
        ENTRY;
 
        /* cobj == NULL means only delete name entry */
@@ -1640,6 +1601,11 @@ static int mdd_unlink(const struct lu_env *env, struct md_object *pobj,
                        RETURN(rc);
 
                is_dir = S_ISDIR(cattr->la_mode);
+               /* search for an existing archive.
+                * we should check ahead as the object
+                * can be destroyed in this transaction */
+               if (mdd_hsm_archive_exists(env, mdd_cobj, ma))
+                       cl_flags |= CLF_UNLINK_HSM_EXISTS;
        }
 
        rc = mdd_unlink_sanity_check(env, mdd_pobj, pattr, mdd_cobj, cattr);
@@ -1717,11 +1683,10 @@ static int mdd_unlink(const struct lu_env *env, struct md_object *pobj,
        ma->ma_attr = *cattr;
        ma->ma_valid |= MA_INODE;
        rc = mdd_finish_unlink(env, mdd_cobj, ma, mdd_pobj, lname, handle);
-
-       /* fetch updated nlink */
        if (rc != 0)
                GOTO(cleanup, rc);
 
+       /* fetch updated nlink */
        rc = mdd_la_get(env, mdd_cobj, cattr);
        /* if object is removed then we can't get its attrs,
         * use last get */
@@ -1741,14 +1706,10 @@ cleanup:
                mdd_write_unlock(env, mdd_cobj);
 
        if (rc == 0) {
-               int cl_flags = 0;
-
-               if (cattr->la_nlink == 0) {
+               if (cattr->la_nlink == 0)
                        cl_flags |= CLF_UNLINK_LAST;
-                       /* search for an existing archive */
-                       if (mdd_hsm_archive_exists(env, mdd_cobj, ma))
-                               cl_flags |= CLF_UNLINK_HSM_EXISTS;
-               }
+               else
+                       cl_flags &= ~CLF_UNLINK_HSM_EXISTS;
 
                rc = mdd_changelog_ns_store(env, mdd,
                        is_dir ? CL_RMDIR : CL_UNLINK, cl_flags,
@@ -1757,7 +1718,7 @@ cleanup:
        }
 
 stop:
-       mdd_trans_stop(env, mdd, rc, handle);
+       rc = mdd_trans_stop(env, mdd, rc, handle);
 
        return rc;
 }
@@ -1777,9 +1738,11 @@ static int mdd_cd_sanity_check(const struct lu_env *env,
         RETURN(0);
 }
 
-static int mdd_create_data(const struct lu_env *env, struct md_object *pobj,
-                           struct md_object *cobj, const struct md_op_spec *spec,
-                           struct md_attr *ma)
+static int mdd_create_data(const struct lu_env *env,
+                          struct md_object *pobj,
+                          struct md_object *cobj,
+                          const struct md_op_spec *spec,
+                          struct md_attr *ma)
 {
        struct mdd_device *mdd = mdo2mdd(cobj);
        struct mdd_object *mdd_pobj = md2mdd_obj(pobj);
@@ -1811,15 +1774,15 @@ static int mdd_create_data(const struct lu_env *env, struct md_object *pobj,
        /* calling ->ah_make_hint() is used to transfer information from parent */
        mdd_object_make_hint(env, mdd_pobj, son, attr, spec, hint);
 
-        handle = mdd_trans_create(env, mdd);
-        if (IS_ERR(handle))
-                GOTO(out_free, rc = PTR_ERR(handle));
+       handle = mdd_trans_create(env, mdd);
+       if (IS_ERR(handle))
+               GOTO(out_free, rc = PTR_ERR(handle));
 
-        /*
-         * XXX: Setting the lov ea is not locked but setting the attr is locked?
-         * Should this be fixed?
-         */
-       CDEBUG(D_OTHER, "ea %p/%u, cr_flags "LPO64", no_create %u\n",
+       /*
+        * XXX: Setting the lov ea is not locked but setting the attr is locked?
+        * Should this be fixed?
+        */
+       CDEBUG(D_OTHER, "ea %p/%u, cr_flags %#llo, no_create %u\n",
               spec->u.sp_ea.eadata, spec->u.sp_ea.eadatalen,
               spec->sp_cr_flags, spec->no_create);
 
@@ -1852,7 +1815,8 @@ static int mdd_create_data(const struct lu_env *env, struct md_object *pobj,
        rc = mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, son, handle);
 
 stop:
-       mdd_trans_stop(env, mdd, rc, handle);
+       rc = mdd_trans_stop(env, mdd, rc, handle);
+
 out_free:
        RETURN(rc);
 }
@@ -1860,24 +1824,15 @@ out_free:
 static int mdd_declare_object_initialize(const struct lu_env *env,
                                         struct mdd_object *parent,
                                         struct mdd_object *child,
-                                        struct lu_attr *attr,
+                                        const struct lu_attr *attr,
                                         struct thandle *handle)
 {
        int rc;
        ENTRY;
 
-       /*
-        * inode mode has been set in creation time, and it's based on umask,
-        * la_mode and acl, don't set here again! (which will go wrong
-        * because below function doesn't consider umask).
-        * I'd suggest set all object attributes in creation time, see above.
-        */
        LASSERT(attr->la_valid & (LA_MODE | LA_TYPE));
-       attr->la_valid &= ~(LA_MODE | LA_TYPE);
-       rc = mdo_declare_attr_set(env, child, attr, handle);
-       attr->la_valid |= LA_MODE | LA_TYPE;
-       if (rc != 0 || !S_ISDIR(attr->la_mode))
-               RETURN(rc);
+       if (!S_ISDIR(attr->la_mode))
+               RETURN(0);
 
        rc = mdo_declare_index_insert(env, child, mdo2fid(child), S_IFDIR,
                                      dot, handle);
@@ -1992,14 +1947,14 @@ static int mdd_create_sanity_check(const struct lu_env *env,
                RETURN(rc);
 
        switch (cattr->la_mode & S_IFMT) {
-        case S_IFLNK: {
-                unsigned int symlen = strlen(spec->u.sp_symname) + 1;
+       case S_IFLNK: {
+               unsigned int symlen = strlen(spec->u.sp_symname) + 1;
 
-                if (symlen > (1 << m->mdd_dt_conf.ddp_block_shift))
-                        RETURN(-ENAMETOOLONG);
-                else
-                        RETURN(0);
-        }
+               if (symlen > m->mdd_dt_conf.ddp_symlink_max)
+                       RETURN(-ENAMETOOLONG);
+               else
+                       RETURN(0);
+       }
         case S_IFDIR:
         case S_IFREG:
         case S_IFCHR:
@@ -2025,6 +1980,7 @@ static int mdd_declare_object_create(const struct lu_env *env,
                                     struct lu_buf *acl_buf,
                                     struct dt_allocation_hint *hint)
 {
+       const struct lu_buf *buf;
        int rc;
 
        rc = mdd_declare_object_create_internal(env, p, c, attr, handle, spec,
@@ -2060,8 +2016,6 @@ static int mdd_declare_object_create(const struct lu_env *env,
        /* replay case, create LOV EA from client data */
        if (spec->no_create ||
            (spec->sp_cr_flags & MDS_OPEN_HAS_EA && S_ISREG(attr->la_mode))) {
-               const struct lu_buf *buf;
-
                buf = mdd_buf_get_const(env, spec->u.sp_ea.eadata,
                                        spec->u.sp_ea.eadatalen);
                rc = mdo_declare_xattr_set(env, c, buf, XATTR_NAME_LOV, 0,
@@ -2081,6 +2035,16 @@ static int mdd_declare_object_create(const struct lu_env *env,
                 if (rc)
                         GOTO(out, rc);
         }
+
+       if (spec->sp_cr_file_secctx_name != NULL) {
+               buf = mdd_buf_get_const(env, spec->sp_cr_file_secctx,
+                                       spec->sp_cr_file_secctx_size);
+               rc = mdo_declare_xattr_set(env, c, buf,
+                                          spec->sp_cr_file_secctx_name, 0,
+                                          handle);
+               if (rc < 0)
+                       GOTO(out, rc);
+       }
 out:
        return rc;
 }
@@ -2121,7 +2085,7 @@ static int mdd_declare_create(const struct lu_env *env, struct mdd_device *mdd,
                if (rc != 0)
                        return rc;
 
-               rc = mdd_declare_links_add(env, c, handle, ldata, MLAO_IGNORE);
+               rc = mdd_declare_links_add(env, c, handle, ldata);
                if (rc)
                        return rc;
 
@@ -2191,6 +2155,7 @@ static int mdd_object_create(const struct lu_env *env, struct mdd_object *pobj,
                             struct dt_allocation_hint *hint,
                             struct thandle *handle)
 {
+       const struct lu_buf    *buf;
        int                     rc;
 
        mdd_write_lock(env, son, MOR_TGT_CHILD);
@@ -2226,8 +2191,6 @@ static int mdd_object_create(const struct lu_env *env, struct mdd_object *pobj,
        if (spec->no_create ||
            (S_ISREG(attr->la_mode) && spec->sp_cr_flags & MDS_OPEN_HAS_EA) ||
            S_ISDIR(attr->la_mode)) {
-               const struct lu_buf *buf;
-
                buf = mdd_buf_get_const(env, spec->u.sp_ea.eadata,
                                        spec->u.sp_ea.eadatalen);
                rc = mdo_xattr_set(env, son, buf,
@@ -2277,6 +2240,15 @@ static int mdd_object_create(const struct lu_env *env, struct mdd_object *pobj,
                        GOTO(err_initlized, rc = -EFAULT);
        }
 
+       if (spec->sp_cr_file_secctx_name != NULL) {
+               buf = mdd_buf_get_const(env, spec->sp_cr_file_secctx,
+                                       spec->sp_cr_file_secctx_size);
+               rc = mdo_xattr_set(env, son, buf, spec->sp_cr_file_secctx_name,
+                                  0, handle);
+               if (rc < 0)
+                       GOTO(err_initlized, rc);
+       }
+
 err_initlized:
        if (unlikely(rc != 0)) {
                int rc2;
@@ -2335,7 +2307,8 @@ static int mdd_index_delete(const struct lu_env *env,
        if (rc)
                GOTO(stop, rc);
 stop:
-       mdd_trans_stop(env, mdd, rc, handle);
+       rc = mdd_trans_stop(env, mdd, rc, handle);
+
        RETURN(rc);
 }
 
@@ -2477,8 +2450,6 @@ static int mdd_create(const struct lu_env *env, struct md_object *pobj,
        EXIT;
 err_insert:
        if (rc != 0) {
-               int rc2;
-
                if (spec->sp_cr_flags & MDS_OPEN_VOLATILE)
                        rc2 = __mdd_orphan_del(env, son, handle);
                else
@@ -2638,8 +2609,8 @@ static int mdd_declare_rename(const struct lu_env *env,
                              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,
+                             const struct lu_name *tname,
                              struct md_attr *ma,
                              struct linkea_data *ldata,
                              struct thandle *handle)
@@ -2700,8 +2671,7 @@ static int mdd_declare_rename(const struct lu_env *env,
        if (rc)
                return rc;
 
-       rc = mdd_declare_links_add(env, mdd_sobj, handle, ldata,
-               S_ISREG(mdd_object_type(mdd_sobj)) ? MLAO_CHECK : MLAO_IGNORE);
+       rc = mdd_declare_links_add(env, mdd_sobj, handle, ldata);
        if (rc)
                return rc;
 
@@ -2802,6 +2772,11 @@ static int mdd_rename(const struct lu_env *env,
                rc = mdd_la_get(env, mdd_tobj, tattr);
                if (rc)
                        GOTO(out_pending, rc);
+               /* search for an existing archive.
+                * we should check ahead as the object
+                * can be destroyed in this transaction */
+               if (mdd_hsm_archive_exists(env, mdd_tobj, ma))
+                       cl_flags |= CLF_RENAME_LAST_EXISTS;
        }
 
        rc = mdd_la_get(env, mdd_tpobj, tpattr);
@@ -2827,8 +2802,12 @@ static int mdd_rename(const struct lu_env *env,
                 GOTO(out_pending, rc = PTR_ERR(handle));
 
        memset(ldata, 0, sizeof(*ldata));
-       mdd_linkea_prepare(env, mdd_sobj, mdd_object_fid(mdd_spobj), lsname,
-                          mdd_object_fid(mdd_tpobj), ltname, 1, 0, ldata);
+       rc = mdd_linkea_prepare(env, mdd_sobj, mdd_object_fid(mdd_spobj),
+                               lsname, mdd_object_fid(mdd_tpobj), ltname,
+                               1, 0, ldata);
+       if (rc)
+               GOTO(stop, rc);
+
        rc = mdd_declare_rename(env, mdd, mdd_spobj, mdd_tpobj, mdd_sobj,
                                mdd_tobj, lsname, ltname, ma, ldata, handle);
        if (rc)
@@ -2965,11 +2944,10 @@ static int mdd_rename(const struct lu_env *env,
                ma->ma_attr = *tattr;
                ma->ma_valid |= MA_INODE;
 
-               if (tattr->la_nlink == 0) {
+               if (tattr->la_nlink == 0)
                        cl_flags |= CLF_RENAME_LAST;
-                       if (mdd_hsm_archive_exists(env, mdd_tobj, ma))
-                               cl_flags |= CLF_RENAME_LAST_EXISTS;
-               }
+               else
+                       cl_flags &= ~CLF_RENAME_LAST_EXISTS;
         }
 
        la->la_valid = LA_CTIME | LA_MTIME;
@@ -3043,7 +3021,7 @@ cleanup:
                                            ltname, lsname, handle);
 
 stop:
-       mdd_trans_stop(env, mdd, rc, handle);
+       rc = mdd_trans_stop(env, mdd, rc, handle);
 
 out_pending:
        mdd_object_put(env, mdd_sobj);
@@ -3108,8 +3086,7 @@ static int mdd_linkea_update_child_internal(const struct lu_env *env,
                linkea_entry_pack(ldata.ld_lee, &lname,
                                  mdd_object_fid(newparent));
                if (declare)
-                       rc = mdd_declare_links_add(env, child, handle, &ldata,
-                                                  MLAO_IGNORE);
+                       rc = mdd_declare_links_add(env, child, handle, &ldata);
                else
                        rc = mdd_links_write(env, child, &ldata, handle);
                break;
@@ -3156,11 +3133,10 @@ static int mdd_update_linkea_internal(const struct lu_env *env,
        ENTRY;
 
        LASSERT(ldata->ld_buf != NULL);
+       LASSERT(ldata->ld_leh != NULL);
 
-again:
        /* If it is mulitple links file, we need update the name entry for
         * all parent */
-       LASSERT(ldata->ld_leh != NULL);
        ldata->ld_lee = (struct link_ea_entry *)(ldata->ld_leh + 1);
        for (count = 0; count < ldata->ld_leh->leh_reccount; count++) {
                struct mdd_device       *mdd = mdo2mdd(&mdd_sobj->mod_obj);
@@ -3175,16 +3151,13 @@ again:
                        CWARN("%s: cannot find obj "DFID": rc = %ld\n",
                              mdd2obd_dev(mdd)->obd_name, PFID(&fid),
                              PTR_ERR(pobj));
-                       linkea_del_buf(ldata, &lname);
-                       goto again;
+                       continue;
                }
 
                if (!mdd_object_exists(pobj)) {
                        CDEBUG(D_INFO, "%s: obj "DFID" does not exist\n",
                              mdd2obd_dev(mdd)->obd_name, PFID(&fid));
-                       linkea_del_buf(ldata, &lname);
-                       mdd_object_put(env, pobj);
-                       goto again;
+                       goto next_put;
                }
 
                if (pobj == mdd_pobj &&
@@ -3194,9 +3167,7 @@ again:
                        CDEBUG(D_INFO, "%s: skip its own %s: "DFID"\n",
                              mdd2obd_dev(mdd)->obd_name, child_name->ln_name,
                              PFID(&fid));
-                       linkea_del_buf(ldata, &lname);
-                       mdd_object_put(env, pobj);
-                       goto again;
+                       goto next_put;
                }
 
                CDEBUG(D_INFO, "%s: update "DFID" with "DNAME":"DFID"\n",
@@ -3232,9 +3203,11 @@ again:
                                /* lnamelen is too big(> NAME_MAX + 16),
                                 * something wrong about this linkea, let's
                                 * skip it */
-                               linkea_del_buf(ldata, &lname);
-                               mdd_object_put(env, pobj);
-                               goto again;
+                               CWARN("%s: the name %.*s is too long under "
+                                     DFID"\n", mdd2obd_dev(mdd)->obd_name,
+                                     lname.ln_namelen, lname.ln_name,
+                                     PFID(&fid));
+                               goto next_put;
                        }
 
                        /* Note: lname might be without \0 at the end, see
@@ -3248,14 +3221,9 @@ again:
                         * it might be packed into the RPC buffer. */
                        rc = mdd_lookup(env, &pobj->mod_obj, &lname,
                                        &info->mti_fid, NULL);
-                       if (rc < 0 ||
-                           !lu_fid_eq(&info->mti_fid,
-                                       mdd_object_fid(mdd_sobj))) {
-                               /* skip invalid linkea entry */
-                               linkea_del_buf(ldata, &lname);
-                               mdd_object_put(env, pobj);
-                               goto again;
-                       }
+                       if (rc < 0 || !lu_fid_eq(&info->mti_fid,
+                                                mdd_object_fid(mdd_sobj)))
+                               GOTO(next_put, rc == -ENOENT ? 0 : rc);
 
                        rc = __mdd_index_delete(env, pobj, tmp_name, 0, handle);
                        if (rc != 0)
@@ -3305,7 +3273,6 @@ static int mdd_migrate_xattrs(const struct lu_env *env,
        int                     list_xsize;
        struct lu_buf           list_xbuf;
        int                     rc;
-       int                     rc1;
 
        /* retrieve xattr list from the old object */
        list_xsize = mdo_xattr_list(env, mdd_sobj, &LU_BUF_NULL);
@@ -3329,8 +3296,7 @@ static int mdd_migrate_xattrs(const struct lu_env *env,
        xname = list_xbuf.lb_buf;
        while (rem > 0) {
                xlen = strnlen(xname, rem - 1) + 1;
-               if (strcmp(XATTR_NAME_LINK, xname) == 0 ||
-                   strcmp(XATTR_NAME_LMA, xname) == 0 ||
+               if (strcmp(XATTR_NAME_LMA, xname) == 0 ||
                    strcmp(XATTR_NAME_LMV, xname) == 0)
                        goto next;
 
@@ -3373,16 +3339,25 @@ static int mdd_migrate_xattrs(const struct lu_env *env,
                if (rc != 0)
                        GOTO(stop_trans, rc);
 
+again:
                rc = mdo_xattr_set(env, mdd_tobj, &xbuf, xname, 0, handle);
                if (rc == -EEXIST)
                        GOTO(stop_trans, rc = 0);
 
+               if (unlikely(rc == -ENOSPC &&
+                            strcmp(xname, XATTR_NAME_LINK) == 0)) {
+                       rc = linkea_overflow_shrink(
+                                       (struct linkea_data *)(xbuf.lb_buf));
+                       if (likely(rc > 0)) {
+                               xbuf.lb_len = rc;
+                               goto again;
+                       }
+               }
+
                if (rc != 0)
                        GOTO(stop_trans, rc);
 stop_trans:
-               rc1 = mdd_trans_stop(env, mdd, rc, handle);
-               if (rc == 0)
-                       rc = rc1;
+               rc = mdd_trans_stop(env, mdd, rc, handle);
                if (rc != 0)
                        GOTO(out, rc);
 next:
@@ -3429,8 +3404,7 @@ static int mdd_declare_migrate_create(const struct lu_env *env,
                if (rc != 0)
                        return rc;
        } else if (S_ISDIR(la->la_mode) && ldata != NULL) {
-               rc = mdd_declare_links_add(env, mdd_tobj, handle, ldata,
-                                          MLAO_IGNORE);
+               rc = mdd_declare_links_add(env, mdd_tobj, handle, ldata);
                if (rc != 0)
                        return rc;
        }
@@ -3511,7 +3485,7 @@ static int mdd_migrate_create(const struct lu_env *env,
                        spec->sp_cr_flags |= MDS_OPEN_HAS_EA;
                }
        } else if (S_ISDIR(la->la_mode)) {
-               rc = mdd_links_read(env, mdd_sobj, ldata);
+               rc = mdd_links_read_with_rec(env, mdd_sobj, ldata);
                if (rc == -ENODATA) {
                        /* ignore the non-linkEA error */
                        ldata = NULL;
@@ -3584,13 +3558,8 @@ static int mdd_migrate_create(const struct lu_env *env,
        la_flag->la_flags = la->la_flags | LUSTRE_IMMUTABLE_FL;
        rc = mdo_attr_set(env, mdd_sobj, la_flag, handle);
 stop_trans:
-       if (handle != NULL) {
-               int rc1;
-
-               rc1 = mdd_trans_stop(env, mdd, rc, handle);
-               if (rc == 0)
-                       rc = rc1;
-       }
+       if (handle != NULL)
+               rc = mdd_trans_stop(env, mdd, rc, handle);
 out_free:
        if (lmm_buf.lb_buf != NULL)
                OBD_FREE(lmm_buf.lb_buf, lmm_buf.lb_len);
@@ -3607,9 +3576,9 @@ static int mdd_migrate_entries(const struct lu_env *env,
        struct thandle          *handle;
        struct dt_it            *it;
        const struct dt_it_ops  *iops;
-       int                      rc;
        int                      result;
        struct lu_dirent        *ent;
+       int                      rc;
        ENTRY;
 
        OBD_ALLOC(ent, NAME_MAX + sizeof(*ent) + 1);
@@ -3642,10 +3611,8 @@ static int mdd_migrate_entries(const struct lu_env *env,
                struct mdd_object       *child;
                char                    *name = mdd_env_info(env)->mti_key;
                int                     len;
-               int                     recsize;
                int                     is_dir;
                bool                    target_exist = false;
-               int                     rc1;
 
                len = iops->key_size(env, it);
                if (len == 0)
@@ -3661,7 +3628,6 @@ static int mdd_migrate_entries(const struct lu_env *env,
                }
 
                fid_le_to_cpu(&ent->lde_fid, &ent->lde_fid);
-               recsize = le16_to_cpu(ent->lde_reclen);
 
                /* Insert new fid with target name into target dir */
                if ((ent->lde_namelen == 1 && ent->lde_name[0] == '.') ||
@@ -3779,10 +3745,7 @@ static int mdd_migrate_entries(const struct lu_env *env,
 out_put:
                mdd_write_unlock(env, child);
                mdd_object_put(env, child);
-               rc1 = mdd_trans_stop(env, mdd, rc, handle);
-               if (rc == 0)
-                       rc = rc1;
-
+               rc = mdd_trans_stop(env, mdd, rc, handle);
                if (rc != 0)
                        GOTO(out, rc);
 next:
@@ -3885,7 +3848,7 @@ static int mdd_declare_migrate_update_name(const struct lu_env *env,
        if (rc != 0)
                return rc;
 
-       rc = mdd_declare_links_add(env, mdd_tobj, handle, NULL, MLAO_IGNORE);
+       rc = mdd_declare_links_add(env, mdd_tobj, handle, NULL);
        if (rc != 0)
                return rc;
 
@@ -4027,21 +3990,14 @@ static int mdd_migrate_update_name(const struct lu_env *env,
        if (rc != 0)
                GOTO(stop_trans, rc);
 
-       linkea_add_buf(ldata, lname, mdd_object_fid(mdd_pobj));
-       rc = mdd_links_add(env, mdd_tobj, mdo2fid(mdd_pobj), lname, handle,
-                          ldata, 1);
-       if (rc != 0)
-               GOTO(stop_trans, rc);
-
        mdd_write_lock(env, mdd_sobj, MOR_TGT_CHILD);
 
-       /* Increase mod_count to add the source object to the orphan list,
-        * so if other clients still send RPC to the old object, then these
-        * objects can help the request to find the new object, see
-        * mdt_reint_open() */
-       mdd_sobj->mod_count++;
-       rc = mdd_finish_unlink(env, mdd_sobj, ma, mdd_pobj, lname, handle);
-       mdd_sobj->mod_count--;
+       mdd_sobj->mod_flags |= DEAD_OBJ;
+       rc = mdd_mark_orphan_object(env, mdd_sobj, handle, false);
+       if (rc != 0)
+               GOTO(out_unlock, rc);
+
+       rc = __mdd_orphan_add(env, mdd_sobj, handle);
        if (rc != 0)
                GOTO(out_unlock, rc);
 
@@ -4078,7 +4034,7 @@ out_unlock:
        mdd_write_unlock(env, mdd_sobj);
 
 stop_trans:
-       mdd_trans_stop(env, mdd, rc, handle);
+       rc = mdd_trans_stop(env, mdd, rc, handle);
 
        RETURN(rc);
 }
@@ -4175,6 +4131,12 @@ static int mdd_migrate_sanity_check(const struct lu_env *env,
        /* If there are still links locally, then the file will not be
         * migrated. */
        LASSERT(ldata->ld_leh != NULL);
+
+       /* If the linkEA is overflow, then means there are some unknown name
+        * entries under unknown parents, that will prevent the migration. */
+       if (unlikely(ldata->ld_leh->leh_overflow_time))
+               RETURN(1);
+
        ldata->ld_lee = (struct link_ea_entry *)(ldata->ld_leh + 1);
        for (count = 0; count < ldata->ld_leh->leh_reccount; count++) {
                struct lu_name          lname;
@@ -4215,6 +4177,7 @@ static int mdd_migrate(const struct lu_env *env, struct md_object *pobj,
        struct mdd_object       *mdd_tobj = md2mdd_obj(tobj);
        struct lu_attr          *so_attr = MDD_ENV_VAR(env, cattr);
        struct lu_attr          *pattr = MDD_ENV_VAR(env, pattr);
+       bool                    created = false;
        int                     rc;
 
        ENTRY;
@@ -4222,7 +4185,8 @@ static int mdd_migrate(const struct lu_env *env, struct md_object *pobj,
         * the file is being opened by someone else right now */
        mdd_read_lock(env, mdd_sobj, MOR_SRC_CHILD);
        if (mdd_sobj->mod_count > 0) {
-               CERROR("%s: "DFID"%s is already opened count %d: rc = %d\n",
+               CDEBUG(D_OTHER,
+                      "%s: "DFID"%s is already opened count %d: rc = %d\n",
                       mdd2obd_dev(mdd)->obd_name,
                       PFID(mdd_object_fid(mdd_sobj)), lname->ln_name,
                       mdd_sobj->mod_count, -EBUSY);
@@ -4273,6 +4237,7 @@ static int mdd_migrate(const struct lu_env *env, struct md_object *pobj,
                                        lname, so_attr);
                if (rc != 0)
                        GOTO(put, rc);
+               created = true;
        }
 
        LASSERT(mdd_object_exists(mdd_tobj));
@@ -4299,6 +4264,10 @@ static int mdd_migrate(const struct lu_env *env, struct md_object *pobj,
                                     ma);
        if (rc != 0)
                GOTO(put, rc);
+
+       /* newly created target was not locked, don't cache its attributes */
+       if (created)
+               mdd_invalidate(env, tobj);
 put:
        RETURN(rc);
 }