[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 4/5] esp: revert 75ef849696 "esp: correctly fill bus id with requ
From: |
Mark Cave-Ayland |
Subject: |
[PATCH 4/5] esp: revert 75ef849696 "esp: correctly fill bus id with requested lun" |
Date: |
Wed, 19 May 2021 11:08:02 +0100 |
This commit from nearly 10 years ago no longer appears to be required and in its
current form prevents the MacOS CDROM driver from detecting the CDROM drive. The
error is caused by the MacOS CDROM driver sending this CDB without DMA:
0x12 0x00 0x00 0x00 0x05 0x00 (INQUIRY)
This is a valid INQUIRY command, however with this logic present the 3rd byte
(0x0) is copied over the 1st byte (0x12) which silently converts the INQUIRY
command to a TEST UNIT READY command before passing it to the QEMU SCSI layer.
Since the TEST UNIT READY command has a zero length response the MacOS CDROM
driver never receives a response and assumes the CDROM is not present.
Resolve the issue by reverting the original commit which I'm fairly sure is now
obsolete so that the original MacOS CDB is passed unaltered to the QEMU SCSI
layer.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
---
hw/scsi/esp.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/hw/scsi/esp.c b/hw/scsi/esp.c
index afb4a7f9f1..a6f7c6c1bf 100644
--- a/hw/scsi/esp.c
+++ b/hw/scsi/esp.c
@@ -260,9 +260,6 @@ static uint32_t get_cmd(ESPState *s, uint32_t maxlen)
return 0;
}
n = esp_fifo_pop_buf(&s->fifo, buf, dmalen);
- if (n >= 3) {
- buf[0] = buf[2] >> 5;
- }
n = MIN(fifo8_num_free(&s->cmdfifo), n);
fifo8_push_all(&s->cmdfifo, buf, n);
}
--
2.20.1
- [PATCH 0/5] esp: fixes for MacOS toolbox ROM, Mark Cave-Ayland, 2021/05/19
- [PATCH 2/5] esp: handle non-DMA transfers from the target one byte at a time, Mark Cave-Ayland, 2021/05/19
- [PATCH 3/5] esp: ensure PDMA write transfers are flushed from the FIFO to the target immediately, Mark Cave-Ayland, 2021/05/19
- [PATCH 4/5] esp: revert 75ef849696 "esp: correctly fill bus id with requested lun",
Mark Cave-Ayland <=
- [PATCH 5/5] esp: correctly accumulate extended messages for PDMA, Mark Cave-Ayland, 2021/05/19
- [PATCH 1/5] esp: allow non-DMA callback in esp_transfer_data() initial transfer, Mark Cave-Ayland, 2021/05/19
- Re: [PATCH 0/5] esp: fixes for MacOS toolbox ROM, Mark Cave-Ayland, 2021/05/28