[openib-general] [CM] what happen if the path in the REQ packet (primary or alternate) is not reversible?

[Michael S. Tsirkin]

> The reversible bit needs to be set as well.

Like this, then?

---

SRP must set IB_SA_PATH_REC_REVERSIBLE since that's the only kind of path
CM currently supports.

Untested.

Signed-off-by: Michael S. Tsirkin <mst at mellanox.co.il>


diff --git a/drivers/infiniband/ulp/srp/ib_srp.c b/drivers/infiniband/ulp/srp/ib_srp.c
index 4b09147..df98754 100644
--- a/drivers/infiniband/ulp/srp/ib_srp.c
+++ b/drivers/infiniband/ulp/srp/ib_srp.c
@@ -266,6 +266,7 @@ static void srp_path_rec_completion(int
 static int srp_lookup_path(struct srp_target_port *target)
 {
 	target->path.numb_path = 1;
+	target->path.reversible = 1;
 
 	init_completion(&target->done);
 
@@ -276,6 +277,7 @@ static int srp_lookup_path(struct srp_ta
 						   IB_SA_PATH_REC_DGID		|
 						   IB_SA_PATH_REC_SGID		|
 						   IB_SA_PATH_REC_NUMB_PATH	|
+						   IB_SA_PATH_REC_REVERSIBLE    |
 						   IB_SA_PATH_REC_PKEY,
 						   SRP_PATH_REC_TIMEOUT_MS,
 						   GFP_KERNEL,

-- 
MST