On 16/11/20 20:05, Hannes Reinecke wrote:
+    if (sreq->host_status == SCSI_HOST_OK) {
+        SCSISense sense;
+
+        sreq->status = scsi_sense_from_host_status(sreq->host_status, &sense);
+        if (sreq->status == CHECK_CONDITION) {
+            scsi_req_build_sense(sreq, sense);
+        }
+    }

Should be != of course.

No.
scsi_req_build_sense() transfers the sense code from the second argument
into a proper SCSI sense. Which is only set if the status is CHECK_CONDITION...

I mean sreq->host_status != SCSI_HOST_OK. I might be wrong, but every other HBA is using that...

Paolo


Reply via email to