Whamcloud - gitweb
LU-3569 ofd: packing ost_idx in IDIF
[fs/lustre-release.git] / lustre / ofd / ofd_lvb.c
index ae80fdb..8fedc75 100644 (file)
@@ -27,7 +27,7 @@
  * Copyright (c) 2009, 2010, Oracle and/or its affiliates. All rights reserved.
  * Use is subject to license terms.
  *
- * Copyright (c) 2011, 2012, Whamcloud, Inc.
+ * Copyright (c) 2012, 2013, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
@@ -54,7 +54,7 @@ static int ofd_lvbo_free(struct ldlm_resource *res)
 /* Called with res->lr_lvb_sem held */
 static int ofd_lvbo_init(struct ldlm_resource *res)
 {
-       struct ost_lvb          *lvb = NULL;
+       struct ost_lvb          *lvb;
        struct ofd_device       *ofd;
        struct ofd_object       *fo;
        struct ofd_thread_info  *info;
@@ -64,7 +64,7 @@ static int ofd_lvbo_init(struct ldlm_resource *res)
        ENTRY;
 
        LASSERT(res);
-       LASSERT_MUTEX_LOCKED(&res->lr_lvb_mutex);
+       LASSERT(mutex_is_locked(&res->lr_lvb_mutex));
 
        if (res->lr_lvb_data != NULL)
                RETURN(0);
@@ -72,26 +72,30 @@ static int ofd_lvbo_init(struct ldlm_resource *res)
        ofd = ldlm_res_to_ns(res)->ns_lvbp;
        LASSERT(ofd != NULL);
 
+       if (OBD_FAIL_CHECK(OBD_FAIL_LDLM_OST_LVB))
+               RETURN(-ENOMEM);
+
        rc = lu_env_init(&env, LCT_DT_THREAD);
        if (rc)
                RETURN(rc);
 
        OBD_ALLOC_PTR(lvb);
        if (lvb == NULL)
-               RETURN(-ENOMEM);
+               GOTO(out_env, rc = -ENOMEM);
 
        res->lr_lvb_data = lvb;
        res->lr_lvb_len = sizeof(*lvb);
 
        info = ofd_info_init(&env, NULL);
-       ofd_fid_from_resid(&info->fti_fid, &res->lr_name);
+       ost_fid_from_resid(&info->fti_fid, &res->lr_name,
+                          ofd->ofd_lut.lut_lsd.lsd_osd_index);
        fo = ofd_object_find(&env, ofd, &info->fti_fid);
        if (IS_ERR(fo))
-               GOTO(out, rc = PTR_ERR(fo));
+               GOTO(out_lvb, rc = PTR_ERR(fo));
 
        rc = ofd_attr_get(&env, fo, &info->fti_attr);
        if (rc)
-               GOTO(out_put, rc);
+               GOTO(out_obj, rc);
 
        lvb->lvb_size = info->fti_attr.la_size;
        lvb->lvb_blocks = info->fti_attr.la_blocks;
@@ -99,18 +103,19 @@ static int ofd_lvbo_init(struct ldlm_resource *res)
        lvb->lvb_atime = info->fti_attr.la_atime;
        lvb->lvb_ctime = info->fti_attr.la_ctime;
 
-       CDEBUG(D_DLMTRACE, "res: "LPX64" initial lvb size: "LPX64", "
+       CDEBUG(D_DLMTRACE, "res: "DFID" initial lvb size: "LPU64", "
               "mtime: "LPX64", blocks: "LPX64"\n",
-              res->lr_name.name[0], lvb->lvb_size,
+              PFID(&info->fti_fid), lvb->lvb_size,
               lvb->lvb_mtime, lvb->lvb_blocks);
 
        EXIT;
-out_put:
+out_obj:
        ofd_object_put(&env, fo);
-out:
-       lu_env_fini(&env);
-       if (rc)
+out_lvb:
+       if (rc != 0)
                OST_LVB_SET_ERR(lvb->lvb_blocks, rc);
+out_env:
+       lu_env_fini(&env);
        /* Don't free lvb data on lookup error */
        return rc;
 }
@@ -139,59 +144,89 @@ static int ofd_lvbo_update(struct ldlm_resource *res,
        ofd = ldlm_res_to_ns(res)->ns_lvbp;
        LASSERT(ofd != NULL);
 
-       lvb = res->lr_lvb_data;
-       if (lvb == NULL) {
-               CERROR("%s: no lvb when running lvbo_update, res: "LPU64"!\n",
-                      ofd_obd(ofd)->obd_name, res->lr_name.name[0]);
-               RETURN(0);
-       }
-
        rc = lu_env_init(&env, LCT_DT_THREAD);
        if (rc)
-               GOTO(out_unlock, rc);
+               RETURN(rc);
 
        info = ofd_info_init(&env, NULL);
+       fid_extract_from_res_name(&info->fti_fid, &res->lr_name);
+
+       lvb = res->lr_lvb_data;
+       if (lvb == NULL) {
+               CERROR("%s: no LVB data for "DFID"\n",
+                      ofd_obd(ofd)->obd_name, PFID(&info->fti_fid));
+               GOTO(out_env, rc = 0);
+       }
+
        /* Update the LVB from the network message */
        if (req != NULL) {
                struct ost_lvb *rpc_lvb;
-
-               /* XXX update always from reply buffer */
-               rpc_lvb = req_capsule_server_get(&req->rq_pill, &RMF_DLM_LVB);
-               if (rpc_lvb == NULL) {
-                       CERROR("lustre_swab_buf failed\n");
-                       goto disk_update;
+               bool lvb_type;
+
+               if (req->rq_import != NULL)
+                       lvb_type = imp_connect_lvb_type(req->rq_import);
+               else
+                       lvb_type = exp_connect_lvb_type(req->rq_export);
+
+               if (!lvb_type) {
+                       struct ost_lvb_v1 *lvb_v1;
+
+                       lvb_v1 = req_capsule_server_swab_get(&req->rq_pill,
+                                       &RMF_DLM_LVB, lustre_swab_ost_lvb_v1);
+                       if (lvb_v1 == NULL)
+                               goto disk_update;
+
+                       rpc_lvb = &info->fti_lvb;
+                       memcpy(rpc_lvb, lvb_v1, sizeof *lvb_v1);
+                       rpc_lvb->lvb_mtime_ns = 0;
+                       rpc_lvb->lvb_atime_ns = 0;
+                       rpc_lvb->lvb_ctime_ns = 0;
+               } else {
+                       rpc_lvb = req_capsule_server_swab_get(&req->rq_pill,
+                                                             &RMF_DLM_LVB,
+                                                       lustre_swab_ost_lvb);
+                       if (rpc_lvb == NULL)
+                               goto disk_update;
                }
+
                lock_res(res);
                if (rpc_lvb->lvb_size > lvb->lvb_size || !increase_only) {
-                       CDEBUG(D_DLMTRACE, "res: "LPU64" updating lvb size: "
-                              LPU64" -> "LPU64"\n", res->lr_name.name[0],
+                       CDEBUG(D_DLMTRACE, "res: "DFID" updating lvb size: "
+                              LPU64" -> "LPU64"\n", PFID(&info->fti_fid),
                               lvb->lvb_size, rpc_lvb->lvb_size);
                        lvb->lvb_size = rpc_lvb->lvb_size;
                }
                if (rpc_lvb->lvb_mtime > lvb->lvb_mtime || !increase_only) {
-                       CDEBUG(D_DLMTRACE, "res: "LPU64" updating lvb mtime: "
-                              LPU64" -> "LPU64"\n", res->lr_name.name[0],
+                       CDEBUG(D_DLMTRACE, "res: "DFID" updating lvb mtime: "
+                              LPU64" -> "LPU64"\n", PFID(&info->fti_fid),
                               lvb->lvb_mtime, rpc_lvb->lvb_mtime);
                        lvb->lvb_mtime = rpc_lvb->lvb_mtime;
                }
                if (rpc_lvb->lvb_atime > lvb->lvb_atime || !increase_only) {
-                       CDEBUG(D_DLMTRACE, "res: "LPU64" updating lvb atime: "
-                              LPU64" -> "LPU64"\n", res->lr_name.name[0],
+                       CDEBUG(D_DLMTRACE, "res: "DFID" updating lvb atime: "
+                              LPU64" -> "LPU64"\n", PFID(&info->fti_fid),
                               lvb->lvb_atime, rpc_lvb->lvb_atime);
                        lvb->lvb_atime = rpc_lvb->lvb_atime;
                }
                if (rpc_lvb->lvb_ctime > lvb->lvb_ctime || !increase_only) {
-                       CDEBUG(D_DLMTRACE, "res: "LPU64" updating lvb ctime: "
-                              LPU64" -> "LPU64"\n", res->lr_name.name[0],
+                       CDEBUG(D_DLMTRACE, "res: "DFID" updating lvb ctime: "
+                              LPU64" -> "LPU64"\n", PFID(&info->fti_fid),
                               lvb->lvb_ctime, rpc_lvb->lvb_ctime);
                        lvb->lvb_ctime = rpc_lvb->lvb_ctime;
                }
+               if (rpc_lvb->lvb_blocks > lvb->lvb_blocks || !increase_only) {
+                       CDEBUG(D_DLMTRACE, "res: "DFID" updating lvb blocks: "
+                              LPU64" -> "LPU64"\n", PFID(&info->fti_fid),
+                              lvb->lvb_blocks, rpc_lvb->lvb_blocks);
+                       lvb->lvb_blocks = rpc_lvb->lvb_blocks;
+               }
                unlock_res(res);
        }
 
 disk_update:
        /* Update the LVB from the disk inode */
-       ofd_fid_from_resid(&info->fti_fid, &res->lr_name);
+       ost_fid_from_resid(&info->fti_fid, &res->lr_name,
+                          ofd->ofd_lut.lut_lsd.lsd_osd_index);
        fo = ofd_object_find(&env, ofd, &info->fti_fid);
        if (IS_ERR(fo))
                GOTO(out_env, rc = PTR_ERR(fo));
@@ -202,34 +237,33 @@ disk_update:
 
        lock_res(res);
        if (info->fti_attr.la_size > lvb->lvb_size || !increase_only) {
-               CDEBUG(D_DLMTRACE, "res: "LPU64" updating lvb size from disk: "
-                      LPU64" -> %llu\n", res->lr_name.name[0],
+               CDEBUG(D_DLMTRACE, "res: "DFID" updating lvb size from disk: "
+                      LPU64" -> %llu\n", PFID(&info->fti_fid),
                       lvb->lvb_size, info->fti_attr.la_size);
                lvb->lvb_size = info->fti_attr.la_size;
        }
 
        if (info->fti_attr.la_mtime >lvb->lvb_mtime || !increase_only) {
-               CDEBUG(D_DLMTRACE, "res: "LPU64" updating lvb mtime from disk: "
-                      LPU64" -> "LPU64"\n", res->lr_name.name[0],
+               CDEBUG(D_DLMTRACE, "res: "DFID" updating lvb mtime from disk: "
+                      LPU64" -> "LPU64"\n", PFID(&info->fti_fid),
                       lvb->lvb_mtime, info->fti_attr.la_mtime);
                lvb->lvb_mtime = info->fti_attr.la_mtime;
        }
        if (info->fti_attr.la_atime >lvb->lvb_atime || !increase_only) {
-               CDEBUG(D_DLMTRACE, "res: "LPU64" updating lvb atime from disk: "
-                      LPU64" -> "LPU64"\n", res->lr_name.name[0],
+               CDEBUG(D_DLMTRACE, "res: "DFID" updating lvb atime from disk: "
+                      LPU64" -> "LPU64"\n", PFID(&info->fti_fid),
                       lvb->lvb_atime, info->fti_attr.la_atime);
                lvb->lvb_atime = info->fti_attr.la_atime;
        }
        if (info->fti_attr.la_ctime >lvb->lvb_ctime || !increase_only) {
-               CDEBUG(D_DLMTRACE, "res: "LPU64" updating lvb ctime from disk: "
-                      LPU64" -> "LPU64"\n", res->lr_name.name[0],
+               CDEBUG(D_DLMTRACE, "res: "DFID" updating lvb ctime from disk: "
+                      LPU64" -> "LPU64"\n", PFID(&info->fti_fid),
                       lvb->lvb_ctime, info->fti_attr.la_ctime);
                lvb->lvb_ctime = info->fti_attr.la_ctime;
        }
-       if (lvb->lvb_blocks != info->fti_attr.la_blocks) {
-               CDEBUG(D_DLMTRACE,"res: "LPU64" updating lvb blocks from disk: "
-                      LPU64" -> %llu\n", res->lr_name.name[0],
-                      lvb->lvb_blocks,
+       if (info->fti_attr.la_blocks > lvb->lvb_blocks || !increase_only) {
+               CDEBUG(D_DLMTRACE, "res: "DFID" updating lvb blocks from disk: "
+                      LPU64" -> %llu\n", PFID(&info->fti_fid), lvb->lvb_blocks,
                       (unsigned long long)info->fti_attr.la_blocks);
                lvb->lvb_blocks = info->fti_attr.la_blocks;
        }
@@ -239,32 +273,43 @@ out_obj:
        ofd_object_put(&env, fo);
 out_env:
        lu_env_fini(&env);
-out_unlock:
        return rc;
 }
 
-static int ofd_lvbo_size(struct ldlm_lock *unused)
+static int ofd_lvbo_size(struct ldlm_lock *lock)
 {
-       return sizeof(struct ost_lvb);
+       if (lock->l_export != NULL && exp_connect_lvb_type(lock->l_export))
+               return sizeof(struct ost_lvb);
+       else
+               return sizeof(struct ost_lvb_v1);
 }
 
 static int ofd_lvbo_fill(struct ldlm_lock *lock, void *buf, int buflen)
 {
        struct ldlm_resource *res = lock->l_resource;
+       int lvb_len;
+
+       /* Former lvbo_init not allocate the "LVB". */
+       if (unlikely(res->lr_lvb_len == 0))
+               return 0;
+
+       lvb_len = ofd_lvbo_size(lock);
+       LASSERT(lvb_len <= res->lr_lvb_len);
+
+       if (lvb_len > buflen)
+               lvb_len = buflen;
 
        lock_res(res);
-       LASSERTF(buflen >= res->lr_lvb_len,
-                "actual %d, want %d\n", buflen, res->lr_lvb_len);
-       memcpy(buf, res->lr_lvb_data, res->lr_lvb_len);
+       memcpy(buf, res->lr_lvb_data, lvb_len);
        unlock_res(res);
 
-       return res->lr_lvb_len;
+       return lvb_len;
 }
 
 struct ldlm_valblock_ops ofd_lvbo = {
-       lvbo_init:      ofd_lvbo_init,
-       lvbo_update:    ofd_lvbo_update,
-       lvbo_free:      ofd_lvbo_free,
-       lvbo_size:      ofd_lvbo_size,
-       lvbo_fill:      ofd_lvbo_fill
+       .lvbo_init      = ofd_lvbo_init,
+       .lvbo_update    = ofd_lvbo_update,
+       .lvbo_free      = ofd_lvbo_free,
+       .lvbo_size      = ofd_lvbo_size,
+       .lvbo_fill      = ofd_lvbo_fill
 };