diff options
| author | Chuck Lever <chuck.lever@oracle.com> | 2016-01-07 14:49:45 -0500 |
|---|---|---|
| committer | Doug Ledford <dledford@redhat.com> | 2016-01-19 15:30:48 -0500 |
| commit | 78da2b3cea05b059d23814509bb46a29101e0f77 (patch) | |
| tree | 3fdf2a902edcb30ca5abb9022e3136c1fcb68b05 /net/sunrpc/xprtrdma/svc_rdma_sendto.c | |
| parent | 39b09a1a121cb22820c374f4e92f7ca34be1b75d (diff) | |
svcrdma: Remove last two __GFP_NOFAIL call sites
Clean up.
These functions can otherwise fail, so check for page allocation
failures too.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Acked-by: Bruce Fields <bfields@fieldses.org>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'net/sunrpc/xprtrdma/svc_rdma_sendto.c')
| -rw-r--r-- | net/sunrpc/xprtrdma/svc_rdma_sendto.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/net/sunrpc/xprtrdma/svc_rdma_sendto.c b/net/sunrpc/xprtrdma/svc_rdma_sendto.c index aeaec7aa34df..de7df7b4d855 100644 --- a/net/sunrpc/xprtrdma/svc_rdma_sendto.c +++ b/net/sunrpc/xprtrdma/svc_rdma_sendto.c @@ -598,7 +598,10 @@ int svc_rdma_sendto(struct svc_rqst *rqstp) inline_bytes = rqstp->rq_res.len; /* Create the RDMA response header */ - res_page = alloc_page(GFP_KERNEL | __GFP_NOFAIL); + ret = -ENOMEM; + res_page = alloc_page(GFP_KERNEL); + if (!res_page) + goto err0; rdma_resp = page_address(res_page); reply_ary = svc_rdma_get_reply_array(rdma_argp); if (reply_ary) |