Whamcloud - gitweb
LU-3254 llite: Update layout_gen only if layout change succeed
[fs/lustre-release.git] / lustre / llite / vvp_object.c
index 65a6e45..498d3b2 100644 (file)
  */
 
 static int vvp_object_print(const struct lu_env *env, void *cookie,
-                            lu_printer_t p, const struct lu_object *o)
+                           lu_printer_t p, const struct lu_object *o)
 {
-        struct ccc_object    *obj   = lu2ccc(o);
-        struct inode         *inode = obj->cob_inode;
-        struct ll_inode_info *lli;
-
-        (*p)(env, cookie, "(%s %d %d) inode: %p ",
-             cfs_list_empty(&obj->cob_pending_list) ? "-" : "+",
-             obj->cob_transient_pages, cfs_atomic_read(&obj->cob_mmap_cnt),
-             inode);
-        if (inode) {
-                lli = ll_i2info(inode);
-                (*p)(env, cookie, "%lu/%u %o %u %d %p "DFID,
-                     inode->i_ino, inode->i_generation, inode->i_mode,
-                     inode->i_nlink, atomic_read(&inode->i_count),
-                     lli->lli_clob, PFID(&lli->lli_fid));
-        }
-        return 0;
+       struct ccc_object    *obj   = lu2ccc(o);
+       struct inode         *inode = obj->cob_inode;
+       struct ll_inode_info *lli;
+
+       (*p)(env, cookie, "(%s %d %d) inode: %p ",
+            list_empty(&obj->cob_pending_list) ? "-" : "+",
+            obj->cob_transient_pages, atomic_read(&obj->cob_mmap_cnt),
+            inode);
+       if (inode) {
+               lli = ll_i2info(inode);
+               (*p)(env, cookie, "%lu/%u %o %u %d %p "DFID,
+                    inode->i_ino, inode->i_generation, inode->i_mode,
+                    inode->i_nlink, atomic_read(&inode->i_count),
+                    lli->lli_clob, PFID(&lli->lli_fid));
+       }
+       return 0;
 }
 
 static int vvp_attr_get(const struct lu_env *env, struct cl_object *obj,
                         struct cl_attr *attr)
 {
-        struct inode *inode = ccc_object_inode(obj);
-
-        /*
-         * lov overwrites most of these fields in
-         * lov_attr_get()->...lov_merge_lvb_kms(), except when inode
-         * attributes are newer.
-         */
-
-        attr->cat_size = i_size_read(inode);
-        attr->cat_mtime = LTIME_S(inode->i_mtime);
-        attr->cat_atime = LTIME_S(inode->i_atime);
-        attr->cat_ctime = LTIME_S(inode->i_ctime);
-        attr->cat_blocks = inode->i_blocks;
-        attr->cat_uid = inode->i_uid;
-        attr->cat_gid = inode->i_gid;
-        /* KMS is not known by this layer */
-        return 0; /* layers below have to fill in the rest */
+       struct inode *inode = ccc_object_inode(obj);
+
+       /*
+        * lov overwrites most of these fields in
+        * lov_attr_get()->...lov_merge_lvb_kms(), except when inode
+        * attributes are newer.
+        */
+
+       attr->cat_size = i_size_read(inode);
+       attr->cat_mtime = LTIME_S(inode->i_mtime);
+       attr->cat_atime = LTIME_S(inode->i_atime);
+       attr->cat_ctime = LTIME_S(inode->i_ctime);
+       attr->cat_blocks = inode->i_blocks;
+       attr->cat_uid = from_kuid(&init_user_ns, inode->i_uid);
+       attr->cat_gid = from_kgid(&init_user_ns, inode->i_gid);
+       /* KMS is not known by this layer */
+       return 0; /* layers below have to fill in the rest */
 }
 
 static int vvp_attr_set(const struct lu_env *env, struct cl_object *obj,
                         const struct cl_attr *attr, unsigned valid)
 {
-        struct inode *inode = ccc_object_inode(obj);
-
-        if (valid & CAT_UID)
-                inode->i_uid = attr->cat_uid;
-        if (valid & CAT_GID)
-                inode->i_gid = attr->cat_gid;
-        if (valid & CAT_ATIME)
-                LTIME_S(inode->i_atime) = attr->cat_atime;
-        if (valid & CAT_MTIME)
-                LTIME_S(inode->i_mtime) = attr->cat_mtime;
-        if (valid & CAT_CTIME)
-                LTIME_S(inode->i_ctime) = attr->cat_ctime;
-        if (0 && valid & CAT_SIZE)
-                cl_isize_write_nolock(inode, attr->cat_size);
-        /* not currently necessary */
-        if (0 && valid & (CAT_UID|CAT_GID|CAT_SIZE))
-                mark_inode_dirty(inode);
-        return 0;
+       struct inode *inode = ccc_object_inode(obj);
+
+       if (valid & CAT_UID)
+               inode->i_uid = make_kuid(&init_user_ns, attr->cat_uid);
+       if (valid & CAT_GID)
+               inode->i_gid = make_kgid(&init_user_ns, attr->cat_gid);
+       if (valid & CAT_ATIME)
+               LTIME_S(inode->i_atime) = attr->cat_atime;
+       if (valid & CAT_MTIME)
+               LTIME_S(inode->i_mtime) = attr->cat_mtime;
+       if (valid & CAT_CTIME)
+               LTIME_S(inode->i_ctime) = attr->cat_ctime;
+       if (0 && valid & CAT_SIZE)
+               cl_isize_write_nolock(inode, attr->cat_size);
+       /* not currently necessary */
+       if (0 && valid & (CAT_UID|CAT_GID|CAT_SIZE))
+               mark_inode_dirty(inode);
+       return 0;
 }
 
-int vvp_conf_set(const struct lu_env *env, struct cl_object *obj,
-               const struct cl_object_conf *conf)
+static int vvp_conf_set(const struct lu_env *env, struct cl_object *obj,
+                       const struct cl_object_conf *conf)
 {
        struct ll_inode_info *lli = ll_i2info(conf->coc_inode);
 
        if (conf->coc_opc == OBJECT_CONF_INVALIDATE) {
-               lli->lli_layout_gen = LL_LAYOUT_GEN_NONE;
-               return 0;
+               CDEBUG(D_VFSTRACE, DFID ": losing layout lock\n",
+                      PFID(&lli->lli_fid));
+
+               ll_layout_version_set(lli, LL_LAYOUT_GEN_NONE);
+
+               /* Clean up page mmap for this inode.
+                * The reason for us to do this is that if the page has
+                * already been installed into memory space, the process
+                * can access it without interacting with lustre, so this
+                * page may be stale due to layout change, and the process
+                * will never be notified.
+                * This operation is expensive but mmap processes have to pay
+                * a price themselves. */
+               unmap_mapping_range(conf->coc_inode->i_mapping,
+                                   0, OBD_OBJECT_EOF, 0);
        }
+       return 0;
+}
 
-       if (conf->coc_opc != OBJECT_CONF_SET)
-               return 0;
-
-       if (conf->u.coc_md != NULL && conf->u.coc_md->lsm != NULL) {
-               CDEBUG(D_VFSTRACE, "layout lock change: %u -> %u\n",
-                       lli->lli_layout_gen,
-                       conf->u.coc_md->lsm->lsm_layout_gen);
+static int vvp_prune(const struct lu_env *env, struct cl_object *obj)
+{
+       struct inode *inode = ccc_object_inode(obj);
+       int rc;
+       ENTRY;
 
-               lli->lli_has_smd = lsm_has_objects(conf->u.coc_md->lsm);
-               lli->lli_layout_gen = conf->u.coc_md->lsm->lsm_layout_gen;
-       } else {
-               CDEBUG(D_VFSTRACE, "layout lock destroyed: %u.\n",
-                       lli->lli_layout_gen);
+       rc = cl_sync_file_range(inode, 0, OBD_OBJECT_EOF, CL_FSYNC_ALL, 1);
+       if (rc == 0)
+               truncate_inode_pages(inode->i_mapping, 0);
 
-               lli->lli_has_smd = false;
-               lli->lli_layout_gen = LL_LAYOUT_GEN_EMPTY;
-       }
-       return 0;
+       RETURN(rc);
 }
 
 static const struct cl_object_operations vvp_ops = {
-        .coo_page_init = vvp_page_init,
-        .coo_lock_init = vvp_lock_init,
-        .coo_io_init   = vvp_io_init,
-        .coo_attr_get  = vvp_attr_get,
-        .coo_attr_set  = vvp_attr_set,
-        .coo_conf_set  = vvp_conf_set,
-        .coo_glimpse   = ccc_object_glimpse
+       .coo_page_init = vvp_page_init,
+       .coo_lock_init = vvp_lock_init,
+       .coo_io_init   = vvp_io_init,
+       .coo_attr_get  = vvp_attr_get,
+       .coo_attr_set  = vvp_attr_set,
+       .coo_conf_set  = vvp_conf_set,
+       .coo_prune     = vvp_prune,
+       .coo_glimpse   = ccc_object_glimpse
 };
 
 static const struct lu_object_operations vvp_lu_obj_ops = {