diff options
Diffstat (limited to 'fs/nfs')
-rw-r--r-- | fs/nfs/pagelist.c | 14 | ||||
-rw-r--r-- | fs/nfs/write.c | 6 |
2 files changed, 11 insertions, 9 deletions
diff --git a/fs/nfs/pagelist.c b/fs/nfs/pagelist.c index f8a4ba533930..257a7f8b2362 100644 --- a/fs/nfs/pagelist.c +++ b/fs/nfs/pagelist.c | |||
@@ -85,9 +85,8 @@ nfs_create_request(struct nfs_open_context *ctx, struct inode *inode, | |||
85 | req->wb_offset = offset; | 85 | req->wb_offset = offset; |
86 | req->wb_pgbase = offset; | 86 | req->wb_pgbase = offset; |
87 | req->wb_bytes = count; | 87 | req->wb_bytes = count; |
88 | atomic_set(&req->wb_count, 1); | ||
89 | req->wb_context = get_nfs_open_context(ctx); | 88 | req->wb_context = get_nfs_open_context(ctx); |
90 | 89 | kref_init(&req->wb_kref); | |
91 | return req; | 90 | return req; |
92 | } | 91 | } |
93 | 92 | ||
@@ -160,11 +159,9 @@ void nfs_clear_request(struct nfs_page *req) | |||
160 | * | 159 | * |
161 | * Note: Should never be called with the spinlock held! | 160 | * Note: Should never be called with the spinlock held! |
162 | */ | 161 | */ |
163 | void | 162 | static void nfs_free_request(struct kref *kref) |
164 | nfs_release_request(struct nfs_page *req) | ||
165 | { | 163 | { |
166 | if (!atomic_dec_and_test(&req->wb_count)) | 164 | struct nfs_page *req = container_of(kref, struct nfs_page, wb_kref); |
167 | return; | ||
168 | 165 | ||
169 | /* Release struct file or cached credential */ | 166 | /* Release struct file or cached credential */ |
170 | nfs_clear_request(req); | 167 | nfs_clear_request(req); |
@@ -172,6 +169,11 @@ nfs_release_request(struct nfs_page *req) | |||
172 | nfs_page_free(req); | 169 | nfs_page_free(req); |
173 | } | 170 | } |
174 | 171 | ||
172 | void nfs_release_request(struct nfs_page *req) | ||
173 | { | ||
174 | kref_put(&req->wb_kref, nfs_free_request); | ||
175 | } | ||
176 | |||
175 | static int nfs_wait_bit_interruptible(void *word) | 177 | static int nfs_wait_bit_interruptible(void *word) |
176 | { | 178 | { |
177 | int ret = 0; | 179 | int ret = 0; |
diff --git a/fs/nfs/write.c b/fs/nfs/write.c index 9e7c21da864f..e9404328ac02 100644 --- a/fs/nfs/write.c +++ b/fs/nfs/write.c | |||
@@ -117,7 +117,7 @@ static struct nfs_page *nfs_page_find_request_locked(struct page *page) | |||
117 | if (PagePrivate(page)) { | 117 | if (PagePrivate(page)) { |
118 | req = (struct nfs_page *)page_private(page); | 118 | req = (struct nfs_page *)page_private(page); |
119 | if (req != NULL) | 119 | if (req != NULL) |
120 | atomic_inc(&req->wb_count); | 120 | kref_get(&req->wb_kref); |
121 | } | 121 | } |
122 | return req; | 122 | return req; |
123 | } | 123 | } |
@@ -398,7 +398,7 @@ static int nfs_inode_add_request(struct inode *inode, struct nfs_page *req) | |||
398 | if (PageDirty(req->wb_page)) | 398 | if (PageDirty(req->wb_page)) |
399 | set_bit(PG_NEED_FLUSH, &req->wb_flags); | 399 | set_bit(PG_NEED_FLUSH, &req->wb_flags); |
400 | nfsi->npages++; | 400 | nfsi->npages++; |
401 | atomic_inc(&req->wb_count); | 401 | kref_get(&req->wb_kref); |
402 | return 0; | 402 | return 0; |
403 | } | 403 | } |
404 | 404 | ||
@@ -531,7 +531,7 @@ static int nfs_wait_on_requests_locked(struct inode *inode, pgoff_t idx_start, u | |||
531 | next = req->wb_index + 1; | 531 | next = req->wb_index + 1; |
532 | BUG_ON(!NFS_WBACK_BUSY(req)); | 532 | BUG_ON(!NFS_WBACK_BUSY(req)); |
533 | 533 | ||
534 | atomic_inc(&req->wb_count); | 534 | kref_get(&req->wb_kref); |
535 | spin_unlock(&nfsi->req_lock); | 535 | spin_unlock(&nfsi->req_lock); |
536 | error = nfs_wait_on_request(req); | 536 | error = nfs_wait_on_request(req); |
537 | nfs_release_request(req); | 537 | nfs_release_request(req); |