diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-05-09 14:04:55 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-05-09 15:17:43 -0400 |
commit | 1d1afcbc294cc7c788eb5c7b6b98e8d63caf002c (patch) | |
tree | 88abc7e7018f733410fa51039351e4f0e2dd84c3 /fs/nfs/direct.c | |
parent | 7ad84aa9448571678c243f0c5ef383fbe5b50f4f (diff) |
NFS: Clean up - Rename nfs_unlock_request and nfs_unlock_request_dont_release
Function rename to ensure that the functionality of nfs_unlock_request()
mirrors that of nfs_lock_request(). Then let nfs_unlock_and_release_request()
do the work of what used to be called nfs_unlock_request()...
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: Fred Isaman <iisaman@netapp.com>
Diffstat (limited to 'fs/nfs/direct.c')
-rw-r--r-- | fs/nfs/direct.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/fs/nfs/direct.c b/fs/nfs/direct.c index 465ea84a2874..845e20196803 100644 --- a/fs/nfs/direct.c +++ b/fs/nfs/direct.c | |||
@@ -488,7 +488,7 @@ static void nfs_direct_write_reschedule(struct nfs_direct_req *dreq) | |||
488 | 488 | ||
489 | while (!list_empty(&failed)) { | 489 | while (!list_empty(&failed)) { |
490 | nfs_release_request(req); | 490 | nfs_release_request(req); |
491 | nfs_unlock_request(req); | 491 | nfs_unlock_and_release_request(req); |
492 | } | 492 | } |
493 | 493 | ||
494 | if (put_dreq(dreq)) | 494 | if (put_dreq(dreq)) |
@@ -521,7 +521,7 @@ static void nfs_direct_commit_complete(struct nfs_commit_data *data) | |||
521 | nfs_mark_request_commit(req, NULL, &cinfo); | 521 | nfs_mark_request_commit(req, NULL, &cinfo); |
522 | } else | 522 | } else |
523 | nfs_release_request(req); | 523 | nfs_release_request(req); |
524 | nfs_unlock_request(req); | 524 | nfs_unlock_and_release_request(req); |
525 | } | 525 | } |
526 | 526 | ||
527 | if (atomic_dec_and_test(&cinfo.mds->rpcs_out)) | 527 | if (atomic_dec_and_test(&cinfo.mds->rpcs_out)) |
@@ -662,7 +662,7 @@ static ssize_t nfs_direct_write_schedule_segment(struct nfs_pageio_descriptor *d | |||
662 | req->wb_offset = pos & ~PAGE_MASK; | 662 | req->wb_offset = pos & ~PAGE_MASK; |
663 | if (!nfs_pageio_add_request(desc, req)) { | 663 | if (!nfs_pageio_add_request(desc, req)) { |
664 | result = desc->pg_error; | 664 | result = desc->pg_error; |
665 | nfs_unlock_request(req); | 665 | nfs_unlock_and_release_request(req); |
666 | nfs_release_request(req); | 666 | nfs_release_request(req); |
667 | break; | 667 | break; |
668 | } | 668 | } |
@@ -739,7 +739,7 @@ static void nfs_direct_write_completion(struct nfs_pgio_header *hdr) | |||
739 | default: | 739 | default: |
740 | nfs_release_request(req); | 740 | nfs_release_request(req); |
741 | } | 741 | } |
742 | nfs_unlock_request(req); | 742 | nfs_unlock_and_release_request(req); |
743 | } | 743 | } |
744 | 744 | ||
745 | out_put: | 745 | out_put: |
@@ -756,7 +756,7 @@ static void nfs_write_sync_pgio_error(struct list_head *head) | |||
756 | req = nfs_list_entry(head->next); | 756 | req = nfs_list_entry(head->next); |
757 | nfs_list_remove_request(req); | 757 | nfs_list_remove_request(req); |
758 | nfs_release_request(req); | 758 | nfs_release_request(req); |
759 | nfs_unlock_request(req); | 759 | nfs_unlock_and_release_request(req); |
760 | } | 760 | } |
761 | } | 761 | } |
762 | 762 | ||