[PATCH 2/5] Allocate a callback request only if you have a mempool<br>

[Rahul]

<br>
This patch adds a check for xprt-&gt;bc_mempool != NULL in xs_tcp_read_request.<br>
This is for cases when you may receive a packet with the call header set. If<br>
you&#39;re not expecting callbacks on the channel (e.g NFSv4.0), the code will<br>
panic<br>
<br>
i don&#39;t get when NFSv4.0 would set the call header.<br>
<br>
Signed-off-by: Rahul Iyer &lt;<a href="mailto:iyer at netapp.com">iyer at netapp.com</a>&gt;<br>
---<br>
&nbsp;net/sunrpc/xprtsock.c |&nbsp;&nbsp;&nbsp; 9 ++++++++-<br>
&nbsp;1 files changed, 8 insertions(+), 1 deletions(-)<br>
<br>
diff --git a/net/sunrpc/xprtsock.c b/net/sunrpc/xprtsock.c<br>
index 94cc6dd..8c24516 100644<br>
--- a/net/sunrpc/xprtsock.c<br>
+++ b/net/sunrpc/xprtsock.c<br>
@@ -691,7 +691,14 @@ static inline void xs_tcp_read_request(struct rpc_xprt *xprt, skb_reader_t *desc<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; calldir = ntohl(xprt-&gt;tcp_calldir);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if (calldir == RPC_CALL) {<br>
<br>
-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; req = xprt_alloc_bc_request(xprt);<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if (xprt-&gt;bc_mempool)<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
req = xprt_alloc_bc_request(xprt);<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; else {<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
/*<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
* Received callback when none is expected<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
*/<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
req = NULL;<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }<br>
<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if (!req) {<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
printk(KERN_NOTICE &quot;Couldn&#39;t get rpc_rqst for the allback! Dropping
callback...\n&quot;);<br>
--<br>
<br>
don&#39;t you want to move the KERN_NOTICE into the case where xprt_alloc_bc_request fails?<br>
<br>
--&gt;Andy<br>
&nbsp;</div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Thanks<br>Regards<br>Rahul<br><br>
</blockquote></div><br>

------=_Part_12247_26634023.1180624924764--