Return-Path: linux-nfs-owner@vger.kernel.org Received: from aserp1040.oracle.com ([141.146.126.69]:34117 "EHLO aserp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751581AbbAPVEG convert rfc822-to-8bit (ORCPT ); Fri, 16 Jan 2015 16:04:06 -0500 Content-Type: text/plain; charset=windows-1252 Mime-Version: 1.0 (Mac OS X Mail 7.3 \(1878.6\)) Subject: Re: [PATCH v2 00/20] NFS/RDMA client for 3.20 From: Chuck Lever In-Reply-To: <54B97C73.9000602@Netapp.com> Date: Fri, 16 Jan 2015 16:04:01 -0500 Cc: linux-rdma , Linux NFS Mailing List Message-Id: <59D943E3-C7F8-48C4-BDB6-8309E74D4EDB@oracle.com> References: <20150113161440.14086.24801.stgit@manet.1015granger.net> <54B97C73.9000602@Netapp.com> To: Anna Schumaker Sender: linux-nfs-owner@vger.kernel.org List-ID: On Jan 16, 2015, at 4:02 PM, Anna Schumaker wrote: > Hi Chuck, > > You already answered my one question for this patch series, and I don't have any other comments. Are these ready for me to take now? Are you planning a v3? Yes, one or two minor changes. I?ll post late next week (after MLK day). > > Thanks, > Anna > > On 01/13/2015 11:24 AM, Chuck Lever wrote: >> The following series of patches for the Linux NFS client breaks up >> the per-transport buffer pool data structures to help them scale >> better with the size of SEND/RECV buffers (the inline threshold), >> the maximum NFS r/wsize, and the number of RDMA credits (concurrent >> RPC requests). >> >> The primary change is that the header send buffers have been split >> from struct rpcrdma_req. Specific benefits are outlined in the patch >> descriptions. >> >> More pre-requisites are required. Changes to raise the maximum >> r/wsize and other limits are left for a future merge window. >> >> See the topic branch "nfs-rdma-for-3.20" at: >> >> git://git.linux-nfs.org/projects/cel/cel-2.6.git >> >> Changes since v1: >> - Rebased on v3.19-rc4 >> - One short description fixed >> - linux-rdma included this time (sorry for the noise) >> >> --- >> >> Chuck Lever (20): >> xprtrdma: human-readable completion status >> xprtrdma: Modernize htonl and ntohl >> xprtrdma: Display XIDs in host byte order >> xprtrdma: Clean up hdrlen >> xprtrdma: Rename "xprt" and "rdma_connect" fields in struct rpcrdma_xprt >> xprtrdma: Remove rpcrdma_ep::rep_ia >> xprtrdma: Remove rl_mr field, and the mr_chunk union >> xprtrdma: Move credit update to RPC reply handler >> xprtrdma: Remove rpcrdma_ep::rep_func and ::rep_xprt >> xprtrdma: Free the pd if ib_query_qp() fails >> xprtrdma: Take struct ib_device_attr off the stack >> xprtrdma: Take struct ib_qp_attr and ib_qp_init_attr off the stack >> xprtrdma: Simplify synopsis of rpcrdma_buffer_create() >> xprtrdma: Refactor rpcrdma_buffer_create() and rpcrdma_buffer_destroy() >> xprtrdma: Add struct rpcrdma_regbuf and helpers >> xprtrdma: Allocate RPC send buffer separately from struct rpcrdma_req >> xprtrdma: Allocate RPC/RDMA send buffer separately from struct rpcrdma_req >> xprtrdma: Allocate RPC/RDMA receive buffer separately from struct rpcrdma_rep >> xprtrdma: Allocate zero pad separately from rpcrdma_buffer >> xprtrdma: Clean up after adding regbuf management >> >> >> include/linux/sunrpc/rpc_rdma.h | 14 + >> include/linux/sunrpc/svc_rdma.h | 2 >> net/sunrpc/xprtrdma/rpc_rdma.c | 108 ++++++---- >> net/sunrpc/xprtrdma/transport.c | 179 +++++++---------- >> net/sunrpc/xprtrdma/verbs.c | 411 ++++++++++++++++++++++++--------------- >> net/sunrpc/xprtrdma/xprt_rdma.h | 111 +++++++---- >> 6 files changed, 478 insertions(+), 347 deletions(-) >> >> -- >> Chuck Lever >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in >> the body of a message to majordomo@vger.kernel.org >> 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@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Chuck Lever chuck[dot]lever[at]oracle[dot]com