On Aug 25, 2015, at 2:41 PM, Trond Myklebust <trond.myklebust@xxxxxxxxxxxxxxx> wrote: > On Tue, Aug 25, 2015 at 2:18 PM, Chuck Lever <chuck.lever@xxxxxxxxxx> wrote: >> >> On Aug 25, 2015, at 1:04 PM, Trond Myklebust <trond.myklebust@xxxxxxxxxxxxxxx> wrote: >> >>> On Tue, Aug 25, 2015 at 12:31 PM, Chuck Lever <chuck.lever@xxxxxxxxxx> wrote: >>>> >>>> On Aug 20, 2015, at 9:59 PM, Trond Myklebust <trond.myklebust@xxxxxxxxxxxxxxx> wrote: >>>> >>>>> If the ctime or mtime or change attribute have changed because >>>>> of an operation we initiated, we should make sure that we force >>>>> an attribute update. However we do not want to mark the page cache >>>>> for revalidation. >>>> >>>> I've tested your linux-next branch (tip is aebbe9d73169 >>>> ("NFS41/flexfiles: zero out DS write wcc") against Solaris 12 >>>> with write delegation enabled (over RDMA, even!). >>>> >>>> I was not able to reproduce the write append failures I saw >>>> before. >>>> >>> >>> Perfect. Thanks for testing! >> >> Would it be possible to label some of these for stable? >> > > I think this one is the only one that is missing such a label. I've > reworded the commit message and pushed out a revised patch. Not related to the write append problem, but I've also seen missing opens on delegated files during reboot recovery in 4.0 kernels. Olga reported it before I got to it. Is that one also appropriate for stable? -- Chuck Lever -- 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