diff options
author | Ralph Campbell <ralph.campbell@qlogic.com> | 2007-11-14 13:34:14 -0800 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2007-11-20 11:05:42 -0800 |
commit | 4187b915a0f7eaa69707715e80d9fc253ff6167a (patch) | |
tree | 8ad1bada5abd9ef982a05a08b54a3dfdf0086187 /drivers/infiniband/hw/ipath/ipath_verbs.c | |
parent | 14de986a0ba560b54340fd277a3579e95a2d3838 (diff) | |
download | lwn-4187b915a0f7eaa69707715e80d9fc253ff6167a.tar.gz lwn-4187b915a0f7eaa69707715e80d9fc253ff6167a.zip |
IB/ipath: Normalize error return codes for posting work requests
The error codes for ib_post_send(), ib_post_recv(), and ib_post_srq_recv()
were inconsistent. Use EINVAL for too many SGEs and ENOMEM for too many
WRs.
Signed-off-by: Ralph Campbell <ralph.campbell@qlogic.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers/infiniband/hw/ipath/ipath_verbs.c')
-rw-r--r-- | drivers/infiniband/hw/ipath/ipath_verbs.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/drivers/infiniband/hw/ipath/ipath_verbs.c b/drivers/infiniband/hw/ipath/ipath_verbs.c index 74f77e7c2c1b..c4c998446c7b 100644 --- a/drivers/infiniband/hw/ipath/ipath_verbs.c +++ b/drivers/infiniband/hw/ipath/ipath_verbs.c @@ -302,8 +302,10 @@ static int ipath_post_one_send(struct ipath_qp *qp, struct ib_send_wr *wr) next = qp->s_head + 1; if (next >= qp->s_size) next = 0; - if (next == qp->s_last) - goto bail_inval; + if (next == qp->s_last) { + ret = -ENOMEM; + goto bail; + } wqe = get_swqe_ptr(qp, qp->s_head); wqe->wr = *wr; @@ -404,7 +406,7 @@ static int ipath_post_receive(struct ib_qp *ibqp, struct ib_recv_wr *wr, if ((unsigned) wr->num_sge > qp->r_rq.max_sge) { *bad_wr = wr; - ret = -ENOMEM; + ret = -EINVAL; goto bail; } |