diff options
| author | J. Bruce Fields <bfields@redhat.com> | 2010-08-26 13:22:27 -0400 |
|---|---|---|
| committer | J. Bruce Fields <bfields@redhat.com> | 2010-08-26 13:22:27 -0400 |
| commit | f632265d0ffb5acf331252d98c64939849d96bb2 (patch) | |
| tree | 31187d9a726bf1ca6ca12e26ad8e7c609eaf4d8b /fs/nfs/write.c | |
| parent | 7d94784293096c0a46897acdb83be5abd9278ece (diff) | |
| parent | da5cabf80e2433131bf0ed8993abc0f7ea618c73 (diff) | |
Merge commit 'v2.6.36-rc1' into HEAD
Diffstat (limited to 'fs/nfs/write.c')
| -rw-r--r-- | fs/nfs/write.c | 39 |
1 files changed, 24 insertions, 15 deletions
diff --git a/fs/nfs/write.c b/fs/nfs/write.c index 91679e2631ee..874972d9427c 100644 --- a/fs/nfs/write.c +++ b/fs/nfs/write.c | |||
| @@ -222,7 +222,7 @@ static void nfs_end_page_writeback(struct page *page) | |||
| 222 | clear_bdi_congested(&nfss->backing_dev_info, BLK_RW_ASYNC); | 222 | clear_bdi_congested(&nfss->backing_dev_info, BLK_RW_ASYNC); |
| 223 | } | 223 | } |
| 224 | 224 | ||
| 225 | static struct nfs_page *nfs_find_and_lock_request(struct page *page) | 225 | static struct nfs_page *nfs_find_and_lock_request(struct page *page, bool nonblock) |
| 226 | { | 226 | { |
| 227 | struct inode *inode = page->mapping->host; | 227 | struct inode *inode = page->mapping->host; |
| 228 | struct nfs_page *req; | 228 | struct nfs_page *req; |
| @@ -241,7 +241,10 @@ static struct nfs_page *nfs_find_and_lock_request(struct page *page) | |||
| 241 | * request as dirty (in which case we don't care). | 241 | * request as dirty (in which case we don't care). |
| 242 | */ | 242 | */ |
| 243 | spin_unlock(&inode->i_lock); | 243 | spin_unlock(&inode->i_lock); |
| 244 | ret = nfs_wait_on_request(req); | 244 | if (!nonblock) |
| 245 | ret = nfs_wait_on_request(req); | ||
| 246 | else | ||
| 247 | ret = -EAGAIN; | ||
| 245 | nfs_release_request(req); | 248 | nfs_release_request(req); |
| 246 | if (ret != 0) | 249 | if (ret != 0) |
| 247 | return ERR_PTR(ret); | 250 | return ERR_PTR(ret); |
| @@ -256,12 +259,12 @@ static struct nfs_page *nfs_find_and_lock_request(struct page *page) | |||
| 256 | * May return an error if the user signalled nfs_wait_on_request(). | 259 | * May return an error if the user signalled nfs_wait_on_request(). |
| 257 | */ | 260 | */ |
| 258 | static int nfs_page_async_flush(struct nfs_pageio_descriptor *pgio, | 261 | static int nfs_page_async_flush(struct nfs_pageio_descriptor *pgio, |
| 259 | struct page *page) | 262 | struct page *page, bool nonblock) |
| 260 | { | 263 | { |
| 261 | struct nfs_page *req; | 264 | struct nfs_page *req; |
| 262 | int ret = 0; | 265 | int ret = 0; |
| 263 | 266 | ||
| 264 | req = nfs_find_and_lock_request(page); | 267 | req = nfs_find_and_lock_request(page, nonblock); |
| 265 | if (!req) | 268 | if (!req) |
| 266 | goto out; | 269 | goto out; |
| 267 | ret = PTR_ERR(req); | 270 | ret = PTR_ERR(req); |
| @@ -283,12 +286,20 @@ out: | |||
| 283 | static int nfs_do_writepage(struct page *page, struct writeback_control *wbc, struct nfs_pageio_descriptor *pgio) | 286 | static int nfs_do_writepage(struct page *page, struct writeback_control *wbc, struct nfs_pageio_descriptor *pgio) |
| 284 | { | 287 | { |
| 285 | struct inode *inode = page->mapping->host; | 288 | struct inode *inode = page->mapping->host; |
| 289 | int ret; | ||
| 286 | 290 | ||
| 287 | nfs_inc_stats(inode, NFSIOS_VFSWRITEPAGE); | 291 | nfs_inc_stats(inode, NFSIOS_VFSWRITEPAGE); |
| 288 | nfs_add_stats(inode, NFSIOS_WRITEPAGES, 1); | 292 | nfs_add_stats(inode, NFSIOS_WRITEPAGES, 1); |
| 289 | 293 | ||
| 290 | nfs_pageio_cond_complete(pgio, page->index); | 294 | nfs_pageio_cond_complete(pgio, page->index); |
| 291 | return nfs_page_async_flush(pgio, page); | 295 | ret = nfs_page_async_flush(pgio, page, |
| 296 | wbc->sync_mode == WB_SYNC_NONE || | ||
| 297 | wbc->nonblocking != 0); | ||
| 298 | if (ret == -EAGAIN) { | ||
| 299 | redirty_page_for_writepage(wbc, page); | ||
| 300 | ret = 0; | ||
| 301 | } | ||
| 302 | return ret; | ||
| 292 | } | 303 | } |
| 293 | 304 | ||
| 294 | /* | 305 | /* |
| @@ -689,7 +700,9 @@ int nfs_flush_incompatible(struct file *file, struct page *page) | |||
| 689 | req = nfs_page_find_request(page); | 700 | req = nfs_page_find_request(page); |
| 690 | if (req == NULL) | 701 | if (req == NULL) |
| 691 | return 0; | 702 | return 0; |
| 692 | do_flush = req->wb_page != page || req->wb_context != ctx; | 703 | do_flush = req->wb_page != page || req->wb_context != ctx || |
| 704 | req->wb_lock_context->lockowner != current->files || | ||
| 705 | req->wb_lock_context->pid != current->tgid; | ||
| 693 | nfs_release_request(req); | 706 | nfs_release_request(req); |
| 694 | if (!do_flush) | 707 | if (!do_flush) |
| 695 | return 0; | 708 | return 0; |
| @@ -813,6 +826,7 @@ static int nfs_write_rpcsetup(struct nfs_page *req, | |||
| 813 | data->args.pages = data->pagevec; | 826 | data->args.pages = data->pagevec; |
| 814 | data->args.count = count; | 827 | data->args.count = count; |
| 815 | data->args.context = get_nfs_open_context(req->wb_context); | 828 | data->args.context = get_nfs_open_context(req->wb_context); |
| 829 | data->args.lock_context = req->wb_lock_context; | ||
| 816 | data->args.stable = NFS_UNSTABLE; | 830 | data->args.stable = NFS_UNSTABLE; |
| 817 | if (how & FLUSH_STABLE) { | 831 | if (how & FLUSH_STABLE) { |
| 818 | data->args.stable = NFS_DATA_SYNC; | 832 | data->args.stable = NFS_DATA_SYNC; |
| @@ -1036,9 +1050,9 @@ out: | |||
| 1036 | void nfs_write_prepare(struct rpc_task *task, void *calldata) | 1050 | void nfs_write_prepare(struct rpc_task *task, void *calldata) |
| 1037 | { | 1051 | { |
| 1038 | struct nfs_write_data *data = calldata; | 1052 | struct nfs_write_data *data = calldata; |
| 1039 | struct nfs_client *clp = (NFS_SERVER(data->inode))->nfs_client; | ||
| 1040 | 1053 | ||
| 1041 | if (nfs4_setup_sequence(clp, &data->args.seq_args, | 1054 | if (nfs4_setup_sequence(NFS_SERVER(data->inode), |
| 1055 | &data->args.seq_args, | ||
| 1042 | &data->res.seq_res, 1, task)) | 1056 | &data->res.seq_res, 1, task)) |
| 1043 | return; | 1057 | return; |
| 1044 | rpc_call_start(task); | 1058 | rpc_call_start(task); |
| @@ -1379,7 +1393,7 @@ static const struct rpc_call_ops nfs_commit_ops = { | |||
| 1379 | .rpc_release = nfs_commit_release, | 1393 | .rpc_release = nfs_commit_release, |
| 1380 | }; | 1394 | }; |
| 1381 | 1395 | ||
| 1382 | static int nfs_commit_inode(struct inode *inode, int how) | 1396 | int nfs_commit_inode(struct inode *inode, int how) |
| 1383 | { | 1397 | { |
| 1384 | LIST_HEAD(head); | 1398 | LIST_HEAD(head); |
| 1385 | int may_wait = how & FLUSH_SYNC; | 1399 | int may_wait = how & FLUSH_SYNC; |
| @@ -1443,11 +1457,6 @@ out_mark_dirty: | |||
| 1443 | return ret; | 1457 | return ret; |
| 1444 | } | 1458 | } |
| 1445 | #else | 1459 | #else |
| 1446 | static int nfs_commit_inode(struct inode *inode, int how) | ||
| 1447 | { | ||
| 1448 | return 0; | ||
| 1449 | } | ||
| 1450 | |||
| 1451 | static int nfs_commit_unstable_pages(struct inode *inode, struct writeback_control *wbc) | 1460 | static int nfs_commit_unstable_pages(struct inode *inode, struct writeback_control *wbc) |
| 1452 | { | 1461 | { |
| 1453 | return 0; | 1462 | return 0; |
| @@ -1546,7 +1555,7 @@ int nfs_migrate_page(struct address_space *mapping, struct page *newpage, | |||
| 1546 | 1555 | ||
| 1547 | nfs_fscache_release_page(page, GFP_KERNEL); | 1556 | nfs_fscache_release_page(page, GFP_KERNEL); |
| 1548 | 1557 | ||
| 1549 | req = nfs_find_and_lock_request(page); | 1558 | req = nfs_find_and_lock_request(page, false); |
| 1550 | ret = PTR_ERR(req); | 1559 | ret = PTR_ERR(req); |
| 1551 | if (IS_ERR(req)) | 1560 | if (IS_ERR(req)) |
| 1552 | goto out; | 1561 | goto out; |
