diff options
author | Ira Weiny <weiny2@llnl.gov> | 2011-05-19 21:19:28 -0400 |
---|---|---|
committer | Roland Dreier <roland@purestorage.com> | 2011-05-23 14:15:48 -0400 |
commit | c8367c4cd9de512d296fc557f121be62a43987f3 (patch) | |
tree | 36b9c185a8ebda7dfbe1a61884291f6ae04766d8 | |
parent | 257313b2a87795e07a0bdf58d0fffbdba8b31051 (diff) |
IB/mad: Return EPROTONOSUPPORT when an RDMA device lacks the QP required
We had a script which was looping through the devices returned from
ibstat and attempted to register a SMI agent on an ethernet device.
This caused a kernel panic for IBoE devices that don't have QP0.
Fix this by checking if the QP exists before using it.
Signed-off-by: Ira Weiny <weiny2@llnl.gov>
Acked-by: Sean Hefty <sean.hefty@intel.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
-rw-r--r-- | drivers/infiniband/core/mad.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/drivers/infiniband/core/mad.c b/drivers/infiniband/core/mad.c index 822cfdcd9f78..b4d8672a3e4e 100644 --- a/drivers/infiniband/core/mad.c +++ b/drivers/infiniband/core/mad.c | |||
@@ -276,6 +276,13 @@ struct ib_mad_agent *ib_register_mad_agent(struct ib_device *device, | |||
276 | goto error1; | 276 | goto error1; |
277 | } | 277 | } |
278 | 278 | ||
279 | /* Verify the QP requested is supported. For example, Ethernet devices | ||
280 | * will not have QP0 */ | ||
281 | if (!port_priv->qp_info[qpn].qp) { | ||
282 | ret = ERR_PTR(-EPROTONOSUPPORT); | ||
283 | goto error1; | ||
284 | } | ||
285 | |||
279 | /* Allocate structures */ | 286 | /* Allocate structures */ |
280 | mad_agent_priv = kzalloc(sizeof *mad_agent_priv, GFP_KERNEL); | 287 | mad_agent_priv = kzalloc(sizeof *mad_agent_priv, GFP_KERNEL); |
281 | if (!mad_agent_priv) { | 288 | if (!mad_agent_priv) { |