From f5b6ffcf2a94337df31e801dd11b34896bd4fe2b Mon Sep 17 00:00:00 2001 From: Paolo Bonzini Date: Wed, 24 Aug 2011 05:28:52 +0000 Subject: vscsi: send the CHECK_CONDITION status down together with autosense data I introduced this bug in commit 05751d3 (vscsi: always use get_sense, 2011-08-03) because at the time there was no way to expose a sense condition to SLOF and Linux manages to work around the bug. However, the bug becomes evident now that SCSI devices also report unit attention on reset. SLOF also has problems dealing with unit attention conditions, so it still will not boot even with this fix (just like OpenBIOS). IBM folks are aware of their part of the bug. :-) Reported-by: Thomas Huth Signed-off-by: Paolo Bonzini Acked-by: David Gibson Signed-off-by: Alexander Graf --- hw/spapr_vscsi.c | 1 - 1 file changed, 1 deletion(-) (limited to 'hw/spapr_vscsi.c') diff --git a/hw/spapr_vscsi.c b/hw/spapr_vscsi.c index 6fc82f6bd7..e8426d7c5e 100644 --- a/hw/spapr_vscsi.c +++ b/hw/spapr_vscsi.c @@ -483,7 +483,6 @@ static void vscsi_command_complete(SCSIRequest *sreq, uint32_t status) if (status == CHECK_CONDITION) { req->senselen = scsi_req_get_sense(req->sreq, req->sense, sizeof(req->sense)); - status = 0; dprintf("VSCSI: Sense data, %d bytes:\n", len); dprintf(" %02x %02x %02x %02x %02x %02x %02x %02x\n", req->sense[0], req->sense[1], req->sense[2], req->sense[3], -- cgit v1.2.1