>>>>> "Hiral" == Hiral Shah <[email protected]> writes:

Hiral> We added changes in fnic driver patch 1.6.0.16 to acquire
Hiral> io_req_lock in fnic_queuecommand() before issuing I/O so that io
Hiral> completion is serialized. But when releasing the lock we check
Hiral> for the I/O flag and this could be modified if IO abort occurs
Hiral> before I/O completion. In this case we wont release the lock and
Hiral> causes deadlock in some scenerios. Using the local variable to
Hiral> check the IO lock status will resolve the problem.

Maybe bool instead of int?

Otherwise OK.

Reviewed-by: Martin K. Petersen <[email protected]>

-- 
Martin K. Petersen      Oracle Linux Engineering
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to