diff options
author | Andreas Gruenbacher <agruenba@redhat.com> | 2016-06-14 19:24:50 +0200 |
---|---|---|
committer | Bob Peterson <rpeterso@redhat.com> | 2016-06-27 16:47:08 +0200 |
commit | cda9dd4207aeb29d0aa2298085cc2d1ebcb87e04 (patch) | |
tree | 734cc4bf36228f8ddfc2f85ebb87328cef383746 | |
parent | gfs2: Get rid of gfs2_ilookup (diff) | |
download | linux-cda9dd4207aeb29d0aa2298085cc2d1ebcb87e04.tar.xz linux-cda9dd4207aeb29d0aa2298085cc2d1ebcb87e04.zip |
gfs2: Large-filesystem fix for 32-bit systems
Commit ff34245d switched from iget5_locked to iget_locked among other
things, but iget_locked doesn't work for filesystems larger than 2^32
blocks on 32-bit systems. Switch back to iget5_locked. Filesystems
larger than 2^32 blocks are unrealistic to work well on 32-bit systems,
so this is mostly a code cleanliness fix.
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
-rw-r--r-- | fs/gfs2/inode.c | 19 |
1 files changed, 17 insertions, 2 deletions
diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c index ebff26ee6865..481b6496727d 100644 --- a/fs/gfs2/inode.c +++ b/fs/gfs2/inode.c @@ -37,19 +37,34 @@ #include "super.h" #include "glops.h" +static int iget_test(struct inode *inode, void *opaque) +{ + u64 no_addr = *(u64 *)opaque; + + return GFS2_I(inode)->i_no_addr == no_addr; +} + +static int iget_set(struct inode *inode, void *opaque) +{ + u64 no_addr = *(u64 *)opaque; + + GFS2_I(inode)->i_no_addr = no_addr; + inode->i_ino = no_addr; + return 0; +} + static struct inode *gfs2_iget(struct super_block *sb, u64 no_addr) { struct inode *inode; repeat: - inode = iget_locked(sb, no_addr); + inode = iget5_locked(sb, no_addr, iget_test, iget_set, &no_addr); if (!inode) return inode; if (is_bad_inode(inode)) { iput(inode); goto repeat; } - GFS2_I(inode)->i_no_addr = no_addr; return inode; } |