On 6/22/2017 12:41 AM, Johannes Thumshirn wrote:
On Wed, Jun 21, 2017 at 10:37:58AM -0700, James Smart wrote:
[...]
-       /*
-        * Catch race where our node has transitioned, but the
-        * transport is still transitioning.
-        */


The patch looks fine but can you please explain why the above race can't
happen anymore?

Thanks,
Reviewed-by: Johannes Thumshirn <jthumsh...@suse.de>


well, the primary reason is the code that was removed was bogus to begin with. Abort of an io/exchange, if its active, needs to occur on the link regardless of the ndlp state. This it simply needed to occur. As not checks on ndlp state are needed, there is no race.

-- james

Reply via email to