LU-3403 test: error of listxattr when buffer is small

[fs/lustre-release.git] / lustre / lov / lov_merge.c

diff --git a/lustre/lov/lov_merge.c b/lustre/lov/lov_merge.c
index 3607283..fdd160f 100644 (file)
--- a/lustre/lov/lov_merge.c
+++ b/lustre/lov/lov_merge.c
@@ -27,7 +27,7 @@
  * Copyright (c) 2005, 2010, Oracle and/or its affiliates. All rights reserved.
  * Use is subject to license terms.
  *
- * Copyright (c) 2012, Intel Corporation.
+ * Copyright (c) 2012, 2013, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
@@ -63,17 +63,16 @@ int lov_merge_lvb_kms(struct lov_stripe_md *lsm,
        obd_time current_ctime = lvb->lvb_ctime;
        int i;
        int rc = 0;
-       struct lu_fid fid = {0};
 
-       LASSERT_SPIN_LOCKED(&lsm->lsm_lock);
+       LASSERT(spin_is_locked(&lsm->lsm_lock));
 #ifdef __KERNEL__
-       LASSERT(lsm->lsm_lock_owner == cfs_curproc_pid());
+       LASSERT(lsm->lsm_lock_owner == current_pid());
 #endif
 
-       ostid_to_fid(&fid, &lsm->lsm_oi, 0);
-       CDEBUG(D_INODE, "MDT FID "DFID" initial value: s="LPU64" m="LPU64
-              " a="LPU64" c="LPU64" b="LPU64"\n", PFID(&fid), lvb->lvb_size,
-              lvb->lvb_mtime, lvb->lvb_atime, lvb->lvb_ctime, lvb->lvb_blocks);
+       CDEBUG(D_INODE, "MDT ID "DOSTID" initial value: s="LPU64" m="LPU64
+              " a="LPU64" c="LPU64" b="LPU64"\n", POSTID(&lsm->lsm_oi),
+              lvb->lvb_size, lvb->lvb_mtime, lvb->lvb_atime, lvb->lvb_ctime,
+              lvb->lvb_blocks);
        for (i = 0; i < lsm->lsm_stripe_count; i++) {
                 struct lov_oinfo *loi = lsm->lsm_oinfo[i];
                 obd_size lov_size, tmpsize;
@@ -103,8 +102,8 @@ int lov_merge_lvb_kms(struct lov_stripe_md *lsm,
                 if (loi->loi_lvb.lvb_ctime > current_ctime)
                         current_ctime = loi->loi_lvb.lvb_ctime;
 
-               CDEBUG(D_INODE, "MDT FID "DFID" on OST[%u]: s="LPU64" m="LPU64
-                      " a="LPU64" c="LPU64" b="LPU64"\n", PFID(&fid),
+               CDEBUG(D_INODE, "MDT ID "DOSTID" on OST[%u]: s="LPU64" m="LPU64
+                      " a="LPU64" c="LPU64" b="LPU64"\n", POSTID(&lsm->lsm_oi),
                       loi->loi_ost_idx, loi->loi_lvb.lvb_size,
                       loi->loi_lvb.lvb_mtime, loi->loi_lvb.lvb_atime,
                       loi->loi_lvb.lvb_ctime, loi->loi_lvb.lvb_blocks);
@@ -131,7 +130,6 @@ int lov_merge_lvb_kms(struct lov_stripe_md *lsm,
 int lov_merge_lvb(struct obd_export *exp,
                   struct lov_stripe_md *lsm, struct ost_lvb *lvb, int kms_only)
 {
-       struct lu_fid fid = {0};
        int   rc;
        __u64 kms;
 
@@ -142,9 +140,8 @@ int lov_merge_lvb(struct obd_export *exp,
        if (kms_only)
                lvb->lvb_size = kms;
 
-       ostid_to_fid(&fid, &lsm->lsm_oi, 0);
-       CDEBUG(D_INODE, "merged for FID "DFID" s="LPU64" m="LPU64" a="LPU64
-              " c="LPU64" b="LPU64"\n", PFID(&fid), lvb->lvb_size,
+       CDEBUG(D_INODE, "merged for ID "DOSTID" s="LPU64" m="LPU64" a="LPU64
+              " c="LPU64" b="LPU64"\n", POSTID(&lsm->lsm_oi), lvb->lvb_size,
               lvb->lvb_mtime, lvb->lvb_atime, lvb->lvb_ctime, lvb->lvb_blocks);
        RETURN(rc);
 }
@@ -158,9 +155,9 @@ int lov_adjust_kms(struct obd_export *exp, struct lov_stripe_md *lsm,
         __u64 kms;
         ENTRY;
 
-        LASSERT_SPIN_LOCKED(&lsm->lsm_lock);
+       LASSERT(spin_is_locked(&lsm->lsm_lock));
 #ifdef __KERNEL__
-        LASSERT(lsm->lsm_lock_owner == cfs_curproc_pid());
+       LASSERT(lsm->lsm_lock_owner == current_pid());
 #endif
 
         if (shrink) {
@@ -192,38 +189,49 @@ int lov_adjust_kms(struct obd_export *exp, struct lov_stripe_md *lsm,
 void lov_merge_attrs(struct obdo *tgt, struct obdo *src, obd_valid valid,
                      struct lov_stripe_md *lsm, int stripeno, int *set)
 {
-        valid &= src->o_valid;
-
-        if (*set) {
-                if (valid & OBD_MD_FLSIZE) {
-                        /* this handles sparse files properly */
-                        obd_size lov_size;
-
-                        lov_size = lov_stripe_size(lsm, src->o_size, stripeno);
-                        if (lov_size > tgt->o_size)
-                                tgt->o_size = lov_size;
-                }
-                if (valid & OBD_MD_FLBLOCKS)
-                        tgt->o_blocks += src->o_blocks;
-                if (valid & OBD_MD_FLBLKSZ)
-                        tgt->o_blksize += src->o_blksize;
-                if (valid & OBD_MD_FLCTIME && tgt->o_ctime < src->o_ctime)
-                        tgt->o_ctime = src->o_ctime;
-                if (valid & OBD_MD_FLMTIME && tgt->o_mtime < src->o_mtime)
-                        tgt->o_mtime = src->o_mtime;
-                if (valid & OBD_MD_FLDATAVERSION)
-                        tgt->o_data_version += src->o_data_version;
+       valid &= src->o_valid;
+
+       if (*set != 0) {
+               tgt->o_valid &= valid;
+               if (valid & OBD_MD_FLSIZE) {
+                       /* this handles sparse files properly */
+                       obd_size lov_size;
+
+                       lov_size = lov_stripe_size(lsm, src->o_size, stripeno);
+                       if (lov_size > tgt->o_size)
+                               tgt->o_size = lov_size;
+               }
+               if (valid & OBD_MD_FLBLOCKS)
+                       tgt->o_blocks += src->o_blocks;
+               if (valid & OBD_MD_FLBLKSZ)
+                       tgt->o_blksize += src->o_blksize;
+               if (valid & OBD_MD_FLCTIME && tgt->o_ctime < src->o_ctime)
+                       tgt->o_ctime = src->o_ctime;
+               if (valid & OBD_MD_FLMTIME && tgt->o_mtime < src->o_mtime)
+                       tgt->o_mtime = src->o_mtime;
+               if (valid & OBD_MD_FLDATAVERSION)
+                       tgt->o_data_version += src->o_data_version;
+
+               /* handle flags */
+               if (valid & OBD_MD_FLFLAGS)
+                       tgt->o_flags &= src->o_flags;
+               else
+                       tgt->o_flags = 0;
        } else {
                memcpy(tgt, src, sizeof(*tgt));
                tgt->o_oi = lsm->lsm_oi;
+               tgt->o_valid = valid;
                if (valid & OBD_MD_FLSIZE)
                        tgt->o_size = lov_stripe_size(lsm, src->o_size,
                                                      stripeno);
+               tgt->o_flags = 0;
+               if (valid & OBD_MD_FLFLAGS)
+                       tgt->o_flags = src->o_flags;
        }
 
-        /* data_version needs to be valid on all stripes to be correct! */
-        if (!(valid & OBD_MD_FLDATAVERSION))
-                tgt->o_valid &= ~OBD_MD_FLDATAVERSION;
+       /* data_version needs to be valid on all stripes to be correct! */
+       if (!(valid & OBD_MD_FLDATAVERSION))
+               tgt->o_valid &= ~OBD_MD_FLDATAVERSION;
 
-        *set += 1;
+       *set += 1;
 }