On Wed, 2018-04-04 at 14:14 -0400, Douglas Gilbert wrote: > Patch e39a97353e53 titled "scsi: core: return BLK_STS_OK for DID_OK in > __scsi_error_from_host_byte()" attempted to make that function return > BLK_STS_OK when it was given host_byte(result)==DID_OK. While that seems > sensible, it failed to take into account that there may be errors present > in the driver_byte and the status byte. Add those checks and expand > description of function accordingly. Hello Doug, Please clarify why you posted a new patch instead of reviewing "[PATCH v2] Fix DID_OK handling in __scsi_error_from_host_byte()". Did you perhaps miss that patch? Thanks, Bart.