diff options
| author | Yannick Cote <yannick.cote@qlogic.com> | 2008-09-30 00:24:04 -0400 |
|---|---|---|
| committer | Roland Dreier <rolandd@cisco.com> | 2008-09-30 00:24:04 -0400 |
| commit | e441d6342890838bfc6d64ca2f0964aca08ae2a2 (patch) | |
| tree | 94a3fa3aba43ddf3b7381435c347d1699e755ab1 | |
| parent | 940358967599ba9057b3c51ba906e1cd5b984729 (diff) | |
IB/ipath: Fix hang on module unload
Handle the case where posting a send is requested when the link is
down. This fixes <https://bugs.openfabrics.org/show_bug.cgi?id=1117>.
Signed-off-by: Yannick Cote <yannick.cote@qlogic.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
| -rw-r--r-- | drivers/infiniband/hw/ipath/ipath_verbs.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/drivers/infiniband/hw/ipath/ipath_verbs.c b/drivers/infiniband/hw/ipath/ipath_verbs.c index b766e40e9ebf..eabc4247860b 100644 --- a/drivers/infiniband/hw/ipath/ipath_verbs.c +++ b/drivers/infiniband/hw/ipath/ipath_verbs.c | |||
| @@ -340,9 +340,16 @@ static int ipath_post_one_send(struct ipath_qp *qp, struct ib_send_wr *wr) | |||
| 340 | int acc; | 340 | int acc; |
| 341 | int ret; | 341 | int ret; |
| 342 | unsigned long flags; | 342 | unsigned long flags; |
| 343 | struct ipath_devdata *dd = to_idev(qp->ibqp.device)->dd; | ||
| 343 | 344 | ||
| 344 | spin_lock_irqsave(&qp->s_lock, flags); | 345 | spin_lock_irqsave(&qp->s_lock, flags); |
| 345 | 346 | ||
| 347 | if (qp->ibqp.qp_type != IB_QPT_SMI && | ||
| 348 | !(dd->ipath_flags & IPATH_LINKACTIVE)) { | ||
| 349 | ret = -ENETDOWN; | ||
| 350 | goto bail; | ||
| 351 | } | ||
| 352 | |||
| 346 | /* Check that state is OK to post send. */ | 353 | /* Check that state is OK to post send. */ |
| 347 | if (unlikely(!(ib_ipath_state_ops[qp->state] & IPATH_POST_SEND_OK))) | 354 | if (unlikely(!(ib_ipath_state_ops[qp->state] & IPATH_POST_SEND_OK))) |
| 348 | goto bail_inval; | 355 | goto bail_inval; |
