]> Pileus Git - ~andy/linux/blobdiff - fs/gfs2/inode.c
[GFS2] inode-diet: Eliminate i_blksize from the inode structure
[~andy/linux] / fs / gfs2 / inode.c
index c19feb9697b030f98f7e473c83bc8159336421f2..57a4c8b68587134059e4c505cd2cf3c56ffb79c7 100644 (file)
@@ -4,7 +4,7 @@
  *
  * This copyrighted material is made available to anyone wishing to use,
  * modify, copy, or redistribute it subject to the terms and conditions
- * of the GNU General Public License v.2.
+ * of the GNU General Public License version 2.
  */
 
 #include <linux/sched.h>
@@ -16,9 +16,9 @@
 #include <linux/sort.h>
 #include <linux/gfs2_ondisk.h>
 #include <linux/crc32.h>
+#include <linux/lm_interface.h>
 
 #include "gfs2.h"
-#include "lm_interface.h"
 #include "incore.h"
 #include "acl.h"
 #include "bmap.h"
 #include "util.h"
 
 /**
- * inode_attr_in - Copy attributes from the dinode into the VFS inode
+ * gfs2_inode_attr_in - Copy attributes from the dinode into the VFS inode
  * @ip: The GFS2 inode (with embedded disk inode data)
  * @inode:  The Linux VFS inode
  *
  */
 
-static void inode_attr_in(struct gfs2_inode *ip, struct inode *inode)
+void gfs2_inode_attr_in(struct gfs2_inode *ip)
 {
-       inode->i_ino = ip->i_num.no_formal_ino;
+       struct inode *inode = &ip->i_inode;
+       struct gfs2_dinode *di = &ip->i_di;
 
-       switch (ip->i_di.di_mode & S_IFMT) {
+       inode->i_ino = ip->i_num.no_addr;
+
+       switch (di->di_mode & S_IFMT) {
        case S_IFBLK:
        case S_IFCHR:
-               inode->i_rdev = MKDEV(ip->i_di.di_major, ip->i_di.di_minor);
+               inode->i_rdev = MKDEV(di->di_major, di->di_minor);
                break;
        default:
                inode->i_rdev = 0;
                break;
        };
 
-       inode->i_mode = ip->i_di.di_mode;
-       inode->i_nlink = ip->i_di.di_nlink;
-       inode->i_uid = ip->i_di.di_uid;
-       inode->i_gid = ip->i_di.di_gid;
-       i_size_write(inode, ip->i_di.di_size);
-       inode->i_atime.tv_sec = ip->i_di.di_atime;
-       inode->i_mtime.tv_sec = ip->i_di.di_mtime;
-       inode->i_ctime.tv_sec = ip->i_di.di_ctime;
+       inode->i_mode = di->di_mode;
+       inode->i_nlink = di->di_nlink;
+       inode->i_uid = di->di_uid;
+       inode->i_gid = di->di_gid;
+       i_size_write(inode, di->di_size);
+       inode->i_atime.tv_sec = di->di_atime;
+       inode->i_mtime.tv_sec = di->di_mtime;
+       inode->i_ctime.tv_sec = di->di_ctime;
        inode->i_atime.tv_nsec = 0;
        inode->i_mtime.tv_nsec = 0;
        inode->i_ctime.tv_nsec = 0;
-       inode->i_blksize = PAGE_SIZE;
-       inode->i_blocks = ip->i_di.di_blocks <<
+       inode->i_blocks = di->di_blocks <<
                (GFS2_SB(inode)->sd_sb.sb_bsize_shift - GFS2_BASIC_BLOCK_SHIFT);
 
-       if (ip->i_di.di_flags & GFS2_DIF_IMMUTABLE)
+       if (di->di_flags & GFS2_DIF_IMMUTABLE)
                inode->i_flags |= S_IMMUTABLE;
        else
                inode->i_flags &= ~S_IMMUTABLE;
 
-       if (ip->i_di.di_flags & GFS2_DIF_APPENDONLY)
+       if (di->di_flags & GFS2_DIF_APPENDONLY)
                inode->i_flags |= S_APPEND;
        else
                inode->i_flags &= ~S_APPEND;
 }
 
-/**
- * gfs2_inode_attr_in - Copy attributes from the dinode into the VFS inode
- * @ip: The GFS2 inode (with embedded disk inode data)
- *
- */
-
-void gfs2_inode_attr_in(struct gfs2_inode *ip)
-{
-       struct inode *inode = &ip->i_inode;
-       inode_attr_in(ip, inode);
-}
-
 /**
  * gfs2_inode_attr_out - Copy attributes from VFS inode into the dinode
  * @ip: The GFS2 inode
@@ -107,15 +97,15 @@ void gfs2_inode_attr_in(struct gfs2_inode *ip)
 void gfs2_inode_attr_out(struct gfs2_inode *ip)
 {
        struct inode *inode = &ip->i_inode;
-
+       struct gfs2_dinode *di = &ip->i_di;
        gfs2_assert_withdraw(GFS2_SB(inode),
-               (ip->i_di.di_mode & S_IFMT) == (inode->i_mode & S_IFMT));
-       ip->i_di.di_mode = inode->i_mode;
-       ip->i_di.di_uid = inode->i_uid;
-       ip->i_di.di_gid = inode->i_gid;
-       ip->i_di.di_atime = inode->i_atime.tv_sec;
-       ip->i_di.di_mtime = inode->i_mtime.tv_sec;
-       ip->i_di.di_ctime = inode->i_ctime.tv_sec;
+               (di->di_mode & S_IFMT) == (inode->i_mode & S_IFMT));
+       di->di_mode = inode->i_mode;
+       di->di_uid = inode->i_uid;
+       di->di_gid = inode->i_gid;
+       di->di_atime = inode->i_atime.tv_sec;
+       di->di_mtime = inode->i_mtime.tv_sec;
+       di->di_ctime = inode->i_ctime.tv_sec;
 }
 
 static int iget_test(struct inode *inode, void *opaque)
@@ -169,7 +159,7 @@ struct inode *gfs2_inode_lookup(struct super_block *sb, struct gfs2_inum *inum,
        if (inode->i_state & I_NEW) {
                struct gfs2_sbd *sdp = GFS2_SB(inode);
                umode_t mode = DT2IF(type);
-               inode->u.generic_ip = ip;
+               inode->i_private = ip;
                inode->i_mode = mode;
 
                if (S_ISREG(mode)) {
@@ -236,7 +226,6 @@ int gfs2_inode_refresh(struct gfs2_inode *ip)
        }
 
        gfs2_dinode_in(&ip->i_di, dibh->b_data);
-       set_bit(GIF_MIN_INIT, &ip->i_flags);
 
        brelse(dibh);
 
@@ -288,8 +277,7 @@ int gfs2_dinode_dealloc(struct gfs2_inode *ip)
        if (error)
                goto out_rindex_relse;
 
-       error = gfs2_trans_begin(sdp, RES_RG_BIT + 
-                                RES_STATFS + RES_QUOTA, 1);
+       error = gfs2_trans_begin(sdp, RES_RG_BIT + RES_STATFS + RES_QUOTA, 1);
        if (error)
                goto out_rg_gunlock;
 
@@ -323,9 +311,10 @@ int gfs2_change_nlink(struct gfs2_inode *ip, int diff)
 {
        struct gfs2_sbd *sdp = ip->i_inode.i_sb->s_fs_info;
        struct buffer_head *dibh;
-       uint32_t nlink;
+       u32 nlink;
        int error;
 
+       BUG_ON(ip->i_di.di_nlink != ip->i_inode.i_nlink);
        nlink = ip->i_di.di_nlink + diff;
 
        /* If we are reducing the nlink count, but the new value ends up being
@@ -342,6 +331,7 @@ int gfs2_change_nlink(struct gfs2_inode *ip, int diff)
 
        ip->i_di.di_nlink = nlink;
        ip->i_di.di_ctime = get_seconds();
+       ip->i_inode.i_nlink = nlink;
 
        gfs2_trans_add_bh(ip->i_gl, dibh, 1);
        gfs2_dinode_out(&ip->i_di, dibh->b_data);
@@ -395,7 +385,6 @@ struct inode *gfs2_lookup_simple(struct inode *dip, const char *name)
 
 struct inode *gfs2_lookupi(struct inode *dir, const struct qstr *name,
                           int is_root, struct nameidata *nd)
-                
 {
        struct super_block *sb = dir->i_sb;
        struct gfs2_inode *dip = GFS2_I(dir);
@@ -438,7 +427,7 @@ out:
        return inode;
 }
 
-static int pick_formal_ino_1(struct gfs2_sbd *sdp, uint64_t *formal_ino)
+static int pick_formal_ino_1(struct gfs2_sbd *sdp, u64 *formal_ino)
 {
        struct gfs2_inode *ip = GFS2_I(sdp->sd_ir_inode);
        struct buffer_head *bh;
@@ -479,7 +468,7 @@ static int pick_formal_ino_1(struct gfs2_sbd *sdp, uint64_t *formal_ino)
        return 1;
 }
 
-static int pick_formal_ino_2(struct gfs2_sbd *sdp, uint64_t *formal_ino)
+static int pick_formal_ino_2(struct gfs2_sbd *sdp, u64 *formal_ino)
 {
        struct gfs2_inode *ip = GFS2_I(sdp->sd_ir_inode);
        struct gfs2_inode *m_ip = GFS2_I(sdp->sd_inum_inode);
@@ -500,18 +489,18 @@ static int pick_formal_ino_2(struct gfs2_sbd *sdp, uint64_t *formal_ino)
        error = gfs2_meta_inode_buffer(ip, &bh);
        if (error)
                goto out_end_trans;
-       
+
        gfs2_inum_range_in(&ir, bh->b_data + sizeof(struct gfs2_dinode));
 
        if (!ir.ir_length) {
                struct buffer_head *m_bh;
-               uint64_t x, y;
+               u64 x, y;
 
                error = gfs2_meta_inode_buffer(m_ip, &m_bh);
                if (error)
                        goto out_brelse;
 
-               x = *(uint64_t *)(m_bh->b_data + sizeof(struct gfs2_dinode));
+               x = *(u64 *)(m_bh->b_data + sizeof(struct gfs2_dinode));
                x = y = be64_to_cpu(x);
                ir.ir_start = x;
                ir.ir_length = GFS2_INUM_QUANTUM;
@@ -520,7 +509,7 @@ static int pick_formal_ino_2(struct gfs2_sbd *sdp, uint64_t *formal_ino)
                        gfs2_consist_inode(m_ip);
                x = cpu_to_be64(x);
                gfs2_trans_add_bh(m_ip->i_gl, m_bh, 1);
-               *(uint64_t *)(m_bh->b_data + sizeof(struct gfs2_dinode)) = x;
+               *(u64 *)(m_bh->b_data + sizeof(struct gfs2_dinode)) = x;
 
                brelse(m_bh);
        }
@@ -531,20 +520,17 @@ static int pick_formal_ino_2(struct gfs2_sbd *sdp, uint64_t *formal_ino)
        gfs2_trans_add_bh(ip->i_gl, bh, 1);
        gfs2_inum_range_out(&ir, bh->b_data + sizeof(struct gfs2_dinode));
 
- out_brelse:
+out_brelse:
        brelse(bh);
-
- out_end_trans:
+out_end_trans:
        mutex_unlock(&sdp->sd_inum_mutex);
        gfs2_trans_end(sdp);
-
- out:
+out:
        gfs2_glock_dq_uninit(&gh);
-
        return error;
 }
 
-static int pick_formal_ino(struct gfs2_sbd *sdp, uint64_t *inum)
+static int pick_formal_ino(struct gfs2_sbd *sdp, u64 *inum)
 {
        int error;
 
@@ -590,9 +576,9 @@ static int create_ok(struct gfs2_inode *dip, const struct qstr *name,
                return error;
        }
 
-       if (dip->i_di.di_entries == (uint32_t)-1)
+       if (dip->i_di.di_entries == (u32)-1)
                return -EFBIG;
-       if (S_ISDIR(mode) && dip->i_di.di_nlink == (uint32_t)-1)
+       if (S_ISDIR(mode) && dip->i_di.di_nlink == (u32)-1)
                return -EMLINK;
 
        return 0;
@@ -602,8 +588,7 @@ static void munge_mode_uid_gid(struct gfs2_inode *dip, unsigned int *mode,
                               unsigned int *uid, unsigned int *gid)
 {
        if (GFS2_SB(&dip->i_inode)->sd_args.ar_suiddir &&
-           (dip->i_di.di_mode & S_ISUID) &&
-           dip->i_di.di_uid) {
+           (dip->i_di.di_mode & S_ISUID) && dip->i_di.di_uid) {
                if (S_ISDIR(*mode))
                        *mode |= S_ISUID;
                else if (dip->i_di.di_uid != current->fsuid)
@@ -620,7 +605,8 @@ static void munge_mode_uid_gid(struct gfs2_inode *dip, unsigned int *mode,
                *gid = current->fsgid;
 }
 
-static int alloc_dinode(struct gfs2_inode *dip, struct gfs2_inum *inum)
+static int alloc_dinode(struct gfs2_inode *dip, struct gfs2_inum *inum,
+                       u64 *generation)
 {
        struct gfs2_sbd *sdp = GFS2_SB(&dip->i_inode);
        int error;
@@ -636,16 +622,14 @@ static int alloc_dinode(struct gfs2_inode *dip, struct gfs2_inum *inum)
        if (error)
                goto out_ipreserv;
 
-       inum->no_addr = gfs2_alloc_di(dip);
+       inum->no_addr = gfs2_alloc_di(dip, generation);
 
        gfs2_trans_end(sdp);
 
- out_ipreserv:
+out_ipreserv:
        gfs2_inplace_release(dip);
-
- out:
+out:
        gfs2_alloc_put(dip);
-
        return error;
 }
 
@@ -661,8 +645,9 @@ static int alloc_dinode(struct gfs2_inode *dip, struct gfs2_inum *inum)
  */
 
 static void init_dinode(struct gfs2_inode *dip, struct gfs2_glock *gl,
-                       struct gfs2_inum *inum, unsigned int mode,
-                       unsigned int uid, unsigned int gid)
+                       const struct gfs2_inum *inum, unsigned int mode,
+                       unsigned int uid, unsigned int gid,
+                       const u64 *generation)
 {
        struct gfs2_sbd *sdp = GFS2_SB(&dip->i_inode);
        struct gfs2_dinode *di;
@@ -685,7 +670,7 @@ static void init_dinode(struct gfs2_inode *dip, struct gfs2_glock *gl,
        di->di_atime = di->di_mtime = di->di_ctime = cpu_to_be64(get_seconds());
        di->di_major = di->di_minor = cpu_to_be32(0);
        di->di_goal_meta = di->di_goal_data = cpu_to_be64(inum->no_addr);
-       di->__pad[0] = di->__pad[1] = 0;
+       di->di_generation = cpu_to_be64(*generation);
        di->di_flags = cpu_to_be32(0);
 
        if (S_ISREG(mode)) {
@@ -703,6 +688,7 @@ static void init_dinode(struct gfs2_inode *dip, struct gfs2_glock *gl,
        }
 
        di->__pad1 = 0;
+       di->di_payload_format = cpu_to_be32(0);
        di->di_height = cpu_to_be32(0);
        di->__pad2 = 0;
        di->__pad3 = 0;
@@ -716,7 +702,8 @@ static void init_dinode(struct gfs2_inode *dip, struct gfs2_glock *gl,
 }
 
 static int make_dinode(struct gfs2_inode *dip, struct gfs2_glock *gl,
-                      unsigned int mode, struct gfs2_inum *inum)
+                      unsigned int mode, const struct gfs2_inum *inum,
+                      const u64 *generation)
 {
        struct gfs2_sbd *sdp = GFS2_SB(&dip->i_inode);
        unsigned int uid, gid;
@@ -737,7 +724,7 @@ static int make_dinode(struct gfs2_inode *dip, struct gfs2_glock *gl,
        if (error)
                goto out_quota;
 
-       init_dinode(dip, gl, inum, mode, uid, gid);
+       init_dinode(dip, gl, inum, mode, uid, gid, generation);
        gfs2_quota_change(dip, +1, uid, gid);
        gfs2_trans_end(sdp);
 
@@ -780,7 +767,7 @@ static int link_dinode(struct gfs2_inode *dip, const struct qstr *name,
 
                error = gfs2_trans_begin(sdp, sdp->sd_max_dirres +
                                         al->al_rgd->rd_ri.ri_length +
-                                        2 * RES_DINODE + 
+                                        2 * RES_DINODE +
                                         RES_STATFS + RES_QUOTA, 0);
                if (error)
                        goto fail_ipreserv;
@@ -843,6 +830,7 @@ struct inode *gfs2_createi(struct gfs2_holder *ghs, const struct qstr *name,
        struct gfs2_sbd *sdp = GFS2_SB(&dip->i_inode);
        struct gfs2_inum inum;
        int error;
+       u64 generation;
 
        if (!name->len || name->len > GFS2_FNAMESIZE)
                return ERR_PTR(-ENAMETOOLONG);
@@ -860,7 +848,7 @@ struct inode *gfs2_createi(struct gfs2_holder *ghs, const struct qstr *name,
        if (error)
                goto fail_gunlock;
 
-       error = alloc_dinode(dip, &inum);
+       error = alloc_dinode(dip, &inum, &generation);
        if (error)
                goto fail_gunlock;
 
@@ -892,7 +880,7 @@ struct inode *gfs2_createi(struct gfs2_holder *ghs, const struct qstr *name,
                        goto fail_gunlock;
        }
 
-       error = make_dinode(dip, ghs[1].gh_gl, mode, &inum);
+       error = make_dinode(dip, ghs[1].gh_gl, mode, &inum, &generation);
        if (error)
                goto fail_gunlock2;
 
@@ -1142,7 +1130,7 @@ int gfs2_glock_nq_atime(struct gfs2_holder *gh)
        struct gfs2_glock *gl = gh->gh_gl;
        struct gfs2_sbd *sdp = gl->gl_sbd;
        struct gfs2_inode *ip = gl->gl_object;
-       int64_t curtime, quantum = gfs2_tune_get(sdp, gt_atime_quantum);
+       s64 curtime, quantum = gfs2_tune_get(sdp, gt_atime_quantum);
        unsigned int state;
        int flags;
        int error;
@@ -1224,31 +1212,26 @@ fail:
  *
  * Returns: 1 if A > B
  *         -1 if A < B
- *          0 if A = B
+ *          0 if A == B
  */
 
 static int glock_compare_atime(const void *arg_a, const void *arg_b)
 {
-       struct gfs2_holder *gh_a = *(struct gfs2_holder **)arg_a;
-       struct gfs2_holder *gh_b = *(struct gfs2_holder **)arg_b;
-       struct lm_lockname *a = &gh_a->gh_gl->gl_name;
-       struct lm_lockname *b = &gh_b->gh_gl->gl_name;
-       int ret = 0;
+       const struct gfs2_holder *gh_a = *(const struct gfs2_holder **)arg_a;
+       const struct gfs2_holder *gh_b = *(const struct gfs2_holder **)arg_b;
+       const struct lm_lockname *a = &gh_a->gh_gl->gl_name;
+       const struct lm_lockname *b = &gh_b->gh_gl->gl_name;
 
        if (a->ln_number > b->ln_number)
-               ret = 1;
-       else if (a->ln_number < b->ln_number)
-               ret = -1;
-       else {
-               if (gh_a->gh_state == LM_ST_SHARED &&
-                   gh_b->gh_state == LM_ST_EXCLUSIVE)
-                       ret = 1;
-               else if (gh_a->gh_state == LM_ST_SHARED &&
-                        (gh_b->gh_flags & GL_ATIME))
-                       ret = 1;
-       }
+               return 1;
+       if (a->ln_number < b->ln_number)
+               return -1;
+       if (gh_a->gh_state == LM_ST_SHARED && gh_b->gh_state == LM_ST_EXCLUSIVE)
+               return 1;
+       if (gh_a->gh_state == LM_ST_SHARED && (gh_b->gh_flags & GL_ATIME))
+               return 1;
 
-       return ret;
+       return 0;
 }
 
 /**
@@ -1304,7 +1287,6 @@ int gfs2_glock_nq_m_atime(unsigned int num_gh, struct gfs2_holder *ghs)
        }
 
        kfree(p);
-
        return error;
 }
 
@@ -1350,9 +1332,7 @@ int gfs2_setattr_simple(struct gfs2_inode *ip, struct iattr *attr)
                return error;
 
        error = __gfs2_setattr_simple(ip, attr);
-
        gfs2_trans_end(GFS2_SB(&ip->i_inode));
-
        return error;
 }