The file layout does not use nr_pages. The calculation is easy. No need to pass the extra parameter. Why not just call nfs_page_array_len() in the layout drivers that need it? -->Andy On Tue, Jul 20, 2010 at 1:57 AM, Benny Halevy <bhalevy@xxxxxxxxxxx> wrote: > Signed-off-by: Benny Halevy <bhalevy@xxxxxxxxxxx> > --- > fs/nfs/nfs4filelayout.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/fs/nfs/nfs4filelayout.c b/fs/nfs/nfs4filelayout.c > index 9b6ba8d..b2ce478 100644 > --- a/fs/nfs/nfs4filelayout.c > +++ b/fs/nfs/nfs4filelayout.c > @@ -244,7 +244,7 @@ filelayout_read_pagelist(struct nfs_read_data *data, unsigned nr_pages) > > /* Perform async writes. */ > static enum pnfs_try_status > -filelayout_write_pagelist(struct nfs_write_data *data, int sync) > +filelayout_write_pagelist(struct nfs_write_data *data, unsigned nr_pages, int sync) > { > struct pnfs_layout_segment *lseg = data->pdata.lseg; > struct nfs4_pnfs_ds *ds; > -- > 1.7.1.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-nfs" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html