diff options
-rw-r--r-- | fs/nfs/direct.c | 435 | ||||
-rw-r--r-- | include/linux/nfs_xdr.h | 2 |
2 files changed, 203 insertions, 234 deletions
diff --git a/fs/nfs/direct.c b/fs/nfs/direct.c index 8ca9707be6c9..9ae7b6f6bf30 100644 --- a/fs/nfs/direct.c +++ b/fs/nfs/direct.c | |||
@@ -68,25 +68,19 @@ struct nfs_direct_req { | |||
68 | struct kref kref; /* release manager */ | 68 | struct kref kref; /* release manager */ |
69 | 69 | ||
70 | /* I/O parameters */ | 70 | /* I/O parameters */ |
71 | struct list_head list, /* nfs_read/write_data structs */ | ||
72 | rewrite_list; /* saved nfs_write_data structs */ | ||
73 | struct nfs_open_context *ctx; /* file open context info */ | 71 | struct nfs_open_context *ctx; /* file open context info */ |
74 | struct kiocb * iocb; /* controlling i/o request */ | 72 | struct kiocb * iocb; /* controlling i/o request */ |
75 | struct inode * inode; /* target file of i/o */ | 73 | struct inode * inode; /* target file of i/o */ |
76 | unsigned long user_addr; /* location of user's buffer */ | ||
77 | size_t user_count; /* total bytes to move */ | ||
78 | loff_t pos; /* starting offset in file */ | ||
79 | struct page ** pages; /* pages in our buffer */ | ||
80 | unsigned int npages; /* count of pages */ | ||
81 | 74 | ||
82 | /* completion state */ | 75 | /* completion state */ |
76 | atomic_t io_count; /* i/os we're waiting for */ | ||
83 | spinlock_t lock; /* protect completion state */ | 77 | spinlock_t lock; /* protect completion state */ |
84 | int outstanding; /* i/os we're waiting for */ | ||
85 | ssize_t count, /* bytes actually processed */ | 78 | ssize_t count, /* bytes actually processed */ |
86 | error; /* any reported error */ | 79 | error; /* any reported error */ |
87 | struct completion completion; /* wait for i/o completion */ | 80 | struct completion completion; /* wait for i/o completion */ |
88 | 81 | ||
89 | /* commit state */ | 82 | /* commit state */ |
83 | struct list_head rewrite_list; /* saved nfs_write_data structs */ | ||
90 | struct nfs_write_data * commit_data; /* special write_data for commits */ | 84 | struct nfs_write_data * commit_data; /* special write_data for commits */ |
91 | int flags; | 85 | int flags; |
92 | #define NFS_ODIRECT_DO_COMMIT (1) /* an unstable reply was received */ | 86 | #define NFS_ODIRECT_DO_COMMIT (1) /* an unstable reply was received */ |
@@ -94,8 +88,37 @@ struct nfs_direct_req { | |||
94 | struct nfs_writeverf verf; /* unstable write verifier */ | 88 | struct nfs_writeverf verf; /* unstable write verifier */ |
95 | }; | 89 | }; |
96 | 90 | ||
97 | static void nfs_direct_write_schedule(struct nfs_direct_req *dreq, int sync); | ||
98 | static void nfs_direct_write_complete(struct nfs_direct_req *dreq, struct inode *inode); | 91 | static void nfs_direct_write_complete(struct nfs_direct_req *dreq, struct inode *inode); |
92 | static const struct rpc_call_ops nfs_write_direct_ops; | ||
93 | |||
94 | static inline void get_dreq(struct nfs_direct_req *dreq) | ||
95 | { | ||
96 | atomic_inc(&dreq->io_count); | ||
97 | } | ||
98 | |||
99 | static inline int put_dreq(struct nfs_direct_req *dreq) | ||
100 | { | ||
101 | return atomic_dec_and_test(&dreq->io_count); | ||
102 | } | ||
103 | |||
104 | /* | ||
105 | * "size" is never larger than rsize or wsize. | ||
106 | */ | ||
107 | static inline int nfs_direct_count_pages(unsigned long user_addr, size_t size) | ||
108 | { | ||
109 | int page_count; | ||
110 | |||
111 | page_count = (user_addr + size + PAGE_SIZE - 1) >> PAGE_SHIFT; | ||
112 | page_count -= user_addr >> PAGE_SHIFT; | ||
113 | BUG_ON(page_count < 0); | ||
114 | |||
115 | return page_count; | ||
116 | } | ||
117 | |||
118 | static inline unsigned int nfs_max_pages(unsigned int size) | ||
119 | { | ||
120 | return (size + PAGE_CACHE_SIZE - 1) >> PAGE_CACHE_SHIFT; | ||
121 | } | ||
99 | 122 | ||
100 | /** | 123 | /** |
101 | * nfs_direct_IO - NFS address space operation for direct I/O | 124 | * nfs_direct_IO - NFS address space operation for direct I/O |
@@ -119,50 +142,21 @@ ssize_t nfs_direct_IO(int rw, struct kiocb *iocb, const struct iovec *iov, loff_ | |||
119 | return -EINVAL; | 142 | return -EINVAL; |
120 | } | 143 | } |
121 | 144 | ||
122 | static void nfs_free_user_pages(struct page **pages, int npages, int do_dirty) | 145 | static void nfs_direct_dirty_pages(struct page **pages, int npages) |
123 | { | 146 | { |
124 | int i; | 147 | int i; |
125 | for (i = 0; i < npages; i++) { | 148 | for (i = 0; i < npages; i++) { |
126 | struct page *page = pages[i]; | 149 | struct page *page = pages[i]; |
127 | if (do_dirty && !PageCompound(page)) | 150 | if (!PageCompound(page)) |
128 | set_page_dirty_lock(page); | 151 | set_page_dirty_lock(page); |
129 | page_cache_release(page); | ||
130 | } | 152 | } |
131 | kfree(pages); | ||
132 | } | 153 | } |
133 | 154 | ||
134 | static inline int nfs_get_user_pages(int rw, unsigned long user_addr, size_t size, struct page ***pages) | 155 | static void nfs_direct_release_pages(struct page **pages, int npages) |
135 | { | 156 | { |
136 | int result = -ENOMEM; | 157 | int i; |
137 | unsigned long page_count; | 158 | for (i = 0; i < npages; i++) |
138 | size_t array_size; | 159 | page_cache_release(pages[i]); |
139 | |||
140 | page_count = (user_addr + size + PAGE_SIZE - 1) >> PAGE_SHIFT; | ||
141 | page_count -= user_addr >> PAGE_SHIFT; | ||
142 | |||
143 | array_size = (page_count * sizeof(struct page *)); | ||
144 | *pages = kmalloc(array_size, GFP_KERNEL); | ||
145 | if (*pages) { | ||
146 | down_read(¤t->mm->mmap_sem); | ||
147 | result = get_user_pages(current, current->mm, user_addr, | ||
148 | page_count, (rw == READ), 0, | ||
149 | *pages, NULL); | ||
150 | up_read(¤t->mm->mmap_sem); | ||
151 | if (result != page_count) { | ||
152 | /* | ||
153 | * If we got fewer pages than expected from | ||
154 | * get_user_pages(), the user buffer runs off the | ||
155 | * end of a mapping; return EFAULT. | ||
156 | */ | ||
157 | if (result >= 0) { | ||
158 | nfs_free_user_pages(*pages, result, 0); | ||
159 | result = -EFAULT; | ||
160 | } else | ||
161 | kfree(*pages); | ||
162 | *pages = NULL; | ||
163 | } | ||
164 | } | ||
165 | return result; | ||
166 | } | 160 | } |
167 | 161 | ||
168 | static inline struct nfs_direct_req *nfs_direct_req_alloc(void) | 162 | static inline struct nfs_direct_req *nfs_direct_req_alloc(void) |
@@ -174,13 +168,13 @@ static inline struct nfs_direct_req *nfs_direct_req_alloc(void) | |||
174 | return NULL; | 168 | return NULL; |
175 | 169 | ||
176 | kref_init(&dreq->kref); | 170 | kref_init(&dreq->kref); |
171 | kref_get(&dreq->kref); | ||
177 | init_completion(&dreq->completion); | 172 | init_completion(&dreq->completion); |
178 | INIT_LIST_HEAD(&dreq->list); | ||
179 | INIT_LIST_HEAD(&dreq->rewrite_list); | 173 | INIT_LIST_HEAD(&dreq->rewrite_list); |
180 | dreq->iocb = NULL; | 174 | dreq->iocb = NULL; |
181 | dreq->ctx = NULL; | 175 | dreq->ctx = NULL; |
182 | spin_lock_init(&dreq->lock); | 176 | spin_lock_init(&dreq->lock); |
183 | dreq->outstanding = 0; | 177 | atomic_set(&dreq->io_count, 0); |
184 | dreq->count = 0; | 178 | dreq->count = 0; |
185 | dreq->error = 0; | 179 | dreq->error = 0; |
186 | dreq->flags = 0; | 180 | dreq->flags = 0; |
@@ -221,18 +215,11 @@ out: | |||
221 | } | 215 | } |
222 | 216 | ||
223 | /* | 217 | /* |
224 | * We must hold a reference to all the pages in this direct read request | 218 | * Synchronous I/O uses a stack-allocated iocb. Thus we can't trust |
225 | * until the RPCs complete. This could be long *after* we are woken up in | 219 | * the iocb is still valid here if this is a synchronous request. |
226 | * nfs_direct_wait (for instance, if someone hits ^C on a slow server). | ||
227 | * | ||
228 | * In addition, synchronous I/O uses a stack-allocated iocb. Thus we | ||
229 | * can't trust the iocb is still valid here if this is a synchronous | ||
230 | * request. If the waiter is woken prematurely, the iocb is long gone. | ||
231 | */ | 220 | */ |
232 | static void nfs_direct_complete(struct nfs_direct_req *dreq) | 221 | static void nfs_direct_complete(struct nfs_direct_req *dreq) |
233 | { | 222 | { |
234 | nfs_free_user_pages(dreq->pages, dreq->npages, 1); | ||
235 | |||
236 | if (dreq->iocb) { | 223 | if (dreq->iocb) { |
237 | long res = (long) dreq->error; | 224 | long res = (long) dreq->error; |
238 | if (!res) | 225 | if (!res) |
@@ -245,48 +232,10 @@ static void nfs_direct_complete(struct nfs_direct_req *dreq) | |||
245 | } | 232 | } |
246 | 233 | ||
247 | /* | 234 | /* |
248 | * Note we also set the number of requests we have in the dreq when we are | 235 | * We must hold a reference to all the pages in this direct read request |
249 | * done. This prevents races with I/O completion so we will always wait | 236 | * until the RPCs complete. This could be long *after* we are woken up in |
250 | * until all requests have been dispatched and completed. | 237 | * nfs_direct_wait (for instance, if someone hits ^C on a slow server). |
251 | */ | 238 | */ |
252 | static struct nfs_direct_req *nfs_direct_read_alloc(size_t nbytes, size_t rsize) | ||
253 | { | ||
254 | struct list_head *list; | ||
255 | struct nfs_direct_req *dreq; | ||
256 | unsigned int rpages = (rsize + PAGE_CACHE_SIZE - 1) >> PAGE_CACHE_SHIFT; | ||
257 | |||
258 | dreq = nfs_direct_req_alloc(); | ||
259 | if (!dreq) | ||
260 | return NULL; | ||
261 | |||
262 | list = &dreq->list; | ||
263 | for(;;) { | ||
264 | struct nfs_read_data *data = nfs_readdata_alloc(rpages); | ||
265 | |||
266 | if (unlikely(!data)) { | ||
267 | while (!list_empty(list)) { | ||
268 | data = list_entry(list->next, | ||
269 | struct nfs_read_data, pages); | ||
270 | list_del(&data->pages); | ||
271 | nfs_readdata_free(data); | ||
272 | } | ||
273 | kref_put(&dreq->kref, nfs_direct_req_release); | ||
274 | return NULL; | ||
275 | } | ||
276 | |||
277 | INIT_LIST_HEAD(&data->pages); | ||
278 | list_add(&data->pages, list); | ||
279 | |||
280 | data->req = (struct nfs_page *) dreq; | ||
281 | dreq->outstanding++; | ||
282 | if (nbytes <= rsize) | ||
283 | break; | ||
284 | nbytes -= rsize; | ||
285 | } | ||
286 | kref_get(&dreq->kref); | ||
287 | return dreq; | ||
288 | } | ||
289 | |||
290 | static void nfs_direct_read_result(struct rpc_task *task, void *calldata) | 239 | static void nfs_direct_read_result(struct rpc_task *task, void *calldata) |
291 | { | 240 | { |
292 | struct nfs_read_data *data = calldata; | 241 | struct nfs_read_data *data = calldata; |
@@ -295,6 +244,9 @@ static void nfs_direct_read_result(struct rpc_task *task, void *calldata) | |||
295 | if (nfs_readpage_result(task, data) != 0) | 244 | if (nfs_readpage_result(task, data) != 0) |
296 | return; | 245 | return; |
297 | 246 | ||
247 | nfs_direct_dirty_pages(data->pagevec, data->npages); | ||
248 | nfs_direct_release_pages(data->pagevec, data->npages); | ||
249 | |||
298 | spin_lock(&dreq->lock); | 250 | spin_lock(&dreq->lock); |
299 | 251 | ||
300 | if (likely(task->tk_status >= 0)) | 252 | if (likely(task->tk_status >= 0)) |
@@ -302,13 +254,10 @@ static void nfs_direct_read_result(struct rpc_task *task, void *calldata) | |||
302 | else | 254 | else |
303 | dreq->error = task->tk_status; | 255 | dreq->error = task->tk_status; |
304 | 256 | ||
305 | if (--dreq->outstanding) { | ||
306 | spin_unlock(&dreq->lock); | ||
307 | return; | ||
308 | } | ||
309 | |||
310 | spin_unlock(&dreq->lock); | 257 | spin_unlock(&dreq->lock); |
311 | nfs_direct_complete(dreq); | 258 | |
259 | if (put_dreq(dreq)) | ||
260 | nfs_direct_complete(dreq); | ||
312 | } | 261 | } |
313 | 262 | ||
314 | static const struct rpc_call_ops nfs_read_direct_ops = { | 263 | static const struct rpc_call_ops nfs_read_direct_ops = { |
@@ -317,41 +266,60 @@ static const struct rpc_call_ops nfs_read_direct_ops = { | |||
317 | }; | 266 | }; |
318 | 267 | ||
319 | /* | 268 | /* |
320 | * For each nfs_read_data struct that was allocated on the list, dispatch | 269 | * For each rsize'd chunk of the user's buffer, dispatch an NFS READ |
321 | * an NFS READ operation | 270 | * operation. If nfs_readdata_alloc() or get_user_pages() fails, |
271 | * bail and stop sending more reads. Read length accounting is | ||
272 | * handled automatically by nfs_direct_read_result(). Otherwise, if | ||
273 | * no requests have been sent, just return an error. | ||
322 | */ | 274 | */ |
323 | static void nfs_direct_read_schedule(struct nfs_direct_req *dreq) | 275 | static ssize_t nfs_direct_read_schedule(struct nfs_direct_req *dreq, unsigned long user_addr, size_t count, loff_t pos) |
324 | { | 276 | { |
325 | struct nfs_open_context *ctx = dreq->ctx; | 277 | struct nfs_open_context *ctx = dreq->ctx; |
326 | struct inode *inode = ctx->dentry->d_inode; | 278 | struct inode *inode = ctx->dentry->d_inode; |
327 | struct list_head *list = &dreq->list; | ||
328 | struct page **pages = dreq->pages; | ||
329 | size_t count = dreq->user_count; | ||
330 | loff_t pos = dreq->pos; | ||
331 | size_t rsize = NFS_SERVER(inode)->rsize; | 279 | size_t rsize = NFS_SERVER(inode)->rsize; |
332 | unsigned int curpage, pgbase; | 280 | unsigned int rpages = nfs_max_pages(rsize); |
281 | unsigned int pgbase; | ||
282 | int result; | ||
283 | ssize_t started = 0; | ||
284 | |||
285 | get_dreq(dreq); | ||
333 | 286 | ||
334 | curpage = 0; | 287 | pgbase = user_addr & ~PAGE_MASK; |
335 | pgbase = dreq->user_addr & ~PAGE_MASK; | ||
336 | do { | 288 | do { |
337 | struct nfs_read_data *data; | 289 | struct nfs_read_data *data; |
338 | size_t bytes; | 290 | size_t bytes; |
339 | 291 | ||
292 | result = -ENOMEM; | ||
293 | data = nfs_readdata_alloc(rpages); | ||
294 | if (unlikely(!data)) | ||
295 | break; | ||
296 | |||
340 | bytes = rsize; | 297 | bytes = rsize; |
341 | if (count < rsize) | 298 | if (count < rsize) |
342 | bytes = count; | 299 | bytes = count; |
343 | 300 | ||
344 | BUG_ON(list_empty(list)); | 301 | data->npages = nfs_direct_count_pages(user_addr, bytes); |
345 | data = list_entry(list->next, struct nfs_read_data, pages); | 302 | down_read(¤t->mm->mmap_sem); |
346 | list_del_init(&data->pages); | 303 | result = get_user_pages(current, current->mm, user_addr, |
304 | data->npages, 1, 0, data->pagevec, NULL); | ||
305 | up_read(¤t->mm->mmap_sem); | ||
306 | if (unlikely(result < data->npages)) { | ||
307 | if (result > 0) | ||
308 | nfs_direct_release_pages(data->pagevec, result); | ||
309 | nfs_readdata_release(data); | ||
310 | break; | ||
311 | } | ||
312 | |||
313 | get_dreq(dreq); | ||
347 | 314 | ||
315 | data->req = (struct nfs_page *) dreq; | ||
348 | data->inode = inode; | 316 | data->inode = inode; |
349 | data->cred = ctx->cred; | 317 | data->cred = ctx->cred; |
350 | data->args.fh = NFS_FH(inode); | 318 | data->args.fh = NFS_FH(inode); |
351 | data->args.context = ctx; | 319 | data->args.context = ctx; |
352 | data->args.offset = pos; | 320 | data->args.offset = pos; |
353 | data->args.pgbase = pgbase; | 321 | data->args.pgbase = pgbase; |
354 | data->args.pages = &pages[curpage]; | 322 | data->args.pages = data->pagevec; |
355 | data->args.count = bytes; | 323 | data->args.count = bytes; |
356 | data->res.fattr = &data->fattr; | 324 | data->res.fattr = &data->fattr; |
357 | data->res.eof = 0; | 325 | data->res.eof = 0; |
@@ -374,33 +342,35 @@ static void nfs_direct_read_schedule(struct nfs_direct_req *dreq) | |||
374 | bytes, | 342 | bytes, |
375 | (unsigned long long)data->args.offset); | 343 | (unsigned long long)data->args.offset); |
376 | 344 | ||
345 | started += bytes; | ||
346 | user_addr += bytes; | ||
377 | pos += bytes; | 347 | pos += bytes; |
378 | pgbase += bytes; | 348 | pgbase += bytes; |
379 | curpage += pgbase >> PAGE_SHIFT; | ||
380 | pgbase &= ~PAGE_MASK; | 349 | pgbase &= ~PAGE_MASK; |
381 | 350 | ||
382 | count -= bytes; | 351 | count -= bytes; |
383 | } while (count != 0); | 352 | } while (count != 0); |
384 | BUG_ON(!list_empty(list)); | 353 | |
354 | if (put_dreq(dreq)) | ||
355 | nfs_direct_complete(dreq); | ||
356 | |||
357 | if (started) | ||
358 | return 0; | ||
359 | return result < 0 ? (ssize_t) result : -EFAULT; | ||
385 | } | 360 | } |
386 | 361 | ||
387 | static ssize_t nfs_direct_read(struct kiocb *iocb, unsigned long user_addr, size_t count, loff_t pos, struct page **pages, unsigned int nr_pages) | 362 | static ssize_t nfs_direct_read(struct kiocb *iocb, unsigned long user_addr, size_t count, loff_t pos) |
388 | { | 363 | { |
389 | ssize_t result; | 364 | ssize_t result = 0; |
390 | sigset_t oldset; | 365 | sigset_t oldset; |
391 | struct inode *inode = iocb->ki_filp->f_mapping->host; | 366 | struct inode *inode = iocb->ki_filp->f_mapping->host; |
392 | struct rpc_clnt *clnt = NFS_CLIENT(inode); | 367 | struct rpc_clnt *clnt = NFS_CLIENT(inode); |
393 | struct nfs_direct_req *dreq; | 368 | struct nfs_direct_req *dreq; |
394 | 369 | ||
395 | dreq = nfs_direct_read_alloc(count, NFS_SERVER(inode)->rsize); | 370 | dreq = nfs_direct_req_alloc(); |
396 | if (!dreq) | 371 | if (!dreq) |
397 | return -ENOMEM; | 372 | return -ENOMEM; |
398 | 373 | ||
399 | dreq->user_addr = user_addr; | ||
400 | dreq->user_count = count; | ||
401 | dreq->pos = pos; | ||
402 | dreq->pages = pages; | ||
403 | dreq->npages = nr_pages; | ||
404 | dreq->inode = inode; | 374 | dreq->inode = inode; |
405 | dreq->ctx = get_nfs_open_context((struct nfs_open_context *)iocb->ki_filp->private_data); | 375 | dreq->ctx = get_nfs_open_context((struct nfs_open_context *)iocb->ki_filp->private_data); |
406 | if (!is_sync_kiocb(iocb)) | 376 | if (!is_sync_kiocb(iocb)) |
@@ -408,8 +378,9 @@ static ssize_t nfs_direct_read(struct kiocb *iocb, unsigned long user_addr, size | |||
408 | 378 | ||
409 | nfs_add_stats(inode, NFSIOS_DIRECTREADBYTES, count); | 379 | nfs_add_stats(inode, NFSIOS_DIRECTREADBYTES, count); |
410 | rpc_clnt_sigmask(clnt, &oldset); | 380 | rpc_clnt_sigmask(clnt, &oldset); |
411 | nfs_direct_read_schedule(dreq); | 381 | result = nfs_direct_read_schedule(dreq, user_addr, count, pos); |
412 | result = nfs_direct_wait(dreq); | 382 | if (!result) |
383 | result = nfs_direct_wait(dreq); | ||
413 | rpc_clnt_sigunmask(clnt, &oldset); | 384 | rpc_clnt_sigunmask(clnt, &oldset); |
414 | 385 | ||
415 | return result; | 386 | return result; |
@@ -417,10 +388,10 @@ static ssize_t nfs_direct_read(struct kiocb *iocb, unsigned long user_addr, size | |||
417 | 388 | ||
418 | static void nfs_direct_free_writedata(struct nfs_direct_req *dreq) | 389 | static void nfs_direct_free_writedata(struct nfs_direct_req *dreq) |
419 | { | 390 | { |
420 | list_splice_init(&dreq->rewrite_list, &dreq->list); | 391 | while (!list_empty(&dreq->rewrite_list)) { |
421 | while (!list_empty(&dreq->list)) { | 392 | struct nfs_write_data *data = list_entry(dreq->rewrite_list.next, struct nfs_write_data, pages); |
422 | struct nfs_write_data *data = list_entry(dreq->list.next, struct nfs_write_data, pages); | ||
423 | list_del(&data->pages); | 393 | list_del(&data->pages); |
394 | nfs_direct_release_pages(data->pagevec, data->npages); | ||
424 | nfs_writedata_release(data); | 395 | nfs_writedata_release(data); |
425 | } | 396 | } |
426 | } | 397 | } |
@@ -428,14 +399,51 @@ static void nfs_direct_free_writedata(struct nfs_direct_req *dreq) | |||
428 | #if defined(CONFIG_NFS_V3) || defined(CONFIG_NFS_V4) | 399 | #if defined(CONFIG_NFS_V3) || defined(CONFIG_NFS_V4) |
429 | static void nfs_direct_write_reschedule(struct nfs_direct_req *dreq) | 400 | static void nfs_direct_write_reschedule(struct nfs_direct_req *dreq) |
430 | { | 401 | { |
431 | struct list_head *pos; | 402 | struct inode *inode = dreq->inode; |
403 | struct list_head *p; | ||
404 | struct nfs_write_data *data; | ||
432 | 405 | ||
433 | list_splice_init(&dreq->rewrite_list, &dreq->list); | ||
434 | list_for_each(pos, &dreq->list) | ||
435 | dreq->outstanding++; | ||
436 | dreq->count = 0; | 406 | dreq->count = 0; |
407 | get_dreq(dreq); | ||
408 | |||
409 | list_for_each(p, &dreq->rewrite_list) { | ||
410 | data = list_entry(p, struct nfs_write_data, pages); | ||
411 | |||
412 | get_dreq(dreq); | ||
413 | |||
414 | /* | ||
415 | * Reset data->res. | ||
416 | */ | ||
417 | nfs_fattr_init(&data->fattr); | ||
418 | data->res.count = data->args.count; | ||
419 | memset(&data->verf, 0, sizeof(data->verf)); | ||
420 | |||
421 | /* | ||
422 | * Reuse data->task; data->args should not have changed | ||
423 | * since the original request was sent. | ||
424 | */ | ||
425 | rpc_init_task(&data->task, NFS_CLIENT(inode), RPC_TASK_ASYNC, | ||
426 | &nfs_write_direct_ops, data); | ||
427 | NFS_PROTO(inode)->write_setup(data, FLUSH_STABLE); | ||
428 | |||
429 | data->task.tk_priority = RPC_PRIORITY_NORMAL; | ||
430 | data->task.tk_cookie = (unsigned long) inode; | ||
431 | |||
432 | /* | ||
433 | * We're called via an RPC callback, so BKL is already held. | ||
434 | */ | ||
435 | rpc_execute(&data->task); | ||
436 | |||
437 | dprintk("NFS: %5u rescheduled direct write call (req %s/%Ld, %u bytes @ offset %Lu)\n", | ||
438 | data->task.tk_pid, | ||
439 | inode->i_sb->s_id, | ||
440 | (long long)NFS_FILEID(inode), | ||
441 | data->args.count, | ||
442 | (unsigned long long)data->args.offset); | ||
443 | } | ||
437 | 444 | ||
438 | nfs_direct_write_schedule(dreq, FLUSH_STABLE); | 445 | if (put_dreq(dreq)) |
446 | nfs_direct_write_complete(dreq, inode); | ||
439 | } | 447 | } |
440 | 448 | ||
441 | static void nfs_direct_commit_result(struct rpc_task *task, void *calldata) | 449 | static void nfs_direct_commit_result(struct rpc_task *task, void *calldata) |
@@ -472,8 +480,8 @@ static void nfs_direct_commit_schedule(struct nfs_direct_req *dreq) | |||
472 | data->cred = dreq->ctx->cred; | 480 | data->cred = dreq->ctx->cred; |
473 | 481 | ||
474 | data->args.fh = NFS_FH(data->inode); | 482 | data->args.fh = NFS_FH(data->inode); |
475 | data->args.offset = dreq->pos; | 483 | data->args.offset = 0; |
476 | data->args.count = dreq->user_count; | 484 | data->args.count = 0; |
477 | data->res.count = 0; | 485 | data->res.count = 0; |
478 | data->res.fattr = &data->fattr; | 486 | data->res.fattr = &data->fattr; |
479 | data->res.verf = &data->verf; | 487 | data->res.verf = &data->verf; |
@@ -535,47 +543,6 @@ static void nfs_direct_write_complete(struct nfs_direct_req *dreq, struct inode | |||
535 | } | 543 | } |
536 | #endif | 544 | #endif |
537 | 545 | ||
538 | static struct nfs_direct_req *nfs_direct_write_alloc(size_t nbytes, size_t wsize) | ||
539 | { | ||
540 | struct list_head *list; | ||
541 | struct nfs_direct_req *dreq; | ||
542 | unsigned int wpages = (wsize + PAGE_CACHE_SIZE - 1) >> PAGE_CACHE_SHIFT; | ||
543 | |||
544 | dreq = nfs_direct_req_alloc(); | ||
545 | if (!dreq) | ||
546 | return NULL; | ||
547 | |||
548 | list = &dreq->list; | ||
549 | for(;;) { | ||
550 | struct nfs_write_data *data = nfs_writedata_alloc(wpages); | ||
551 | |||
552 | if (unlikely(!data)) { | ||
553 | while (!list_empty(list)) { | ||
554 | data = list_entry(list->next, | ||
555 | struct nfs_write_data, pages); | ||
556 | list_del(&data->pages); | ||
557 | nfs_writedata_free(data); | ||
558 | } | ||
559 | kref_put(&dreq->kref, nfs_direct_req_release); | ||
560 | return NULL; | ||
561 | } | ||
562 | |||
563 | INIT_LIST_HEAD(&data->pages); | ||
564 | list_add(&data->pages, list); | ||
565 | |||
566 | data->req = (struct nfs_page *) dreq; | ||
567 | dreq->outstanding++; | ||
568 | if (nbytes <= wsize) | ||
569 | break; | ||
570 | nbytes -= wsize; | ||
571 | } | ||
572 | |||
573 | nfs_alloc_commit_data(dreq); | ||
574 | |||
575 | kref_get(&dreq->kref); | ||
576 | return dreq; | ||
577 | } | ||
578 | |||
579 | static void nfs_direct_write_result(struct rpc_task *task, void *calldata) | 546 | static void nfs_direct_write_result(struct rpc_task *task, void *calldata) |
580 | { | 547 | { |
581 | struct nfs_write_data *data = calldata; | 548 | struct nfs_write_data *data = calldata; |
@@ -605,8 +572,6 @@ static void nfs_direct_write_result(struct rpc_task *task, void *calldata) | |||
605 | } | 572 | } |
606 | } | 573 | } |
607 | } | 574 | } |
608 | /* In case we have to resend */ | ||
609 | data->args.stable = NFS_FILE_SYNC; | ||
610 | 575 | ||
611 | spin_unlock(&dreq->lock); | 576 | spin_unlock(&dreq->lock); |
612 | } | 577 | } |
@@ -620,14 +585,8 @@ static void nfs_direct_write_release(void *calldata) | |||
620 | struct nfs_write_data *data = calldata; | 585 | struct nfs_write_data *data = calldata; |
621 | struct nfs_direct_req *dreq = (struct nfs_direct_req *) data->req; | 586 | struct nfs_direct_req *dreq = (struct nfs_direct_req *) data->req; |
622 | 587 | ||
623 | spin_lock(&dreq->lock); | 588 | if (put_dreq(dreq)) |
624 | if (--dreq->outstanding) { | 589 | nfs_direct_write_complete(dreq, data->inode); |
625 | spin_unlock(&dreq->lock); | ||
626 | return; | ||
627 | } | ||
628 | spin_unlock(&dreq->lock); | ||
629 | |||
630 | nfs_direct_write_complete(dreq, data->inode); | ||
631 | } | 590 | } |
632 | 591 | ||
633 | static const struct rpc_call_ops nfs_write_direct_ops = { | 592 | static const struct rpc_call_ops nfs_write_direct_ops = { |
@@ -636,41 +595,62 @@ static const struct rpc_call_ops nfs_write_direct_ops = { | |||
636 | }; | 595 | }; |
637 | 596 | ||
638 | /* | 597 | /* |
639 | * For each nfs_write_data struct that was allocated on the list, dispatch | 598 | * For each wsize'd chunk of the user's buffer, dispatch an NFS WRITE |
640 | * an NFS WRITE operation | 599 | * operation. If nfs_writedata_alloc() or get_user_pages() fails, |
600 | * bail and stop sending more writes. Write length accounting is | ||
601 | * handled automatically by nfs_direct_write_result(). Otherwise, if | ||
602 | * no requests have been sent, just return an error. | ||
641 | */ | 603 | */ |
642 | static void nfs_direct_write_schedule(struct nfs_direct_req *dreq, int sync) | 604 | static ssize_t nfs_direct_write_schedule(struct nfs_direct_req *dreq, unsigned long user_addr, size_t count, loff_t pos, int sync) |
643 | { | 605 | { |
644 | struct nfs_open_context *ctx = dreq->ctx; | 606 | struct nfs_open_context *ctx = dreq->ctx; |
645 | struct inode *inode = ctx->dentry->d_inode; | 607 | struct inode *inode = ctx->dentry->d_inode; |
646 | struct list_head *list = &dreq->list; | ||
647 | struct page **pages = dreq->pages; | ||
648 | size_t count = dreq->user_count; | ||
649 | loff_t pos = dreq->pos; | ||
650 | size_t wsize = NFS_SERVER(inode)->wsize; | 608 | size_t wsize = NFS_SERVER(inode)->wsize; |
651 | unsigned int curpage, pgbase; | 609 | unsigned int wpages = nfs_max_pages(wsize); |
610 | unsigned int pgbase; | ||
611 | int result; | ||
612 | ssize_t started = 0; | ||
652 | 613 | ||
653 | curpage = 0; | 614 | get_dreq(dreq); |
654 | pgbase = dreq->user_addr & ~PAGE_MASK; | 615 | |
616 | pgbase = user_addr & ~PAGE_MASK; | ||
655 | do { | 617 | do { |
656 | struct nfs_write_data *data; | 618 | struct nfs_write_data *data; |
657 | size_t bytes; | 619 | size_t bytes; |
658 | 620 | ||
621 | result = -ENOMEM; | ||
622 | data = nfs_writedata_alloc(wpages); | ||
623 | if (unlikely(!data)) | ||
624 | break; | ||
625 | |||
659 | bytes = wsize; | 626 | bytes = wsize; |
660 | if (count < wsize) | 627 | if (count < wsize) |
661 | bytes = count; | 628 | bytes = count; |
662 | 629 | ||
663 | BUG_ON(list_empty(list)); | 630 | data->npages = nfs_direct_count_pages(user_addr, bytes); |
664 | data = list_entry(list->next, struct nfs_write_data, pages); | 631 | down_read(¤t->mm->mmap_sem); |
632 | result = get_user_pages(current, current->mm, user_addr, | ||
633 | data->npages, 0, 0, data->pagevec, NULL); | ||
634 | up_read(¤t->mm->mmap_sem); | ||
635 | if (unlikely(result < data->npages)) { | ||
636 | if (result > 0) | ||
637 | nfs_direct_release_pages(data->pagevec, result); | ||
638 | nfs_writedata_release(data); | ||
639 | break; | ||
640 | } | ||
641 | |||
642 | get_dreq(dreq); | ||
643 | |||
665 | list_move_tail(&data->pages, &dreq->rewrite_list); | 644 | list_move_tail(&data->pages, &dreq->rewrite_list); |
666 | 645 | ||
646 | data->req = (struct nfs_page *) dreq; | ||
667 | data->inode = inode; | 647 | data->inode = inode; |
668 | data->cred = ctx->cred; | 648 | data->cred = ctx->cred; |
669 | data->args.fh = NFS_FH(inode); | 649 | data->args.fh = NFS_FH(inode); |
670 | data->args.context = ctx; | 650 | data->args.context = ctx; |
671 | data->args.offset = pos; | 651 | data->args.offset = pos; |
672 | data->args.pgbase = pgbase; | 652 | data->args.pgbase = pgbase; |
673 | data->args.pages = &pages[curpage]; | 653 | data->args.pages = data->pagevec; |
674 | data->args.count = bytes; | 654 | data->args.count = bytes; |
675 | data->res.fattr = &data->fattr; | 655 | data->res.fattr = &data->fattr; |
676 | data->res.count = bytes; | 656 | data->res.count = bytes; |
@@ -694,19 +674,26 @@ static void nfs_direct_write_schedule(struct nfs_direct_req *dreq, int sync) | |||
694 | bytes, | 674 | bytes, |
695 | (unsigned long long)data->args.offset); | 675 | (unsigned long long)data->args.offset); |
696 | 676 | ||
677 | started += bytes; | ||
678 | user_addr += bytes; | ||
697 | pos += bytes; | 679 | pos += bytes; |
698 | pgbase += bytes; | 680 | pgbase += bytes; |
699 | curpage += pgbase >> PAGE_SHIFT; | ||
700 | pgbase &= ~PAGE_MASK; | 681 | pgbase &= ~PAGE_MASK; |
701 | 682 | ||
702 | count -= bytes; | 683 | count -= bytes; |
703 | } while (count != 0); | 684 | } while (count != 0); |
704 | BUG_ON(!list_empty(list)); | 685 | |
686 | if (put_dreq(dreq)) | ||
687 | nfs_direct_write_complete(dreq, inode); | ||
688 | |||
689 | if (started) | ||
690 | return 0; | ||
691 | return result < 0 ? (ssize_t) result : -EFAULT; | ||
705 | } | 692 | } |
706 | 693 | ||
707 | static ssize_t nfs_direct_write(struct kiocb *iocb, unsigned long user_addr, size_t count, loff_t pos, struct page **pages, int nr_pages) | 694 | static ssize_t nfs_direct_write(struct kiocb *iocb, unsigned long user_addr, size_t count, loff_t pos) |
708 | { | 695 | { |
709 | ssize_t result; | 696 | ssize_t result = 0; |
710 | sigset_t oldset; | 697 | sigset_t oldset; |
711 | struct inode *inode = iocb->ki_filp->f_mapping->host; | 698 | struct inode *inode = iocb->ki_filp->f_mapping->host; |
712 | struct rpc_clnt *clnt = NFS_CLIENT(inode); | 699 | struct rpc_clnt *clnt = NFS_CLIENT(inode); |
@@ -714,17 +701,14 @@ static ssize_t nfs_direct_write(struct kiocb *iocb, unsigned long user_addr, siz | |||
714 | size_t wsize = NFS_SERVER(inode)->wsize; | 701 | size_t wsize = NFS_SERVER(inode)->wsize; |
715 | int sync = 0; | 702 | int sync = 0; |
716 | 703 | ||
717 | dreq = nfs_direct_write_alloc(count, wsize); | 704 | dreq = nfs_direct_req_alloc(); |
718 | if (!dreq) | 705 | if (!dreq) |
719 | return -ENOMEM; | 706 | return -ENOMEM; |
707 | nfs_alloc_commit_data(dreq); | ||
708 | |||
720 | if (dreq->commit_data == NULL || count < wsize) | 709 | if (dreq->commit_data == NULL || count < wsize) |
721 | sync = FLUSH_STABLE; | 710 | sync = FLUSH_STABLE; |
722 | 711 | ||
723 | dreq->user_addr = user_addr; | ||
724 | dreq->user_count = count; | ||
725 | dreq->pos = pos; | ||
726 | dreq->pages = pages; | ||
727 | dreq->npages = nr_pages; | ||
728 | dreq->inode = inode; | 712 | dreq->inode = inode; |
729 | dreq->ctx = get_nfs_open_context((struct nfs_open_context *)iocb->ki_filp->private_data); | 713 | dreq->ctx = get_nfs_open_context((struct nfs_open_context *)iocb->ki_filp->private_data); |
730 | if (!is_sync_kiocb(iocb)) | 714 | if (!is_sync_kiocb(iocb)) |
@@ -735,8 +719,9 @@ static ssize_t nfs_direct_write(struct kiocb *iocb, unsigned long user_addr, siz | |||
735 | nfs_begin_data_update(inode); | 719 | nfs_begin_data_update(inode); |
736 | 720 | ||
737 | rpc_clnt_sigmask(clnt, &oldset); | 721 | rpc_clnt_sigmask(clnt, &oldset); |
738 | nfs_direct_write_schedule(dreq, sync); | 722 | result = nfs_direct_write_schedule(dreq, user_addr, count, pos, sync); |
739 | result = nfs_direct_wait(dreq); | 723 | if (!result) |
724 | result = nfs_direct_wait(dreq); | ||
740 | rpc_clnt_sigunmask(clnt, &oldset); | 725 | rpc_clnt_sigunmask(clnt, &oldset); |
741 | 726 | ||
742 | return result; | 727 | return result; |
@@ -766,8 +751,6 @@ static ssize_t nfs_direct_write(struct kiocb *iocb, unsigned long user_addr, siz | |||
766 | ssize_t nfs_file_direct_read(struct kiocb *iocb, char __user *buf, size_t count, loff_t pos) | 751 | ssize_t nfs_file_direct_read(struct kiocb *iocb, char __user *buf, size_t count, loff_t pos) |
767 | { | 752 | { |
768 | ssize_t retval = -EINVAL; | 753 | ssize_t retval = -EINVAL; |
769 | int page_count; | ||
770 | struct page **pages; | ||
771 | struct file *file = iocb->ki_filp; | 754 | struct file *file = iocb->ki_filp; |
772 | struct address_space *mapping = file->f_mapping; | 755 | struct address_space *mapping = file->f_mapping; |
773 | 756 | ||
@@ -789,14 +772,7 @@ ssize_t nfs_file_direct_read(struct kiocb *iocb, char __user *buf, size_t count, | |||
789 | if (retval) | 772 | if (retval) |
790 | goto out; | 773 | goto out; |
791 | 774 | ||
792 | retval = nfs_get_user_pages(READ, (unsigned long) buf, | 775 | retval = nfs_direct_read(iocb, (unsigned long) buf, count, pos); |
793 | count, &pages); | ||
794 | if (retval < 0) | ||
795 | goto out; | ||
796 | page_count = retval; | ||
797 | |||
798 | retval = nfs_direct_read(iocb, (unsigned long) buf, count, pos, | ||
799 | pages, page_count); | ||
800 | if (retval > 0) | 776 | if (retval > 0) |
801 | iocb->ki_pos = pos + retval; | 777 | iocb->ki_pos = pos + retval; |
802 | 778 | ||
@@ -832,8 +808,6 @@ out: | |||
832 | ssize_t nfs_file_direct_write(struct kiocb *iocb, const char __user *buf, size_t count, loff_t pos) | 808 | ssize_t nfs_file_direct_write(struct kiocb *iocb, const char __user *buf, size_t count, loff_t pos) |
833 | { | 809 | { |
834 | ssize_t retval; | 810 | ssize_t retval; |
835 | int page_count; | ||
836 | struct page **pages; | ||
837 | struct file *file = iocb->ki_filp; | 811 | struct file *file = iocb->ki_filp; |
838 | struct address_space *mapping = file->f_mapping; | 812 | struct address_space *mapping = file->f_mapping; |
839 | 813 | ||
@@ -861,14 +835,7 @@ ssize_t nfs_file_direct_write(struct kiocb *iocb, const char __user *buf, size_t | |||
861 | if (retval) | 835 | if (retval) |
862 | goto out; | 836 | goto out; |
863 | 837 | ||
864 | retval = nfs_get_user_pages(WRITE, (unsigned long) buf, | 838 | retval = nfs_direct_write(iocb, (unsigned long) buf, count, pos); |
865 | count, &pages); | ||
866 | if (retval < 0) | ||
867 | goto out; | ||
868 | page_count = retval; | ||
869 | |||
870 | retval = nfs_direct_write(iocb, (unsigned long) buf, count, | ||
871 | pos, pages, page_count); | ||
872 | 839 | ||
873 | /* | 840 | /* |
874 | * XXX: nfs_end_data_update() already ensures this file's | 841 | * XXX: nfs_end_data_update() already ensures this file's |
diff --git a/include/linux/nfs_xdr.h b/include/linux/nfs_xdr.h index 7c7320fa51aa..2d3fb6416d91 100644 --- a/include/linux/nfs_xdr.h +++ b/include/linux/nfs_xdr.h | |||
@@ -729,6 +729,7 @@ struct nfs_read_data { | |||
729 | struct list_head pages; /* Coalesced read requests */ | 729 | struct list_head pages; /* Coalesced read requests */ |
730 | struct nfs_page *req; /* multi ops per nfs_page */ | 730 | struct nfs_page *req; /* multi ops per nfs_page */ |
731 | struct page **pagevec; | 731 | struct page **pagevec; |
732 | unsigned int npages; /* active pages in pagevec */ | ||
732 | struct nfs_readargs args; | 733 | struct nfs_readargs args; |
733 | struct nfs_readres res; | 734 | struct nfs_readres res; |
734 | #ifdef CONFIG_NFS_V4 | 735 | #ifdef CONFIG_NFS_V4 |
@@ -747,6 +748,7 @@ struct nfs_write_data { | |||
747 | struct list_head pages; /* Coalesced requests we wish to flush */ | 748 | struct list_head pages; /* Coalesced requests we wish to flush */ |
748 | struct nfs_page *req; /* multi ops per nfs_page */ | 749 | struct nfs_page *req; /* multi ops per nfs_page */ |
749 | struct page **pagevec; | 750 | struct page **pagevec; |
751 | unsigned int npages; /* active pages in pagevec */ | ||
750 | struct nfs_writeargs args; /* argument struct */ | 752 | struct nfs_writeargs args; /* argument struct */ |
751 | struct nfs_writeres res; /* result struct */ | 753 | struct nfs_writeres res; /* result struct */ |
752 | #ifdef CONFIG_NFS_V4 | 754 | #ifdef CONFIG_NFS_V4 |