Signed-off-by: Fred Isaman <iisaman@xxxxxxxxxx>
---
fs/nfs/pnfs.c | 27 +++++++++------------------
fs/nfs/pnfs.h | 11 ++++-------
2 files changed, 13 insertions(+), 25 deletions(-)
diff --git a/fs/nfs/pnfs.c b/fs/nfs/pnfs.c
index e37b71e..ae76e88 100644
--- a/fs/nfs/pnfs.c
+++ b/fs/nfs/pnfs.c
@@ -1050,7 +1050,7 @@ pnfs_has_layout(struct pnfs_layout_type *lo,
* If lsegpp is given, the appropriate layout segment is referenced and
* returned to the caller.
*/
-int
+void
_pnfs_update_layout(struct inode *ino,
struct nfs_open_context *ctx,
u64 count,
@@ -1068,14 +1068,12 @@ _pnfs_update_layout(struct inode *ino,
struct pnfs_layout_segment *lseg = NULL;
bool take_ref = (lsegpp != NULL);
DEFINE_WAIT(__wait);
- int result = 0;
if (take_ref)
*lsegpp = NULL;
lo = get_lock_alloc_layout(ino);
if (IS_ERR(lo)) {
dprintk("%s ERROR: can't get pnfs_layout_type\n", __func__);
- result = PTR_ERR(lo);
goto out;
}
@@ -1086,7 +1084,6 @@ _pnfs_update_layout(struct inode *ino,
put_lseg_locked(lseg);
/* someone is cleaning the layout */
lseg = NULL;
- result = -EAGAIN;
goto out_put;
}
@@ -1109,21 +1106,18 @@ _pnfs_update_layout(struct inode *ino,
clear_bit(lo_fail_bit(iomode),
&nfsi->layout.pnfs_layout_state);
nfsi->layout.pnfs_layout_suspend = 0;
- } else {
- result = 1;
+ } else
goto out_put;
- }
}
/* Lose lock, but not reference, match this with pnfs_layout_release */
spin_unlock(&nfsi->lo_lock);
- result = get_layout(ino, ctx, &arg, lsegpp, lo);
+ get_layout(ino, ctx, &arg, lsegpp, lo);
out:
- dprintk("%s end (err:%d) state 0x%lx lseg %p\n",
- __func__, result, nfsi->layout.pnfs_layout_state,
- lseg);
- return result;
+ dprintk("%s end, state 0x%lx lseg %p\n", __func__,
+ nfsi->layout.pnfs_layout_state, lseg);
+ return;
out_put:
if (lsegpp)
*lsegpp = lseg;
@@ -1364,7 +1358,6 @@ pnfs_pageio_init_read(struct nfs_pageio_descriptor *pgio,
struct nfs_server *nfss = NFS_SERVER(inode);
size_t count = 0;
loff_t loff;
- int status = 0;
pgio->pg_iswrite = 0;
pgio->pg_boundary = 0;
@@ -1378,11 +1371,9 @@ pnfs_pageio_init_read(struct nfs_pageio_descriptor *pgio,
readahead_range(inode, pages, &loff, &count);
if (count > 0) {
- status = _pnfs_update_layout(inode, ctx, count,
- loff, IOMODE_READ,
- &pgio->pg_lseg);
- dprintk("%s virt update returned %d\n", __func__, status);
- if (status != 0)
+ _pnfs_update_layout(inode, ctx, count, loff, IOMODE_READ,
+ &pgio->pg_lseg);
+ if (!pgio->pg_lseg)
return;
pgio->pg_boundary = pnfs_getboundary(inode);
diff --git a/fs/nfs/pnfs.h b/fs/nfs/pnfs.h
index bdf11bc..09de7a3 100644
--- a/fs/nfs/pnfs.h
+++ b/fs/nfs/pnfs.h
@@ -32,7 +32,7 @@ extern int pnfs4_proc_layoutreturn(struct nfs4_pnfs_layoutreturn *lrp, bool wait
extern const nfs4_stateid zero_stateid;
void put_lseg(struct pnfs_layout_segment *lseg);
-int _pnfs_update_layout(struct inode *ino, struct nfs_open_context *ctx,
+void _pnfs_update_layout(struct inode *ino, struct nfs_open_context *ctx,
u64 count, loff_t pos, enum pnfs_iomode access_type,
struct pnfs_layout_segment **lsegpp);
@@ -175,7 +175,7 @@ static inline int pnfs_return_layout(struct inode *ino,
return 0;
}
-static inline int pnfs_update_layout(struct inode *ino,
+static inline void pnfs_update_layout(struct inode *ino,
struct nfs_open_context *ctx,
u64 count, loff_t pos, enum pnfs_iomode access_type,
struct pnfs_layout_segment **lsegpp)
@@ -183,12 +183,10 @@ static inline int pnfs_update_layout(struct inode *ino,
struct nfs_server *nfss = NFS_SERVER(ino);
if (pnfs_enabled_sb(nfss))
- return _pnfs_update_layout(ino, ctx, count, pos,
- access_type, lsegpp);
+ _pnfs_update_layout(ino, ctx, count, pos, access_type, lsegpp);
else {
if (lsegpp)
*lsegpp = NULL;
- return 0;
}
}
@@ -220,14 +218,13 @@ static inline void put_lseg(struct pnfs_layout_segment *lseg)
{
}
-static inline int
+static inline void
pnfs_update_layout(struct inode *ino, struct nfs_open_context *ctx,
u64 count, loff_t pos, enum pnfs_iomode access_type,
struct pnfs_layout_segment **lsegpp)
{
if (lsegpp)
*lsegpp = NULL;
- return 0;
}
static inline enum pnfs_try_status
--
1.6.6.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
