Whamcloud - gitweb
LU-6280 lod: delete xattr on striped dir
[fs/lustre-release.git] / lustre / lod / lod_object.c
index dba607c..791466c 100644 (file)
@@ -6,13 +6,13 @@
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 2 only,
  * as published by the Free Software Foundation.
-
+ *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License version 2 for more details.  A copy is
  * included in the COPYING file that accompanied this code.
-
+ *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
@@ -23,7 +23,7 @@
  * Copyright  2009 Sun Microsystems, Inc. All rights reserved
  * Use is subject to license terms.
  *
- * Copyright (c) 2012, 2013, Intel Corporation.
+ * Copyright (c) 2012, 2014, Intel Corporation.
  */
 /*
  * lustre/lod/lod_object.c
@@ -58,7 +58,6 @@
 static const char dot[] = ".";
 static const char dotdot[] = "..";
 
-extern struct kmem_cache *lod_object_kmem;
 static const struct dt_body_operations lod_body_lnk_ops;
 
 /**
@@ -188,7 +187,7 @@ do {                                                                \
  *
  * \see dt_index_operations::dio_it.fini() in the API description for details.
  */
-void lod_it_fini(const struct lu_env *env, struct dt_it *di)
+static void lod_it_fini(const struct lu_env *env, struct dt_it *di)
 {
        struct lod_it *it = (struct lod_it *)di;
 
@@ -207,8 +206,8 @@ void lod_it_fini(const struct lu_env *env, struct dt_it *di)
  *
  * \see dt_it_ops::get() in the API description for details.
  */
-int lod_it_get(const struct lu_env *env, struct dt_it *di,
-              const struct dt_key *key)
+static int lod_it_get(const struct lu_env *env, struct dt_it *di,
+                     const struct dt_key *key)
 {
        const struct lod_it *it = (const struct lod_it *)di;
 
@@ -223,7 +222,7 @@ int lod_it_get(const struct lu_env *env, struct dt_it *di,
  *
  * \see dt_it_ops::put() in the API description for details.
  */
-void lod_it_put(const struct lu_env *env, struct dt_it *di)
+static void lod_it_put(const struct lu_env *env, struct dt_it *di)
 {
        struct lod_it *it = (struct lod_it *)di;
 
@@ -238,7 +237,7 @@ void lod_it_put(const struct lu_env *env, struct dt_it *di)
  *
  * \see dt_it_ops::next() in the API description for details.
  */
-int lod_it_next(const struct lu_env *env, struct dt_it *di)
+static int lod_it_next(const struct lu_env *env, struct dt_it *di)
 {
        struct lod_it *it = (struct lod_it *)di;
 
@@ -253,7 +252,8 @@ int lod_it_next(const struct lu_env *env, struct dt_it *di)
  *
  * \see dt_it_ops::key() in the API description for details.
  */
-struct dt_key *lod_it_key(const struct lu_env *env, const struct dt_it *di)
+static struct dt_key *lod_it_key(const struct lu_env *env,
+                                const struct dt_it *di)
 {
        const struct lod_it *it = (const struct lod_it *)di;
 
@@ -268,7 +268,7 @@ struct dt_key *lod_it_key(const struct lu_env *env, const struct dt_it *di)
  *
  * \see dt_it_ops::key_size() in the API description for details.
  */
-int lod_it_key_size(const struct lu_env *env, const struct dt_it *di)
+static int lod_it_key_size(const struct lu_env *env, const struct dt_it *di)
 {
        struct lod_it *it = (struct lod_it *)di;
 
@@ -283,8 +283,8 @@ int lod_it_key_size(const struct lu_env *env, const struct dt_it *di)
  *
  * \see dt_it_ops::rec() in the API description for details.
  */
-int lod_it_rec(const struct lu_env *env, const struct dt_it *di,
-              struct dt_rec *rec, __u32 attr)
+static int lod_it_rec(const struct lu_env *env, const struct dt_it *di,
+                     struct dt_rec *rec, __u32 attr)
 {
        const struct lod_it *it = (const struct lod_it *)di;
 
@@ -300,8 +300,8 @@ int lod_it_rec(const struct lu_env *env, const struct dt_it *di,
  *
  * \see dt_it_ops::rec_size() in the API description for details.
  */
-int lod_it_rec_size(const struct lu_env *env, const struct dt_it *di,
-                   __u32 attr)
+static int lod_it_rec_size(const struct lu_env *env, const struct dt_it *di,
+                          __u32 attr)
 {
        const struct lod_it *it = (const struct lod_it *)di;
 
@@ -317,7 +317,7 @@ int lod_it_rec_size(const struct lu_env *env, const struct dt_it *di,
  *
  * \see dt_it_ops::store() in the API description for details.
  */
-__u64 lod_it_store(const struct lu_env *env, const struct dt_it *di)
+static __u64 lod_it_store(const struct lu_env *env, const struct dt_it *di)
 {
        const struct lod_it *it = (const struct lod_it *)di;
 
@@ -332,7 +332,8 @@ __u64 lod_it_store(const struct lu_env *env, const struct dt_it *di)
  *
  * \see dt_it_ops::load() in the API description for details.
  */
-int lod_it_load(const struct lu_env *env, const struct dt_it *di, __u64 hash)
+static int lod_it_load(const struct lu_env *env, const struct dt_it *di,
+                      __u64 hash)
 {
        const struct lod_it *it = (const struct lod_it *)di;
 
@@ -347,8 +348,8 @@ int lod_it_load(const struct lu_env *env, const struct dt_it *di, __u64 hash)
  *
  * \see dt_it_ops::rec() in the API description for details.
  */
-int lod_it_key_rec(const struct lu_env *env, const struct dt_it *di,
-                  void *key_rec)
+static int lod_it_key_rec(const struct lu_env *env, const struct dt_it *di,
+                         void *key_rec)
 {
        const struct lod_it *it = (const struct lod_it *)di;
 
@@ -1432,7 +1433,7 @@ static int lod_xattr_get(const struct lu_env *env, struct dt_object *dt,
 /**
  * Verify LVM EA.
  *
- * Checks that the magic and the number of the stripes are sane.
+ * Checks that the magic of the stripe is sane.
  *
  * \param[in] lod      lod device
  * \param[in] lum      a buffer storing LMV EA to verify
@@ -1443,22 +1444,16 @@ static int lod_xattr_get(const struct lu_env *env, struct dt_object *dt,
 static int lod_verify_md_striping(struct lod_device *lod,
                                  const struct lmv_user_md_v1 *lum)
 {
-       int     rc = 0;
-       ENTRY;
-
-       if (unlikely(le32_to_cpu(lum->lum_magic) != LMV_USER_MAGIC))
-               GOTO(out, rc = -EINVAL);
-
-       if (unlikely(le32_to_cpu(lum->lum_stripe_count) == 0))
-               GOTO(out, rc = -EINVAL);
-out:
-       if (rc != 0)
+       if (unlikely(le32_to_cpu(lum->lum_magic) != LMV_USER_MAGIC)) {
                CERROR("%s: invalid lmv_user_md: magic = %x, "
                       "stripe_offset = %d, stripe_count = %u: rc = %d\n",
                       lod2obd(lod)->obd_name, le32_to_cpu(lum->lum_magic),
                       (int)le32_to_cpu(lum->lum_stripe_offset),
-                      le32_to_cpu(lum->lum_stripe_count), rc);
-       return rc;
+                      le32_to_cpu(lum->lum_stripe_count), -EINVAL);
+               return -EINVAL;
+       }
+
+       return 0;
 }
 
 /**
@@ -1490,8 +1485,8 @@ static void lod_prep_slave_lmv_md(struct lmv_mds_md_v1 *slave_lmv,
  * \retval             0 on success
  * \retval             negative if failed
  */
-int lod_prep_lmv_md(const struct lu_env *env, struct dt_object *dt,
-                   struct lu_buf *lmv_buf)
+static int lod_prep_lmv_md(const struct lu_env *env, struct dt_object *dt,
+                          struct lu_buf *lmv_buf)
 {
        struct lod_thread_info  *info = lod_env_info(env);
        struct lod_device       *lod = lu2lod_dev(dt->do_lu.lo_dev);
@@ -1527,7 +1522,6 @@ int lod_prep_lmv_md(const struct lu_env *env, struct dt_object *dt,
        lmm1->lmv_master_mdt_index = cpu_to_le32(mdtidx);
        lmv_buf->lb_buf = info->lti_ea_store;
        lmv_buf->lb_len = sizeof(*lmm1);
-       lo->ldo_dir_striping_cached = 1;
 
        RETURN(rc);
 }
@@ -1810,7 +1804,7 @@ next:
                struct dt_object        *dto            = stripe[i];
                char                    *stripe_name    = info->lti_key;
                struct lu_name          *sname;
-               struct linkea_data       ldata          = { 0 };
+               struct linkea_data       ldata          = { NULL };
                struct lu_buf            linkea_buf;
 
                rc = dt_declare_create(env, dto, attr, NULL, dof, th);
@@ -1820,6 +1814,10 @@ next:
                if (!dt_try_as_dir(env, dto))
                        GOTO(out_put, rc = -EINVAL);
 
+               rc = dt_declare_ref_add(env, dto, th);
+               if (rc != 0)
+                       GOTO(out_put, rc);
+
                rec->rec_fid = lu_object_fid(&dto->do_lu);
                rc = dt_declare_insert(env, dto, (const struct dt_rec *)rec,
                                       (const struct dt_key *)dot, th);
@@ -1834,7 +1832,7 @@ next:
                        GOTO(out_put, rc);
 
                /* probably nothing to inherite */
-               if (lo->ldo_striping_cached &&
+               if (lo->ldo_def_striping_set &&
                    !LOVEA_DELETE_VALUES(lo->ldo_def_stripe_size,
                                         lo->ldo_def_stripenr,
                                         lo->ldo_def_stripe_offset,
@@ -2126,13 +2124,13 @@ static int lod_declare_xattr_set(const struct lu_env *env,
  */
 static void lod_lov_stripe_cache_clear(struct lod_object *lo)
 {
-       lo->ldo_striping_cached = 0;
        lo->ldo_def_striping_set = 0;
+       lo->ldo_def_striping_cached = 0;
        lod_object_set_pool(lo, NULL);
        lo->ldo_def_stripe_size = 0;
        lo->ldo_def_stripenr = 0;
        if (lo->ldo_dir_stripe != NULL)
-               lo->ldo_dir_striping_cached = 0;
+               lo->ldo_dir_def_striping_cached = 0;
 }
 
 /**
@@ -2359,10 +2357,7 @@ static int lod_xattr_set_default_lmv_on_dir(const struct lu_env *env,
                        RETURN(-ENOMEM);
        }
 
-       l->ldo_dir_striping_cached = 0;
-       l->ldo_dir_def_striping_set = 1;
-       l->ldo_dir_def_stripenr = le32_to_cpu(lum->lum_stripe_count);
-
+       l->ldo_dir_def_striping_cached = 0;
        RETURN(rc);
 }
 
@@ -2416,7 +2411,8 @@ static int lod_xattr_set_lmv(const struct lu_env *env, struct dt_object *dt,
        if (rc != 0)
                RETURN(rc);
 
-       attr->la_valid = LA_TYPE | LA_MODE;
+       attr->la_valid = LA_ATIME | LA_MTIME | LA_CTIME |
+                        LA_MODE | LA_UID | LA_GID | LA_TYPE;
        dof->dof_type = DFT_DIR;
 
        rc = lod_prep_lmv_md(env, dt, &lmv_buf);
@@ -2437,12 +2433,18 @@ static int lod_xattr_set_lmv(const struct lu_env *env, struct dt_object *dt,
                struct dt_object        *dto;
                char                    *stripe_name    = info->lti_key;
                struct lu_name          *sname;
-               struct linkea_data       ldata          = { 0 };
+               struct linkea_data       ldata          = { NULL };
                struct lu_buf            linkea_buf;
 
                dto = lo->ldo_stripe[i];
                dt_write_lock(env, dto, MOR_TGT_CHILD);
                rc = dt_create(env, dto, attr, NULL, dof, th);
+               if (rc != 0) {
+                       dt_write_unlock(env, dto);
+                       RETURN(rc);
+               }
+
+               rc = dt_ref_add(env, dto, th);
                dt_write_unlock(env, dto);
                if (rc != 0)
                        RETURN(rc);
@@ -2459,7 +2461,7 @@ static int lod_xattr_set_lmv(const struct lu_env *env, struct dt_object *dt,
                if (rc != 0)
                        RETURN(rc);
 
-               if (lo->ldo_striping_cached &&
+               if (lo->ldo_def_striping_set &&
                    !LOVEA_DELETE_VALUES(lo->ldo_def_stripe_size,
                                         lo->ldo_def_stripenr,
                                         lo->ldo_def_stripe_offset,
@@ -2578,12 +2580,12 @@ out:
  * \retval             0 on success
  * \retval             negative if failed
  */
-int lod_dir_striping_create_internal(const struct lu_env *env,
-                                    struct dt_object *dt,
-                                    struct lu_attr *attr,
-                                    struct dt_object_format *dof,
-                                    struct thandle *th,
-                                    bool declare)
+static int lod_dir_striping_create_internal(const struct lu_env *env,
+                                           struct dt_object *dt,
+                                           struct lu_attr *attr,
+                                           struct dt_object_format *dof,
+                                           struct thandle *th,
+                                           bool declare)
 {
        struct lod_thread_info  *info = lod_env_info(env);
        struct lod_object       *lo = lod_dt_obj(dt);
@@ -2623,7 +2625,7 @@ int lod_dir_striping_create_internal(const struct lu_env *env,
        }
 
        /* Transfer default LMV striping from the parent */
-       if (lo->ldo_dir_striping_cached &&
+       if (lo->ldo_dir_def_striping_set &&
            !LMVEA_DELETE_VALUES(lo->ldo_dir_def_stripenr,
                                 lo->ldo_dir_def_stripe_offset)) {
                struct lmv_user_md_v1 *v1 = info->lti_ea_store;
@@ -2660,7 +2662,7 @@ int lod_dir_striping_create_internal(const struct lu_env *env,
        }
 
        /* Transfer default LOV striping from the parent */
-       if (lo->ldo_striping_cached &&
+       if (lo->ldo_def_striping_set &&
            !LOVEA_DELETE_VALUES(lo->ldo_def_stripe_size,
                                 lo->ldo_def_stripenr,
                                 lo->ldo_def_stripe_offset,
@@ -2715,7 +2717,14 @@ static int lod_dir_striping_create(const struct lu_env *env,
                                   struct dt_object_format *dof,
                                   struct thandle *th)
 {
-       return lod_dir_striping_create_internal(env, dt, attr, dof, th, false);
+       struct lod_object *lo = lod_dt_obj(dt);
+       int rc;
+
+       rc = lod_dir_striping_create_internal(env, dt, attr, dof, th, false);
+       if (rc == 0)
+               lo->ldo_striping_cached = 1;
+
+       return rc;
 }
 
 /**
@@ -2802,7 +2811,34 @@ static int lod_declare_xattr_del(const struct lu_env *env,
                                 struct dt_object *dt, const char *name,
                                 struct thandle *th)
 {
-       return dt_declare_xattr_del(env, dt_object_child(dt), name, th);
+       struct lod_object       *lo = lod_dt_obj(dt);
+       int                     rc;
+       int                     i;
+       ENTRY;
+
+       rc = dt_declare_xattr_del(env, dt_object_child(dt), name, th);
+       if (rc != 0)
+               RETURN(rc);
+
+       if (!S_ISDIR(dt->do_lu.lo_header->loh_attr))
+               RETURN(0);
+
+       /* set xattr to each stripes, if needed */
+       rc = lod_load_striping(env, lo);
+       if (rc != 0)
+               RETURN(rc);
+
+       if (lo->ldo_stripenr == 0)
+               RETURN(0);
+
+       for (i = 0; i < lo->ldo_stripenr; i++) {
+               LASSERT(lo->ldo_stripe[i]);
+               rc = dt_declare_xattr_del(env, lo->ldo_stripe[i], name, th);
+               if (rc != 0)
+                       break;
+       }
+
+       RETURN(rc);
 }
 
 /**
@@ -2817,9 +2853,30 @@ static int lod_xattr_del(const struct lu_env *env, struct dt_object *dt,
                         const char *name, struct thandle *th,
                         struct lustre_capa *capa)
 {
+       struct dt_object        *next = dt_object_child(dt);
+       struct lod_object       *lo = lod_dt_obj(dt);
+       int                     rc;
+       int                     i;
+       ENTRY;
+
        if (!strcmp(name, XATTR_NAME_LOV))
                lod_object_free_striping(env, lod_dt_obj(dt));
-       return dt_xattr_del(env, dt_object_child(dt), name, th, capa);
+
+       rc = dt_xattr_del(env, next, name, th, capa);
+       if (rc != 0 || !S_ISDIR(dt->do_lu.lo_header->loh_attr))
+               RETURN(rc);
+
+       if (lo->ldo_stripenr == 0)
+               RETURN(0);
+
+       for (i = 0; i < lo->ldo_stripenr; i++) {
+               LASSERT(lo->ldo_stripe[i]);
+               rc = dt_xattr_del(env, lo->ldo_stripe[i], name, th, capa);
+               if (rc != 0)
+                       break;
+       }
+
+       RETURN(rc);
 }
 
 /**
@@ -2904,7 +2961,7 @@ static int lod_cache_parent_lov_striping(const struct lu_env *env,
        if (rc < (typeof(rc))sizeof(struct lov_user_md)) {
                /* don't lookup for non-existing or invalid striping */
                lp->ldo_def_striping_set = 0;
-               lp->ldo_striping_cached = 1;
+               lp->ldo_def_striping_cached = 1;
                lp->ldo_def_stripe_size = 0;
                lp->ldo_def_stripenr = 0;
                lp->ldo_def_stripe_offset = (typeof(v1->lmm_stripe_offset))(-1);
@@ -2934,7 +2991,7 @@ static int lod_cache_parent_lov_striping(const struct lu_env *env,
        lp->ldo_def_stripenr = v1->lmm_stripe_count;
        lp->ldo_def_stripe_size = v1->lmm_stripe_size;
        lp->ldo_def_stripe_offset = v1->lmm_stripe_offset;
-       lp->ldo_striping_cached = 1;
+       lp->ldo_def_striping_cached = 1;
        lp->ldo_def_striping_set = 1;
        if (v1->lmm_magic == LOV_USER_MAGIC_V3) {
                /* XXX: sanity check here */
@@ -2979,7 +3036,7 @@ static int lod_cache_parent_lmv_striping(const struct lu_env *env,
        if (rc < (typeof(rc))sizeof(struct lmv_user_md)) {
                /* don't lookup for non-existing or invalid striping */
                lp->ldo_dir_def_striping_set = 0;
-               lp->ldo_dir_striping_cached = 1;
+               lp->ldo_dir_def_striping_cached = 1;
                lp->ldo_dir_def_stripenr = 0;
                lp->ldo_dir_def_stripe_offset =
                                        (typeof(v1->lum_stripe_offset))(-1);
@@ -2994,7 +3051,7 @@ static int lod_cache_parent_lmv_striping(const struct lu_env *env,
        lp->ldo_dir_def_stripe_offset = le32_to_cpu(v1->lum_stripe_offset);
        lp->ldo_dir_def_hash_type = le32_to_cpu(v1->lum_hash_type);
        lp->ldo_dir_def_striping_set = 1;
-       lp->ldo_dir_striping_cached = 1;
+       lp->ldo_dir_def_striping_cached = 1;
 
        EXIT;
 unlock:
@@ -3025,11 +3082,7 @@ static int lod_cache_parent_striping(const struct lu_env *env,
        int rc = 0;
        ENTRY;
 
-       rc = lod_load_striping(env, lp);
-       if (rc != 0)
-               RETURN(rc);
-
-       if (!lp->ldo_striping_cached) {
+       if (!lp->ldo_def_striping_cached) {
                /* we haven't tried to get default striping for
                 * the directory yet, let's cache it in the object */
                rc = lod_cache_parent_lov_striping(env, lp);
@@ -3037,7 +3090,12 @@ static int lod_cache_parent_striping(const struct lu_env *env,
                        RETURN(rc);
        }
 
-       if (S_ISDIR(child_mode) && !lp->ldo_dir_striping_cached)
+       /* If the parent is on the remote MDT, we should always
+        * try to refresh the default stripeEA cache, because we
+        * do not cache default striping information for remote
+        * object. */
+       if (S_ISDIR(child_mode) && (!lp->ldo_dir_def_striping_cached ||
+                                   dt_object_remote(&lp->ldo_obj)))
                rc = lod_cache_parent_lmv_striping(env, lp);
 
        RETURN(rc);
@@ -3104,6 +3162,7 @@ static void lod_ah_init(const struct lu_env *env,
                                return;
                }
 
+               LASSERT(lp != NULL);
                if (lp->ldo_dir_stripe == NULL) {
                        OBD_ALLOC_PTR(lp->ldo_dir_stripe);
                        if (lp->ldo_dir_stripe == NULL)
@@ -3115,14 +3174,14 @@ static void lod_ah_init(const struct lu_env *env,
                        return;
 
                /* transfer defaults to new directory */
-               if (lp->ldo_striping_cached) {
+               if (lp->ldo_def_striping_set) {
                        if (lp->ldo_pool)
                                lod_object_set_pool(lc, lp->ldo_pool);
                        lc->ldo_def_stripenr = lp->ldo_def_stripenr;
                        lc->ldo_def_stripe_size = lp->ldo_def_stripe_size;
                        lc->ldo_def_stripe_offset = lp->ldo_def_stripe_offset;
-                       lc->ldo_striping_cached = 1;
                        lc->ldo_def_striping_set = 1;
+                       lc->ldo_def_striping_cached = 1;
                        CDEBUG(D_OTHER, "inherite EA sz:%d off:%d nr:%d\n",
                               (int)lc->ldo_def_stripe_size,
                               (int)lc->ldo_def_stripe_offset,
@@ -3130,14 +3189,14 @@ static void lod_ah_init(const struct lu_env *env,
                }
 
                /* transfer dir defaults to new directory */
-               if (lp->ldo_dir_striping_cached) {
+               if (lp->ldo_dir_def_striping_set) {
                        lc->ldo_dir_def_stripenr = lp->ldo_dir_def_stripenr;
                        lc->ldo_dir_def_stripe_offset =
                                                  lp->ldo_dir_def_stripe_offset;
                        lc->ldo_dir_def_hash_type =
                                                  lp->ldo_dir_def_hash_type;
-                       lc->ldo_dir_striping_cached = 1;
                        lc->ldo_dir_def_striping_set = 1;
+                       lc->ldo_dir_def_striping_cached = 1;
                        CDEBUG(D_INFO, "inherit default EA nr:%d off:%d t%u\n",
                               (int)lc->ldo_dir_def_stripenr,
                               (int)lc->ldo_dir_def_stripe_offset,
@@ -3420,6 +3479,24 @@ static int lod_declare_object_create(const struct lu_env *env,
                        rc = lod_declare_striped_object(env, dt, attr,
                                                        NULL, th);
        } else if (dof->dof_type == DFT_DIR) {
+               struct seq_server_site *ss;
+
+               ss = lu_site2seq(dt->do_lu.lo_dev->ld_site);
+
+               /* If the parent has default stripeEA, and client
+                * did not find it before sending create request,
+                * then MDT will return -EREMOTE, and client will
+                * retrieve the default stripeEA and re-create the
+                * sub directory.
+                *
+                * Note: if dah_eadata != NULL, it means creating the
+                * striped directory with specified stripeEA, then it
+                * should ignore the default stripeEA */
+               if ((hint == NULL || hint->dah_eadata == NULL) &&
+                   lo->ldo_dir_stripe_offset != -1 &&
+                   lo->ldo_dir_stripe_offset != ss->ss_node_id)
+                       GOTO(out, rc = -EREMOTE);
+
                /* Orphan object (like migrating object) does not have
                 * lod_dir_stripe, see lod_ah_init */
                if (lo->ldo_dir_stripe != NULL)
@@ -3467,8 +3544,12 @@ int lod_striping_create(const struct lu_env *env, struct dt_object *dt,
                if (rc)
                        break;
        }
-       if (rc == 0)
+
+       if (rc == 0) {
                rc = lod_generate_and_set_lovea(env, lo, th);
+               if (rc == 0)
+                       lo->ldo_striping_cached = 1;
+       }
 
        RETURN(rc);
 }
@@ -3568,6 +3649,13 @@ static int lod_declare_object_destroy(const struct lu_env *env,
        /* declare destroy all striped objects */
        for (i = 0; i < lo->ldo_stripenr; i++) {
                if (likely(lo->ldo_stripe[i] != NULL)) {
+                       if (S_ISDIR(dt->do_lu.lo_header->loh_attr)) {
+                               rc = dt_declare_ref_del(env, lo->ldo_stripe[i],
+                                                       th);
+                               if (rc != 0)
+                                       RETURN(rc);
+                       }
+
                        rc = dt_declare_destroy(env, lo->ldo_stripe[i], th);
                        if (rc != 0)
                                break;
@@ -3636,6 +3724,15 @@ static int lod_object_destroy(const struct lu_env *env,
                if (likely(lo->ldo_stripe[i] != NULL) &&
                    (!OBD_FAIL_CHECK(OBD_FAIL_LFSCK_LOST_SPEOBJ) ||
                     i == cfs_fail_val)) {
+                       if (S_ISDIR(dt->do_lu.lo_header->loh_attr)) {
+                               dt_write_lock(env, lo->ldo_stripe[i],
+                                             MOR_TGT_CHILD);
+                               rc = dt_ref_del(env, lo->ldo_stripe[i], th);
+                               dt_write_unlock(env, lo->ldo_stripe[i]);
+                               if (rc != 0)
+                                       break;
+                       }
+
                        rc = dt_destroy(env, lo->ldo_stripe[i], th);
                        if (rc != 0)
                                break;
@@ -4070,6 +4167,7 @@ void lod_object_free_striping(const struct lu_env *env, struct lod_object *lo)
                lo->ldo_stripe = NULL;
                lo->ldo_stripes_allocated = 0;
        }
+       lo->ldo_striping_cached = 0;
        lo->ldo_stripenr = 0;
        lo->ldo_pattern = 0;
 }