* + svcrdma-check-num_sge-when-setting-last_ctxt-bit.patch added to -mm tree
@ 2008-03-26 6:39 akpm
0 siblings, 0 replies; only message in thread
From: akpm @ 2008-03-26 6:39 UTC (permalink / raw)
To: mm-commits; +Cc: tom, bfields, rolandd
The patch titled
svcrdma: check num_sge when setting LAST_CTXT bit
has been added to the -mm tree. Its filename is
svcrdma-check-num_sge-when-setting-last_ctxt-bit.patch
Before you just go and hit "reply", please:
a) Consider who else should be cc'ed
b) Prefer to cc a suitable mailing list as well
c) Ideally: find the original patch on the mailing list and do a
reply-to-all to that, adding suitable additional cc's
*** Remember to use Documentation/SubmitChecklist when testing your code ***
See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find
out what to do about this
The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/
------------------------------------------------------
Subject: svcrdma: check num_sge when setting LAST_CTXT bit
From: Tom Tucker <tom@opengridcomputing.com>
The RDMACTXT_F_LAST_CTXT bit was getting set incorrectly when the last chunk
in the read-list spanned multiple pages. This resulted in a kernel panic when
the wrong context was used to build the RPC iovec page list.
RDMA_READ is used to fetch RPC data from the client for NFS_WRITE requests. A
scatter-gather is used to map the advertised client side buffer to the
server-side iovec and associated page list.
WR contexts are used to convey which scatter-gather entries are handled by
each WR. When the write data is large, a single RPC may require multiple
RDMA_READ requests so the contexts for a single RPC are chained together in a
linked list. The last context in this list is marked with a bit
RDMACTXT_F_LAST_CTXT so that when this WR completes, the CQ handler code can
enqueue the RPC for processing.
The code in rdma_read_xdr was setting this bit on the last two contexts on
this list when the last read-list chunk spanned multiple pages. This caused
the svc_rdma_recvfrom logic to incorrectly build the RPC and caused the kernel
to crash because the second-to-last context doesn't contain the iovec page
list.
Modified the condition that sets this bit so that it correctly detects the
last context for the RPC.
Signed-off-by: Tom Tucker <tom@opengridcomputing.com>
Tested-by: Roland Dreier <rolandd@cisco.com>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 21 +++++++++++----------
1 file changed, 11 insertions(+), 10 deletions(-)
diff -puN net/sunrpc/xprtrdma/svc_rdma_recvfrom.c~svcrdma-check-num_sge-when-setting-last_ctxt-bit net/sunrpc/xprtrdma/svc_rdma_recvfrom.c
--- a/net/sunrpc/xprtrdma/svc_rdma_recvfrom.c~svcrdma-check-num_sge-when-setting-last_ctxt-bit
+++ a/net/sunrpc/xprtrdma/svc_rdma_recvfrom.c
@@ -322,15 +322,6 @@ next_sge:
ctxt->direction = DMA_FROM_DEVICE;
clear_bit(RDMACTXT_F_READ_DONE, &ctxt->flags);
clear_bit(RDMACTXT_F_LAST_CTXT, &ctxt->flags);
- if ((ch+1)->rc_discrim == 0) {
- /*
- * Checked in sq_cq_reap to see if we need to
- * be enqueued
- */
- set_bit(RDMACTXT_F_LAST_CTXT, &ctxt->flags);
- ctxt->next = hdr_ctxt;
- hdr_ctxt->next = head;
- }
/* Prepare READ WR */
memset(&read_wr, 0, sizeof read_wr);
@@ -348,7 +339,17 @@ next_sge:
rdma_set_ctxt_sge(ctxt, &sge[ch_sge_ary[ch_no].start],
&sgl_offset,
read_wr.num_sge);
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2008-03-26 6:40 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-03-26 6:39 + svcrdma-check-num_sge-when-setting-last_ctxt-bit.patch added to -mm tree akpm
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.