On Fri, Nov 03, 2017 at 05:19:30PM -0400, Chuck Lever wrote: > Hi Bruce- > > > On Oct 28, 2017, at 3:43 AM, Devesh Sharma <devesh.sharma@xxxxxxxxxxxx> wrote: > > > > looks good > > > > Reveiwed-By: Devesh Sharma <devesh.sharma@xxxxxxxxxxxx> > > > > On Fri, Oct 27, 2017 at 8:19 PM, Chuck Lever <chuck.lever@xxxxxxxxxx> wrote: > >> I noticed the server was sometimes not closing the connection after > >> a flushed Send. For example, if the client responds with an RNR NAK > >> to a Reply from the server, that client might be deadlocked, and > >> thus wouldn't send any more traffic. Thus the server wouldn't have > >> any opportunity to notice the XPT_CLOSE bit has been set. > >> > >> Enqueue the transport so that svcxprt notices the bit even if there > >> is no more transport activity after a flushed completion, QP access > >> error, or device removal event. > >> > >> Signed-off-by: Chuck Lever <chuck.lever@xxxxxxxxxx> > >> --- > >> net/sunrpc/xprtrdma/svc_rdma_transport.c | 11 ++++++++--- > >> 1 file changed, 8 insertions(+), 3 deletions(-) > >> > >> Hi Bruce- > >> > >> Please consider this patch for v4.15. Thanks! > > I notice nfsd-next does not have this patch. Is there anything > else I need to do? Should I resend with Devesh's Reviewed-by > tag? Thanks for the reminder; applied for 4.15 with Devesh's Reviewed-by. --b. -- 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
