From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Steve Wise" Subject: RE: [PATCH 2/3] svcrdma: handle rdma read with a non-zero initial page offset Date: Mon, 28 Sep 2015 16:49:21 -0500 Message-ID: <009001d0fa37$8871a980$9954fc80$@opengridcomputing.com> References: <20150921172423.9761.92399.stgit@build2.ogc.int> <20150921172428.9761.27838.stgit@build2.ogc.int> <56094F3D.2030002@opengridcomputing.com> <20150928210459.GC3190@fieldses.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20150928210459.GC3190-uC3wQj2KruNg9hUCZPvPmw@public.gmane.org> Content-Language: en-us Sender: linux-nfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: "'J. Bruce Fields'" Cc: trond.myklebust-7I+n7zu2hftEKMMhf/gKZA@public.gmane.org, linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-rdma@vger.kernel.org > -----Original Message----- > From: J. Bruce Fields [mailto:bfields-uC3wQj2KruNg9hUCZPvPmw@public.gmane.org] > Sent: Monday, September 28, 2015 4:05 PM > To: Steve Wise > Cc: trond.myklebust-7I+n7zu2hftEKMMhf/gKZA@public.gmane.org; linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > Subject: Re: [PATCH 2/3] svcrdma: handle rdma read with a non-zero initial page offset > > On Mon, Sep 28, 2015 at 09:31:25AM -0500, Steve Wise wrote: > > On 9/21/2015 12:24 PM, Steve Wise wrote: > > >The server rdma_read_chunk_lcl() and rdma_read_chunk_frmr() functions > > >were not taking into account the initial page_offset when determining > > >the rdma read length. This resulted in a read who's starting address > > >and length exceeded the base/bounds of the frmr. > > > > > >Most work loads don't tickle this bug apparently, but one test hit it > > >every time: building the linux kernel on a 16 core node with 'make -j > > >16 O=/mnt/0' where /mnt/0 is a ramdisk mounted via NFSRDMA. > > > > > >This bug seems to only be tripped with devices having small fastreg page > > >list depths. I didn't see it with mlx4, for instance. > > > > > >Fixes: 0bf4828983df ('svcrdma: refactor marshalling logic') > > >Signed-off-by: Steve Wise > > >Tested-by: Chuck Lever > > >--- > > > > > > > > > > Hey Bruce, can this make 4.3-rc? Also, what do you think about > > pushing it to stable? > > It looks like a reasonable candidate for stable. Apologies, somehow I > missed it when you posted it--would you mind resending? > > --b. resent this one patch. What is your process for pushing to stable? Thanks, Steve. -- 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