From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MW9Yh-0004Ca-Bo for qemu-devel@nongnu.org; Wed, 29 Jul 2009 09:51:07 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MW9Yb-00048h-Gq for qemu-devel@nongnu.org; Wed, 29 Jul 2009 09:51:06 -0400 Received: from [199.232.76.173] (port=59700 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MW9Yb-00048Q-78 for qemu-devel@nongnu.org; Wed, 29 Jul 2009 09:51:01 -0400 Received: from mx2.redhat.com ([66.187.237.31]:53225) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MW9Ya-0003v4-IY for qemu-devel@nongnu.org; Wed, 29 Jul 2009 09:51:00 -0400 Received: from int-mx2.corp.redhat.com (int-mx2.corp.redhat.com [172.16.27.26]) by mx2.redhat.com (8.13.8/8.13.8) with ESMTP id n6TDoxs7012821 for ; Wed, 29 Jul 2009 09:50:59 -0400 Received: from ns3.rdu.redhat.com (ns3.rdu.redhat.com [10.11.255.199]) by int-mx2.corp.redhat.com (8.13.1/8.13.1) with ESMTP id n6TDowlL031403 for ; Wed, 29 Jul 2009 09:50:59 -0400 Received: from dhcp-1-237.tlv.redhat.com (dhcp-1-237.tlv.redhat.com [10.35.1.237]) by ns3.rdu.redhat.com (8.13.8/8.13.8) with ESMTP id n6TDowWH001718 for ; Wed, 29 Jul 2009 09:50:58 -0400 Date: Wed, 29 Jul 2009 16:50:57 +0300 From: Gleb Natapov Message-ID: <20090729135057.GY30449@redhat.com> References: <20090729120719.GN30449@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20090729120719.GN30449@redhat.com> Subject: [Qemu-devel] [PATCH v2] make windows notice media change List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Windows seems to be very stupid about cdrom media change. It polls cdrom status and if status goes ready->media not present->ready it assumes that media was changed. If "media not present" step doesn't happen even if "medium may have changed" was seen it assumes media haven't changed. Fake "media not present" step. Signed-off-by: Gleb Natapov --- v1->v2: do not reuse media_changed. Save/restore on migration. diff --git a/hw/ide.c b/hw/ide.c index 1e56786..b8f35ad 100644 --- a/hw/ide.c +++ b/hw/ide.c @@ -418,6 +418,7 @@ typedef struct IDEState { /* ATAPI specific */ uint8_t sense_key; uint8_t asc; + uint8_t cdrom_changed; int packet_transfer_size; int elementary_transfer_size; int io_buffer_index; @@ -1644,9 +1645,10 @@ static void ide_atapi_cmd(IDEState *s) } switch(s->io_buffer[0]) { case GPCMD_TEST_UNIT_READY: - if (bdrv_is_inserted(s->bs)) { + if (bdrv_is_inserted(s->bs) && !s->cdrom_changed) { ide_atapi_cmd_ok(s); } else { + s->cdrom_changed = 0; ide_atapi_cmd_error(s, SENSE_NOT_READY, ASC_MEDIUM_NOT_PRESENT); } @@ -2106,7 +2108,7 @@ static void cdrom_change_cb(void *opaque) s->sense_key = SENSE_UNIT_ATTENTION; s->asc = ASC_MEDIUM_MAY_HAVE_CHANGED; - + s->cdrom_changed = 1; ide_set_irq(s); } @@ -2870,6 +2872,7 @@ static void ide_save(QEMUFile* f, IDEState *s) qemu_put_8s(f, &s->sense_key); qemu_put_8s(f, &s->asc); + qemu_put_8s(f, &s->cdrom_changed); /* XXX: if a transfer is pending, we do not save it yet */ } @@ -2898,6 +2901,7 @@ static void ide_load(QEMUFile* f, IDEState *s) qemu_get_8s(f, &s->sense_key); qemu_get_8s(f, &s->asc); + qemu_get_8s(f, &s->cdrom_changed); /* XXX: if a transfer is pending, we do not save it yet */ } @@ -3219,7 +3223,7 @@ static int pci_ide_load(QEMUFile* f, void *opaque, int version_id) PCIIDEState *d = opaque; int ret, i; - if (version_id != 2) + if (version_id != 3) return -EINVAL; ret = pci_device_load(&d->dev, f); if (ret < 0) @@ -3339,7 +3343,7 @@ void pci_cmd646_ide_init(PCIBus *bus, BlockDriverState **hd_table, ide_init2(&d->ide_if[0], hd_table[0], hd_table[1], irq[0]); ide_init2(&d->ide_if[2], hd_table[2], hd_table[3], irq[1]); - register_savevm("ide", 0, 2, pci_ide_save, pci_ide_load, d); + register_savevm("ide", 0, 3, pci_ide_save, pci_ide_load, d); qemu_register_reset(cmd646_reset, d); cmd646_reset(d); } -- Gleb.