Re: [Qemu-devel] LSI53C895A, IDE HDD detection under SCO OpenServer 5.0.5

[Justin Chevrier]

Hey guys,

Well I've continued to work on the:

...
WARNING: A_DeviceReset interrupt on ha=0 id=0 lun=5 tag=A6
...

errors (in the emulated OS) when installing Openserver.

I believe I have narrowed down the problem. Openserver does an INQUIRY on 
various LUNs above 0 (which we don't support). We return 0x7f (nothing here, 
move along). Openserver then proceeds to send another command to the SCSI disk. 
The disk then returns:

0x02 - STATUS_CHECK_CONDITION
0x05 - SENSE_ILLEGAL_REQUEST

which is as expected. The problem comes along when the controller goes into 
Status Phase and compares the resulting sense returned by the drive to see what 
the status of the drive is. What I've found is that the controller is using the 
status value (STATUS_CHECK_CONDITION) as the sense value. So it thinks we've 
recieved a sense return of 'SENSE_NOT_READY' and then it keeps probing the 
drive until it finally gives up, and we get the above error. See the log below:

...
lsi_scsi: SCRIPTS dsp=f20021a4 opcode 1a000000 arg1 00000018 arg2 810b0000
lsi_scsi: Send command len=6
scsi-disk: Command: lun=1 tag=0x100c1 data=0x00 0x20 0x00 0x00 0x00 0x00
scsi-disk: Unimplemented LUN 1
scsi-disk: Command complete tag=0x100c1 status=2 sense=5 <--****
lsi_scsi: Command reason: 0
lsi_scsi: Command complete sense=2 <-- ****
...

The patch below changes 'scsi_command_complete' (in scsi-disk.c) to pass a 
sense value along to the callbak instead of status.

I'm not sure this is where we want to go however as this basically reverses 
part of rev 5455 (which made a point of specifically changing this line).

Do we need to update the LSI emulation to make use of status as well?

Either way, at least we know the cause of these errors and this can be a hack 
at least if it's not the direction we want to take.

Alas this does not fix Openserver's install problem (not too surprised). Onto 
the next bug...

Justin

--- hw/scsi-disk.c      (revision 5722)
+++ hw/scsi-disk.c      (working copy)
@@ -135,7 +135,7 @@
     s->sense = sense;
     tag = r->tag;
     scsi_remove_request(r);
-    s->completion(s->opaque, SCSI_REASON_DONE, tag, status);
+    s->completion(s->opaque, SCSI_REASON_DONE, tag, sense);
 }

 /* Cancel a pending data transfer.  */