diff options
| author | Andrew Price <anprice@redhat.com> | 2012-04-16 16:40:55 +0100 | 
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2012-04-24 16:44:37 +0100 | 
| commit | 4306629e1c0fd098d52f499e5a60aaa03e30df0d (patch) | |
| tree | 82e2f8768dd0d273d31bb83eb8e10c1501eceece | |
| parent | c50b91c4bd511dfe844e2aa7be429b6b88406353 (diff) | |
GFS2: Remove unused argument from gfs2_internal_read
gfs2_internal_read accepts an unused ra_state argument, left over from
when we did readahead on the rindex. Since there are currently no plans
to add back this readahead, this patch removes the ra_state parameter
and updates the functions which call gfs2_internal_read accordingly.
Signed-off-by: Andrew Price <anprice@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
| -rw-r--r-- | fs/gfs2/aops.c | 5 | ||||
| -rw-r--r-- | fs/gfs2/inode.h | 1 | ||||
| -rw-r--r-- | fs/gfs2/quota.c | 4 | ||||
| -rw-r--r-- | fs/gfs2/rgrp.c | 17 | 
4 files changed, 9 insertions, 18 deletions
| diff --git a/fs/gfs2/aops.c b/fs/gfs2/aops.c index a627b3ee5c9f..695bbe16d1f0 100644 --- a/fs/gfs2/aops.c +++ b/fs/gfs2/aops.c @@ -517,15 +517,14 @@ out:  /**   * gfs2_internal_read - read an internal file   * @ip: The gfs2 inode - * @ra_state: The readahead state (or NULL for no readahead)   * @buf: The buffer to fill   * @pos: The file position   * @size: The amount to read   *   */ -int gfs2_internal_read(struct gfs2_inode *ip, struct file_ra_state *ra_state, -                       char *buf, loff_t *pos, unsigned size) +int gfs2_internal_read(struct gfs2_inode *ip, char *buf, loff_t *pos, +                       unsigned size)  {  	struct address_space *mapping = ip->i_inode.i_mapping;  	unsigned long index = *pos / PAGE_CACHE_SIZE; diff --git a/fs/gfs2/inode.h b/fs/gfs2/inode.h index 5d345b402773..c53c7477f6da 100644 --- a/fs/gfs2/inode.h +++ b/fs/gfs2/inode.h @@ -17,7 +17,6 @@  extern int gfs2_releasepage(struct page *page, gfp_t gfp_mask);  extern int gfs2_internal_read(struct gfs2_inode *ip, -			      struct file_ra_state *ra_state,  			      char *buf, loff_t *pos, unsigned size);  extern void gfs2_set_aops(struct inode *inode); diff --git a/fs/gfs2/quota.c b/fs/gfs2/quota.c index 6019da3dcaed..45d9171e078b 100644 --- a/fs/gfs2/quota.c +++ b/fs/gfs2/quota.c @@ -652,7 +652,7 @@ static int gfs2_adjust_quota(struct gfs2_inode *ip, loff_t loc,  	}  	memset(&q, 0, sizeof(struct gfs2_quota)); -	err = gfs2_internal_read(ip, NULL, (char *)&q, &loc, sizeof(q)); +	err = gfs2_internal_read(ip, (char *)&q, &loc, sizeof(q));  	if (err < 0)  		return err; @@ -852,7 +852,7 @@ static int update_qd(struct gfs2_sbd *sdp, struct gfs2_quota_data *qd)  	memset(&q, 0, sizeof(struct gfs2_quota));  	pos = qd2offset(qd); -	error = gfs2_internal_read(ip, NULL, (char *)&q, &pos, sizeof(q)); +	error = gfs2_internal_read(ip, (char *)&q, &pos, sizeof(q));  	if (error < 0)  		return error; diff --git a/fs/gfs2/rgrp.c b/fs/gfs2/rgrp.c index 7a1cf67d7db2..b550e5c1e797 100644 --- a/fs/gfs2/rgrp.c +++ b/fs/gfs2/rgrp.c @@ -541,16 +541,14 @@ u64 gfs2_ri_total(struct gfs2_sbd *sdp)  	struct inode *inode = sdp->sd_rindex;  	struct gfs2_inode *ip = GFS2_I(inode);  	char buf[sizeof(struct gfs2_rindex)]; -	struct file_ra_state ra_state;  	int error, rgrps; -	file_ra_state_init(&ra_state, inode->i_mapping);  	for (rgrps = 0;; rgrps++) {  		loff_t pos = rgrps * sizeof(struct gfs2_rindex);  		if (pos + sizeof(struct gfs2_rindex) > i_size_read(inode))  			break; -		error = gfs2_internal_read(ip, &ra_state, buf, &pos, +		error = gfs2_internal_read(ip, buf, &pos,  					   sizeof(struct gfs2_rindex));  		if (error != sizeof(struct gfs2_rindex))  			break; @@ -586,14 +584,12 @@ static int rgd_insert(struct gfs2_rgrpd *rgd)  /**   * read_rindex_entry - Pull in a new resource index entry from the disk - * @ip: The GFS2 inode - * @ra_state: The read-ahead state + * @ip: Pointer to the rindex inode   *   * Returns: 0 on success, > 0 on EOF, error code otherwise   */ -static int read_rindex_entry(struct gfs2_inode *ip, -			     struct file_ra_state *ra_state) +static int read_rindex_entry(struct gfs2_inode *ip)  {  	struct gfs2_sbd *sdp = GFS2_SB(&ip->i_inode);  	loff_t pos = sdp->sd_rgrps * sizeof(struct gfs2_rindex); @@ -604,7 +600,7 @@ static int read_rindex_entry(struct gfs2_inode *ip,  	if (pos >= i_size_read(&ip->i_inode))  		return 1; -	error = gfs2_internal_read(ip, ra_state, (char *)&buf, &pos, +	error = gfs2_internal_read(ip, (char *)&buf, &pos,  				   sizeof(struct gfs2_rindex));  	if (error != sizeof(struct gfs2_rindex)) @@ -660,13 +656,10 @@ fail:  static int gfs2_ri_update(struct gfs2_inode *ip)  {  	struct gfs2_sbd *sdp = GFS2_SB(&ip->i_inode); -	struct inode *inode = &ip->i_inode; -	struct file_ra_state ra_state;  	int error; -	file_ra_state_init(&ra_state, inode->i_mapping);  	do { -		error = read_rindex_entry(ip, &ra_state); +		error = read_rindex_entry(ip);  	} while (error == 0);  	if (error < 0) | 
