diff options
author | Ralph Campbell <ralph.campbell@qlogic.com> | 2008-04-17 00:09:25 -0400 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2008-04-17 00:09:25 -0400 |
commit | 69bd74c6968f45a864a394338a63da4b768da6ec (patch) | |
tree | 6b55ef7ca606c97337992f56876008afad38d440 /drivers/infiniband/hw/ipath | |
parent | 72708a0a2b60e83255631f2557a85ac7daf33fac (diff) |
IB/ipath: Remove useless comments
Remove useless comment about list removal since locks are held and
the code checks that the QP is on the list before removing it.
Signed-off-by: Ralph Campbell <ralph.campbell@qlogic.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers/infiniband/hw/ipath')
-rw-r--r-- | drivers/infiniband/hw/ipath/ipath_qp.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/infiniband/hw/ipath/ipath_qp.c b/drivers/infiniband/hw/ipath/ipath_qp.c index 5c8094a03643..9125e02eadaf 100644 --- a/drivers/infiniband/hw/ipath/ipath_qp.c +++ b/drivers/infiniband/hw/ipath/ipath_qp.c | |||
@@ -392,7 +392,6 @@ int ipath_error_qp(struct ipath_qp *qp, enum ib_wc_status err) | |||
392 | qp->ibqp.qp_num, qp->remote_qpn, err); | 392 | qp->ibqp.qp_num, qp->remote_qpn, err); |
393 | 393 | ||
394 | spin_lock(&dev->pending_lock); | 394 | spin_lock(&dev->pending_lock); |
395 | /* XXX What if its already removed by the timeout code? */ | ||
396 | if (!list_empty(&qp->timerwait)) | 395 | if (!list_empty(&qp->timerwait)) |
397 | list_del_init(&qp->timerwait); | 396 | list_del_init(&qp->timerwait); |
398 | if (!list_empty(&qp->piowait)) | 397 | if (!list_empty(&qp->piowait)) |
@@ -1021,7 +1020,6 @@ void ipath_sqerror_qp(struct ipath_qp *qp, struct ib_wc *wc) | |||
1021 | qp->ibqp.qp_num, qp->remote_qpn, wc->status); | 1020 | qp->ibqp.qp_num, qp->remote_qpn, wc->status); |
1022 | 1021 | ||
1023 | spin_lock(&dev->pending_lock); | 1022 | spin_lock(&dev->pending_lock); |
1024 | /* XXX What if its already removed by the timeout code? */ | ||
1025 | if (!list_empty(&qp->timerwait)) | 1023 | if (!list_empty(&qp->timerwait)) |
1026 | list_del_init(&qp->timerwait); | 1024 | list_del_init(&qp->timerwait); |
1027 | if (!list_empty(&qp->piowait)) | 1025 | if (!list_empty(&qp->piowait)) |