mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-05-16 00:51:51 -04:00
nfsd: update mtime/ctime on COPY in presence of delegated attributes
When delegated attributes are given on open, the file is opened with
NOCMTIME and modifying operations do not update mtime/ctime as to not get
out-of-sync with the client's delegated view. However, for COPY operation,
the server should update its view of mtime/ctime and reflect that in any
GETATTR queries.
Fixes: e5e9b24ab8 ("nfsd: freeze c/mtime updates with outstanding WRITE_ATTRS delegation")
Cc: stable@vger.kernel.org
Signed-off-by: Olga Kornievskaia <okorniev@redhat.com>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
This commit is contained in:
committed by
Chuck Lever
parent
2863bac7f4
commit
4183cf383b
@@ -2121,8 +2121,10 @@ static int nfsd4_do_async_copy(void *data)
|
||||
|
||||
set_bit(NFSD4_COPY_F_COMPLETED, ©->cp_flags);
|
||||
trace_nfsd_copy_async_done(copy);
|
||||
nfsd4_send_cb_offload(copy);
|
||||
atomic_dec(©->cp_nn->pending_async_copies);
|
||||
if (copy->cp_res.wr_bytes_written > 0 && copy->attr_update)
|
||||
nfsd_update_cmtime_attr(copy->nf_dst->nf_file, 0);
|
||||
nfsd4_send_cb_offload(copy);
|
||||
return 0;
|
||||
}
|
||||
|
||||
@@ -2182,6 +2184,9 @@ nfsd4_copy(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
|
||||
memcpy(&result->cb_stateid, ©->cp_stateid.cs_stid,
|
||||
sizeof(result->cb_stateid));
|
||||
dup_copy_fields(copy, async_copy);
|
||||
if ((READ_ONCE(copy->nf_dst->nf_file->f_mode) &
|
||||
FMODE_NOCMTIME) != 0)
|
||||
async_copy->attr_update = true;
|
||||
memcpy(async_copy->cp_cb_offload.co_referring_sessionid.data,
|
||||
cstate->session->se_sessionid.data,
|
||||
NFS4_MAX_SESSIONID_LEN);
|
||||
@@ -2200,6 +2205,10 @@ nfsd4_copy(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
|
||||
} else {
|
||||
status = nfsd4_do_copy(copy, copy->nf_src->nf_file,
|
||||
copy->nf_dst->nf_file, true);
|
||||
if ((READ_ONCE(copy->nf_dst->nf_file->f_mode) &
|
||||
FMODE_NOCMTIME) != 0 &&
|
||||
copy->cp_res.wr_bytes_written > 0)
|
||||
nfsd_update_cmtime_attr(copy->nf_dst->nf_file, 0);
|
||||
}
|
||||
out:
|
||||
trace_nfsd_copy_done(copy, status);
|
||||
|
||||
@@ -752,6 +752,7 @@ struct nfsd4_copy {
|
||||
|
||||
struct nfsd_file *nf_src;
|
||||
struct nfsd_file *nf_dst;
|
||||
bool attr_update;
|
||||
|
||||
copy_stateid_t cp_stateid;
|
||||
|
||||
|
||||
Reference in New Issue
Block a user