On 12/02/2016 12:28 PM, Trond Myklebust wrote: > >> On Dec 2, 2016, at 12:05, Anna Schumaker <schumaker.anna@xxxxxxxxx> wrote: >> >> Hi Trond, >> >> On 12/01/2016 05:06 PM, Trond Myklebust wrote: >>> We don't need to ask for the change attribute when returning a delegation >>> or recovering from a server reboot, and it could actually cause us to >>> obtain an incorrect value if we're using a pNFS flavour that requires >>> LAYOUTCOMMIT. >>> >>> Signed-off-by: Trond Myklebust <trond.myklebust@xxxxxxxxxxxxxxx> >>> --- >>> fs/nfs/nfs4proc.c | 1 - >>> 1 file changed, 1 deletion(-) >>> >>> diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c >>> index b582df89c083..c0628f78ed98 100644 >>> --- a/fs/nfs/nfs4proc.c >>> +++ b/fs/nfs/nfs4proc.c >>> @@ -226,7 +226,6 @@ static const u32 nfs4_pnfs_open_bitmap[3] = { >>> >>> static const u32 nfs4_open_noattr_bitmap[3] = { >>> FATTR4_WORD0_TYPE >>> - | FATTR4_WORD0_CHANGE >> >> Do these patches depend on another patch series? I'm having trouble applying this patch since my tree doesn't have FATTR4_WORD0_CHANGE yet. I'm having trouble with patch 4, too. >> > > I’ve pushed out the linux-next branch http://git.linux-nfs.org/?p=trondmy/linux-nfs.git;a=shortlog;h=refs/heads/linux-next I'll look there, then! Thanks! > >> Thanks, >> Anna >> >>> | FATTR4_WORD0_FILEID, >>> }; > -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html