diff options
Diffstat (limited to 'fs/nfs')
-rw-r--r-- | fs/nfs/nfs4proc.c | 15 | ||||
-rw-r--r-- | fs/nfs/nfs4state.c | 3 | ||||
-rw-r--r-- | fs/nfs/nfs4trace.h | 2 | ||||
-rw-r--r-- | fs/nfs/write.c | 14 |
4 files changed, 23 insertions, 11 deletions
diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c index f93b9cdb4934..5133bb18830e 100644 --- a/fs/nfs/nfs4proc.c +++ b/fs/nfs/nfs4proc.c | |||
@@ -1458,12 +1458,18 @@ nfs4_opendata_check_deleg(struct nfs4_opendata *data, struct nfs4_state *state) | |||
1458 | if (delegation) | 1458 | if (delegation) |
1459 | delegation_flags = delegation->flags; | 1459 | delegation_flags = delegation->flags; |
1460 | rcu_read_unlock(); | 1460 | rcu_read_unlock(); |
1461 | if (data->o_arg.claim == NFS4_OPEN_CLAIM_DELEGATE_CUR) { | 1461 | switch (data->o_arg.claim) { |
1462 | default: | ||
1463 | break; | ||
1464 | case NFS4_OPEN_CLAIM_DELEGATE_CUR: | ||
1465 | case NFS4_OPEN_CLAIM_DELEG_CUR_FH: | ||
1462 | pr_err_ratelimited("NFS: Broken NFSv4 server %s is " | 1466 | pr_err_ratelimited("NFS: Broken NFSv4 server %s is " |
1463 | "returning a delegation for " | 1467 | "returning a delegation for " |
1464 | "OPEN(CLAIM_DELEGATE_CUR)\n", | 1468 | "OPEN(CLAIM_DELEGATE_CUR)\n", |
1465 | clp->cl_hostname); | 1469 | clp->cl_hostname); |
1466 | } else if ((delegation_flags & 1UL<<NFS_DELEGATION_NEED_RECLAIM) == 0) | 1470 | return; |
1471 | } | ||
1472 | if ((delegation_flags & 1UL<<NFS_DELEGATION_NEED_RECLAIM) == 0) | ||
1467 | nfs_inode_set_delegation(state->inode, | 1473 | nfs_inode_set_delegation(state->inode, |
1468 | data->owner->so_cred, | 1474 | data->owner->so_cred, |
1469 | &data->o_res); | 1475 | &data->o_res); |
@@ -1771,6 +1777,9 @@ int nfs4_open_delegation_recall(struct nfs_open_context *ctx, | |||
1771 | if (IS_ERR(opendata)) | 1777 | if (IS_ERR(opendata)) |
1772 | return PTR_ERR(opendata); | 1778 | return PTR_ERR(opendata); |
1773 | nfs4_stateid_copy(&opendata->o_arg.u.delegation, stateid); | 1779 | nfs4_stateid_copy(&opendata->o_arg.u.delegation, stateid); |
1780 | write_seqlock(&state->seqlock); | ||
1781 | nfs4_stateid_copy(&state->stateid, &state->open_stateid); | ||
1782 | write_sequnlock(&state->seqlock); | ||
1774 | clear_bit(NFS_DELEGATED_STATE, &state->flags); | 1783 | clear_bit(NFS_DELEGATED_STATE, &state->flags); |
1775 | switch (type & (FMODE_READ|FMODE_WRITE)) { | 1784 | switch (type & (FMODE_READ|FMODE_WRITE)) { |
1776 | case FMODE_READ|FMODE_WRITE: | 1785 | case FMODE_READ|FMODE_WRITE: |
@@ -1863,6 +1872,8 @@ static int _nfs4_proc_open_confirm(struct nfs4_opendata *data) | |||
1863 | data->rpc_done = 0; | 1872 | data->rpc_done = 0; |
1864 | data->rpc_status = 0; | 1873 | data->rpc_status = 0; |
1865 | data->timestamp = jiffies; | 1874 | data->timestamp = jiffies; |
1875 | if (data->is_recover) | ||
1876 | nfs4_set_sequence_privileged(&data->c_arg.seq_args); | ||
1866 | task = rpc_run_task(&task_setup_data); | 1877 | task = rpc_run_task(&task_setup_data); |
1867 | if (IS_ERR(task)) | 1878 | if (IS_ERR(task)) |
1868 | return PTR_ERR(task); | 1879 | return PTR_ERR(task); |
diff --git a/fs/nfs/nfs4state.c b/fs/nfs/nfs4state.c index 5db324635e92..d854693a15b0 100644 --- a/fs/nfs/nfs4state.c +++ b/fs/nfs/nfs4state.c | |||
@@ -1725,7 +1725,8 @@ restart: | |||
1725 | if (!test_and_clear_bit(ops->owner_flag_bit, | 1725 | if (!test_and_clear_bit(ops->owner_flag_bit, |
1726 | &sp->so_flags)) | 1726 | &sp->so_flags)) |
1727 | continue; | 1727 | continue; |
1728 | atomic_inc(&sp->so_count); | 1728 | if (!atomic_inc_not_zero(&sp->so_count)) |
1729 | continue; | ||
1729 | spin_unlock(&clp->cl_lock); | 1730 | spin_unlock(&clp->cl_lock); |
1730 | rcu_read_unlock(); | 1731 | rcu_read_unlock(); |
1731 | 1732 | ||
diff --git a/fs/nfs/nfs4trace.h b/fs/nfs/nfs4trace.h index 28df12e525ba..671cf68fe56b 100644 --- a/fs/nfs/nfs4trace.h +++ b/fs/nfs/nfs4trace.h | |||
@@ -409,7 +409,7 @@ DECLARE_EVENT_CLASS(nfs4_open_event, | |||
409 | __entry->flags = flags; | 409 | __entry->flags = flags; |
410 | __entry->fmode = (__force unsigned int)ctx->mode; | 410 | __entry->fmode = (__force unsigned int)ctx->mode; |
411 | __entry->dev = ctx->dentry->d_sb->s_dev; | 411 | __entry->dev = ctx->dentry->d_sb->s_dev; |
412 | if (!IS_ERR(state)) | 412 | if (!IS_ERR_OR_NULL(state)) |
413 | inode = state->inode; | 413 | inode = state->inode; |
414 | if (inode != NULL) { | 414 | if (inode != NULL) { |
415 | __entry->fileid = NFS_FILEID(inode); | 415 | __entry->fileid = NFS_FILEID(inode); |
diff --git a/fs/nfs/write.c b/fs/nfs/write.c index 72624dc4a623..75ab7622e0cc 100644 --- a/fs/nfs/write.c +++ b/fs/nfs/write.c | |||
@@ -569,19 +569,17 @@ static int nfs_page_async_flush(struct nfs_pageio_descriptor *pgio, | |||
569 | if (!nfs_pageio_add_request(pgio, req)) { | 569 | if (!nfs_pageio_add_request(pgio, req)) { |
570 | nfs_redirty_request(req); | 570 | nfs_redirty_request(req); |
571 | ret = pgio->pg_error; | 571 | ret = pgio->pg_error; |
572 | } | 572 | } else |
573 | nfs_add_stats(page_file_mapping(page)->host, | ||
574 | NFSIOS_WRITEPAGES, 1); | ||
573 | out: | 575 | out: |
574 | return ret; | 576 | return ret; |
575 | } | 577 | } |
576 | 578 | ||
577 | static int nfs_do_writepage(struct page *page, struct writeback_control *wbc, struct nfs_pageio_descriptor *pgio) | 579 | static int nfs_do_writepage(struct page *page, struct writeback_control *wbc, struct nfs_pageio_descriptor *pgio) |
578 | { | 580 | { |
579 | struct inode *inode = page_file_mapping(page)->host; | ||
580 | int ret; | 581 | int ret; |
581 | 582 | ||
582 | nfs_inc_stats(inode, NFSIOS_VFSWRITEPAGE); | ||
583 | nfs_add_stats(inode, NFSIOS_WRITEPAGES, 1); | ||
584 | |||
585 | nfs_pageio_cond_complete(pgio, page_file_index(page)); | 583 | nfs_pageio_cond_complete(pgio, page_file_index(page)); |
586 | ret = nfs_page_async_flush(pgio, page, wbc->sync_mode == WB_SYNC_NONE); | 584 | ret = nfs_page_async_flush(pgio, page, wbc->sync_mode == WB_SYNC_NONE); |
587 | if (ret == -EAGAIN) { | 585 | if (ret == -EAGAIN) { |
@@ -597,9 +595,11 @@ static int nfs_do_writepage(struct page *page, struct writeback_control *wbc, st | |||
597 | static int nfs_writepage_locked(struct page *page, struct writeback_control *wbc) | 595 | static int nfs_writepage_locked(struct page *page, struct writeback_control *wbc) |
598 | { | 596 | { |
599 | struct nfs_pageio_descriptor pgio; | 597 | struct nfs_pageio_descriptor pgio; |
598 | struct inode *inode = page_file_mapping(page)->host; | ||
600 | int err; | 599 | int err; |
601 | 600 | ||
602 | nfs_pageio_init_write(&pgio, page->mapping->host, wb_priority(wbc), | 601 | nfs_inc_stats(inode, NFSIOS_VFSWRITEPAGE); |
602 | nfs_pageio_init_write(&pgio, inode, wb_priority(wbc), | ||
603 | false, &nfs_async_write_completion_ops); | 603 | false, &nfs_async_write_completion_ops); |
604 | err = nfs_do_writepage(page, wbc, &pgio); | 604 | err = nfs_do_writepage(page, wbc, &pgio); |
605 | nfs_pageio_complete(&pgio); | 605 | nfs_pageio_complete(&pgio); |
@@ -1223,7 +1223,7 @@ static int nfs_can_extend_write(struct file *file, struct page *page, struct ino | |||
1223 | return 1; | 1223 | return 1; |
1224 | if (!flctx || (list_empty_careful(&flctx->flc_flock) && | 1224 | if (!flctx || (list_empty_careful(&flctx->flc_flock) && |
1225 | list_empty_careful(&flctx->flc_posix))) | 1225 | list_empty_careful(&flctx->flc_posix))) |
1226 | return 0; | 1226 | return 1; |
1227 | 1227 | ||
1228 | /* Check to see if there are whole file write locks */ | 1228 | /* Check to see if there are whole file write locks */ |
1229 | ret = 0; | 1229 | ret = 0; |