[openib-general] [PATCH] [CM] 1/2 Fix CM redirection

[Sean Hefty]

John Kingman wrote:
> I found that CM handling for SRP is broken when handling a REJ with 
> reason 24 (Port and CM Redirection) with a RedirectLID supplied.  As 
> stated in the spec, if RedirectLID is non-zero, it is the DLID a 
> requester _shall_ use to access the class services.  I believe that 
> without this support, CM does not comply with C13-28 with respect to 
> RedirectLID.  
> 
> In my testing, the following patches seem to fix the problem.  If there 
> is a better way to fix the problem, I would appreciate the input.  

Thanks for the feedback.  I didn't realize anyone was using redirection.

Many months ago we had some discussion around supporting QP redirection.  I 
don't think that we reached any conclusion on the best way to handle it.  So, 
you are correct in that the current code base does not handle QP redirection.

I'll need to spend some time thinking about what we may want to do here.  Your 
CM patch seems good for handling a single connection, but won't result in 
automatically redirecting future connection requests to the redirected port.

I'm not sure if QP redirection should be provided by a library or integrated 
with the MAD layer.

- Sean