diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2011-06-15 11:59:10 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2011-07-12 13:40:29 -0400 |
commit | a56aaa02b1f723e28b41d339ddff02e958d32d43 (patch) | |
tree | 3e24ad69ceb5ee66e53baec133ef7bb950795c54 /fs/nfs/pnfs.c | |
parent | aa5c01446610f0305f96251d0f9621866b8e5a14 (diff) |
NFSv4.1: Clean up layoutreturn
Since we take a reference to it, we really ought to pass the a pointer to
the layout header in the arguments instead of assuming that
NFS_I(inode)->layout will forever point to the correct object.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs/pnfs.c')
-rw-r--r-- | fs/nfs/pnfs.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/nfs/pnfs.c b/fs/nfs/pnfs.c index a7dc3367a857..5fc2e5d755a5 100644 --- a/fs/nfs/pnfs.c +++ b/fs/nfs/pnfs.c | |||
@@ -667,6 +667,7 @@ _pnfs_return_layout(struct inode *ino) | |||
667 | lrp->args.stateid = stateid; | 667 | lrp->args.stateid = stateid; |
668 | lrp->args.layout_type = NFS_SERVER(ino)->pnfs_curr_ld->id; | 668 | lrp->args.layout_type = NFS_SERVER(ino)->pnfs_curr_ld->id; |
669 | lrp->args.inode = ino; | 669 | lrp->args.inode = ino; |
670 | lrp->args.layout = lo; | ||
670 | lrp->clp = NFS_SERVER(ino)->nfs_client; | 671 | lrp->clp = NFS_SERVER(ino)->nfs_client; |
671 | 672 | ||
672 | status = nfs4_proc_layoutreturn(lrp); | 673 | status = nfs4_proc_layoutreturn(lrp); |