ide: fix cmd_write_pio when nsectors > 1
authorJohn Snow <jsnow@redhat.com>
Fri, 20 Mar 2015 00:24:13 +0000 (20:24 -0400)
committerJohn Snow <jsnow@redhat.com>
Mon, 23 Mar 2015 16:24:16 +0000 (12:24 -0400)
We need to adjust the sector being written to
prior to calling ide_transfer_start, otherwise
we'll write to the same sector again.

Signed-off-by: John Snow <jsnow@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Tested-by: Andreas Färber <afaerber@suse.de>
Message-id: 1426811056-2202-2-git-send-email-jsnow@redhat.com

hw/ide/core.c

index ef52f3516f0fb31c96cd3b00aedb696d5024c1b8..0e9da648ea99a00185eb6f03d8c201b8bfe5ecf2 100644 (file)
@@ -846,6 +846,7 @@ static void ide_sector_write_cb(void *opaque, int ret)
     s->nsector -= n;
     s->io_buffer_offset += 512 * n;
 
+    ide_set_sector(s, ide_get_sector(s) + n);
     if (s->nsector == 0) {
         /* no more sectors to write */
         ide_transfer_stop(s);
@@ -857,7 +858,6 @@ static void ide_sector_write_cb(void *opaque, int ret)
         ide_transfer_start(s, s->io_buffer, n1 * BDRV_SECTOR_SIZE,
                            ide_sector_write);
     }
-    ide_set_sector(s, ide_get_sector(s) + n);
 
     if (win2k_install_hack && ((++s->irq_count % 16) == 0)) {
         /* It seems there is a bug in the Windows 2000 installer HDD