IB/hfi1: Use VL15 for SM packets

Subnet Management Packets (SMP) should exclusively use VL15 and their SL
is ignored (IBTA v1.3, Section 3.5.8.2). Therefore, when an SMP is posted,
the SL in the address handle can be set to 0 by a user
application. Consequently, when an address handle is created by the IB
core, some fields in struct rvt_ah may not be set correctly by using the
SL2SC and SC2VL tables at the time. Subsequently, when the request is post
sent, the incoming swqe may fail the validation check, resulting in the
rejection of the send request.

This patch fixes the problem by using VL15 for any validation, ignoring
the SL in the address handle.

Reviewed-by: Mike Marciniszyn <mike.marciniszyn@intel.com>
Signed-off-by: Kaike Wan <kaike.wan@intel.com>
Signed-off-by: Dennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>

diff --git a/drivers/infiniband/hw/hfi1/qp.c b/drivers/infiniband/hw/hfi1/qp.c
index 126e9739e44fb2234d971294cc86401a0e5a98e6..6f3bc4dab858999740333d28f78c84f5adfa1117 100644 (file)
--- a/drivers/infiniband/hw/hfi1/qp.c
+++ b/drivers/infiniband/hw/hfi1/qp.c
@@ -306,6 +306,8 @@ int hfi1_setup_wqe(struct rvt_qp *qp, struct rvt_swqe *wqe, bool *call_send)
 {
        struct hfi1_ibport *ibp = to_iport(qp->ibqp.device, qp->port_num);
        struct rvt_ah *ah;
+       struct hfi1_pportdata *ppd;
+       struct hfi1_devdata *dd;
 
        switch (qp->ibqp.qp_type) {
        case IB_QPT_RC:
@@ -316,8 +318,16 @@ int hfi1_setup_wqe(struct rvt_qp *qp, struct rvt_swqe *wqe, bool *call_send)
                        *call_send = false;
                break;
        case IB_QPT_SMI:
-               ah = ibah_to_rvtah(wqe->ud_wr.ah);
-               if (wqe->length > (1 << ah->log_pmtu))
+               /*
+                * SM packets should exclusively use VL15 and their SL is
+                * ignored (IBTA v1.3, Section 3.5.8.2). Therefore, when ah
+                * is created, SL is 0 in most cases and as a result some
+                * fields (vl and pmtu) in ah may not be set correctly,
+                * depending on the SL2SC and SC2VL tables at the time.
+                */
+               ppd = ppd_from_ibp(ibp);
+               dd = dd_from_ppd(ppd);
+               if (wqe->length > dd->vld[15].mtu)
                        return -EINVAL;
                break;
        case IB_QPT_GSI: