summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJuan Quintela <quintela@redhat.com>2009-07-31 19:01:48 +0200
committerEduardo Habkost <ehabkost@redhat.com>2009-07-31 17:37:07 -0300
commitb4294916b1fc3fc789478d8c73017666969f2ec4 (patch)
treef61b4bfa683f10eb641c9e4c0baa97fdaebfe0a0
parent32bf73073ae7d6c9b509001ad05f8eeaf20b0f66 (diff)
ide calls register_savevm() three times for pci_ide_save/load
Hi In this commit, it was changed the save/load version for ide to 3 in only 1 place, but pci_ide_save/load is called from three places. This patch fixes it. commit ee49f80b360dd57acc61df1694fc7ea798ffa485 Author: Gleb Natapov <gleb@redhat.com> Date: Wed Jul 29 19:15:11 2009 +0300 As funny as it can get, the problem that we are fixing is that keyboard didn't worked after migration. If you really like strange bugs :) Later, Juan. bugzilla 514887 [ehabkost: more detailed description from Markus Armbruster below] ide.c defines a couple of chips: ISA, CMD646, PIIX3, PIIX4, PowerMac, MMIO, DSCM-1XXXX. Gleb's commit changed pci_ide_save() and pci_ide_load() to put/get an additional four bytes. The two functions are used with CMD646, PIIX3 and PIIX3. The commit bumped the savevm version_id only for CMD646, not for PIIX3 and PIIX4. Juan's patch bumps the other two. Signed-off-by: Juan Quintela <quintela@redhat.com> Message-Id: <1249059708-32209-1-git-send-email-quintela@redhat.com> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com> Bugzilla: 514887 Acked-by: Eduardo Habkost <ehabkost@redhat.com> Acked-by: Markus Armbruster <armbru@redhat.com> Acked-by: Rik van Riel <riel@redhat.com>
-rw-r--r--qemu/hw/ide.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/qemu/hw/ide.c b/qemu/hw/ide.c
index e9911625..1e6b2b4d 100644
--- a/qemu/hw/ide.c
+++ b/qemu/hw/ide.c
@@ -3381,7 +3381,7 @@ void pci_piix3_ide_init(PCIBus *bus, BlockDriverState **hd_table, int devfn,
qemu_register_reset(piix3_reset, d);
piix3_reset(d);
- register_savevm("ide", 0, 2, pci_ide_save, pci_ide_load, d);
+ register_savevm("ide", 0, 3, pci_ide_save, pci_ide_load, d);
}
/* hd_table must contain 4 block drivers */
@@ -3420,7 +3420,7 @@ void pci_piix4_ide_init(PCIBus *bus, BlockDriverState **hd_table, int devfn,
qemu_register_reset(piix3_reset, d);
piix3_reset(d);
- register_savevm("ide", 0, 2, pci_ide_save, pci_ide_load, d);
+ register_savevm("ide", 0, 3, pci_ide_save, pci_ide_load, d);
}
/***********************************************************/