Re: [PATCH v3 0/2] nfsd update mtime/ctime on CLONE/COPY with delegated attritutes

[Jeff Layton <jlayton@kernel.org>]

On Thu, 2026-04-09 at 12:43 -0400, Olga Kornievskaia wrote:
> generic/407 is failing in presence of delegated attributes and this patch series
> explores the solution where the CLONE compound returns updated mtime/ctime
> in the GETATTR op in the compound.
> 
> Both CLONE and COPY need to update destination file mtime/ctime
> (in presence of delegated timestamp) when file was modified.
> 
> v3 changes:
> --- Consolidated synchronous and asynchronous COPY patches into single
> patch.
> patch1: pulled out notify_change() code from nfsd4_finalize_deleg_timestamps as
> it's shared with nfsd_update_cmtime_attr and both paths log an error if
> notify_change() fails
> patch2: dentry can be pulled from copy->nf_dst->nf_file.fh_path.dentry and
> no longer requires additional tracking.
> 
> Olga Kornievskaia (2):
>   nfsd: update mtime/ctime on CLONE in presense of delegated attributes
>   nfsd: update mtime/ctime on COPY in presence of delegated attributes
> 
>  fs/nfsd/nfs4proc.c  | 30 +++++++++++++++++++++++++++++-
>  fs/nfsd/nfs4state.c | 14 +-------------
>  fs/nfsd/state.h     | 16 ++++++++++++++++
>  3 files changed, 46 insertions(+), 14 deletions(-)

Nice work.

Reviewed-by: Jeff Layton <jlayton@kernel.org>