Thanks Bruce On Mon, Nov 6, 2017 at 8:49 PM, Bruce Fields <bfields@xxxxxxxxxxxx> wrote: > 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-rdma" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- 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
