[ofa-general] Re: [PATCH] ib/cm: fix stale connection detection

[Michael S. Tsirkin]

> >>Also, I left the duplicate request handling
> >>as it was, since that should go in as a separate patch.
> >
> >Could you please describe what is missing currently?
> >Is the missing handling likely to cause timeouts?
> 
> If two REQs are received with matching local IDs, but the REQs 
> themselves differ on one or more fields, such as the QPN, the second REQ 
> is dropped as a duplicate.

Why do you speak about dropping duplicates as a valid response?
As far as I can tell, the 2 legal responses to a duplicate REQ
are resending a REP and rejecting with code 30.

> This causes timeouts, so I need to figure 
> out what the correct behavior should be here.

I agree that it seems that we could use this as a hint that remote has
rebooted.

-- 
MST