diff options
| author |   Sean Hefty <sean.hefty@intel.com> | 2007-07-02 16:14:14 -0700 |
|---|---|---|
| committer |   Roland Dreier <rolandd@cisco.com> | 2007-07-10 21:52:28 -0700 |
| commit | 6164c8cd1333403a28202f7c7e64ff9086d8f1aa (patch) | |
| tree | 13f7156734c73a7f456572b252b8a8de87a65217 /drivers/infiniband/core/cm.c | |
| parent | IB/cm: Fix handling of duplicate SIDR REQs (diff) | |
| download | linux-dev-6164c8cd1333403a28202f7c7e64ff9086d8f1aa.tar.xz linux-dev-6164c8cd1333403a28202f7c7e64ff9086d8f1aa.zip | |
IB/cm: Send no match if a SIDR REQ does not match a listen
If a SIDR REQ does not match a listen, we should reply with status
value 1 (service ID not supported), rather than dropping through to
the default case of status 2 (rejected by service provider).
Doing this also fixes a bug where the cm_id_priv is removed from the
remote_sidr_table twice.
Signed-off-by: Sean Hefty <sean.hefty@intel.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers/infiniband/core/cm.c')
| -rw-r--r-- | drivers/infiniband/core/cm.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/infiniband/core/cm.c b/drivers/infiniband/core/cm.c index 9135a8c1d4ac..9820c67ba47d 100644 --- a/drivers/infiniband/core/cm.c +++ b/drivers/infiniband/core/cm.c @@ -2808,9 +2808,8 @@ static int cm_sidr_req_handler(struct cm_work *work) sidr_req_msg->service_id, sidr_req_msg->private_data); if (!cur_cm_id_priv) { - rb_erase(&cm_id_priv->sidr_id_node, &cm.remote_sidr_table); spin_unlock_irq(&cm.lock); - /* todo: reply with no match */ + cm_reject_sidr_req(cm_id_priv, IB_SIDR_UNSUPPORTED); goto out; /* No match. */ } atomic_inc(&cur_cm_id_priv->refcount); |