From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Layton Subject: Re: [PATCH v2 19/19] sunrpc: Disable splice for krb5i Date: Thu, 22 Jun 2017 13:13:03 -0400 Message-ID: <1498151583.30388.7.camel@redhat.com> References: <20170616151535.14210.34926.stgit@klimt.1015granger.net> <20170616152254.14210.48071.stgit@klimt.1015granger.net> <20170617150705.GA27917@infradead.org> <20170618073438.GA15358@infradead.org> <14AD4311-5092-441C-8A3B-F7EC7E801358@oracle.com> <20170619153018.GA2667@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20170619153018.GA2667-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org> Sender: linux-nfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Christoph Hellwig , Chuck Lever Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Linux NFS Mailing List List-Id: linux-rdma@vger.kernel.org On Mon, 2017-06-19 at 08:30 -0700, Christoph Hellwig wrote: > On Mon, Jun 19, 2017 at 11:22:41AM -0400, Chuck Lever wrote: > > I'm going to submit this again with a comment change and > > Jeff's Reviewed-by. Can I add your Reviewed-by as well? > > It looks ok as a quick fix for me but at least needs a big comment. > I'd also be much happier if we could find and fix the root cause. Don't we already know the root cause? There's nothing that holds the data stable, so we calculate the checksum but by the time we get to transmitting the frame it's no longer valid. Is there something else that should be preventing this? In any case, I do agree that some comments explaining this would be a good thing. -- Jeff Layton -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html