diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-06-08 11:56:09 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-06-28 17:20:50 -0400 |
commit | 2f2c63bc221c5fcded24de2704575d0abf96b910 (patch) | |
tree | dacf9f2bdbd101de6525a53571f0f1e48a75902e /fs/nfs/write.c | |
parent | 98d9452448122486f81030c6c70f29471f65e1ce (diff) |
NFS: Cleanup - only store the write verifier in struct nfs_page
The 'committed' field is not needed once we have put the struct nfs_page
on the right list.
Also correct the type of the verifier: it is not an array of __be32, but
simply an 8 byte long opaque array.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs/write.c')
-rw-r--r-- | fs/nfs/write.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/nfs/write.c b/fs/nfs/write.c index 4d6861c0dc14..ee929e5e1f7b 100644 --- a/fs/nfs/write.c +++ b/fs/nfs/write.c | |||
@@ -620,7 +620,7 @@ static void nfs_write_completion(struct nfs_pgio_header *hdr) | |||
620 | goto next; | 620 | goto next; |
621 | } | 621 | } |
622 | if (test_bit(NFS_IOHDR_NEED_COMMIT, &hdr->flags)) { | 622 | if (test_bit(NFS_IOHDR_NEED_COMMIT, &hdr->flags)) { |
623 | memcpy(&req->wb_verf, hdr->verf, sizeof(req->wb_verf)); | 623 | memcpy(&req->wb_verf, &hdr->verf->verifier, sizeof(req->wb_verf)); |
624 | nfs_mark_request_commit(req, hdr->lseg, &cinfo); | 624 | nfs_mark_request_commit(req, hdr->lseg, &cinfo); |
625 | goto next; | 625 | goto next; |
626 | } | 626 | } |
@@ -1547,7 +1547,7 @@ static void nfs_commit_release_pages(struct nfs_commit_data *data) | |||
1547 | 1547 | ||
1548 | /* Okay, COMMIT succeeded, apparently. Check the verifier | 1548 | /* Okay, COMMIT succeeded, apparently. Check the verifier |
1549 | * returned by the server against all stored verfs. */ | 1549 | * returned by the server against all stored verfs. */ |
1550 | if (!memcmp(req->wb_verf.verifier, data->verf.verifier, sizeof(data->verf.verifier))) { | 1550 | if (!memcmp(&req->wb_verf, &data->verf.verifier, sizeof(req->wb_verf))) { |
1551 | /* We have a match */ | 1551 | /* We have a match */ |
1552 | nfs_inode_remove_request(req); | 1552 | nfs_inode_remove_request(req); |
1553 | dprintk(" OK\n"); | 1553 | dprintk(" OK\n"); |