iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[GFS2] Tidy up bmap/inode code

Browse Source 
As per Jan Engelhardt's third set of comments, this make various
code style changes and moves the structures from format.h into
super.c, which was the only place that format.h was actually used.

Cc: Jan Engelhardt <jengelh@linux01.gwdg.de>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
This commit is contained in:
Steven Whitehouse 2006-09-04 11:41:31 -04:00
parent 31e77ac55f
commit 75d3b817a0
5 changed files with 41 additions and 64 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
fs/gfs2/bmap.c
View File

@ -94,7 +94,7 @@ static int gfs2_unstuffer_page(struct gfs2_inode *ip, struct buffer_head *dibh,
map_bh(bh, inode->i_sb, block); map_bh(bh, inode->i_sb, block);
set_buffer_uptodate(bh); set_buffer_uptodate(bh);
if ((sdp->sd_args.ar_data == GFS2_DATA_ORDERED) || gfs2_is_jdata(ip)) if (sdp->sd_args.ar_data == GFS2_DATA_ORDERED || gfs2_is_jdata(ip))
gfs2_trans_add_bh(ip->i_gl, bh, 0); gfs2_trans_add_bh(ip->i_gl, bh, 0);
mark_buffer_dirty(bh); mark_buffer_dirty(bh);
@ -369,7 +369,7 @@ static inline u64 *metapointer(struct buffer_head *bh, int *boundary,
u64 *ptr; u64 *ptr;
*boundary = 0; *boundary = 0;
ptr = ((u64 *)(bh->b_data + head_size)) + mp->mp_list[height]; ptr = ((u64 *)(bh->b_data + head_size)) + mp->mp_list[height];
if (ptr + 1 == (u64*)(bh->b_data + bh->b_size)) if (ptr + 1 == (u64 *)(bh->b_data + bh->b_size))
*boundary = 1; *boundary = 1;
return ptr; return ptr;
} }
@ -456,7 +456,7 @@ static struct buffer_head *gfs2_block_pointers(struct inode *inode, u64 lblock,
if (gfs2_assert_warn(sdp, !gfs2_is_stuffed(ip))) if (gfs2_assert_warn(sdp, !gfs2_is_stuffed(ip)))
goto out; goto out;
bsize = (gfs2_is_dir(ip)) ? sdp->sd_jbsize : sdp->sd_sb.sb_bsize; bsize = gfs2_is_dir(ip) ? sdp->sd_jbsize : sdp->sd_sb.sb_bsize;
height = calc_tree_height(ip, (lblock + 1) * bsize); height = calc_tree_height(ip, (lblock + 1) * bsize);
if (ip->i_di.di_height < height) { if (ip->i_di.di_height < height) {
@ -554,7 +554,7 @@ int gfs2_extent_map(struct inode *inode, u64 lblock, int *new, u64 *dblock, unsi
bh = gfs2_block_pointers(inode, lblock, new, dblock, &boundary, &mp); bh = gfs2_block_pointers(inode, lblock, new, dblock, &boundary, &mp);
*extlen = 1; *extlen = 1;
if (bh && !IS_ERR(bh) && *dblock && !*new) { if (bh != NULL && !IS_ERR(bh) && *dblock != 0 && *new == 0) {
u64 tmp_dblock; u64 tmp_dblock;
int tmp_new; int tmp_new;
unsigned int nptrs; unsigned int nptrs;
@ -565,7 +565,7 @@ int gfs2_extent_map(struct inode *inode, u64 lblock, int *new, u64 *dblock, unsi
lookup_block(ip, bh, end_of_metadata, &mp, 0, &tmp_new, &tmp_dblock); lookup_block(ip, bh, end_of_metadata, &mp, 0, &tmp_new, &tmp_dblock);
if (*dblock + *extlen != tmp_dblock) if (*dblock + *extlen != tmp_dblock)
break; break;
(*extlen)++; ++*extlen;
} }
} }
bmap_unlock(inode, create); bmap_unlock(inode, create);
@ -612,10 +612,8 @@ static int recursive_scan(struct gfs2_inode *ip, struct buffer_head *dibh,
return error; return error;
dibh = bh; dibh = bh;
top = (uint64_t *)(bh->b_data + sizeof(struct gfs2_dinode)) + top = (u64 *)(bh->b_data + sizeof(struct gfs2_dinode)) + mp->mp_list[0];
mp->mp_list[0]; bottom = (u64 *)(bh->b_data + sizeof(struct gfs2_dinode)) + sdp->sd_diptrs;
bottom = (uint64_t *)(bh->b_data + sizeof(struct gfs2_dinode)) +
sdp->sd_diptrs;
} else { } else {
error = gfs2_meta_indirect_buffer(ip, height, block, 0, &bh); error = gfs2_meta_indirect_buffer(ip, height, block, 0, &bh);
if (error) if (error)

21
fs/gfs2/format.h
View File

@ -1,21 +0,0 @@
/*
* Copyright (C) Sistina Software, Inc. 1997-2003 All rights reserved.
* Copyright (C) 2004-2006 Red Hat, Inc. All rights reserved.
*
* 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 version 2.
*/
#ifndef __FORMAT_DOT_H__
#define __FORMAT_DOT_H__
static const uint32_t gfs2_old_fs_formats[] = {
0
};
static const uint32_t gfs2_old_multihost_formats[] = {
0
};
#endif /* __FORMAT_DOT_H__ */

12
fs/gfs2/glops.c
View File

@ -234,7 +234,6 @@ static void inode_go_sync(struct gfs2_glock *gl, int flags)
if (flags & DIO_RELEASE) if (flags & DIO_RELEASE)
gfs2_ail_empty_gl(gl); gfs2_ail_empty_gl(gl);
} }
} }
/** /**
@ -324,12 +323,11 @@ static void inode_go_unlock(struct gfs2_holder *gh)
struct gfs2_glock *gl = gh->gh_gl; struct gfs2_glock *gl = gh->gh_gl;
struct gfs2_inode *ip = gl->gl_object; struct gfs2_inode *ip = gl->gl_object;
if (ip) { if (ip == NULL)
if (test_bit(GLF_DIRTY, &gl->gl_flags)) return;
gfs2_inode_attr_in(ip); if (test_bit(GLF_DIRTY, &gl->gl_flags))
gfs2_inode_attr_in(ip);
gfs2_meta_cache_flush(ip); gfs2_meta_cache_flush(ip);
}
} }
/** /**

47
fs/gfs2/inode.c
View File

@ -97,15 +97,15 @@ void gfs2_inode_attr_in(struct gfs2_inode *ip)
void gfs2_inode_attr_out(struct gfs2_inode *ip) void gfs2_inode_attr_out(struct gfs2_inode *ip)
{ {
struct inode *inode = &ip->i_inode; struct inode *inode = &ip->i_inode;
struct gfs2_dinode *di = &ip->i_di;
gfs2_assert_withdraw(GFS2_SB(inode), gfs2_assert_withdraw(GFS2_SB(inode),
(ip->i_di.di_mode & S_IFMT) == (inode->i_mode & S_IFMT)); (di->di_mode & S_IFMT) == (inode->i_mode & S_IFMT));
ip->i_di.di_mode = inode->i_mode; di->di_mode = inode->i_mode;
ip->i_di.di_uid = inode->i_uid; di->di_uid = inode->i_uid;
ip->i_di.di_gid = inode->i_gid; di->di_gid = inode->i_gid;
ip->i_di.di_atime = inode->i_atime.tv_sec; di->di_atime = inode->i_atime.tv_sec;
ip->i_di.di_mtime = inode->i_mtime.tv_sec; di->di_mtime = inode->i_mtime.tv_sec;
ip->i_di.di_ctime = inode->i_ctime.tv_sec; di->di_ctime = inode->i_ctime.tv_sec;
} }
static int iget_test(struct inode *inode, void *opaque) static int iget_test(struct inode *inode, void *opaque)
@ -1213,31 +1213,26 @@ fail:
* *
* Returns: 1 if A > B * Returns: 1 if A > B
* -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) static int glock_compare_atime(const void *arg_a, const void *arg_b)
{ {
struct gfs2_holder *gh_a = *(struct gfs2_holder **)arg_a; const struct gfs2_holder *gh_a = *(const struct gfs2_holder **)arg_a;
struct gfs2_holder *gh_b = *(struct gfs2_holder **)arg_b; const struct gfs2_holder *gh_b = *(const struct gfs2_holder **)arg_b;
struct lm_lockname *a = &gh_a->gh_gl->gl_name; const struct lm_lockname *a = &gh_a->gh_gl->gl_name;
struct lm_lockname *b = &gh_b->gh_gl->gl_name; const struct lm_lockname *b = &gh_b->gh_gl->gl_name;
int ret = 0;
if (a->ln_number > b->ln_number) if (a->ln_number > b->ln_number)
ret = 1; return 1;
else if (a->ln_number < b->ln_number) if (a->ln_number < b->ln_number)
ret = -1; return -1;
else { if (gh_a->gh_state == LM_ST_SHARED && gh_b->gh_state == LM_ST_EXCLUSIVE)
if (gh_a->gh_state == LM_ST_SHARED && return 1;
gh_b->gh_state == LM_ST_EXCLUSIVE) if (gh_a->gh_state == LM_ST_SHARED && (gh_b->gh_flags & GL_ATIME))
ret = 1; return 1;
else if (gh_a->gh_state == LM_ST_SHARED &&
(gh_b->gh_flags & GL_ATIME))
ret = 1;
}
return ret; return 0;
} }
/** /**

9
fs/gfs2/super.c
View File

@ -21,7 +21,6 @@
#include "incore.h" #include "incore.h"
#include "bmap.h" #include "bmap.h"
#include "dir.h" #include "dir.h"
#include "format.h"
#include "glock.h" #include "glock.h"
#include "glops.h" #include "glops.h"
#include "inode.h" #include "inode.h"
@ -34,6 +33,14 @@
#include "trans.h" #include "trans.h"
#include "util.h" #include "util.h"
static const uint32_t gfs2_old_fs_formats[] = {
0
};
static const uint32_t gfs2_old_multihost_formats[] = {
0
};
/** /**
* gfs2_tune_init - Fill a gfs2_tune structure with default values * gfs2_tune_init - Fill a gfs2_tune structure with default values
* @gt: tune * @gt: tune