Commit 15303d9e authored by Chuck Lever's avatar Chuck Lever Committed by Anna Schumaker

xprtrdma: Recognize XDRBUF_SPARSE_PAGES

Commit 431f6eb3 ("SUNRPC: Add a label for RPC calls that require
allocation on receive") didn't update similar logic in rpc_rdma.c.
I don't think this is a bug, per-se; the commit just adds more
careful checking for broken upper layer behavior.
Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
Signed-off-by: default avatarAnna Schumaker <Anna.Schumaker@Netapp.com>
parent 0dfbb5f0
...@@ -218,10 +218,11 @@ rpcrdma_convert_iovs(struct rpcrdma_xprt *r_xprt, struct xdr_buf *xdrbuf, ...@@ -218,10 +218,11 @@ rpcrdma_convert_iovs(struct rpcrdma_xprt *r_xprt, struct xdr_buf *xdrbuf,
ppages = xdrbuf->pages + (xdrbuf->page_base >> PAGE_SHIFT); ppages = xdrbuf->pages + (xdrbuf->page_base >> PAGE_SHIFT);
page_base = offset_in_page(xdrbuf->page_base); page_base = offset_in_page(xdrbuf->page_base);
while (len) { while (len) {
if (unlikely(!*ppages)) { /* ACL likes to be lazy in allocating pages - ACLs
/* XXX: Certain upper layer operations do * are small by default but can get huge.
* not provide receive buffer pages.
*/ */
if (unlikely(xdrbuf->flags & XDRBUF_SPARSE_PAGES)) {
if (!*ppages)
*ppages = alloc_page(GFP_ATOMIC); *ppages = alloc_page(GFP_ATOMIC);
if (!*ppages) if (!*ppages)
return -ENOBUFS; return -ENOBUFS;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment