From: aliguori Date: Thu, 9 Oct 2008 14:19:34 +0000 (+0000) Subject: Fix IDE DIAGNOSE for packet devices (Vincent Sanders) X-Git-Tag: release_0_10_0~1234 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=33256a25b3b31915b9038eefe7923c68bb034118;p=thirdparty%2Fqemu.git Fix IDE DIAGNOSE for packet devices (Vincent Sanders) The DIAGNOSE command in the qemu IDE implementation has an error when use dfor packet devices. The status register value is dependant on the drive being a packet device or not, this patch corrects the returned status. From the ATA/PI specification (V6 draft): "If the device implements the PACKET command feature set, the device SHALL clear bits 6,5,4,3,2 and 0 in the Status register to zero." A selection of physical devices have been checked and do conform to the specifications behaviour. Signed-off-by: Vincent Sanders Signed-off-by: Anthony Liguori git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@5452 c046a42c-6fe2-441c-8c8c-71466251a162 --- diff --git a/hw/ide.c b/hw/ide.c index 686d372d18c..33e8b393298 100644 --- a/hw/ide.c +++ b/hw/ide.c @@ -2308,8 +2308,15 @@ static void ide_ioport_write(void *opaque, uint32_t addr, uint32_t val) break; case WIN_DIAGNOSE: ide_set_signature(s); - s->status = READY_STAT | SEEK_STAT; - s->error = 0x01; + if (s->is_cdrom) + s->status = 0; /* ATAPI spec (v6) section 9.10 defines packet + * devices to return a clear status register + * with READY_STAT *not* set. */ + else + s->status = READY_STAT | SEEK_STAT; + s->error = 0x01; /* Device 0 passed, Device 1 passed or not + * present. + */ ide_set_irq(s); break; case WIN_SRST: