From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steven Whitehouse Date: Mon, 16 Apr 2012 17:31:15 +0100 Subject: [Cluster-devel] [PATCH] GFS2: Remove unused argument from gfs2_internal_read In-Reply-To: <1334590855-2844-1-git-send-email-anprice@redhat.com> References: <1334590855-2844-1-git-send-email-anprice@redhat.com> Message-ID: <1334593875.2871.19.camel@menhir> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Hi, Now in the -nmw git tree. Thanks, Steve. On Mon, 2012-04-16 at 16:40 +0100, Andrew Price wrote: > 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 > --- > fs/gfs2/aops.c | 5 ++--- > fs/gfs2/inode.h | 1 - > fs/gfs2/quota.c | 4 ++-- > 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 56dc1f0..2f67a17 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 5d345b4..c53c747 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 6019da3..45d9171 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 7a1cf67..b550e5c 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)