diff options
author | Steve Wise <swise@opengridcomputing.com> | 2013-08-06 11:34:37 -0400 |
---|---|---|
committer | Roland Dreier <roland@purestorage.com> | 2013-08-13 14:55:47 -0400 |
commit | b38a0ad8ec1129bc2fdadc3baed45ff70d5726e2 (patch) | |
tree | 425fd70c7a88ad26a6d41bf9abfc734e9ea487e2 /drivers/infiniband | |
parent | 27ca34f54a70cb85895aa7147a6c35f1cd07fa55 (diff) |
RDMA/cxgb4: Set arp error handler for PASS_ACCEPT_RPL messages
accept_cr() failed to set the arp error handler on a reused skb. This
results in a kernel crash if the arp does indeed time out.
Signed-off-by: Steve Wise <swise@opengridcomputing.com>
Signed-off-by: Vipul Pandya <vipul@chelsio.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'drivers/infiniband')
-rw-r--r-- | drivers/infiniband/hw/cxgb4/cm.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/infiniband/hw/cxgb4/cm.c b/drivers/infiniband/hw/cxgb4/cm.c index 8865a3332a62..12fef76c791c 100644 --- a/drivers/infiniband/hw/cxgb4/cm.c +++ b/drivers/infiniband/hw/cxgb4/cm.c | |||
@@ -332,6 +332,7 @@ static struct sk_buff *get_skb(struct sk_buff *skb, int len, gfp_t gfp) | |||
332 | } else { | 332 | } else { |
333 | skb = alloc_skb(len, gfp); | 333 | skb = alloc_skb(len, gfp); |
334 | } | 334 | } |
335 | t4_set_arp_err_handler(skb, NULL, NULL); | ||
335 | return skb; | 336 | return skb; |
336 | } | 337 | } |
337 | 338 | ||
@@ -2025,6 +2026,7 @@ static void accept_cr(struct c4iw_ep *ep, struct sk_buff *skb, | |||
2025 | rpl->opt0 = cpu_to_be64(opt0); | 2026 | rpl->opt0 = cpu_to_be64(opt0); |
2026 | rpl->opt2 = cpu_to_be32(opt2); | 2027 | rpl->opt2 = cpu_to_be32(opt2); |
2027 | set_wr_txq(skb, CPL_PRIORITY_SETUP, ep->ctrlq_idx); | 2028 | set_wr_txq(skb, CPL_PRIORITY_SETUP, ep->ctrlq_idx); |
2029 | t4_set_arp_err_handler(skb, NULL, arp_failure_discard); | ||
2028 | c4iw_l2t_send(&ep->com.dev->rdev, skb, ep->l2t); | 2030 | c4iw_l2t_send(&ep->com.dev->rdev, skb, ep->l2t); |
2029 | 2031 | ||
2030 | return; | 2032 | return; |