Whamcloud - gitweb
LU-14646 flr: write a FLR file downgrade SoM
[fs/lustre-release.git] / lustre / mdd / mdd_object.c
index 29af1ed..0ed2820 100644 (file)
@@ -27,7 +27,6 @@
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
- * Lustre is a trademark of Sun Microsystems, Inc.
  *
  * lustre/mdd/mdd_object.c
  *
@@ -72,6 +71,7 @@ static int mdd_changelog_data_store_by_fid(const struct lu_env *env,
                                           enum changelog_rec_type type,
                                           enum changelog_rec_flags clf_flags,
                                           const struct lu_fid *fid,
+                                          const struct lu_fid *pfid,
                                           const char *xattr_name,
                                           struct thandle *handle);
 
@@ -335,6 +335,7 @@ static void mdd_object_free(const struct lu_env *env, struct lu_object *o)
        }
 
        lu_object_fini(o);
+       /* mdd doesn't contain an lu_object_header, so don't need call_rcu */
        OBD_SLAB_FREE_PTR(mdd, mdd_object_kmem);
 }
 
@@ -431,7 +432,7 @@ static int mdd_xattr_get(const struct lu_env *env,
                struct thandle *handle;
                int rc2;
 
-               LASSERT(mdo2fid(mdd_obj) != NULL);
+               LASSERT(mdd_object_fid(mdd_obj) != NULL);
 
                handle = mdd_trans_create(env, mdd);
                if (IS_ERR(handle))
@@ -447,8 +448,8 @@ static int mdd_xattr_get(const struct lu_env *env,
                        GOTO(stop, rc2);
 
                rc2 = mdd_changelog_data_store_by_fid(env, mdd, CL_GETXATTR, 0,
-                                                     mdo2fid(mdd_obj), name,
-                                                     handle);
+                                                     mdd_object_fid(mdd_obj),
+                                                     NULL, name, handle);
 
 stop:
                rc2 = mdd_trans_stop(env, mdd, rc2, handle);
@@ -683,7 +684,7 @@ static int mdd_fix_attr(const struct lu_env *env, struct mdd_object *obj,
                RETURN(0);
 
        if (is_project_state_change(oattr, la)) {
-               if (!md_capable(uc, CFS_CAP_SYS_RESOURCE) &&
+               if (!md_capable(uc, CAP_SYS_RESOURCE) &&
                    !lustre_in_group_p(uc, ma->ma_enable_chprojid_gid) &&
                    !(ma->ma_enable_chprojid_gid == -1 &&
                      mdd_permission_internal(env, obj, oattr, MAY_WRITE)))
@@ -725,17 +726,26 @@ static int mdd_fix_attr(const struct lu_env *env, struct mdd_object *obj,
                                (LUSTRE_IMMUTABLE_FL | LUSTRE_APPEND_FL);
 
                if ((uc->uc_fsuid != oattr->la_uid) &&
-                   !md_capable(uc, CFS_CAP_FOWNER))
+                   !md_capable(uc, CAP_FOWNER))
                        RETURN(-EPERM);
 
                /* The IMMUTABLE and APPEND_ONLY flags can
                 * only be changed by the relevant capability. */
                if ((oldflags ^ newflags) &&
-                   !md_capable(uc, CFS_CAP_LINUX_IMMUTABLE))
+                   !md_capable(uc, CAP_LINUX_IMMUTABLE))
                        RETURN(-EPERM);
 
-               if (!S_ISDIR(oattr->la_mode))
+               if (!S_ISDIR(oattr->la_mode)) {
                        la->la_flags &= ~(LUSTRE_DIRSYNC_FL | LUSTRE_TOPDIR_FL);
+               } else if (la->la_flags & LUSTRE_ENCRYPT_FL) {
+                       /* when trying to add encryption flag on dir,
+                        * make sure it is empty
+                        */
+                       rc = mdd_dir_is_empty(env, obj);
+                       if (rc)
+                               RETURN(rc);
+                       rc = 0;
+               }
        }
 
        if (oattr->la_flags & (LUSTRE_IMMUTABLE_FL | LUSTRE_APPEND_FL) &&
@@ -747,7 +757,7 @@ static int mdd_fix_attr(const struct lu_env *env, struct mdd_object *obj,
        if ((la->la_valid & (LA_MTIME | LA_ATIME | LA_CTIME)) &&
            !(la->la_valid & ~(LA_MTIME | LA_ATIME | LA_CTIME))) {
                if ((uc->uc_fsuid != oattr->la_uid) &&
-                   !md_capable(uc, CFS_CAP_FOWNER)) {
+                   !md_capable(uc, CAP_FOWNER)) {
                        rc = mdd_permission_internal(env, obj, oattr,
                                                     MAY_WRITE);
                        if (rc)
@@ -780,7 +790,7 @@ static int mdd_fix_attr(const struct lu_env *env, struct mdd_object *obj,
        if (la->la_valid & LA_MODE) {
                if (!(flags & MDS_PERM_BYPASS) &&
                    (uc->uc_fsuid != oattr->la_uid) &&
-                   !md_capable(uc, CFS_CAP_FOWNER))
+                   !md_capable(uc, CAP_FOWNER))
                        RETURN(-EPERM);
 
                if (la->la_mode == (umode_t) -1)
@@ -792,7 +802,7 @@ static int mdd_fix_attr(const struct lu_env *env, struct mdd_object *obj,
                /* Also check the setgid bit! */
                if (!lustre_in_group_p(uc, (la->la_valid & LA_GID) ?
                                       la->la_gid : oattr->la_gid) &&
-                   !md_capable(uc, CFS_CAP_FSETID))
+                   !md_capable(uc, CAP_FSETID))
                        la->la_mode &= ~S_ISGID;
        } else {
               la->la_mode = oattr->la_mode;
@@ -804,7 +814,7 @@ static int mdd_fix_attr(const struct lu_env *env, struct mdd_object *obj,
                        la->la_uid = oattr->la_uid;
                if (((uc->uc_fsuid != oattr->la_uid) ||
                     (la->la_uid != oattr->la_uid)) &&
-                   !md_capable(uc, CFS_CAP_CHOWN))
+                   !md_capable(uc, CAP_CHOWN))
                        RETURN(-EPERM);
 
                /* If the user or group of a non-directory has been
@@ -830,7 +840,7 @@ static int mdd_fix_attr(const struct lu_env *env, struct mdd_object *obj,
                if (((uc->uc_fsuid != oattr->la_uid) ||
                     ((la->la_gid != oattr->la_gid) &&
                      !lustre_in_group_p(uc, la->la_gid))) &&
-                   !md_capable(uc, CFS_CAP_CHOWN))
+                   !md_capable(uc, CAP_CHOWN))
                        RETURN(-EPERM);
 
                /* Likewise, if the user or group of a non-directory
@@ -874,6 +884,7 @@ static int mdd_changelog_data_store_by_fid(const struct lu_env *env,
                                           enum changelog_rec_type type,
                                           enum changelog_rec_flags clf_flags,
                                           const struct lu_fid *fid,
+                                          const struct lu_fid *pfid,
                                           const char *xattr_name,
                                           struct thandle *handle)
 {
@@ -900,7 +911,7 @@ static int mdd_changelog_data_store_by_fid(const struct lu_env *env,
        reclen = llog_data_len(LLOG_CHANGELOG_HDR_SZ +
                               changelog_rec_offset(clf_flags & CLF_SUPPORTED,
                                                    xflags & CLFE_SUPPORTED));
-       buf = lu_buf_check_and_alloc(&mdd_env_info(env)->mti_big_buf, reclen);
+       buf = lu_buf_check_and_alloc(&mdd_env_info(env)->mti_chlg_buf, reclen);
        if (buf->lb_buf == NULL)
                RETURN(-ENOMEM);
        rec = buf->lb_buf;
@@ -909,6 +920,8 @@ static int mdd_changelog_data_store_by_fid(const struct lu_env *env,
        rec->cr.cr_flags = clf_flags;
        rec->cr.cr_type = (__u32)type;
        rec->cr.cr_tfid = *fid;
+       if (pfid)
+               rec->cr.cr_pfid = *pfid;
        rec->cr.cr_namelen = 0;
 
        if (clf_flags & CLF_JOBID)
@@ -940,11 +953,13 @@ static int mdd_changelog_data_store_by_fid(const struct lu_env *env,
  * want the change to commit without the log entry.
  * \param mdd_obj - mdd_object of change
  * \param handle - transaction handle
+ * \param pfid - parent FID for CL_MTIME changelogs
  */
 int mdd_changelog_data_store(const struct lu_env *env, struct mdd_device *mdd,
                             enum changelog_rec_type type,
                             enum changelog_rec_flags clf_flags,
-                            struct mdd_object *mdd_obj, struct thandle *handle)
+                            struct mdd_object *mdd_obj, struct thandle *handle,
+                            const struct lu_fid *pfid)
 {
        int                              rc;
 
@@ -966,7 +981,8 @@ int mdd_changelog_data_store(const struct lu_env *env, struct mdd_device *mdd,
        }
 
        rc = mdd_changelog_data_store_by_fid(env, mdd, type, clf_flags,
-                                            mdo2fid(mdd_obj), NULL, handle);
+                                            mdd_object_fid(mdd_obj), pfid,
+                                            NULL, handle);
        if (rc == 0)
                mdd_obj->mod_cltime = ktime_get();
 
@@ -1002,8 +1018,8 @@ int mdd_changelog_data_store_xattr(const struct lu_env *env,
        }
 
        rc = mdd_changelog_data_store_by_fid(env, mdd, type, clf_flags,
-                                            mdo2fid(mdd_obj), xattr_name,
-                                            handle);
+                                            mdd_object_fid(mdd_obj), NULL,
+                                            xattr_name, handle);
        if (rc == 0)
                mdd_obj->mod_cltime = ktime_get();
 
@@ -1041,7 +1057,7 @@ static int mdd_changelog(const struct lu_env *env, enum changelog_rec_type type,
                GOTO(stop, rc);
 
        rc = mdd_changelog_data_store_by_fid(env, mdd, type, clf_flags,
-                                            fid, NULL, handle);
+                                            fid, NULL, NULL, handle);
 
 stop:
        rc = mdd_trans_stop(env, mdd, rc, handle);
@@ -1062,16 +1078,16 @@ stop:
  * atime and ctime are independent.) */
 static int mdd_attr_set_changelog(const struct lu_env *env,
                                   struct md_object *obj, struct thandle *handle,
-                                  __u64 valid)
+                                 const struct lu_fid *pfid, __u64 valid)
 {
        struct mdd_device *mdd = mdo2mdd(obj);
        int bits, type = 0;
 
-       bits =  (valid & LA_SIZE)  ? 1 << CL_TRUNC : 0;
-       bits |= (valid & ~(LA_CTIME|LA_MTIME|LA_ATIME)) ? 1 << CL_SETATTR : 0;
-       bits |= (valid & LA_MTIME) ? 1 << CL_MTIME : 0;
-       bits |= (valid & LA_CTIME) ? 1 << CL_CTIME : 0;
-       bits |= (valid & LA_ATIME) ? 1 << CL_ATIME : 0;
+       bits =  (valid & LA_SIZE)  ? BIT(CL_TRUNC) : 0;
+       bits |= (valid & ~(LA_CTIME|LA_MTIME|LA_ATIME)) ? BIT(CL_SETATTR) : 0;
+       bits |= (valid & LA_MTIME) ? BIT(CL_MTIME) : 0;
+       bits |= (valid & LA_CTIME) ? BIT(CL_CTIME) : 0;
+       bits |= (valid & LA_ATIME) ? BIT(CL_ATIME) : 0;
        bits = bits & mdd->mdd_cl.mc_mask;
        /* This is an implementation limit rather than a protocol limit */
        BUILD_BUG_ON(CL_LAST > sizeof(int) * 8);
@@ -1082,8 +1098,8 @@ static int mdd_attr_set_changelog(const struct lu_env *env,
        type = __ffs(bits);
 
        /* XXX: we only store the low CLF_FLAGMASK bits of la_valid */
-       return mdd_changelog_data_store(env, mdd, type, valid,
-                                       md2mdd_obj(obj), handle);
+       return mdd_changelog_data_store(env, mdd, type, valid, md2mdd_obj(obj),
+                                       handle, pfid);
 }
 
 static int mdd_declare_attr_set(const struct lu_env *env,
@@ -1218,9 +1234,10 @@ int mdd_attr_set(const struct lu_env *env, struct md_object *obj,
 
        /* no need to setattr anymore */
        if (la_copy->la_valid == 0) {
-               CDEBUG(D_INODE, "%s: no valid attribute on "DFID", previous"
-                      "valid is %#llx\n", mdd2obd_dev(mdd)->obd_name,
-                      PFID(mdo2fid(mdd_obj)), la->la_valid);
+               CDEBUG(D_INODE,
+                      "%s: no valid attribute on "DFID", previous was %#llx\n",
+                      mdd_obj_dev_name(mdd_obj),
+                      PFID(mdd_object_fid(mdd_obj)), la->la_valid);
 
                RETURN(0);
        }
@@ -1308,7 +1325,7 @@ int mdd_attr_set(const struct lu_env *env, struct md_object *obj,
 
 out:
        if (rc == 0)
-               rc = mdd_attr_set_changelog(env, obj, handle,
+               rc = mdd_attr_set_changelog(env, obj, handle, &ma->ma_pfid,
                                            la_copy->la_valid);
 
        if (handle != NULL)
@@ -1334,11 +1351,11 @@ static int mdd_xattr_sanity_check(const struct lu_env *env,
                 * can write attributes. */
                if (S_ISDIR(attr->la_mode) && (attr->la_mode & S_ISVTX) &&
                    (uc->uc_fsuid != attr->la_uid) &&
-                   !md_capable(uc, CFS_CAP_FOWNER))
+                   !md_capable(uc, CAP_FOWNER))
                        RETURN(-EPERM);
        } else if (strcmp(name, XATTR_NAME_SOM) != 0 &&
                   (uc->uc_fsuid != attr->la_uid) &&
-                  !md_capable(uc, CFS_CAP_FOWNER)) {
+                  !md_capable(uc, CAP_FOWNER)) {
                RETURN(-EPERM);
        }
 
@@ -1442,7 +1459,8 @@ static int mdd_hsm_update_locked(const struct lu_env *env,
 
        /* Read HSM attrs from disk */
        current_buf = lu_buf_check_and_alloc(&info->mti_xattr_buf,
-                       MIN(mdd_obj2mdd_dev(mdd_obj)->mdd_dt_conf.ddp_max_ea_size,
+                       min_t(unsigned int,
+                             mdd_obj2mdd_dev(mdd_obj)->mdd_dt_conf.ddp_max_ea_size,
                            XATTR_SIZE_MAX));
        rc = mdo_xattr_get(env, mdd_obj, current_buf, XATTR_NAME_HSM);
        rc = lustre_buf2hsm(current_buf->lb_buf, rc, current_mh);
@@ -1529,7 +1547,7 @@ static int mdd_xattr_merge(const struct lu_env *env, struct md_object *md_obj,
        int rc;
        ENTRY;
 
-       rc = lu_fid_cmp(mdo2fid(obj), mdo2fid(vic));
+       rc = lu_fid_cmp(mdd_object_fid(obj), mdd_object_fid(vic));
        if (rc == 0) /* same fid */
                RETURN(-EPERM);
 
@@ -1588,8 +1606,10 @@ static int mdd_xattr_merge(const struct lu_env *env, struct md_object *md_obj,
        if (rc) /* wtf? */
                GOTO(out_restore, rc);
 
-       (void)mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, obj, handle);
-       (void)mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, vic, handle);
+       (void)mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, obj, handle,
+                                      NULL);
+       (void)mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, vic, handle,
+                                      NULL);
        EXIT;
 
 out_restore:
@@ -1597,9 +1617,9 @@ out_restore:
                int rc2 = mdo_xattr_set(env, obj, buf, XATTR_NAME_LOV,
                                        LU_XATTR_REPLACE, handle);
                if (rc2)
-                       CERROR("%s: failed to rollback of layout of: "DFID
-                              ": %d, file state unknown\n",
-                              mdd_obj_dev_name(obj), PFID(mdo2fid(obj)), rc2);
+                       CERROR("%s: failed rollback of "DFID" layout: file state unknown: rc = %d\n",
+                              mdd_obj_dev_name(obj),
+                              PFID(mdd_object_fid(obj)), rc2);
        }
 
 out:
@@ -1749,7 +1769,7 @@ static int mdd_xattr_split(const struct lu_env *env, struct md_object *md_obj,
 {
        struct mdd_device *mdd = mdo2mdd(md_obj);
        struct mdd_object *obj = md2mdd_obj(md_obj);
-       struct mdd_object *vic = md2mdd_obj(mrd->mrd_obj);
+       struct mdd_object *vic = NULL;
        struct lu_buf *buf = &mdd_env_info(env)->mti_buf[0];
        struct lu_buf *buf_save = &mdd_env_info(env)->mti_buf[1];
        struct lu_buf *buf_vic = &mdd_env_info(env)->mti_buf[2];
@@ -1760,95 +1780,156 @@ static int mdd_xattr_split(const struct lu_env *env, struct md_object *md_obj,
 
        ENTRY;
 
-       rc = lu_fid_cmp(mdo2fid(obj), mdo2fid(vic));
-       if (rc == 0) /* same fid */
-               RETURN(-EPERM);
+       /**
+        * NULL @mrd_obj means mirror deleting, and use NULL vic to indicate
+        * mirror deleting
+        */
+       if (mrd->mrd_obj)
+               vic = md2mdd_obj(mrd->mrd_obj);
 
        handle = mdd_trans_create(env, mdd);
        if (IS_ERR(handle))
                RETURN(PTR_ERR(handle));
 
-       if (rc > 0) {
-               mdd_write_lock(env, obj, DT_TGT_CHILD);
-               mdd_write_lock(env, vic, DT_TGT_CHILD);
-       } else {
-               mdd_write_lock(env, vic, DT_TGT_CHILD);
-               mdd_write_lock(env, obj, DT_TGT_CHILD);
-       }
-
        /* get EA of mirrored file */
        memset(buf_save, 0, sizeof(*buf));
        rc = mdd_stripe_get(env, obj, buf_save, XATTR_NAME_LOV);
        if (rc < 0)
-               GOTO(out, rc);
+               GOTO(stop, rc);
 
        lcm = buf_save->lb_buf;
        if (le32_to_cpu(lcm->lcm_magic) != LOV_MAGIC_COMP_V1)
-               GOTO(out, rc = -EINVAL);
+               GOTO(stop, rc = -EINVAL);
 
        /**
         * Extract the mirror with specified mirror id, and store the splitted
-        * mirror layout to the victim file.
+        * mirror layout to the victim buffer.
         */
        memset(buf, 0, sizeof(*buf));
        memset(buf_vic, 0, sizeof(*buf_vic));
        rc = mdd_split_ea(lcm, mrd->mrd_mirror_id, buf, buf_vic);
        if (rc < 0)
-               GOTO(out, rc);
+               GOTO(stop, rc);
+       /**
+        * @buf stores layout w/o the specified mirror, @buf_vic stores the
+        * splitted mirror
+        */
 
        dom_stripe = mdd_lmm_dom_size(buf_vic->lb_buf) > 0;
 
-       rc = mdd_declare_xattr_set(env, mdd, obj, buf, XATTR_NAME_LOV,
-                                  LU_XATTR_SPLIT, handle);
-       if (rc)
-               GOTO(out, rc);
-       rc = mdd_declare_xattr_set(env, mdd, vic, buf_vic, XATTR_NAME_LOV,
-                                  LU_XATTR_SPLIT, handle);
-       if (rc)
-               GOTO(out, rc);
+       if (vic) {
+               /**
+                * non delete mirror split
+                *
+                * declare obj set remaining layout in @buf, will set obj's
+                * in-memory layout
+                */
+               rc = mdd_declare_xattr_set(env, mdd, obj, buf, XATTR_NAME_LOV,
+                                          LU_XATTR_SPLIT, handle);
+               if (rc)
+                       GOTO(stop, rc);
+
+               /* declare vic set splitted layout in @buf_vic */
+               rc = mdd_declare_xattr_set(env, mdd, vic, buf_vic,
+                                          XATTR_NAME_LOV, LU_XATTR_SPLIT,
+                                          handle);
+               if (rc)
+                       GOTO(stop, rc);
+       } else {
+               /**
+                * declare delete mirror objects in @buf_vic, will change obj's
+                * in-memory layout
+                */
+               rc = mdd_declare_xattr_set(env, mdd, obj, buf_vic,
+                                          XATTR_NAME_LOV, LU_XATTR_PURGE,
+                                          handle);
+               if (rc)
+                       GOTO(stop, rc);
+
+               /* declare obj set remaining layout in @buf */
+               rc = mdd_declare_xattr_set(env, mdd, obj, buf,
+                                          XATTR_NAME_LOV, LU_XATTR_SPLIT,
+                                          handle);
+               if (rc)
+                       GOTO(stop, rc);
+       }
 
        rc = mdd_trans_start(env, mdd, handle);
        if (rc)
-               GOTO(out, rc);
+               GOTO(stop, rc);
 
+       if (vic) {
+               /* don't use the same file to save the splitted mirror */
+               rc = lu_fid_cmp(mdd_object_fid(obj), mdd_object_fid(vic));
+               if (rc == 0)
+                       GOTO(stop, rc = -EPERM);
+
+               if (rc > 0) {
+                       mdd_write_lock(env, obj, DT_TGT_CHILD);
+                       mdd_write_lock(env, vic, DT_TGT_CHILD);
+               } else {
+                       mdd_write_lock(env, vic, DT_TGT_CHILD);
+                       mdd_write_lock(env, obj, DT_TGT_CHILD);
+               }
+       } else {
+               mdd_write_lock(env, obj, DT_TGT_CHILD);
+       }
+
+       /* set obj's layout in @buf */
        rc = mdo_xattr_set(env, obj, buf, XATTR_NAME_LOV, LU_XATTR_REPLACE,
                           handle);
        if (rc)
-               GOTO(out, rc);
+               GOTO(unlock, rc);
 
-       rc = mdo_xattr_set(env, vic, buf_vic, XATTR_NAME_LOV, LU_XATTR_CREATE,
-                          handle);
-       if (rc)
-               GOTO(out_restore, rc);
+       if (vic) {
+               /* set vic's layout in @buf_vic */
+               rc = mdo_xattr_set(env, vic, buf_vic, XATTR_NAME_LOV,
+                                  LU_XATTR_CREATE, handle);
+               if (rc)
+                       GOTO(out_restore, rc);
+       } else {
+               /* delete mirror objects */
+               rc = mdo_xattr_set(env, obj, buf_vic, XATTR_NAME_LOV,
+                                  LU_XATTR_PURGE, handle);
+               if (rc)
+                       GOTO(out_restore, rc);
+       }
 
-       rc = mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, obj, handle);
+       rc = mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, obj, handle,
+                                     NULL);
        if (rc)
-               GOTO(out, rc);
+               GOTO(out_restore, rc);
 
-       rc = mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, vic, handle);
-       if (rc)
-               GOTO(out, rc);
-       EXIT;
+       if (vic) {
+               rc = mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, vic,
+                                             handle, NULL);
+               if (rc)
+                       GOTO(out_restore, rc);
+       }
 
 out_restore:
        if (rc) {
-               /* restore obj's layout */
+               /* restore obj's in-memory and on-disk layout */
                int rc2 = mdo_xattr_set(env, obj, buf_save, XATTR_NAME_LOV,
                                        LU_XATTR_REPLACE, handle);
                if (rc2)
-                       CERROR("%s: failed to rollback of layout of: "DFID
-                              ": %d, file state unkonwn.\n",
-                              mdd_obj_dev_name(obj), PFID(mdo2fid(obj)), rc2);
+                       CERROR("%s: failed rollback "DFID
+                              " layout: file state unknown: rc = %d\n",
+                              mdd_obj_dev_name(obj),
+                              PFID(mdd_object_fid(obj)), rc);
        }
-out:
+
+unlock:
+       mdd_write_unlock(env, obj);
+       if (vic)
+               mdd_write_unlock(env, vic);
+stop:
        rc = mdd_trans_stop(env, mdd, rc, handle);
 
        /* Truncate local DOM data if all went well */
        if (!rc && dom_stripe)
                mdd_dom_data_truncate(env, mdd, obj);
 
-       mdd_write_unlock(env, obj);
-       mdd_write_unlock(env, vic);
        lu_buf_free(buf_save);
        lu_buf_free(buf);
        lu_buf_free(buf_vic);
@@ -1908,20 +1989,16 @@ static int mdd_xattr_set(const struct lu_env *env, struct md_object *obj,
                if (buf->lb_len != sizeof(*mrd))
                        RETURN(-EINVAL);
 
-               rc = mdd_layout_merge_allowed(env, obj, victim);
-               if (rc)
-                       RETURN(rc);
 
-               if (fl == LU_XATTR_MERGE)
+               if (fl == LU_XATTR_MERGE) {
+                       rc = mdd_layout_merge_allowed(env, obj, victim);
+                       if (rc)
+                               RETURN(rc);
                        /* merge layout of victim as a mirror of obj's. */
                        rc = mdd_xattr_merge(env, obj, victim);
-               else
+               } else {
                        rc = mdd_xattr_split(env, obj, mrd);
-               RETURN(rc);
-       }
-
-       if (strcmp(name, XATTR_NAME_LMV) == 0) {
-               rc = mdd_dir_layout_shrink(env, obj, buf);
+               }
                RETURN(rc);
        }
 
@@ -2127,7 +2204,7 @@ static int mdd_xattr_hsm_replace(const struct lu_env *env,
        /* Add a changelog record for release. */
        hsm_set_cl_event(&clf_flags, HE_RELEASE);
        rc = mdd_changelog_data_store(env, mdo2mdd(&o->mod_obj), CL_HSM,
-                                     clf_flags, o, handle);
+                                     clf_flags, o, handle, NULL);
        RETURN(rc);
 }
 
@@ -2146,11 +2223,11 @@ static int mdd_layout_swap_allowed(const struct lu_env *env,
                                   const struct lu_attr *attr2,
                                   __u64 flags)
 {
-       const struct lu_fid     *fid1, *fid2;
+       const struct lu_fid *fid1, *fid2;
        ENTRY;
 
-       fid1 = mdo2fid(o1);
-       fid2 = mdo2fid(o2);
+       fid1 = mdd_object_fid(o1);
+       fid2 = mdd_object_fid(o2);
 
        if (!fid_is_norm(fid1) &&
            (!fid_is_igif(fid1) || IS_ERR(mdd_links_get(env, o1))))
@@ -2291,8 +2368,8 @@ stop:
 out:
        /* Ignore failure but report the error */
        if (rc)
-               CERROR("%s: "DFID" can't truncate DOM inode data, rc = %d\n",
-                      mdd_obj_dev_name(mo), PFID(mdo2fid(mo)), rc);
+               CERROR("%s: can't truncate DOM inode "DFID" data: rc = %d\n",
+                      mdd_obj_dev_name(mo), PFID(mdd_object_fid(mo)), rc);
        return rc;
 }
 
@@ -2328,7 +2405,7 @@ static int mdd_swap_layouts(const struct lu_env *env, struct md_object *obj1,
 
        /* we have to sort the 2 obj, so locking will always
         * be in the same order, even in case of 2 concurrent swaps */
-       rc = lu_fid_cmp(mdo2fid(fst_o), mdo2fid(snd_o));
+       rc = lu_fid_cmp(mdd_object_fid(fst_o), mdd_object_fid(snd_o));
        if (rc == 0) /* same fid ? */
                RETURN(-EPERM);
 
@@ -2393,15 +2470,17 @@ static int mdd_swap_layouts(const struct lu_env *env, struct md_object *obj1,
 
        if (domsize_vlt > 0 && domsize_dom == 0) {
                rc = -EOPNOTSUPP;
-               CDEBUG(D_LAYOUT, "cannot swap layout for "DFID": OST to DOM "
-                      "migration is not supported: rc = %d\n",
-                      PFID(mdo2fid(snd_o)), rc);
+               CDEBUG(D_LAYOUT,
+                      "%s: cannot swap "DFID" layout: OST to DOM migration not supported: rc = %d\n",
+                      mdd_obj_dev_name(snd_o),
+                      PFID(mdd_object_fid(snd_o)), rc);
                GOTO(stop, rc);
        } else if (domsize_vlt > 0 && domsize_dom != domsize_vlt) {
                rc = -EOPNOTSUPP;
-               CDEBUG(D_LAYOUT, "cannot swap layout for "DFID": new layout "
-                      "must have the same DoM component size: rc = %d\n",
-                      PFID(mdo2fid(fst_o)), rc);
+               CDEBUG(D_LAYOUT,
+                      "%s: cannot swap "DFID" layout: new layout must have same DoM component size: rc = %d\n",
+                      mdd_obj_dev_name(fst_o),
+                      PFID(mdd_object_fid(fst_o)), rc);
                GOTO(stop, rc);
        } else if (domsize_vlt > 0) {
                /* Migration with the same DOM component size, no need to
@@ -2459,10 +2538,13 @@ static int mdd_swap_layouts(const struct lu_env *env, struct md_object *obj1,
 
        /* set the file specific informations in lmm */
        if (fst_lmm != NULL) {
+               struct ost_id temp_oi;
+
                saved_oi = &info->mti_oa.o_oi;
                mdd_get_lmm_oi(fst_lmm, saved_oi);
+               mdd_get_lmm_oi(snd_lmm, &temp_oi);
                mdd_set_lmm_gen(fst_lmm, &snd_gen);
-               mdd_set_lmm_oi(fst_lmm, &snd_lmm->lmm_oi);
+               mdd_set_lmm_oi(fst_lmm, &temp_oi);
                mdd_set_lmm_oi(snd_lmm, saved_oi);
        } else {
                if ((snd_lmm->lmm_magic & cpu_to_le32(LOV_MAGIC_MASK)) ==
@@ -2534,9 +2616,9 @@ static int mdd_swap_layouts(const struct lu_env *env, struct md_object *obj1,
                        rc2 = mdd_xattr_hsm_replace(env, fst_o, fst_hsm_buf,
                                                    handle);
                        if (rc2 < 0)
-                               CERROR("%s: restore "DFID" HSM error: %d/%d\n",
+                               CERROR("%s: HSM error restoring "DFID": rc = %d/%d\n",
                                       mdd_obj_dev_name(fst_o),
-                                      PFID(mdo2fid(fst_o)), rc, rc2);
+                                      PFID(mdd_object_fid(fst_o)), rc, rc2);
                        GOTO(stop, rc);
                }
        }
@@ -2558,11 +2640,13 @@ static int mdd_swap_layouts(const struct lu_env *env, struct md_object *obj1,
                GOTO(out_restore, rc);
 
        /* Issue one changelog record per file */
-       rc = mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, fst_o, handle);
+       rc = mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, fst_o, handle,
+                                     NULL);
        if (rc)
                GOTO(stop, rc);
 
-       rc = mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, snd_o, handle);
+       rc = mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, snd_o, handle,
+                                     NULL);
        if (rc)
                GOTO(stop, rc);
        EXIT;
@@ -2595,10 +2679,10 @@ out_restore:
        do_lbug:
                if (rc2 < 0) {
                        /* very bad day */
-                       CERROR("%s: unable to roll back layout swap. FIDs: "
-                              DFID" and "DFID "error: %d/%d, steps: %d\n",
+                       CERROR("%s: unable to roll back layout swap of "DFID" and "DFID", steps: %d: rc = %d/%d\n",
                               mdd_obj_dev_name(fst_o),
-                              PFID(mdo2fid(snd_o)), PFID(mdo2fid(fst_o)),
+                              PFID(mdd_object_fid(snd_o)),
+                              PFID(mdd_object_fid(fst_o)),
                               rc, rc2, steps);
                        /* a solution to avoid journal commit is to panic,
                         * but it has strong consequences so we use LBUG to
@@ -2678,7 +2762,8 @@ mdd_layout_instantiate_component(const struct lu_env *env,
        if (rc)
                RETURN(rc);
 
-       rc = mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, obj, handle);
+       rc = mdd_changelog_data_store(env, mdd, CL_LAYOUT, 0, obj, handle,
+                                     NULL);
        RETURN(rc);
 }
 
@@ -2724,6 +2809,12 @@ mdd_layout_update_rdonly(const struct lu_env *env, struct mdd_object *obj,
                lustre_som_swab(som);
                if (som->lsa_valid & SOM_FL_STRICT)
                        fl = LU_XATTR_REPLACE;
+
+               if (mlc->mlc_opc == MD_LAYOUT_WRITE &&
+                   mlc->mlc_intent->li_extent.e_end > som->lsa_size) {
+                       som->lsa_size = mlc->mlc_intent->li_extent.e_end + 1;
+                       fl = LU_XATTR_REPLACE;
+               }
        }
 
        rc = mdd_declare_layout_change(env, mdd, obj, mlc, handle);
@@ -2762,7 +2853,7 @@ mdd_layout_update_rdonly(const struct lu_env *env, struct mdd_object *obj,
        if (rc)
                GOTO(out, rc);
 
-       rc = mdd_changelog_data_store(env, mdd, CL_FLRW, 0, obj, handle);
+       rc = mdd_changelog_data_store(env, mdd, CL_FLRW, 0, obj, handle, NULL);
        if (rc)
                GOTO(out, rc);
 
@@ -2786,6 +2877,9 @@ mdd_layout_update_write_pending(const struct lu_env *env,
                struct thandle *handle)
 {
        struct mdd_device *mdd = mdd_obj2mdd_dev(obj);
+       struct lu_buf *som_buf = &mdd_env_info(env)->mti_buf[1];
+       struct lustre_som_attrs *som = &mlc->mlc_som;
+       int fl = 0;
        int rc;
        ENTRY;
 
@@ -2798,8 +2892,25 @@ mdd_layout_update_write_pending(const struct lu_env *env,
                 * resync state. */
                break;
        case MD_LAYOUT_WRITE:
-               /* legal race for concurrent write, the file state has been
-                * changed by another client. */
+               /**
+                * legal race for concurrent write, the file state has been
+                * changed by another client. Or a jump over file size and
+                * write.
+                */
+               som_buf->lb_buf = som;
+               som_buf->lb_len = sizeof(*som);
+               rc = mdo_xattr_get(env, obj, som_buf, XATTR_NAME_SOM);
+               if (rc < 0 && rc != -ENODATA)
+                       RETURN(rc);
+
+               if (rc > 0) {
+                       lustre_som_swab(som);
+                       if (mlc->mlc_intent->li_extent.e_end > som->lsa_size) {
+                               som->lsa_size =
+                                       mlc->mlc_intent->li_extent.e_end + 1;
+                               fl = LU_XATTR_REPLACE;
+                       }
+               }
                break;
        default:
                RETURN(-EBUSY);
@@ -2809,6 +2920,13 @@ mdd_layout_update_write_pending(const struct lu_env *env,
        if (rc)
                GOTO(out, rc);
 
+       if (fl) {
+               rc = mdd_declare_xattr_set(env, mdd, obj, som_buf,
+                                          XATTR_NAME_SOM, fl, handle);
+               if (rc)
+                       GOTO(out, rc);
+       }
+
        rc = mdd_trans_start(env, mdd, handle);
        if (rc)
                GOTO(out, rc);
@@ -2818,6 +2936,12 @@ mdd_layout_update_write_pending(const struct lu_env *env,
 
        mdd_write_lock(env, obj, DT_TGT_CHILD);
        rc = mdo_layout_change(env, obj, mlc, handle);
+       if (!rc && fl) {
+               som->lsa_valid = SOM_FL_STALE;
+               lustre_som_swab(som);
+               rc = mdo_xattr_set(env, obj, som_buf, XATTR_NAME_SOM,
+                                  fl, handle);
+       }
        mdd_write_unlock(env, obj);
        if (rc)
                GOTO(out, rc);
@@ -2912,7 +3036,8 @@ mdd_object_update_sync_pending(const struct lu_env *env, struct mdd_object *obj,
                        GOTO(out, rc);
        }
 
-       rc = mdd_changelog_data_store(env, mdd, CL_RESYNC, 0, obj, handle);
+       rc = mdd_changelog_data_store(env, mdd, CL_RESYNC, 0, obj, handle,
+                                     NULL);
        if (rc)
                GOTO(out, rc);
        EXIT;
@@ -2937,8 +3062,24 @@ mdd_layout_change(const struct lu_env *env, struct md_object *o,
        struct thandle          *handle;
        int flr_state;
        int rc;
+
        ENTRY;
 
+       if (S_ISDIR(mdd_object_type(obj))) {
+               switch (mlc->mlc_opc) {
+               case MD_LAYOUT_SHRINK:
+                       rc = mdd_dir_layout_shrink(env, o, mlc);
+                       break;
+               case MD_LAYOUT_SPLIT:
+                       rc = mdd_dir_layout_split(env, o, mlc);
+                       break;
+               default:
+                       LBUG();
+               }
+
+               RETURN(rc);
+       }
+
        /* Verify acceptable operations */
        switch (mlc->mlc_opc) {
        case MD_LAYOUT_WRITE:
@@ -3034,8 +3175,8 @@ void mdd_object_make_hint(const struct lu_env *env, struct mdd_object *parent,
        nc->do_ops->do_ah_init(env, hint, np, nc, attr->la_mode & S_IFMT);
 }
 
-static int accmode(const struct lu_env *env, const struct lu_attr *la,
-                  u64 open_flags)
+static int mdd_accmode(const struct lu_env *env, const struct lu_attr *la,
+                      u64 open_flags)
 {
        /* Sadly, NFSD reopens a file repeatedly during operation, so the
         * "acc_mode = 0" allowance for newly-created files isn't honoured.
@@ -3057,7 +3198,8 @@ static int mdd_open_sanity_check(const struct lu_env *env,
                                 const struct lu_attr *attr, u64 open_flags,
                                 int is_replay)
 {
-       int mode, rc;
+       unsigned int may_mask;
+       int rc;
        ENTRY;
 
        /* EEXIST check, also opening of *open* orphans is allowed so we can
@@ -3070,13 +3212,13 @@ static int mdd_open_sanity_check(const struct lu_env *env,
        if (S_ISLNK(attr->la_mode))
                RETURN(-ELOOP);
 
-       mode = accmode(env, attr, open_flags);
+       may_mask = mdd_accmode(env, attr, open_flags);
 
-       if (S_ISDIR(attr->la_mode) && (mode & MAY_WRITE))
+       if (S_ISDIR(attr->la_mode) && (may_mask & MAY_WRITE))
                RETURN(-EISDIR);
 
        if (!(open_flags & MDS_OPEN_CREATED)) {
-               rc = mdd_permission_internal(env, obj, attr, mode);
+               rc = mdd_permission_internal(env, obj, attr, may_mask);
                if (rc)
                        RETURN(rc);
        }
@@ -3118,7 +3260,7 @@ static int mdd_open(const struct lu_env *env, struct md_object *obj,
 
        rc = mdd_open_sanity_check(env, mdd_obj, attr, open_flags,
                                   spec->no_create);
-       if ((rc == -EACCES) && (mdd->mdd_cl.mc_mask & (1 << CL_DN_OPEN)))
+       if ((rc == -EACCES) && (mdd->mdd_cl.mc_mask & BIT(CL_DN_OPEN)))
                type = CL_DN_OPEN;
        else if (rc != 0)
                GOTO(out, rc);
@@ -3173,7 +3315,7 @@ find:
        }
 
        /* FYI, only the bottom 32 bits of open_flags are recorded */
-       mdd_changelog(env, type, open_flags, md_dev, mdo2fid(mdd_obj));
+       mdd_changelog(env, type, open_flags, md_dev, mdd_object_fid(mdd_obj));
 
        EXIT;
 out:
@@ -3356,10 +3498,10 @@ out:
         * this is not a big deal if we have a CL_CLOSE entry with no matching
         * CL_OPEN. Plus Changelogs mask may not change often.
         */
-       if (((!(mdd->mdd_cl.mc_mask & (1 << CL_OPEN)) &&
+       if (((!(mdd->mdd_cl.mc_mask & BIT(CL_OPEN)) &&
              (open_flags & (MDS_FMODE_WRITE | MDS_OPEN_APPEND |
                             MDS_OPEN_TRUNC))) ||
-            ((mdd->mdd_cl.mc_mask & (1 << CL_OPEN)) && last_close_by_uid)) &&
+            ((mdd->mdd_cl.mc_mask & BIT(CL_OPEN)) && last_close_by_uid)) &&
            !(ma->ma_valid & MA_FLAGS && ma->ma_attr_flags & MDS_RECOV_OPEN)) {
                if (handle == NULL) {
                        handle = mdd_trans_create(env, mdo2mdd(obj));
@@ -3378,7 +3520,7 @@ out:
 
                /* FYI, only the bottom 32 bits of open_flags are recorded */
                mdd_changelog_data_store(env, mdd, CL_CLOSE, open_flags,
-                                        mdd_obj, handle);
+                                        mdd_obj, handle, NULL);
        }
 
 stop: