[openib-general] Completion callback /teardown race

[Or Gerlitz]

>> I understand your driver is a CMA consumer whose QP state transitions 
>> are carried out by the CMA. So you need ***not*** modify the QP state to 
>> error, as the CMA does it for you in rdma_disconnect() before sending 
>> the DREQ or DREP.

> Yes - understood.  It's not actually harmful at this point I think.  Please
> correct me if I'm wrong.

Not that its harmful, but its not needed, so it can confuse people 
looking/debugging this code...

>> Please note that you need to call rdma_disconnect() in both sides, the 
>> one that initiates the disconnection but also on the side that gets the 
>> DREQ, that is suddenly gets RDMA_CM_EVENT_DISCONNECTED event...

> I ensure I always call rdma_disconnect() once, no matter whether I am the
> initiator or not.

cool.

Or.